* [PATCH] fbdev: omapfb: cleanup inconsistent indentation
@ 2023-02-07 10:06 Lucy Mielke
2023-02-25 17:23 ` Helge Deller
0 siblings, 1 reply; 4+ messages in thread
From: Lucy Mielke @ 2023-02-07 10:06 UTC (permalink / raw)
To: deller, linux-fbdev, linux-omap, dri-devel; +Cc: linux-kernel, arnd
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,
},
};
@@ -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;
--
2.39.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] fbdev: omapfb: cleanup inconsistent indentation
2023-02-07 10:06 [PATCH] fbdev: omapfb: cleanup inconsistent indentation Lucy Mielke
@ 2023-02-25 17:23 ` Helge Deller
2023-02-25 18:10 ` Helge Deller
0 siblings, 1 reply; 4+ messages in thread
From: Helge Deller @ 2023-02-25 17:23 UTC (permalink / raw)
To: Lucy Mielke, linux-fbdev, linux-omap, dri-devel; +Cc: linux-kernel, arnd
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;
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] fbdev: omapfb: cleanup inconsistent indentation
2023-02-25 17:23 ` Helge Deller
@ 2023-02-25 18:10 ` Helge Deller
0 siblings, 0 replies; 4+ messages in thread
From: Helge Deller @ 2023-02-25 18:10 UTC (permalink / raw)
To: Lucy Mielke, linux-fbdev, linux-omap, dri-devel; +Cc: linux-kernel, arnd
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;
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] fbdev: omapfb: cleanup inconsistent indentation
@ 2023-02-07 9:36 Lucy Mielke
0 siblings, 0 replies; 4+ messages in thread
From: Lucy Mielke @ 2023-02-07 9:36 UTC (permalink / raw)
To: deller, linux-fbdev, linux-omap, dri-devel
Cc: linux-kernel, arnd, Lucy Mielke, kernel test robot
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,
},
};
@@ -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;
--
2.39.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-02-25 18:10 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-07 10:06 [PATCH] fbdev: omapfb: cleanup inconsistent indentation Lucy Mielke
2023-02-25 17:23 ` Helge Deller
2023-02-25 18:10 ` Helge Deller
-- strict thread matches above, loose matches on Subject: below --
2023-02-07 9:36 Lucy Mielke
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).