lkml.org 
[lkml]   [2025]   [Sep]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v5 3/9] iio: imu: inv_icm45600: add buffer support in iio devices

> >
> >> >> +#define INV_ICM45600_SENSOR_CONF_KEEP_VALUES {U8_MAX, U8_MAX, U8_MAX, U8_MAX, }
> >> >
> >> >When one line, no need to have inner trailing comma, besides that missed
> >> >space.
> >> The trailing comma was a request from Jonathan Cameron on the v2 of patchset.
> >> Let me know, if you disagree with him and I'll fix.
> >
> >I see, then let's ask him, because it's a usual pattern for
> >the one-line arrays like this to have no inner trailing commas.
> Ok let's wait Jonathan Cameron's feedback.

Definitely ignore me on this. For one line arrays any change means that
line changes anyway so, unlike multiline arrays there is no reduction in noise
by having the trailing comma.

\
 
 \ /
  Last update: 2025-09-07 15:35    [from the cache]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and my Meterkast|Read the blog