linux-kernel - RE: [PATCH 2/2] gpio: tegra186: Add support for Tegra234 gpio

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <DM5PR12MB240697948C6DFF64E45AFF00B78C9@DM5PR12MB2406.namprd12.prod.outlook.com>
Date:   Wed, 3 Nov 2021 11:01:46 +0000
From:   Prathamesh Shete <pshete@...dia.com>
To:     Thierry Reding <thierry.reding@...il.com>
CC:     "linus.walleij@...aro.org" <linus.walleij@...aro.org>,
        "bgolaszewski@...libre.com" <bgolaszewski@...libre.com>,
        Jonathan Hunter <jonathanh@...dia.com>,
        "linux-gpio@...r.kernel.org" <linux-gpio@...r.kernel.org>,
        "linux-tegra@...r.kernel.org" <linux-tegra@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        Suresh Mangipudi <smangipudi@...dia.com>
Subject: RE: [PATCH 2/2] gpio: tegra186: Add support for Tegra234 gpio

Hi Thierry,

>>That said, I noticed that you didn't include an update to the device tree bindings documentation. I do have that, in addition to a conversion to the new json-schema format. Do you mind if I resend your two patches here along with the DT >>documentatino update that I have? I've also got the DTS changes for Tegra234 to expose these, which I would also like to include for completeness.

It would be helpful if you share the update to the device tree bindings documentation patch with me I will push all the changes together.
OR 
Can you please resend these patches along with device tree binding document patch that you have.
Anything is fine for me.

Thanks
Prathamesh.

-----Original Message-----
From: Thierry Reding <thierry.reding@...il.com> 
Sent: Tuesday, November 2, 2021 6:52 PM
To: Prathamesh Shete <pshete@...dia.com>
Cc: linus.walleij@...aro.org; bgolaszewski@...libre.com; Jonathan Hunter <jonathanh@...dia.com>; linux-gpio@...r.kernel.org; linux-tegra@...r.kernel.org; linux-kernel@...r.kernel.org; Suresh Mangipudi <smangipudi@...dia.com>
Subject: Re: [PATCH 2/2] gpio: tegra186: Add support for Tegra234 gpio

On Thu, Oct 28, 2021 at 04:47:27PM +0200, Thierry Reding wrote:
> On Thu, Oct 21, 2021 at 06:00:21PM +0530, Prathamesh Shete wrote:
> > Add support for the Tegra234 GPIO bank configuration.
> > 
> > Signed-off-by: Prathamesh Shete <pshete@...dia.com>
> > ---
> >  drivers/gpio/gpio-tegra186.c | 74 
> > ++++++++++++++++++++++++++++++++++++
> >  1 file changed, 74 insertions(+)
> > 
> > diff --git a/drivers/gpio/gpio-tegra186.c 
> > b/drivers/gpio/gpio-tegra186.c index d38980b9923a..edcc91b35e1e 
> > 100644
> > --- a/drivers/gpio/gpio-tegra186.c
> > +++ b/drivers/gpio/gpio-tegra186.c
> > @@ -14,6 +14,7 @@
> >  
> >  #include <dt-bindings/gpio/tegra186-gpio.h>
> >  #include <dt-bindings/gpio/tegra194-gpio.h>
> > +#include <dt-bindings/gpio/tegra234-gpio.h>
> >  
> >  /* security registers */
> >  #define TEGRA186_GPIO_CTL_SCR 0x0c
> > @@ -877,6 +878,73 @@ static const struct tegra_gpio_soc tegra194_aon_soc = {
> >  	.instance = 1,
> >  };
> >  
> > +#define TEGRA234_MAIN_GPIO_PORT(_name, _bank, _port, _pins)	\
> > +	[TEGRA234_MAIN_GPIO_PORT_##_name] = {			\
> > +		.name = #_name,					\
> > +		.bank = _bank,					\
> > +		.port = _port,					\
> > +		.pins = _pins,					\
> > +	}
> > +
> > +static const struct tegra_gpio_port tegra234_main_ports[] = {
> > +	TEGRA234_MAIN_GPIO_PORT(A, 0, 0, 8),
> > +	TEGRA234_MAIN_GPIO_PORT(B, 0, 3, 1),
> > +	TEGRA234_MAIN_GPIO_PORT(C, 5, 1, 8),
> > +	TEGRA234_MAIN_GPIO_PORT(D, 5, 2, 4),
> > +	TEGRA234_MAIN_GPIO_PORT(E, 5, 3, 8),
> > +	TEGRA234_MAIN_GPIO_PORT(F, 5, 4, 6),
> > +	TEGRA234_MAIN_GPIO_PORT(G, 4, 0, 8),
> > +	TEGRA234_MAIN_GPIO_PORT(H, 4, 1, 8),
> > +	TEGRA234_MAIN_GPIO_PORT(I, 4, 2, 7),
> > +	TEGRA234_MAIN_GPIO_PORT(J, 5, 0, 6),
> > +	TEGRA234_MAIN_GPIO_PORT(K, 3, 0, 8),
> > +	TEGRA234_MAIN_GPIO_PORT(L, 3, 1, 4),
> > +	TEGRA234_MAIN_GPIO_PORT(M, 2, 0, 8),
> > +	TEGRA234_MAIN_GPIO_PORT(N, 2, 1, 8),
> > +	TEGRA234_MAIN_GPIO_PORT(P, 2, 2, 8),
> > +	TEGRA234_MAIN_GPIO_PORT(Q, 2, 3, 8),
> > +	TEGRA234_MAIN_GPIO_PORT(R, 2, 4, 6),
> 
> I stumbled across an old patch I had created a couple of months ago 
> that is very similar to this one. However, at the time I had added a 
> couple more ports here, namely S, T, U and V. Is there a reason why 
> you're not including those here?

Nevermind, looks like my patch had been based on out-of-date documentation, so the table here is correct.

That said, I noticed that you didn't include an update to the device tree bindings documentation. I do have that, in addition to a conversion to the new json-schema format. Do you mind if I resend your two patches here along with the DT documentatino update that I have? I've also got the DTS changes for Tegra234 to expose these, which I would also like to include for completeness.

Thierry