From mboxrd@z Thu Jan 1 00:00:00 1970 From: Darren Hart Subject: Re: linux-next: build warning after merge of the drivers-x86 tree Date: Wed, 14 Dec 2016 14:21:38 -0800 Message-ID: <20161214222138.GA116314@f23x64.localdomain> References: <20161214135044.57dc96f5@canb.auug.org.au> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20161214135044.57dc96f5@canb.auug.org.au> Sender: linux-kernel-owner@vger.kernel.org To: Stephen Rothwell Cc: linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, Lyude , Henrique de Moraes Holschuh , Andy Shevchenko List-Id: linux-next.vger.kernel.org On Wed, Dec 14, 2016 at 01:50:44PM +1100, Stephen Rothwell wrote: > Hi Darren, > > After merging the drivers-x86 tree, today's linux-next build (x86_64 > allmodconfig) produced this warning: > > In file included from include/linux/kernel.h:13:0, > from drivers/platform/x86/thinkpad_acpi.c:52: > drivers/platform/x86/thinkpad_acpi.c: In function 'hotkey_init': > include/linux/printk.h:299:2: warning: 'type' may be used uninitialized in this function [-Wmaybe-uninitialized] > printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__) > ^ > drivers/platform/x86/thinkpad_acpi.c:3147:8: note: 'type' was declared here > char *type; > ^ > In file included from include/linux/kernel.h:13:0, > from drivers/platform/x86/thinkpad_acpi.c:52: > include/linux/printk.h:299:2: warning: 'in_tablet_mode' may be used uninitialized in this function [-Wmaybe-uninitialized] > printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__) > ^ > drivers/platform/x86/thinkpad_acpi.c:3146:6: note: 'in_tablet_mode' was declared here > int in_tablet_mode, res; > ^ > > Introduced by commit > > b31800283868 ("platform/x86: thinkpad_acpi: Move tablet detection into separate function") > > I can't tell if this is a false positive or not. That's an uninitialized local variable. Not sure how I missed that. I'll fix it up today. Thank you for the report. -- Darren Hart Intel Open Source Technology Center