Re: [PATCH v2 0/3] staging: iio: replace sprintf() with sysfs_emit()

From: Dan Carpenter

Date: Sat Mar 21 2026 - 11:38:31 EST


On Sat, Mar 21, 2026 at 01:03:07PM +0000, Jonathan Cameron wrote:
> On Fri, 20 Mar 2026 22:24:21 +0000
> Gabriel Rondon <grondon@xxxxxxxxx> wrote:
>
> > Replace the remaining sprintf() calls with sysfs_emit() in sysfs
> > show functions across three staging IIO drivers: ad7816, ad5933, and
> > ad9834.
> >
> > sysfs_emit() is the preferred API for sysfs callbacks as it is aware
> > of the PAGE_SIZE buffer limit. Build tested with gcc 13.3.
> >
> Firstly a process thing. Don't send a v2 in reply to a v1. It should
> be a new email thread. I'm not sure where this style is coming from
> as I've never encountered any kernel subsystem who ask for it.
>

People used to do that in the early days of git, but now no one does.
It breaks patchwork as well.

regards,
dan carpenter