RE: [PATCH v5 2/3] gpio: pca9570: add a platform data structure

From: "Datta, Shubhrajyoti" <shubhrajyoti.datta@amd.com>
To: Bartosz Golaszewski <brgl@bgdev.pl>,
	Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>,
	"linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>,
	"git (AMD-Xilinx)" <git@amd.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"krzysztof.kozlowski+dt@linaro.org" 
	<krzysztof.kozlowski+dt@linaro.org>,
	"robh+dt@kernel.org" <robh+dt@kernel.org>,
	"linus.walleij@linaro.org" <linus.walleij@linaro.org>
Subject: RE: [PATCH v5 2/3] gpio: pca9570: add a platform data structure
Date: Wed, 26 Oct 2022 15:58:55 +0000	[thread overview]
Message-ID: <BY5PR12MB490225682B7B31ED987B97BF81309@BY5PR12MB4902.namprd12.prod.outlook.com> (raw)
In-Reply-To: <CAMRc=MfR14_Pd57AgqyGTRsghb7OjyPNOyoWmvnae5i=Fnznug@mail.gmail.com>

[AMD Official Use Only - General]

> -----Original Message-----
> From: Bartosz Golaszewski <brgl@bgdev.pl>
> Sent: Wednesday, October 26, 2022 6:21 PM
> To: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> Cc: Geert Uytterhoeven <geert@linux-m68k.org>; Datta, Shubhrajyoti
> <shubhrajyoti.datta@amd.com>; linux-gpio@vger.kernel.org; git (AMD-
> Xilinx) <git@amd.com>; devicetree@vger.kernel.org;
> krzysztof.kozlowski+dt@linaro.org; robh+dt@kernel.org;
> linus.walleij@linaro.org
> Subject: Re: [PATCH v5 2/3] gpio: pca9570: add a platform data structure
> 
> Caution: This message originated from an External Source. Use proper
> caution when opening attachments, clicking links, or responding.
> 
> 
> On Wed, Oct 26, 2022 at 1:46 PM Andy Shevchenko
> <andriy.shevchenko@linux.intel.com> wrote:
> >
> > On Wed, Oct 26, 2022 at 12:00:34PM +0200, Geert Uytterhoeven wrote:
> > > Hi Shubhrajyoti,
> > > On Fri, Sep 30, 2022 at 12:41 PM Shubhrajyoti Datta
> > > <shubhrajyoti.datta@amd.com> wrote:
> >
> > ...
> >
> > > Thanks for your patch, which is now commit 35a4bc94a47f2ea6 ("gpio:
> > > pca9570: add a platform data structure") in gpio/gpio/for-next
> > > linux-next/master next-20221026
> >
> > Dunno if Bart rebases his tree...
> >
> 
> I will back it out of next. Shubhrajyoti: can you send a fixed version of this
> series?

Just sent it fixing it up.

> 
> Bart
> 
> > ...
> >
> > > >  static const struct of_device_id pca9570_of_match_table[] = {
> > > > -       { .compatible = "nxp,pca9570", .data = (void *)4 },
> > > > -       { .compatible = "nxp,pca9571", .data = (void *)8 },
> > > > +       { .compatible = "nxp,pca9570", .data = &pca9570_gpio },
> > > > +       { .compatible = "nxp,pca9571", .data = &pca9571_gpio },
> > >
> > > This breaks bisection, as .data is still considered to be the number
> > > of GPIOs:
> > >
> > >     gpio->chip.ngpio =
> > > (uintptr_t)device_get_match_data(&client->dev);
> >
> > You beat me up to it, I have also noticed this.
> >
> > > >         { /* sentinel */ }
> > > >  };
> >
> > --
> > With Best Regards,
> > Andy Shevchenko
> >
> >