diff mbox series

gpio: stmpe: fix semicolon.cocci warnings

Message ID 20180117023405.GA124815@lkp-hsx03
State New
Headers show
Series gpio: stmpe: fix semicolon.cocci warnings | expand

Commit Message

kbuild test robot Jan. 17, 2018, 2:34 a.m. UTC 
From: Fengguang Wu <fengguang.wu@intel.com>

drivers/gpio/gpio-stmpe.c:201:2-3: Unneeded semicolon


 Remove unneeded semicolon.

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

Fixes: bb628176dbc5 ("gpio: stmpe: i2c transfer are forbiden in atomic context")
CC: Patrice Chotard <patrice.chotard@st.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---

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

--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Patrice CHOTARD Jan. 17, 2018, 8:26 a.m. UTC | #1 
Hi Fengguang

On 01/17/2018 03:34 AM, kbuild test robot wrote:
> From: Fengguang Wu <fengguang.wu@intel.com>

> 

> drivers/gpio/gpio-stmpe.c:201:2-3: Unneeded semicolon

> 

> 

>   Remove unneeded semicolon.

> 

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

> 

> Fixes: bb628176dbc5 ("gpio: stmpe: i2c transfer are forbiden in atomic context")

> CC: Patrice Chotard <patrice.chotard@st.com>

> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>

> ---

> 

>   gpio-stmpe.c |    2 +-

>   1 file changed, 1 insertion(+), 1 deletion(-)

> 

> --- a/drivers/gpio/gpio-stmpe.c

> +++ b/drivers/gpio/gpio-stmpe.c

> @@ -198,7 +198,7 @@ static void stmpe_gpio_irq_sync_unlock(s

>   	if (stmpe->partnum == STMPE1600) {

>   		stmpe_reg_read(stmpe, stmpe->regs[STMPE_IDX_GPMR_LSB]);

>   		stmpe_reg_read(stmpe, stmpe->regs[STMPE_IDX_GPMR_CSB]);

> -	};

> +	}

>   

>   	for (i = 0; i < CACHE_NR_REGS; i++) {

>   		/* STMPE801 and STMPE1600 don't have RE and FE registers */

> 


Acked-by: Patrice Chotard <patrice.chotard@st.com>


Thanks
Linus Walleij Jan. 17, 2018, 9:21 a.m. UTC | #2 
On Wed, Jan 17, 2018 at 3:34 AM, kbuild test robot
<fengguang.wu@intel.com> wrote:

> From: Fengguang Wu <fengguang.wu@intel.com>
>
> drivers/gpio/gpio-stmpe.c:201:2-3: Unneeded semicolon
>
>
>  Remove unneeded semicolon.
>
> Generated by: scripts/coccinelle/misc/semicolon.cocci
>
> Fixes: bb628176dbc5 ("gpio: stmpe: i2c transfer are forbiden in atomic context")
> CC: Patrice Chotard <patrice.chotard@st.com>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>

I just edited the offending commit, thanks!

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox series

Patch

--- a/drivers/gpio/gpio-stmpe.c
+++ b/drivers/gpio/gpio-stmpe.c
@@ -198,7 +198,7 @@  static void stmpe_gpio_irq_sync_unlock(s
 	if (stmpe->partnum == STMPE1600) {
 		stmpe_reg_read(stmpe, stmpe->regs[STMPE_IDX_GPMR_LSB]);
 		stmpe_reg_read(stmpe, stmpe->regs[STMPE_IDX_GPMR_CSB]);
-	};
+	}
 
 	for (i = 0; i < CACHE_NR_REGS; i++) {
 		/* STMPE801 and STMPE1600 don't have RE and FE registers */