| Message ID | 1617067518-31091-1-git-send-email-tiantao6@hisilicon.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | drm/komeda: Convert sysfs sprintf/snprintf family to sysfs_emit | expand |
On 3/30/21 2:25 AM, Tian Tao wrote: > Fix the following coccicheck warning: > drivers/gpu/drm/arm/display/komeda/komeda_dev.c:97:8-16: WARNING: > use scnprintf or sprintf > drivers/gpu/drm/arm/display/komeda/komeda_dev.c:88:8-16: WARNING: > use scnprintf or sprintf > drivers/gpu/drm/arm/display/komeda/komeda_dev.c:65:8-16: WARNING: > use scnprintf or sprintf > > Signed-off-by: Tian Tao <tiantao6@hisilicon.com> > --- > drivers/gpu/drm/arm/display/komeda/komeda_dev.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c > index ca891ae..cc7664c 100644 > --- a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c > +++ b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c > @@ -62,7 +62,7 @@ core_id_show(struct device *dev, struct device_attribute *attr, char *buf) > { > struct komeda_dev *mdev = dev_to_mdev(dev); > > - return snprintf(buf, PAGE_SIZE, "0x%08x\n", mdev->chip.core_id); > + return sysfs_emit(buf, "0x%08x\n", mdev->chip.core_id); > } > static DEVICE_ATTR_RO(core_id); > > @@ -85,7 +85,7 @@ config_id_show(struct device *dev, struct device_attribute *attr, char *buf) > if (pipe->layers[i]->layer_type == KOMEDA_FMT_RICH_LAYER) > config_id.n_richs++; > } > - return snprintf(buf, PAGE_SIZE, "0x%08x\n", config_id.value); > + return sysfs_emit(buf, "0x%08x\n", config_id.value); > } > static DEVICE_ATTR_RO(config_id); > > @@ -94,7 +94,7 @@ aclk_hz_show(struct device *dev, struct device_attribute *attr, char *buf) > { > struct komeda_dev *mdev = dev_to_mdev(dev); > > - return snprintf(buf, PAGE_SIZE, "%lu\n", clk_get_rate(mdev->aclk)); > + return sysfs_emit(buf, "%lu\n", clk_get_rate(mdev->aclk)); > } > static DEVICE_ATTR_RO(aclk_hz); > > Looks OK to me.
diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c index ca891ae..cc7664c 100644 --- a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c +++ b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c @@ -62,7 +62,7 @@ core_id_show(struct device *dev, struct device_attribute *attr, char *buf) { struct komeda_dev *mdev = dev_to_mdev(dev); - return snprintf(buf, PAGE_SIZE, "0x%08x\n", mdev->chip.core_id); + return sysfs_emit(buf, "0x%08x\n", mdev->chip.core_id); } static DEVICE_ATTR_RO(core_id); @@ -85,7 +85,7 @@ config_id_show(struct device *dev, struct device_attribute *attr, char *buf) if (pipe->layers[i]->layer_type == KOMEDA_FMT_RICH_LAYER) config_id.n_richs++; } - return snprintf(buf, PAGE_SIZE, "0x%08x\n", config_id.value); + return sysfs_emit(buf, "0x%08x\n", config_id.value); } static DEVICE_ATTR_RO(config_id); @@ -94,7 +94,7 @@ aclk_hz_show(struct device *dev, struct device_attribute *attr, char *buf) { struct komeda_dev *mdev = dev_to_mdev(dev); - return snprintf(buf, PAGE_SIZE, "%lu\n", clk_get_rate(mdev->aclk)); + return sysfs_emit(buf, "%lu\n", clk_get_rate(mdev->aclk)); } static DEVICE_ATTR_RO(aclk_hz);
Fix the following coccicheck warning: drivers/gpu/drm/arm/display/komeda/komeda_dev.c:97:8-16: WARNING: use scnprintf or sprintf drivers/gpu/drm/arm/display/komeda/komeda_dev.c:88:8-16: WARNING: use scnprintf or sprintf drivers/gpu/drm/arm/display/komeda/komeda_dev.c:65:8-16: WARNING: use scnprintf or sprintf Signed-off-by: Tian Tao <tiantao6@hisilicon.com> --- drivers/gpu/drm/arm/display/komeda/komeda_dev.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)