On 1/15/24 20:51, Li Zhijian wrote: > 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(). > >> ./drivers/scsi/ibmvscsi/ibmvfc.c:3483:8-16: WARNING: please use sysfs_emit >> ./drivers/scsi/ibmvscsi/ibmvfc.c:3493:8-16: WARNING: please use sysfs_emit >> ./drivers/scsi/ibmvscsi/ibmvfc.c:3503:8-16: WARNING: please use sysfs_emit >> ./drivers/scsi/ibmvscsi/ibmvfc.c:3513:8-16: WARNING: please use sysfs_emit >> ./drivers/scsi/ibmvscsi/ibmvfc.c:3522:8-16: WARNING: please use sysfs_emit >> ./drivers/scsi/ibmvscsi/ibmvfc.c:3530:8-16: WARNING: please use sysfs_emit > > No functional change intended > > CC: Tyrel Datwyler <tyreld@xxxxxxxxxxxxx> > CC: Michael Ellerman <mpe@xxxxxxxxxxxxxx> > CC: Nicholas Piggin <npiggin@xxxxxxxxx> > CC: Christophe Leroy <christophe.leroy@xxxxxxxxxx> > CC: "Aneesh Kumar K.V" <aneesh.kumar@xxxxxxxxxx> > CC: "Naveen N. Rao" <naveen.n.rao@xxxxxxxxxxxxx> > CC: "James E.J. Bottomley" <jejb@xxxxxxxxxxxxx> > CC: "Martin K. Petersen" <martin.petersen@xxxxxxxxxx> > CC: linux-scsi@xxxxxxxxxxxxxxx > CC: linuxppc-dev@xxxxxxxxxxxxxxxx > Signed-off-by: Li Zhijian <lizhijian@xxxxxxxxxxx> > --- Acked-by: Tyrel Datwyler <tyreld@xxxxxxxxxxxxx>