Re: [PATCH] bus: use sysfs_emit() to instead of scnprintf()

From: Serge Semin
Date: Sun Dec 04 2022 - 19:54:01 EST


On Thu, Dec 01, 2022 at 09:38:14AM +0800, ye.xingchen@xxxxxxxxxx wrote:
> From: ye xingchen <ye.xingchen@xxxxxxxxxx>
>
> Replace the open-code with sysfs_emit() to simplify the code.

Nice cleanup. Thanks.
Acked-by: Serge Semin <fancer.lancer@xxxxxxxxx>

* One note. Next time please use the scripts/get_maintainer.pl script
to get the driver maintainers list and their actual email addresses.

-Sergey

>
> Signed-off-by: ye xingchen <ye.xingchen@xxxxxxxxxx>
> ---
> drivers/bus/bt1-apb.c | 6 +++---
> drivers/bus/bt1-axi.c | 4 ++--
> 2 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/bus/bt1-apb.c b/drivers/bus/bt1-apb.c
> index 63b1b4a76671..bcf10f1d6dc1 100644
> --- a/drivers/bus/bt1-apb.c
> +++ b/drivers/bus/bt1-apb.c
> @@ -265,7 +265,7 @@ static ssize_t count_show(struct device *dev, struct device_attribute *attr,
> {
> struct bt1_apb *apb = dev_get_drvdata(dev);
>
> - return scnprintf(buf, PAGE_SIZE, "%d\n", atomic_read(&apb->count));
> + return sysfs_emit(buf, "%d\n", atomic_read(&apb->count));
> }
> static DEVICE_ATTR_RO(count);
>
> @@ -283,7 +283,7 @@ static ssize_t timeout_show(struct device *dev, struct device_attribute *attr,
>
> timeout = bt1_apb_n_to_timeout_us(apb, n);
>
> - return scnprintf(buf, PAGE_SIZE, "%lu\n", timeout);
> + return sysfs_emit(buf, "%lu\n", timeout);
> }
>
> static ssize_t timeout_store(struct device *dev,
> @@ -310,7 +310,7 @@ static DEVICE_ATTR_RW(timeout);
> static ssize_t inject_error_show(struct device *dev,
> struct device_attribute *attr, char *buf)
> {
> - return scnprintf(buf, PAGE_SIZE, "Error injection: nodev irq\n");
> + return sysfs_emit(buf, "Error injection: nodev irq\n");
> }
>
> static ssize_t inject_error_store(struct device *dev,
> diff --git a/drivers/bus/bt1-axi.c b/drivers/bus/bt1-axi.c
> index 70e49a6e5374..04c14821bb3c 100644
> --- a/drivers/bus/bt1-axi.c
> +++ b/drivers/bus/bt1-axi.c
> @@ -197,14 +197,14 @@ static ssize_t count_show(struct device *dev,
> {
> struct bt1_axi *axi = dev_get_drvdata(dev);
>
> - return scnprintf(buf, PAGE_SIZE, "%d\n", atomic_read(&axi->count));
> + return sysfs_emit(buf, "%d\n", atomic_read(&axi->count));
> }
> static DEVICE_ATTR_RO(count);
>
> static ssize_t inject_error_show(struct device *dev,
> struct device_attribute *attr, char *buf)
> {
> - return scnprintf(buf, PAGE_SIZE, "Error injection: bus unaligned\n");
> + return sysfs_emit(buf, "Error injection: bus unaligned\n");
> }
>
> static ssize_t inject_error_store(struct device *dev,
> --
> 2.25.1
>