From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Torokhov Subject: Re: [PATCH] Input: gpio_keys - fix warning regarding uninitialized 'button' variable Date: Sat, 13 Dec 2014 11:00:35 -0800 Message-ID: <20141213190035.GB22702@dtor-ws> References: <1418136252-14365-1-git-send-email-fabio.estevam@freescale.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-ie0-f170.google.com ([209.85.223.170]:42207 "EHLO mail-ie0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753549AbaLMTAk (ORCPT ); Sat, 13 Dec 2014 14:00:40 -0500 Received: by mail-ie0-f170.google.com with SMTP id rd18so8791376iec.15 for ; Sat, 13 Dec 2014 11:00:39 -0800 (PST) Content-Disposition: inline In-Reply-To: <1418136252-14365-1-git-send-email-fabio.estevam@freescale.com> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Fabio Estevam Cc: linux-input@vger.kernel.org, festevam@gmail.com On Tue, Dec 09, 2014 at 12:44:12PM -0200, Fabio Estevam wrote: > Commit af906faabcf91eb1 ("Input: gpio_keys - fix warning regarding uninitialized > 'irq' variable") introduced the following build warning: > > drivers/input/keyboard/gpio_keys.c:625:16: warning: 'button' may be used uninitialized in this function [-Wmaybe-uninitialized] > > Move the 'button' initialization to a proper location to avoid such warning. Doh! Why didn't I see the warning at least? Argh! Applied, thank you. > > Reported-by: Olof's autobuilder > Signed-off-by: Fabio Estevam > --- > drivers/input/keyboard/gpio_keys.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/input/keyboard/gpio_keys.c b/drivers/input/keyboard/gpio_keys.c > index fb8256f..28ac922 100644 > --- a/drivers/input/keyboard/gpio_keys.c > +++ b/drivers/input/keyboard/gpio_keys.c > @@ -621,6 +621,8 @@ gpio_keys_get_devtree_pdata(struct device *dev) > int gpio = -1; > enum of_gpio_flags flags; > > + button = &pdata->buttons[i++]; > + > if (!of_find_property(pp, "gpios", NULL)) { > button->irq = irq_of_parse_and_map(pp, 0); > if (button->irq == 0) { > @@ -640,8 +642,6 @@ gpio_keys_get_devtree_pdata(struct device *dev) > } > } > > - button = &pdata->buttons[i++]; > - > button->gpio = gpio; > button->active_low = flags & OF_GPIO_ACTIVE_LOW; > > -- > 1.9.1 > -- Dmitry