Re: [PATCH v2] serial: 8259_aspeed_vuart: replace snprintf with sysfs_emit

[Andy Shevchenko]

On Thu, Feb 10, 2022 at 12:54 PM <davidcomponentone@xxxxxxxxx> wrote:
>
> From: Yang Guang <yang.guang5@xxxxxxxxxx>
>
> coccinelle report:
> ./drivers/tty/serial/8250/8250_aspeed_vuart.c:85:8-16:
> WARNING: use scnprintf or sprintf
> ./drivers/tty/serial/8250/8250_aspeed_vuart.c:174:8-16:
> WARNING: use scnprintf or sprintf
> ./drivers/tty/serial/8250/8250_aspeed_vuart.c:127:8-16:
> WARNING: use scnprintf or sprintf
>
> Use sysfs_emit instead of scnprintf or sprintf makes more sense.

sysfs_emit()
scnprintf()
sprintf()

...

> CC: David Yang <davidcomponentone@xxxxxxxxx>

> - Use the CC for patch sender

This is incorrect. The submitter must have (the last) SoB tag.
Question was about authorship just to be clear.

-- 
With Best Regards,
Andy Shevchenko