[PATCH] drm/ssd130x: clean up some inconsistent indenting

Jiapeng Chong posted 1 patch 2 years, 6 months ago
drivers/gpu/drm/solomon/ssd130x.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] drm/ssd130x: clean up some inconsistent indenting
Posted by Jiapeng Chong 2 years, 6 months ago
No functional modification involved.

drivers/gpu/drm/solomon/ssd130x.c:715 ssd130x_primary_plane_duplicate_state() warn: inconsistent indenting.

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=6034
Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
---
 drivers/gpu/drm/solomon/ssd130x.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/solomon/ssd130x.c b/drivers/gpu/drm/solomon/ssd130x.c
index 971c425340c1..5a80b228d18c 100644
--- a/drivers/gpu/drm/solomon/ssd130x.c
+++ b/drivers/gpu/drm/solomon/ssd130x.c
@@ -712,7 +712,7 @@ static struct drm_plane_state *ssd130x_primary_plane_duplicate_state(struct drm_
 
 	new_shadow_plane_state = &ssd130x_state->base;
 
-	 __drm_gem_duplicate_shadow_plane_state(plane, new_shadow_plane_state);
+	__drm_gem_duplicate_shadow_plane_state(plane, new_shadow_plane_state);
 
 	return &new_shadow_plane_state->base;
 }
-- 
2.20.1.7.g153144c
Re: [PATCH] drm/ssd130x: clean up some inconsistent indenting
Posted by Javier Martinez Canillas 2 years, 6 months ago
Jiapeng Chong <jiapeng.chong@linux.alibaba.com> writes:

Hello Jiapeng,

Thanks for your patch.

> No functional modification involved.
>
> drivers/gpu/drm/solomon/ssd130x.c:715 ssd130x_primary_plane_duplicate_state() warn: inconsistent indenting.
>
> Reported-by: Abaci Robot <abaci@linux.alibaba.com>

Interesting that checkpatch didn't catch this, even when running with --strict.

Do you know which tool this bot is using to catch these inconsistencies?

> Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=6034
> Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
> ---

Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>

Pushed to drm-misc (drm-misc-next). Thanks!

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat