On Mon, Mar 22, 2021 at 09:19:38AM +0800, Tian Tao wrote: > Fix the following coccicheck warning: > drivers/gpu/drm/drm_sysfs.c:172:8-16: WARNING: use scnprintf or sprintf > drivers/gpu/drm/drm_sysfs.c:185:8-16: WARNING: use scnprintf or sprintf > drivers/gpu/drm/drm_sysfs.c:159:8-16: WARNING: use scnprintf or sprintf > > Signed-off-by: Tian Tao <tiantao6@xxxxxxxxxxxxx> Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx> > --- > drivers/gpu/drm/drm_sysfs.c | 9 ++++----- > 1 file changed, 4 insertions(+), 5 deletions(-) > > diff --git a/drivers/gpu/drm/drm_sysfs.c b/drivers/gpu/drm/drm_sysfs.c > index f0336c8..968a956 100644 > --- a/drivers/gpu/drm/drm_sysfs.c > +++ b/drivers/gpu/drm/drm_sysfs.c > @@ -156,8 +156,8 @@ static ssize_t status_show(struct device *device, > > status = READ_ONCE(connector->status); > > - return snprintf(buf, PAGE_SIZE, "%s\n", > - drm_get_connector_status_name(status)); > + return sysfs_emit(buf, "%s\n", > + drm_get_connector_status_name(status)); > } > > static ssize_t dpms_show(struct device *device, > @@ -169,8 +169,7 @@ static ssize_t dpms_show(struct device *device, > > dpms = READ_ONCE(connector->dpms); > > - return snprintf(buf, PAGE_SIZE, "%s\n", > - drm_get_dpms_name(dpms)); > + return sysfs_emit(buf, "%s\n", drm_get_dpms_name(dpms)); > } > > static ssize_t enabled_show(struct device *device, > @@ -182,7 +181,7 @@ static ssize_t enabled_show(struct device *device, > > enabled = READ_ONCE(connector->encoder); > > - return snprintf(buf, PAGE_SIZE, enabled ? "enabled\n" : "disabled\n"); > + return sysfs_emit(buf, enabled ? "enabled\n" : "disabled\n"); > } > > static ssize_t edid_show(struct file *filp, struct kobject *kobj, > -- > 2.7.4 > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
