linux-kernel - Re: [kbuild-all] [PATCH] power: act8945a_charger: fix semicolon.cocci warnings

Open Source and information security mailing list archives

Message-ID: <20160729010130.GB7107@wfg-t540p.sh.intel.com>
Date:	Fri, 29 Jul 2016 09:01:30 +0800
From:	Fengguang Wu <lkp@...el.com>
To:	"Yang, Wenyou" <Wenyou.Yang@...el.com>
Cc:	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
	Pawel Moll <pawel.moll@....com>,
	Ian Campbell <ijc+devicetree@...lion.org.uk>,
	Dmitry Eremin-Solenikov <dbaryshkov@...il.com>,
	Mark Brown <broonie@...nel.org>,
	"linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>,
	"Ferre, Nicolas" <Nicolas.FERRE@...el.com>,
	Sebastian Reichel <sre@...nel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Rob Herring <robh+dt@...nel.org>,
	"kbuild-all@...org" <kbuild-all@...org>,
	Kumar Gala <galak@...eaurora.org>,
	David Woodhouse <dwmw2@...radead.org>,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>
Subject: Re: [kbuild-all] [PATCH] power: act8945a_charger: fix
 semicolon.cocci warnings

Hi Wenyou,

On Fri, Jul 29, 2016 at 12:57:20AM +0000, Yang, Wenyou wrote:
>Hi Fengguang,
>
>I would like to merge this patch and add your Signed-off-by, do you agree?
>
>I think it is better.

Yes that'd be fine, too.

Thanks,
Fengguang

>From: kbuild test robot [lkp@...el.com]
>Sent: Friday, June 24, 2016 20:43
>To: Yang, Wenyou
>Cc: kbuild-all@...org; Sebastian Reichel; Dmitry Eremin-Solenikov; David Woodhouse; Rob Herring; Pawel Moll; Mark Brown; Ian Campbell; Kumar Gala; linux-kernel@...r.kernel.org; devicetree@...r.kernel.org; linux-arm-kernel@...ts.infradead.org; Ferre, Nicolas; linux-pm@...r.kernel.org; Yang, Wenyou
>Subject: [PATCH] power: act8945a_charger: fix semicolon.cocci warnings
>
>drivers/power/act8945a_charger.c:115:2-3: Unneeded semicolon
>
>
> Remove unneeded semicolon.
>
>Generated by: scripts/coccinelle/misc/semicolon.cocci
>
>CC: Wenyou Yang <wenyou.yang@...el.com>
>Signed-off-by: Fengguang Wu <fengguang.wu@...el.com>
>---
>
> act8945a_charger.c |    2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>--- a/drivers/power/act8945a_charger.c
>+++ b/drivers/power/act8945a_charger.c
>@@ -112,7 +112,7 @@ static int act8945a_get_charger_state(st
>                else
>                        *val = POWER_SUPPLY_STATUS_NOT_CHARGING;
>                break;
>-       };
>+       }
>
>        return 0;
> }
>_______________________________________________
>kbuild-all mailing list
>kbuild-all@...ts.01.org
>https://lists.01.org/mailman/listinfo/kbuild-all

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives