diff mbox series

fbdev: omapfb: cleanup inconsistent indentation

Message ID 20230207100630.11644-1-mielkesteven@icloud.com
State New
Headers show
Series fbdev: omapfb: cleanup inconsistent indentation | expand

Commit Message

Lucy Mielke Feb. 7, 2023, 10:06 a.m. UTC
This cleans up the indentation according to the Linux kernel coding
style, and should fix the warning created by the kernel test robot.

Fixes: 8b08cf2b64f5a60594b07795b2ad518c6d044566 ("OMAP: add TI OMAP
	framebuffer driver")

Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Lucy Mielke <mielkesteven@icloud.com>
---
 drivers/video/fbdev/omap/omapfb_main.c | 32 +++++++++++++++-----------
 1 file changed, 19 insertions(+), 13 deletions(-)

Comments

Helge Deller Feb. 25, 2023, 5:23 p.m. UTC | #1
Hi Lucy,

On 2/7/23 11:06, Lucy Mielke wrote:
> This cleans up the indentation according to the Linux kernel coding
> style, and should fix the warning created by the kernel test robot.
>
> Fixes: 8b08cf2b64f5a60594b07795b2ad518c6d044566 ("OMAP: add TI OMAP
> 	framebuffer driver")
>
> Reported-by: kernel test robot <lkp@intel.com>
> Signed-off-by: Lucy Mielke <mielkesteven@icloud.com>
> ---
>   drivers/video/fbdev/omap/omapfb_main.c | 32 +++++++++++++++-----------
>   1 file changed, 19 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/video/fbdev/omap/omapfb_main.c b/drivers/video/fbdev/omap/omapfb_main.c
> index 1f3df2055ff0..4b673daa32cb 100644
> --- a/drivers/video/fbdev/omap/omapfb_main.c
> +++ b/drivers/video/fbdev/omap/omapfb_main.c
> @@ -78,7 +78,7 @@ static struct platform_device omapdss_device = {
>   	.name		= "omapdss_dss",
>   	.id		= -1,
>   	.dev            = {
> -		.release = omapdss_release,
> +	.release	= omapdss_release,>   	},

This is wrong. "release" belongs inside the .dev
variable, so the previous indenting is correct.

The changes below seem correct.

Helge

>   };
>
> @@ -544,19 +544,25 @@ static int set_fb_var(struct fb_info *fbi,
>   		var->yoffset = var->yres_virtual - var->yres;
>
>   	if (plane->color_mode == OMAPFB_COLOR_RGB444) {
> -		var->red.offset	  = 8; var->red.length	 = 4;
> -						var->red.msb_right   = 0;
> -		var->green.offset = 4; var->green.length = 4;
> -						var->green.msb_right = 0;
> -		var->blue.offset  = 0; var->blue.length  = 4;
> -						var->blue.msb_right  = 0;
> +		var->red.offset		= 8;
> +		var->red.length		= 4;
> +		var->red.msb_right	= 0;
> +		var->green.offset	= 4;
> +		var->green.length	= 4;
> +		var->green.msb_right	= 0;
> +		var->blue.offset	= 0;
> +		var->blue.length	= 4;
> +		var->blue.msb_right	= 0;
>   	} else {
> -		var->red.offset	 = 11; var->red.length	 = 5;
> -						var->red.msb_right   = 0;
> -		var->green.offset = 5;  var->green.length = 6;
> -						var->green.msb_right = 0;
> -		var->blue.offset = 0;  var->blue.length  = 5;
> -						var->blue.msb_right  = 0;
> +		var->red.offset		= 11;
> +		var->red.length		= 5;
> +		var->red.msb_right	= 0;
> +		var->green.offset	= 5;
> +		var->green.length	= 6;
> +		var->green.msb_right	= 0;
> +		var->blue.offset	= 0;
> +		var->blue.length	= 5;
> +		var->blue.msb_right	= 0;
>   	}
>
>   	var->height		= -1;
Helge Deller Feb. 25, 2023, 6:10 p.m. UTC | #2
Hi Lucy,

there were some more things wrong in your patch:

On 2/25/23 18:23, Helge Deller wrote:
> On 2/7/23 11:06, Lucy Mielke wrote:
>> This cleans up the indentation according to the Linux kernel coding
>> style, and should fix the warning created by the kernel test robot.
>>
>> Fixes: 8b08cf2b64f5a60594b07795b2ad518c6d044566 ("OMAP: add TI OMAP framebuffer driver")

Please use only 12 chars of the sha1 in the Fixes tag.

>> Reported-by: kernel test robot <lkp@intel.com>
>> Signed-off-by: Lucy Mielke <mielkesteven@icloud.com>
>> ---
>>   drivers/video/fbdev/omap/omapfb_main.c | 32 +++++++++++++++-----------
>>   1 file changed, 19 insertions(+), 13 deletions(-)
>>
>> diff --git a/drivers/video/fbdev/omap/omapfb_main.c b/drivers/video/fbdev/omap/omapfb_main.c
>> index 1f3df2055ff0..4b673daa32cb 100644
>> --- a/drivers/video/fbdev/omap/omapfb_main.c
>> +++ b/drivers/video/fbdev/omap/omapfb_main.c
>> @@ -78,7 +78,7 @@ static struct platform_device omapdss_device = {
>>       .name        = "omapdss_dss",
>>       .id        = -1,
>>       .dev            = {
>> -        .release = omapdss_release,
>> +    .release    = omapdss_release,>       },
> 
> This is wrong. "release" belongs inside the .dev
> variable, so the previous indenting is correct.
> 
> The changes below seem correct.
> 
> Helge
> 
>>   };
>>
>> @@ -544,19 +544,25 @@ static int set_fb_var(struct fb_info *fbi,
>>           var->yoffset = var->yres_virtual - var->yres;
>>
>>       if (plane->color_mode == OMAPFB_COLOR_RGB444) {
>> -        var->red.offset      = 8; var->red.length     = 4;
>> -                        var->red.msb_right   = 0;
>> -        var->green.offset = 4; var->green.length = 4;
>> -                        var->green.msb_right = 0;
>> -        var->blue.offset  = 0; var->blue.length  = 4;
>> -                        var->blue.msb_right  = 0;
>> +        var->red.offset        = 8;
>> +        var->red.length        = 4;

At various of those lines you have trailing whitespaces.


I fixed up those issues, dropped the first part of your patch
and added it to the fbdev tree.

Helge


>> +        var->red.msb_right    = 0;
>> +        var->green.offset    = 4;
>> +        var->green.length    = 4;
>> +        var->green.msb_right    = 0;
>> +        var->blue.offset    = 0;
>> +        var->blue.length    = 4;
>> +        var->blue.msb_right    = 0;
>>       } else {
>> -        var->red.offset     = 11; var->red.length     = 5;
>> -                        var->red.msb_right   = 0;
>> -        var->green.offset = 5;  var->green.length = 6;
>> -                        var->green.msb_right = 0;
>> -        var->blue.offset = 0;  var->blue.length  = 5;
>> -                        var->blue.msb_right  = 0;
>> +        var->red.offset        = 11;
>> +        var->red.length        = 5;
>> +        var->red.msb_right    = 0;
>> +        var->green.offset    = 5;
>> +        var->green.length    = 6;
>> +        var->green.msb_right    = 0;
>> +        var->blue.offset    = 0;
>> +        var->blue.length    = 5;
>> +        var->blue.msb_right    = 0;
>>       }
>>
>>       var->height        = -1;
>
diff mbox series

Patch

diff --git a/drivers/video/fbdev/omap/omapfb_main.c b/drivers/video/fbdev/omap/omapfb_main.c
index 1f3df2055ff0..4b673daa32cb 100644
--- a/drivers/video/fbdev/omap/omapfb_main.c
+++ b/drivers/video/fbdev/omap/omapfb_main.c
@@ -78,7 +78,7 @@  static struct platform_device omapdss_device = {
 	.name		= "omapdss_dss",
 	.id		= -1,
 	.dev            = {
-		.release = omapdss_release,
+	.release	= omapdss_release,
 	},
 };
 
@@ -544,19 +544,25 @@  static int set_fb_var(struct fb_info *fbi,
 		var->yoffset = var->yres_virtual - var->yres;
 
 	if (plane->color_mode == OMAPFB_COLOR_RGB444) {
-		var->red.offset	  = 8; var->red.length	 = 4;
-						var->red.msb_right   = 0;
-		var->green.offset = 4; var->green.length = 4;
-						var->green.msb_right = 0;
-		var->blue.offset  = 0; var->blue.length  = 4;
-						var->blue.msb_right  = 0;
+		var->red.offset		= 8; 
+		var->red.length		= 4;
+		var->red.msb_right	= 0;
+		var->green.offset	= 4; 
+		var->green.length	= 4;
+		var->green.msb_right	= 0;
+		var->blue.offset	= 0; 
+		var->blue.length	= 4;
+		var->blue.msb_right	= 0;
 	} else {
-		var->red.offset	 = 11; var->red.length	 = 5;
-						var->red.msb_right   = 0;
-		var->green.offset = 5;  var->green.length = 6;
-						var->green.msb_right = 0;
-		var->blue.offset = 0;  var->blue.length  = 5;
-						var->blue.msb_right  = 0;
+		var->red.offset		= 11; 
+		var->red.length		= 5;
+		var->red.msb_right	= 0;
+		var->green.offset	= 5;  
+		var->green.length	= 6;
+		var->green.msb_right	= 0;
+		var->blue.offset	= 0;
+		var->blue.length	= 5;
+		var->blue.msb_right	= 0;
 	}
 
 	var->height		= -1;