lkml.org 
[lkml]   [2017]   [Aug]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] drivers: video: Fix 80 character limit and remove unnecessary space
Date

Hi,

Please use a more descriptive patch subject, i.e.
"fbdev: vgastate: fix checkpatch.pl reported warnings".

Please also cc: fbdev mailing list (linux-fbdev@vger.kernel.org
on fbdev patches).

On Monday, August 21, 2017 07:33:51 PM Himanshu Jha wrote:
> Fixes the following checkpatch warnings:
> WARNING: line over 80 characters
> WARNING: space prohibited between function name and open parenthesis '('
> WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
>
> Signed-off-by: Himanshu Jha <himanshujha199640@gmail.com>
> ---
> drivers/video/vgastate.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/video/vgastate.c b/drivers/video/vgastate.c
> index 548c751..a52283d 100644
> --- a/drivers/video/vgastate.c
> +++ b/drivers/video/vgastate.c
> @@ -31,8 +31,9 @@ struct regstate {
> __u8 misc;
> };
>
> -static inline unsigned char vga_rcrtcs(void __iomem *regbase, unsigned short iobase,
> - unsigned char reg)
> +static inline unsigned char vga_rcrtcs(void __iomem *regbase,
> + unsigned short iobase,
> + unsigned char reg)

This is not an improvement and doesn't conform to the kernel CodingStyle.

It should be:

static inline unsigned char vga_rcrtcs(void __iomem *regbase,
unsigned short iobase,
unsigned char reg)

> {
> vga_w(regbase, iobase + 0x4, reg);
> return vga_r(regbase, iobase + 0x5);
> @@ -454,8 +455,9 @@ int save_vga(struct vgastate *state)
> }
> return 0;
> }
> +EXPORT_SYMBOL(save_vga);
>
> -int restore_vga (struct vgastate *state)
> +int restore_vga(struct vgastate *state)
> {
> if (state->vidstate == NULL)
> return 1;
> @@ -480,8 +482,6 @@ int restore_vga (struct vgastate *state)
> vga_cleanup(state);
> return 0;
> }
> -
> -EXPORT_SYMBOL(save_vga);
> EXPORT_SYMBOL(restore_vga);
>
> MODULE_AUTHOR("James Simmons <jsimmons@users.sf.net>");

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics

\
 
 \ /
  Last update: 2017-08-23 18:13    [W:0.084 / U:0.136 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site