Re: [PATCH v5 3/4] iio: imu: bmi270: add support for motion events
From: Andy Shevchenko
Date: Sat Aug 30 2025 - 08:47:53 EST
On Sat, Aug 30, 2025 at 3:44 PM Andy Shevchenko
<andy.shevchenko@xxxxxxxxx> wrote:
> On Sat, Aug 30, 2025 at 2:58 PM Gustavo Silva <gustavograzs@xxxxxxxxx> wrote:
...
> > + return sysfs_emit(buf, "[0 %u %u]\n", step, max);
>
> Do we need []? Is it common in IIO? The usual way in SW to have "$min
> $max $step" (see `seq` in shell, range() in Python and so on).
I stand corrected `seq` actually gives what you used. So, the first
question is still relevant. Use what is the common case in the other
IIO drivers / core.
--
With Best Regards,
Andy Shevchenko