Re: [PATCH] staging: iio: adt7316: replace sprintf() with sysfs_emit()

[Jonathan Cameron <jic23@kernel.org>]

On Sun, 16 Nov 2025 13:08:07 +0200
Andy Shevchenko <andy.shevchenko@gmail.com> wrote:

> On Sun, Nov 16, 2025 at 12:16 PM Shi Hao <i.shihao.999@gmail.com> wrote:
> >
> > Convert several sprintf() calls to sysfs_emit() in the
> > sysfs show functions, as it is the preferred helper and
> > prevents potential buffer overruns.
> >
> > No functional changes intended.  
> 
> I briefly looked at it and see no issues,
> Reviewed-by: Andy Shevchenko <andy@kernel.org>
> 

Applied, but I will note that this driver is a long way from suitable
for moving out of staging and I would be surprised if more than
one or two of the lines changed in this patch make it through the
necessary refactors (should anyone actually have another go at
doing them).  Anyhow, I still think this is worth taking just to
reduce the noise of remaining instances of this.

I'd have deleted this long ago except I actually have one somewhere
and it one of the supported parts was still listed as suitable for
new designs when I checked not long ago.

ADI folk, I would like to hear if we should just delete this one.

Thanks,

Jonathan