* Fwd: [PATCH] scsi: be2iscsi: wacom: convert sysfs sprintf/snprintf family to sysfs_emit
[not found] <tencent_A27502E2D5B495E4E319441AB4B3B5F7E708@qq.com>
@ 2023-07-14 8:10 ` hanyu001
2023-07-19 18:05 ` Chris Leech
2023-07-19 18:08 ` Chris Leech
0 siblings, 2 replies; 3+ messages in thread
From: hanyu001 @ 2023-07-14 8:10 UTC (permalink / raw)
To: jejb, martin.petersen; +Cc: linux-scsi, linux-kernel
Fix the following coccicheck warning:
./drivers/hid/wacom_sys.c:1828:8-16: WARNING: use scnprintf or sprintf.
./drivers/scsi/be2iscsi/be_mgmt.c:1251:9-17: WARNING: use scnprintf or
sprintf
./drivers/scsi/be2iscsi/be_mgmt.c:1145:8-16: WARNING: use scnprintf or
sprintf
./drivers/scsi/be2iscsi/be_mgmt.c:1164:8-16: WARNING: use scnprintf or
sprintf
./drivers/scsi/be2iscsi/be_mgmt.c:1280:8-16: WARNING: use scnprintf or
sprintf
Signed-off-by: ztt <1549089851@qq.com>
---
drivers/scsi/be2iscsi/be_mgmt.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/scsi/be2iscsi/be_mgmt.c
b/drivers/scsi/be2iscsi/be_mgmt.c
index 4e899ec1477d..4916ce9c36a6 100644
--- a/drivers/scsi/be2iscsi/be_mgmt.c
+++ b/drivers/scsi/be2iscsi/be_mgmt.c
@@ -1142,7 +1142,7 @@ ssize_t
beiscsi_drvr_ver_disp(struct device *dev, struct device_attribute
*attr,
char *buf)
{
- return snprintf(buf, PAGE_SIZE, BE_NAME "\n");
+ return scnprintf(buf, PAGE_SIZE, BE_NAME "\n");
}
/**
@@ -1161,7 +1161,7 @@ beiscsi_fw_ver_disp(struct device *dev, struct
device_attribute *attr,
struct Scsi_Host *shost = class_to_shost(dev);
struct beiscsi_hba *phba = iscsi_host_priv(shost);
- return snprintf(buf, PAGE_SIZE, "%s\n", phba->fw_ver_str);
+ return scnprintf(buf, PAGE_SIZE, "%s\n", phba->fw_ver_str);
}
/**
@@ -1248,7 +1248,7 @@ beiscsi_adap_family_disp(struct device *dev,
struct device_attribute *attr,
case BE_DEVICE_ID1:
case OC_DEVICE_ID1:
case OC_DEVICE_ID2:
- return snprintf(buf, PAGE_SIZE,
+ return scnprintf(buf, PAGE_SIZE,
"Obsolete/Unsupported BE2 Adapter Family\n");
case BE_DEVICE_ID2:
case OC_DEVICE_ID3:
@@ -1277,7 +1277,7 @@ beiscsi_phys_port_disp(struct device *dev, struct
device_attribute *attr,
struct Scsi_Host *shost = class_to_shost(dev);
struct beiscsi_hba *phba = iscsi_host_priv(shost);
- return snprintf(buf, PAGE_SIZE, "Port Identifier : %u\n",
+ return scnprintf(buf, PAGE_SIZE, "Port Identifier : %u\n",
phba->fw_config.phys_port);
}
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: Fwd: [PATCH] scsi: be2iscsi: wacom: convert sysfs sprintf/snprintf family to sysfs_emit
2023-07-14 8:10 ` Fwd: [PATCH] scsi: be2iscsi: wacom: convert sysfs sprintf/snprintf family to sysfs_emit hanyu001
@ 2023-07-19 18:05 ` Chris Leech
2023-07-19 18:08 ` Chris Leech
1 sibling, 0 replies; 3+ messages in thread
From: Chris Leech @ 2023-07-19 18:05 UTC (permalink / raw)
To: hanyu001; +Cc: jejb, martin.petersen, linux-scsi, linux-kernel
On Fri, Jul 14, 2023 at 04:10:43PM +0800, hanyu001@208suo.com wrote:
> Fix the following coccicheck warning:
>
> ./drivers/hid/wacom_sys.c:1828:8-16: WARNING: use scnprintf or sprintf.
>
> ./drivers/scsi/be2iscsi/be_mgmt.c:1251:9-17: WARNING: use scnprintf or
> sprintf
> ./drivers/scsi/be2iscsi/be_mgmt.c:1145:8-16: WARNING: use scnprintf or
> sprintf
> ./drivers/scsi/be2iscsi/be_mgmt.c:1164:8-16: WARNING: use scnprintf or
> sprintf
> ./drivers/scsi/be2iscsi/be_mgmt.c:1280:8-16: WARNING: use scnprintf or
> sprintf
>
> Signed-off-by: ztt <1549089851@qq.com>
> ---
> drivers/scsi/be2iscsi/be_mgmt.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
The changes here to scnprintf don't match the subject, which mentions
sysfs_emit. It would be better to convert to sysfs_emit.
- Chris Leech
>
> diff --git a/drivers/scsi/be2iscsi/be_mgmt.c
> b/drivers/scsi/be2iscsi/be_mgmt.c
> index 4e899ec1477d..4916ce9c36a6 100644
> --- a/drivers/scsi/be2iscsi/be_mgmt.c
> +++ b/drivers/scsi/be2iscsi/be_mgmt.c
> @@ -1142,7 +1142,7 @@ ssize_t
> beiscsi_drvr_ver_disp(struct device *dev, struct device_attribute *attr,
> char *buf)
> {
> - return snprintf(buf, PAGE_SIZE, BE_NAME "\n");
> + return scnprintf(buf, PAGE_SIZE, BE_NAME "\n");
> }
>
> /**
> @@ -1161,7 +1161,7 @@ beiscsi_fw_ver_disp(struct device *dev, struct
> device_attribute *attr,
> struct Scsi_Host *shost = class_to_shost(dev);
> struct beiscsi_hba *phba = iscsi_host_priv(shost);
>
> - return snprintf(buf, PAGE_SIZE, "%s\n", phba->fw_ver_str);
> + return scnprintf(buf, PAGE_SIZE, "%s\n", phba->fw_ver_str);
> }
>
> /**
> @@ -1248,7 +1248,7 @@ beiscsi_adap_family_disp(struct device *dev, struct
> device_attribute *attr,
> case BE_DEVICE_ID1:
> case OC_DEVICE_ID1:
> case OC_DEVICE_ID2:
> - return snprintf(buf, PAGE_SIZE,
> + return scnprintf(buf, PAGE_SIZE,
> "Obsolete/Unsupported BE2 Adapter Family\n");
> case BE_DEVICE_ID2:
> case OC_DEVICE_ID3:
> @@ -1277,7 +1277,7 @@ beiscsi_phys_port_disp(struct device *dev, struct
> device_attribute *attr,
> struct Scsi_Host *shost = class_to_shost(dev);
> struct beiscsi_hba *phba = iscsi_host_priv(shost);
>
> - return snprintf(buf, PAGE_SIZE, "Port Identifier : %u\n",
> + return scnprintf(buf, PAGE_SIZE, "Port Identifier : %u\n",
> phba->fw_config.phys_port);
> }
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Fwd: [PATCH] scsi: be2iscsi: wacom: convert sysfs sprintf/snprintf family to sysfs_emit
2023-07-14 8:10 ` Fwd: [PATCH] scsi: be2iscsi: wacom: convert sysfs sprintf/snprintf family to sysfs_emit hanyu001
2023-07-19 18:05 ` Chris Leech
@ 2023-07-19 18:08 ` Chris Leech
1 sibling, 0 replies; 3+ messages in thread
From: Chris Leech @ 2023-07-19 18:08 UTC (permalink / raw)
To: hanyu001; +Cc: jejb, martin.petersen, linux-scsi, linux-kernel
On Fri, Jul 14, 2023 at 04:10:43PM +0800, hanyu001@208suo.com wrote:
> Fix the following coccicheck warning:
>
> ./drivers/hid/wacom_sys.c:1828:8-16: WARNING: use scnprintf or sprintf.
>
> ./drivers/scsi/be2iscsi/be_mgmt.c:1251:9-17: WARNING: use scnprintf or
> sprintf
> ./drivers/scsi/be2iscsi/be_mgmt.c:1145:8-16: WARNING: use scnprintf or
> sprintf
> ./drivers/scsi/be2iscsi/be_mgmt.c:1164:8-16: WARNING: use scnprintf or
> sprintf
> ./drivers/scsi/be2iscsi/be_mgmt.c:1280:8-16: WARNING: use scnprintf or
> sprintf
>
> Signed-off-by: ztt <1549089851@qq.com>
These should all be converted to sysfs_emit instead of scnprintf.
Thanks,
- Chris Leech
> ---
> drivers/scsi/be2iscsi/be_mgmt.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/scsi/be2iscsi/be_mgmt.c
> b/drivers/scsi/be2iscsi/be_mgmt.c
> index 4e899ec1477d..4916ce9c36a6 100644
> --- a/drivers/scsi/be2iscsi/be_mgmt.c
> +++ b/drivers/scsi/be2iscsi/be_mgmt.c
> @@ -1142,7 +1142,7 @@ ssize_t
> beiscsi_drvr_ver_disp(struct device *dev, struct device_attribute *attr,
> char *buf)
> {
> - return snprintf(buf, PAGE_SIZE, BE_NAME "\n");
> + return scnprintf(buf, PAGE_SIZE, BE_NAME "\n");
> }
>
> /**
> @@ -1161,7 +1161,7 @@ beiscsi_fw_ver_disp(struct device *dev, struct
> device_attribute *attr,
> struct Scsi_Host *shost = class_to_shost(dev);
> struct beiscsi_hba *phba = iscsi_host_priv(shost);
>
> - return snprintf(buf, PAGE_SIZE, "%s\n", phba->fw_ver_str);
> + return scnprintf(buf, PAGE_SIZE, "%s\n", phba->fw_ver_str);
> }
>
> /**
> @@ -1248,7 +1248,7 @@ beiscsi_adap_family_disp(struct device *dev, struct
> device_attribute *attr,
> case BE_DEVICE_ID1:
> case OC_DEVICE_ID1:
> case OC_DEVICE_ID2:
> - return snprintf(buf, PAGE_SIZE,
> + return scnprintf(buf, PAGE_SIZE,
> "Obsolete/Unsupported BE2 Adapter Family\n");
> case BE_DEVICE_ID2:
> case OC_DEVICE_ID3:
> @@ -1277,7 +1277,7 @@ beiscsi_phys_port_disp(struct device *dev, struct
> device_attribute *attr,
> struct Scsi_Host *shost = class_to_shost(dev);
> struct beiscsi_hba *phba = iscsi_host_priv(shost);
>
> - return snprintf(buf, PAGE_SIZE, "Port Identifier : %u\n",
> + return scnprintf(buf, PAGE_SIZE, "Port Identifier : %u\n",
> phba->fw_config.phys_port);
> }
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-07-19 18:09 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <tencent_A27502E2D5B495E4E319441AB4B3B5F7E708@qq.com>
2023-07-14 8:10 ` Fwd: [PATCH] scsi: be2iscsi: wacom: convert sysfs sprintf/snprintf family to sysfs_emit hanyu001
2023-07-19 18:05 ` Chris Leech
2023-07-19 18:08 ` Chris Leech
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox