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

Andy Shevchenko <[email protected]> Thu, 13 Aug 2026 10:30:20 +0300

Newsgroups	org.kernel.vger.linux-iio,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel
Organization	Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs, Bertel Jungin Aukio 5, 02600 Espoo
Message-ID	<[email protected]>

On Thu, Aug 13, 2026 at 02:27:15AM +0100, Jonathan Cameron wrote:
> On Thu, 13 Aug 2026 02:04:29 +0100
> Jonathan Cameron <[email protected]> wrote:
> > On Wed, 12 Aug 2026 22:27:41 +0200
> > Javier Carrasco <[email protected]> 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