[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAHp75VfQFN+F0xMyhWvHOejD0AefDfBLf9s4eu1bpqCBY7bkdA@mail.gmail.com>
Date: Tue, 12 Aug 2025 22:44:13 +0200
From: Andy Shevchenko <andy.shevchenko@...il.com>
To: linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org,
Jonathan Cameron <jic23@...nel.org>, David Lechner <dlechner@...libre.com>, Nuno Sá <nuno.sa@...og.com>,
Andy Shevchenko <andy@...nel.org>
Subject: Re: [PATCH] iio: iio_format_list() should set stride=1 for IIO_VAL_CHAR
On Tue, Aug 12, 2025 at 1:13 PM Ben Collins <bcollins@...nel.org> wrote:
>
> iio_format_list() sets a stride across the val array of 1 for INT
> type, and 2 for all others. Add IIO_VAL_CHAR so it also gets a
> stride of 1 assuming val is an array of integers with char type
> values.
>
> No drivers currently use this, but I found this issue adding an
> avail callback for IIO_INFO_THERMOCOUPLE_TYPE for a driver I'm
> working on.
>
> Signed-off-by: Ben Collins <bcollins@...nel.org>
Please, keep the Cc list after the '---' line (note, you may have more
than a single one in the patch, hence you may just add it here). This
will reduce the unneeded noise in the commit message as the very same
information will be available in the email headers and in lore
archive.
> Cc: Jonathan Cameron <jic23@...nel.org>
> Cc: David Lechner <dlechner@...libre.com>
> Cc: Nuno Sá <nuno.sa@...log.com>
> Cc: Andy Shevchenko <andy@...nel.org>
>
> ---
> drivers/iio/industrialio-core.c | 1 +
> 1 file changed, 1 insertion(+)
--
With Best Regards,
Andy Shevchenko
Powered by blists - more mailing lists