Re: [PATCH] staging: iio: adc: ad7816: replace sprintf with sysfs_emit
From: Jonathan Cameron
Date: Sun Apr 19 2026 - 07:33:24 EST
On Sat, 18 Apr 2026 20:05:37 -0500
Maxwell Doose <m32285159@xxxxxxxxx> 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@xxxxxxxxx>
Hi Maxwell,
There is a more substantial rework of this code undergoing revisions.
https://lore.kernel.org/all/20260331072453.6530-1-mdmahmudulhasan1511@xxxxxxxxx/
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