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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <DM6PR12MB3164E33542018A69895585CAB5639@DM6PR12MB3164.namprd12.prod.outlook.com>
Date:   Mon, 8 Aug 2022 20:10:09 +0000
From:   Petlozu Pravareshwar <petlozup@...dia.com>
To:     Ben Dooks <ben.dooks@...ethink.co.uk>,
        Jonathan Hunter <jonathanh@...dia.com>,
        "thierry.reding@...il.com" <thierry.reding@...il.com>,
        "p.zabel@...gutronix.de" <p.zabel@...gutronix.de>,
        "dmitry.osipenko@...labora.com" <dmitry.osipenko@...labora.com>,
        "ulf.hansson@...aro.org" <ulf.hansson@...aro.org>,
        "Kartik ." <kkartik@...dia.com>,
        "cai.huoqing@...ux.dev" <cai.huoqing@...ux.dev>,
        Sandipan Patra <spatra@...dia.com>,
        "linux-tegra@...r.kernel.org" <linux-tegra@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] soc/tegra: pmc: Add IO Pad table for tegra234

> 
> 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@...dia.com>
> > ---
> >   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.