2017-03-12 14:47:11

by Fengguang Wu

[permalink] [raw]
Subject: drivers/gpu/drm/tinydrm/mipi-dbi.c:657:2-3: Unneeded semicolon

tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 106e4da60209b508894956b6adf4688f84c1766d
commit: 1f47e6cbf58d10392a2158b0e33ef72ad681e40f drm/tinydrm: Add support for Multi-Inno MI0283QT display
date: 3 weeks ago


coccinelle warnings: (new ones prefixed by >>)

>> drivers/gpu/drm/tinydrm/mipi-dbi.c:657:2-3: Unneeded semicolon
drivers/gpu/drm/tinydrm/mipi-dbi.c:593:2-3: Unneeded semicolon

Please review and possibly fold the followup patch.

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation


2017-03-12 14:47:26

by Fengguang Wu

[permalink] [raw]
Subject: [PATCH] drm/tinydrm: fix semicolon.cocci warnings

drivers/gpu/drm/tinydrm/mipi-dbi.c:657:2-3: Unneeded semicolon
drivers/gpu/drm/tinydrm/mipi-dbi.c:593:2-3: Unneeded semicolon


Remove unneeded semicolon.

Generated by: scripts/coccinelle/misc/semicolon.cocci

Signed-off-by: Fengguang Wu <[email protected]>
---

mipi-dbi.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

--- a/drivers/gpu/drm/tinydrm/mipi-dbi.c
+++ b/drivers/gpu/drm/tinydrm/mipi-dbi.c
@@ -590,7 +590,7 @@ static int mipi_dbi_spi1e_transfer(struc
ret = spi_sync(spi, &m);
if (ret)
return ret;
- };
+ }

return 0;
}
@@ -654,7 +654,7 @@ static int mipi_dbi_spi1_transfer(struct
ret = spi_sync(spi, &m);
if (ret)
return ret;
- };
+ }

return 0;
}

2017-03-14 13:18:26

by Daniel Vetter

[permalink] [raw]
Subject: Re: [PATCH] drm/tinydrm: fix semicolon.cocci warnings

On Sun, Mar 12, 2017 at 10:46:36PM +0800, kbuild test robot wrote:
> drivers/gpu/drm/tinydrm/mipi-dbi.c:657:2-3: Unneeded semicolon
> drivers/gpu/drm/tinydrm/mipi-dbi.c:593:2-3: Unneeded semicolon
>
>
> Remove unneeded semicolon.
>
> Generated by: scripts/coccinelle/misc/semicolon.cocci
>
> Signed-off-by: Fengguang Wu <[email protected]>

Applied to drm-misc-next, thx.
-Daniel

> ---
>
> mipi-dbi.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> --- a/drivers/gpu/drm/tinydrm/mipi-dbi.c
> +++ b/drivers/gpu/drm/tinydrm/mipi-dbi.c
> @@ -590,7 +590,7 @@ static int mipi_dbi_spi1e_transfer(struc
> ret = spi_sync(spi, &m);
> if (ret)
> return ret;
> - };
> + }
>
> return 0;
> }
> @@ -654,7 +654,7 @@ static int mipi_dbi_spi1_transfer(struct
> ret = spi_sync(spi, &m);
> if (ret)
> return ret;
> - };
> + }
>
> return 0;
> }
> _______________________________________________
> dri-devel mailing list
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch