| Message ID | 20201128224114.1033617-29-sam@ravnborg.org |
|---|---|
| State | New |
| Headers | show |
| Series | drivers/video: W=1 warning fixes | expand |
Am 28.11.20 um 23:41 schrieb Sam Ravnborg: > Fix following W=1 warnings: > - Fix set but not nused variables which was used only for logging. s/nused/used s/which was/that were Otherwise Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de> > Fixed by introducing no_printk() to trick compiler to think variables > are used > - Fix kernel-doc warning by deleting an empty comment line > > v2: > - Subject updated (Lee) > > Signed-off-by: Sam Ravnborg <sam@ravnborg.org> > Cc: Kristoffer Ericson <kristoffer.ericson@gmail.com> > Cc: Lee Jones <lee.jones@linaro.org> > --- > drivers/video/fbdev/s1d13xxxfb.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/video/fbdev/s1d13xxxfb.c b/drivers/video/fbdev/s1d13xxxfb.c > index 4541afcf9386..d1b5f965bc96 100644 > --- a/drivers/video/fbdev/s1d13xxxfb.c > +++ b/drivers/video/fbdev/s1d13xxxfb.c > @@ -45,7 +45,7 @@ > #if 0 > #define dbg(fmt, args...) do { printk(KERN_INFO fmt, ## args); } while(0) > #else > -#define dbg(fmt, args...) do { } while (0) > +#define dbg(fmt, args...) do { no_printk(KERN_INFO fmt, ## args); } while (0) > #endif > > /* > @@ -512,7 +512,6 @@ s1d13xxxfb_bitblt_copyarea(struct fb_info *info, const struct fb_copyarea *area) > } > > /** > - * > * s1d13xxxfb_bitblt_solidfill - accelerated solidfill function > * @info : framebuffer structure > * @rect : fb_fillrect structure >
diff --git a/drivers/video/fbdev/s1d13xxxfb.c b/drivers/video/fbdev/s1d13xxxfb.c index 4541afcf9386..d1b5f965bc96 100644 --- a/drivers/video/fbdev/s1d13xxxfb.c +++ b/drivers/video/fbdev/s1d13xxxfb.c @@ -45,7 +45,7 @@ #if 0 #define dbg(fmt, args...) do { printk(KERN_INFO fmt, ## args); } while(0) #else -#define dbg(fmt, args...) do { } while (0) +#define dbg(fmt, args...) do { no_printk(KERN_INFO fmt, ## args); } while (0) #endif /* @@ -512,7 +512,6 @@ s1d13xxxfb_bitblt_copyarea(struct fb_info *info, const struct fb_copyarea *area) } /** - * * s1d13xxxfb_bitblt_solidfill - accelerated solidfill function * @info : framebuffer structure * @rect : fb_fillrect structure
Fix following W=1 warnings: - Fix set but not nused variables which was used only for logging. Fixed by introducing no_printk() to trick compiler to think variables are used - Fix kernel-doc warning by deleting an empty comment line v2: - Subject updated (Lee) Signed-off-by: Sam Ravnborg <sam@ravnborg.org> Cc: Kristoffer Ericson <kristoffer.ericson@gmail.com> Cc: Lee Jones <lee.jones@linaro.org> --- drivers/video/fbdev/s1d13xxxfb.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)