Re: [PATCH v6 2/4] iio: light: add support for veml6031x00 ALS series

From: Andy Shevchenko

Date: Thu Aug 13 2026 - 03:34:03 EST


On Thu, Aug 13, 2026 at 02:27:15AM +0100, Jonathan Cameron wrote:
> On Thu, 13 Aug 2026 02:04:29 +0100
> Jonathan Cameron <jic23@xxxxxxxxxx> wrote:
> > On Wed, 12 Aug 2026 22:27:41 +0200
> > Javier Carrasco <javier.carrasco.cruz@xxxxxxxxx> wrote:

> > > + }
> >
> > Trailing comma. Maybe we'll add more after this and they will fit on one line.
> > Yeah it's a stretch but generally always add comma unless it is a specific
> > terminating entry.

Which is correct suggestion.

> Ah. Not such a stretch. I'd forgotten timestamp channels. This creates
> unnecessary noise in the very next patch!

Exactly, so trailing comma here saves us from unneeded churn in the next patch.

--
With Best Regards,
Andy Shevchenko