Re: [PATCH] video: fbdev: vesafb: fix indentation issue

2019-04-01 Thread Bartlomiej Zolnierkiewicz

On 02/12/2019 02:29 PM, Colin King wrote:
> From: Colin Ian King 
> 
> There are a couple of statements that are indented too deeply, fix
> this by removing tabs. Also add a space after a comma to clean up
> a cppcheck warning.
> 
> Signed-off-by: Colin Ian King 

Patch queued for v5.2, thanks.

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R Institute Poland
Samsung Electronics
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH] video: fbdev: vesafb: fix indentation issue

2019-02-15 Thread Joe Perches
On Tue, 2019-02-12 at 13:29 +, Colin King wrote:
> From: Colin Ian King 
> 
> There are a couple of statements that are indented too deeply, fix
> this by removing tabs. Also add a space after a comma to clean up
> a cppcheck warning.
> 
> Signed-off-by: Colin Ian King 
> ---
>  drivers/video/fbdev/vesafb.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/video/fbdev/vesafb.c b/drivers/video/fbdev/vesafb.c
> index 528fe917dd49..b7388e25aa2a 100644
> --- a/drivers/video/fbdev/vesafb.c
> +++ b/drivers/video/fbdev/vesafb.c
> @@ -336,8 +336,8 @@ static int vesafb_probe(struct platform_device *dev)
>   printk(KERN_INFO "vesafb: pmi: set display start = %p, set 
> palette = %p\n",pmi_start,pmi_pal);
>   if (pmi_base[3]) {
>   printk(KERN_INFO "vesafb: pmi: ports = ");
> - for (i = pmi_base[3]/2; pmi_base[i] != 0x; 
> i++)
> - printk("%x ",pmi_base[i]);
> + for (i = pmi_base[3]/2; pmi_base[i] != 0x; i++)
> + printk("%x ", pmi_base[i]);
>   printk("\n");
>   if (pmi_base[i] != 0x) {
>   /*

Perhaps you could also use pr_cont too because otherwise this
emits each printk on separate lines.




___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel