From mboxrd@z Thu Jan 1 00:00:00 1970 From: Darren Hart Subject: Re: [PATCH] eeepc-laptop: fix a Wuninitialized warning Date: Mon, 15 Sep 2014 10:40:48 -0700 Message-ID: <20140915174048.GA59389@vmdeb7> References: <1410511466-1070-1-git-send-email-chenhanxiao@cn.fujitsu.com> <20140912205337.GB23114@vmdeb7> <5871495633F38949900D2BF2DC04883E5A8782@G08CNEXMBPEKD02.g08.fujitsu.local> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from bombadil.infradead.org ([198.137.202.9]:35833 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753549AbaIORkz (ORCPT ); Mon, 15 Sep 2014 13:40:55 -0400 Content-Disposition: inline In-Reply-To: <5871495633F38949900D2BF2DC04883E5A8782@G08CNEXMBPEKD02.g08.fujitsu.local> Sender: platform-driver-x86-owner@vger.kernel.org List-ID: To: "Chen, Hanxiao" , "Rafael J. Wysocki" Cc: "acpi4asus-user@lists.sourceforge.net" , "platform-driver-x86@vger.kernel.org" , Corentin Chary On Mon, Sep 15, 2014 at 02:16:35AM +0000, Chen, Hanxiao wrote: > > > > -----Original Message----- > > From: Darren Hart [mailto:dvhart@infradead.org] > > On Fri, Sep 12, 2014 at 04:44:26PM +0800, Chen Hanxiao wrote: > > > > Hi Chen, > > > > I don't see such a warning with gcc 4.9.1. Which compiler are you using? > > > > I'm using gcc 4.8.2 on Fedora20. > Should we ignore the warning of old version of gcc? > > If not, I'll resend this with a detail commit message. In my experience, we do not add workarounds for buggy compilers reporting bogus unitialized variable warnings because those workarounds effectively mask legitimate uninitialized variable cases. Rafael, do you agree? Thanks, Darren > > Thanks for your help. > > - Chen > > > Please include these sorts of details (which compiler, etc.) when submitting > > patches. I will never merge a patch without a complete commit message. > > > > > > > Signed-off-by: Chen Hanxiao > > > --- > > > drivers/platform/x86/eeepc-laptop.c | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/drivers/platform/x86/eeepc-laptop.c > > b/drivers/platform/x86/eeepc-laptop.c > > > index bd533c2..738fc0c 100644 > > > --- a/drivers/platform/x86/eeepc-laptop.c > > > +++ b/drivers/platform/x86/eeepc-laptop.c > > > @@ -276,7 +276,7 @@ static ssize_t store_sys_acpi(struct device *dev, int cm, > > > const char *buf, size_t count) > > > { > > > struct eeepc_laptop *eeepc = dev_get_drvdata(dev); > > > - int rv, value; > > > + int rv, value = -1; > > > > > > rv = parse_arg(buf, count, &value); > > > > Inspecting the code, I don't see how this could possibly result in the use of > > an > > uninitialized variable. > > > > -- > > Darren Hart > > Intel Open Source Technology Center -- Darren Hart Intel Open Source Technology Center