RE: [PATCH] soc/tegra: pmc: Add IO Pad table for tegra234

From: Petlozu Pravareshwar
Date: Mon Aug 08 2022 - 16:10:18 EST


>
> On 05/08/2022 11:41, Petlozu Pravareshwar wrote:
> > Add IO PAD table for tegra234 to allow configuring dpd mode and
> > switching the pins to 1.8V or 3.3V as needed.
> >
> > In tegra234, DPD registers are reorganized such that there is a
> > DPD_REQ register and a DPD_STATUS register per pad group.
> > This change accordingly updates the PMC driver.
> >
> > Signed-off-by: Petlozu Pravareshwar <petlozup@xxxxxxxxxx>
> > ---
> > drivers/soc/tegra/pmc.c | 116
> ++++++++++++++++++++++++++++++++++++++--
> > 1 file changed, 112 insertions(+), 4 deletions(-)
>
>
> snip
>
> > /**
> > @@ -1546,6 +1560,14 @@ static int
> tegra_io_pad_get_dpd_register_bit(struct tegra_pmc *pmc,
> > if (pad->dpd == UINT_MAX)
> > return -ENOTSUPP;
> >
> > + if (pmc->soc->has_reorg_hw_dpd_reg_impl) {
> > + *mask = BIT(pad->dpd);
> > + *status = pmc->soc->regs->reorg_dpd_status[pad->reg_index];
> > + *request =
> > + pmc->soc->regs->reorg_dpd_req[pad->reg_index];
> > +
> > + goto done;
> > + }
> > +
> > *mask = BIT(pad->dpd % 32);
> >
> > if (pad->dpd < 32) {
> > @@ -1556,6 +1578,7 @@ static int
> tegra_io_pad_get_dpd_register_bit(struct tegra_pmc *pmc,
> > *request = pmc->soc->regs->dpd2_req;
> > }
> >
> > +done:
> > return 0;
> > }
> >
> > @@ -3132,6 +3155,7 @@ static const struct tegra_pmc_soc
> tegra20_pmc_soc = {
> > .num_pmc_clks = 0,
> > .has_blink_output = true,
> > .has_usb_sleepwalk = true,
> > + .has_reorg_hw_dpd_reg_impl = false,
> > };
>
> I don't think you need to set false values in these blocks, the default should
> be false.
Agree. Will update the patch accordingly.

Thanks.