linux-kernel - Re: [PATCH] iio: bma180: Convert to use sysfs_emit

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20221207095614.z4gv3q76hfx47qs2@CAB-WSD-L081021>
Date:   Wed, 7 Dec 2022 12:56:14 +0300
From:   Dmitry Rokosov <ddrokosov@...rdevices.ru>
To:     <ye.xingchen@....com.cn>
CC:     <andriy.shevchenko@...ux.intel.com>, <jic23@...nel.org>,
        <lars@...afoo.de>, <u.kleine-koenig@...gutronix.de>,
        <petrm@...dia.com>, <olteanv@...il.com>,
        <linux-iio@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] iio: bma180: Convert to use sysfs_emit_at() API

On Wed, Dec 07, 2022 at 10:03:19AM +0800, ye.xingchen@....com.cn wrote:
> From: ye xingchen <ye.xingchen@....com.cn>
> 
> Follow the advice of the Documentation/filesystems/sysfs.rst and show()
> should only use sysfs_emit() or sysfs_emit_at() when formatting the
> value to be returned to user space.
> 
> Signed-off-by: ye xingchen <ye.xingchen@....com.cn>

I think it would be better, if you provide one cover letter as 0/3 patch
and three nested real patches to each iio/accel driver which you want to
rework: adxl372, bma180, mma8452. Or fix all iio drivers (no only accel)
and send all of these changes as nested patches for one cover letter.

Please refer to https://www.kernel.org/doc/html/latest/process/submitting-patches.html

and use the following commands to generate such patches hierarchy:

$ git format-patch --cover-letter -o patches ...
$ git send-email \     
      --annotate \               
      --to="john.doe@...nel.org" \               
      --cc="linux-iio@...r.kernel.org" \
      --cc="linux-kernel@...r.kernel.org" \
      --smtp-debug 1 \                  
      patches/*

> ---
>  drivers/iio/accel/bma180.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/drivers/iio/accel/bma180.c b/drivers/iio/accel/bma180.c
> index eb697eeb4301..9e3e5dfdded7 100644
> --- a/drivers/iio/accel/bma180.c
> +++ b/drivers/iio/accel/bma180.c
> @@ -488,8 +488,7 @@ static ssize_t bma180_show_avail(char *buf, const int *vals, unsigned int n,
>  	for (i = 0; i < n; i++) {
>  		if (!vals[i])
>  			continue;
> -		len += scnprintf(buf + len, PAGE_SIZE - len,
> -			micros ? "0.%06d " : "%d ", vals[i]);
> +		len += sysfs_emit_at(buf, len, micros ? "0.%06d " : "%d ", vals[i]);
>  	}
>  	buf[len - 1] = '\n';
> 
> -- 
> 2.25.1

-- 
Thank you,
Dmitry