Re: [PATCH 1/9] iio: imu: st_lsm6dsx: dynamically initialize iio_chan_spec data
From: Andy Shevchenko
Date: Fri Oct 31 2025 - 04:09:24 EST
On Fri, Oct 31, 2025 at 09:04:58AM +0100, Francesco Lavra wrote:
> On Thu, 2025-10-30 at 17:42 +0100, Lorenzo Bianconi wrote:
[...]
> > > +static int st_lsm6dsx_chan_init(struct iio_chan_spec *channels, struct
> > > st_lsm6dsx_hw *hw,
> > > + enum st_lsm6dsx_sensor_id id, int
> > > index)
> >
> > please try to respect the 79 column limit (I still like it :))
>
> OK
FWIW, the limit is the exact 80. Don't waste that 1 characters when it's the case.
(And moreover there is a note in the documentation that allows, and we follow
that from time to time in IIO, slightly longer lines if it really increases
readability.)
--
With Best Regards,
Andy Shevchenko