Re: [PATCH] staging: iio: adc: ad7816: replace sprintf with sysfs_emit

Jonathan Cameron
2026-04-19T11:33:15Z

On Sat, 18 Apr 2026 20:05:37 -0500
Maxwell Doose <m32285159@gmail.com> wrote:

> This patch replaces sprintf calls in ad7816.c with sysfs_emit to help
> modernize the driver, ensure bounds checking, and to increase stability.
>
> Signed-off-by: Maxwell Doose <m32285159@gmail.com>
Hi Maxwell,

There is a more substantial rework of this code undergoing revisions.
https://lore.kernel.org/all/20260331072453.6530-1-mdmahmudulhasan1511@gmail.com/

Both because that's been underway for a little time and because it makes
a wider set of improvements, I'd rather we took that one forwards.
However, as I noted in that thread, the ABI is likely to go away anyway as
part of dragging this driver into a state suitable for moving out of staging.
I don't mind cleaning it up as an intermediate state however as that graduation
from staging may take a while!

Thanks,

Jonathan