Re: [PATCH] gpiolib: remove trailing comma from sentinel in GPIO_LOOKUP_SINGLE

From: Bartosz Golaszewski

Date: Wed Jul 15 2026 - 05:09:39 EST


On Wed, 15 Jul 2026 10:36:22 +0200, Andy Shevchenko
<andriy.shevchenko@xxxxxxxxxxxxxxx> said:
> On Wed, Jul 15, 2026 at 08:11:06AM +0000, Bartosz Golaszewski wrote:
>> On Tue, 14 Jul 2026 10:41:39 +0200, Andy Shevchenko
>> <andriy.shevchenko@xxxxxxxxxxxxxxx> said:
>> > On Mon, Jul 13, 2026 at 04:29:16PM -0700, Dmitry Torokhov wrote:
>> >> The GPIO_LOOKUP_SINGLE macro defines a lookup table with a single entry
>> >
>> > GPIO_LOOKUP_SINGLE()
>>
>> I fixed it when applying.
>
> Thanks!
>
>> >> followed by an empty sentinel entry. The sentinel entry has a trailing
>> >> comma which is unnecessary. Remove it.
>> >
>> > Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
>>
>> I queued it but I do consider it a bit of a needless churn TBH.
>
> You should really be against Uwe's patches then! :-)
>

He's fixing more stuff and removing trailing commas is just part of it. And as
you can see - I did apply it and will apply Uwe's series too.

Bart