Re:  [PATCH linux-next] net: atm: use sysfs_emit()/sysfs_emit_at() instead of scnprintf().

From: Simon Horman
Date: Thu Mar 20 2025 - 07:48:42 EST


On Mon, Mar 17, 2025 at 03:29:33PM +0800, xie.ludan@xxxxxxxxxx wrote:
> From: XieLudan <xie.ludan@xxxxxxxxxx>
>
> Follow the advice in Documentation/filesystems/sysfs.rst:
> show() should only use sysfs_emit() or sysfs_emit_at() when formatting
> the value to be returned to user space.
>
> Signed-off-by: XieLudan <xie.ludan@xxxxxxxxxx>

Thanks Xie Ludan,

As per or discussion offline,
please coordinate with your colleague
Tang Dongxing who has also posted a patch in this area.

https://lore.kernel.org/r/20250317155102808MZdMkiovw52X0oY7n47wI@xxxxxxxxxx/

It will be much easier for review if there is a single patch
that addresses these issues for ATM.

Also, please consider reading the following guidance on processes
for the networking subsystem of the Linux kernel. These are similar
but different to other subsystems.

https://docs.kernel.org/process/maintainer-netdev.html

--
pw-bot: changes-requested