Re: [PATCH] staging: greybus: light: use sysfs_emit() instead of sprintf()

From: Greg Kroah-Hartman

Date: Sun Sep 20 2026 - 10:03:58 EST


On Sun, Sep 20, 2026 at 10:38:03PM +0900, ts930@xxxxxxxxx wrote:
> From: Park Tae-sun <ts930@xxxxxxxxx>
>
> The sysfs documentation (Documentation/filesystems/sysfs.rst) recommends
> using sysfs_emit() instead of sprintf() in sysfs show() callbacks.
>
> Replace the remaining sprintf() calls in the fade_##__dir##_show()
> macro and color_show() callback with sysfs_emit().
>
> This follows the standard kernel practice and existing use of
> sysfs_emit() in other Greybus sysfs show functions.
>
> No functional change.

How is this different from past patches that have attempted to change
this?

thanks,

greg k-h