2019-06-22 00:58:19

by kernel test robot

Subject: [PATCH] platform/chrome: cros_ec_lpc: fix semicolon.cocci warnings

From: kbuild test robot <[email protected]>

drivers/platform/chrome/wilco_ec/event.c:161:3-4: Unneeded semicolon


Remove unneeded semicolon.

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

Fixes: 22c040fa21b6 ("platform/chrome: cros_ec_lpc: Choose Microchip EC at runtime")
Signed-off-by: kbuild test robot <[email protected]>
---

tree: https://git.kernel.org/pub/scm/linux/kernel/git/chrome-platform/linux.git for-next
head: 9eecd07b34507de9d6a9c264d13d30e1ee5fabe8
commit: 22c040fa21b604b9b3d88645e108fb2f0a74474b [21/22] platform/chrome: cros_ec_lpc: Choose Microchip EC at runtime

event.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

--- a/drivers/platform/chrome/wilco_ec/event.c
+++ b/drivers/platform/chrome/wilco_ec/event.c
@@ -158,7 +158,7 @@ static int enqueue_events(struct acpi_de
dev_err(&adev->dev, "Too many event words: %zu > %d\n",
num_words, EC_ACPI_MAX_EVENT_WORDS);
return -EOVERFLOW;
- };
+ }

/* Ensure event does not overflow the available buffer */
if ((offset + event_size) > length) {

2019-06-24 15:15:21

by Nick Crews

Subject: Re: [PATCH] platform/chrome: cros_ec_lpc: fix semicolon.cocci warnings

I fixed this in the new version.

On Fri, Jun 21, 2019 at 6:55 PM kbuild test robot <[email protected]> wrote:
>
> From: kbuild test robot <[email protected]>
>
> drivers/platform/chrome/wilco_ec/event.c:161:3-4: Unneeded semicolon
>
>
> Remove unneeded semicolon.
>
> Generated by: scripts/coccinelle/misc/semicolon.cocci
>
> Fixes: 22c040fa21b6 ("platform/chrome: cros_ec_lpc: Choose Microchip EC at runtime")
> Signed-off-by: kbuild test robot <[email protected]>
> ---
>
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/chrome-platform/linux.git for-next
> head: 9eecd07b34507de9d6a9c264d13d30e1ee5fabe8
> commit: 22c040fa21b604b9b3d88645e108fb2f0a74474b [21/22] platform/chrome: cros_ec_lpc: Choose Microchip EC at runtime
>
> event.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- a/drivers/platform/chrome/wilco_ec/event.c
> +++ b/drivers/platform/chrome/wilco_ec/event.c
> @@ -158,7 +158,7 @@ static int enqueue_events(struct acpi_de
> dev_err(&adev->dev, "Too many event words: %zu > %d\n",
> num_words, EC_ACPI_MAX_EVENT_WORDS);
> return -EOVERFLOW;
> - };
> + }
>
> /* Ensure event does not overflow the available buffer */
> if ((offset + event_size) > length) {