Messages in this thread | | | From | Kalle Valo <> | Subject | Re: [PATCH v2 3/3] wifi: ti: Convert sprintf/snprintf to sysfs_emit | Date | Mon, 18 Mar 2024 12:05:04 +0200 |
| |
"Zhijian Li (Fujitsu)" <lizhijian@fujitsu.com> writes:
> Kalle, > > > On 18/03/2024 17:16, Kalle Valo wrote: >> Li Zhijian <lizhijian@fujitsu.com> writes: >> >>> Per filesystems/sysfs.rst, show() should only use sysfs_emit() >>> or sysfs_emit_at() when formatting the value to be returned to user space. >>> >>> coccinelle complains that there are still a couple of functions that use >>> snprintf(). Convert them to sysfs_emit(). >>> >>> sprintf() will be converted as weel if they have. >>> >>> Generally, this patch is generated by >>> make coccicheck M=<path/to/file> MODE=patch \ >>> COCCI=scripts/coccinelle/api/device_attr_show.cocci >>> >>> No functional change intended >>> >>> CC: Kalle Valo <kvalo@kernel.org> >>> CC: linux-wireless@vger.kernel.org >>> Signed-off-by: Li Zhijian <lizhijian@fujitsu.com> >>> --- >>> V2: subject updated >>> >>> This is a part of the work "Fix coccicheck device_attr_show warnings"[1] >>> Split them per subsystem so that the maintainer can review it easily >>> [1] https://lore.kernel.org/lkml/20240116041129.3937800-1-lizhijian@fujitsu.com/ >>> Signed-off-by: Li Zhijian <lizhijian@fujitsu.com> >> >> Please resubmit the whole patchset as v3. > > > May I know what's wrong with this V2? or what update should I do in V3
Sorry, my mistake. I didn't see patch 1 and I assumed it was not sent. But I checked patchwork now and I do see all three patches:
https://patchwork.kernel.org/project/linux-wireless/list/?series=835497
So no need to resend anything because of this.
-- https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
| |