Re: [PATCH v4 3/4] gpio: adp5585: Add Analog Devices ADP5585 support
From: Andy Shevchenko
Date: Mon Jun 10 2024 - 12:30:16 EST
On Mon, Jun 10, 2024 at 6:26 PM Laurent Pinchart
<laurent.pinchart@xxxxxxxxxxxxxxxx> wrote:
> On Mon, Jun 10, 2024 at 06:15:40PM +0300, Andy Shevchenko wrote:
> > Sat, Jun 08, 2024 at 05:16:32PM +0300, Laurent Pinchart kirjoitti:
...
> > > +static const struct platform_device_id adp5585_gpio_id_table[] = {
> > > + { "adp5585-gpio" },
> >
> > > + { /* Sentinel */ },
> >
> > Drop the comma.
>
> I prefer keeping it.
For what reason?
The sentinel should be runtime and compile time one. Why should we
make our lives worse by neglecting help from a compiler?
> > > +};
--
With Best Regards,
Andy Shevchenko