Re: [PATCH v3] staging: greybus: audio: Use sysfs_emit in show functions

[Greg Kroah-Hartman <gregkh@linuxfoundation.org>]

On Mon, Feb 23, 2026 at 11:41:58PM +0530, Shubham Chakraborty wrote:
> Refactor sprintf to sysfs_emit in all show functions of the greybus
> audio manager module. This follows the standard kernel practice of
> using sysfs_emit for sysfs attributes, ensuring consistent output
> formatting and newline handling.
> 
> Signed-off-by: Shubham Chakraborty <chakrabortyshubham66@gmail.com>
> ---
> 
> v3:
>  - Add patch history below the --- line as requested by Greg KH.
> 
> v2:
>  - Add missing Signed-off-by line.
>  - Wrap commit message at 72 characters.
> 
>  drivers/staging/greybus/audio_manager_module.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/staging/greybus/audio_manager_module.c b/drivers/staging/greybus/audio_manager_module.c
> index e87b82ca6..f22ee73eb 100644
> --- a/drivers/staging/greybus/audio_manager_module.c
> +++ b/drivers/staging/greybus/audio_manager_module.c
> @@ -76,7 +76,7 @@ static void gb_audio_module_release(struct kobject *kobj)
>  static ssize_t gb_audio_module_name_show(struct gb_audio_manager_module *module,
>                                          struct gb_audio_manager_module_attribute *attr, char *buf)
>  {
> -       return sprintf(buf, "%s", module->desc.name);
> +       return sysfs_emit(buf, "%s\n", module->desc.name);

You just changed the output of all of these sysfs files by adding "\n"
to the end.  Why?

I don't think that's wrong, BUT it is a user/kernel api change, so you
need to document it, and ideally, that would happen in a separate change
from this one.

So can you do the conversion to sysfs_emit() with NO change in the
output, and a second one to add the \n if you really think it is needed.

And is it needed?  If so, why?  What tool uses these files and does it
now still work?

thanks,

greg k-h