diff options
| author | Li Zhijian <[email protected]> | 2024-01-16 12:51:36 +0800 |
|---|---|---|
| committer | Martin K. Petersen <[email protected]> | 2024-01-29 20:40:49 -0500 |
| commit | 5fbf37e53091057fc53f1046ded8a967464c2ecf (patch) | |
| tree | f2ea46ca88fd09f0a8cd2194bc4892b626e8ae47 | |
| parent | 01105c23de4263559919dd44e6ffc557fd261f0f (diff) | |
scsi: isci: Convert snprintf() to sysfs_emit()
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/isci/init.c:140:8-16: WARNING: please use sysfs_emit
No functional change intended
CC: Artur Paszkiewicz <[email protected]>
CC: James E.J. Bottomley <[email protected]>
CC: Martin K. Petersen <[email protected]>
CC: [email protected]
Signed-off-by: Li Zhijian <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Reviewed-by: Artur Paszkiewicz <[email protected]>
Signed-off-by: Martin K. Petersen <[email protected]>
| -rw-r--r-- | drivers/scsi/isci/init.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/isci/init.c b/drivers/scsi/isci/init.c index 6277162a028b..c582a3932cea 100644 --- a/drivers/scsi/isci/init.c +++ b/drivers/scsi/isci/init.c @@ -137,7 +137,7 @@ static ssize_t isci_show_id(struct device *dev, struct device_attribute *attr, c struct sas_ha_struct *sas_ha = SHOST_TO_SAS_HA(shost); struct isci_host *ihost = container_of(sas_ha, typeof(*ihost), sas_ha); - return snprintf(buf, PAGE_SIZE, "%d\n", ihost->id); + return sysfs_emit(buf, "%d\n", ihost->id); } static DEVICE_ATTR(isci_id, S_IRUGO, isci_show_id, NULL); |