* [patch 2.6.25-rc8] omap-keypad: fix build warning
@ 2008-04-11 8:09 David Brownell
2008-04-15 5:36 ` Dmitry Torokhov
0 siblings, 1 reply; 2+ messages in thread
From: David Brownell @ 2008-04-11 8:09 UTC (permalink / raw)
To: Dmitry Torokhov, dmitry.torokhov, linux-input, linux-input
From: David Brownell <dbrownell@users.sourceforge.net>
Build fixes:
drivers/input/keyboard/omap-keypad.c: In function 'omap_kp_probe':
drivers/input/keyboard/omap-keypad.c:418: warning: 'row_idx' is used uninitialized in this function
drivers/input/keyboard/omap-keypad.c:421: warning: 'col_idx' is used uninitialized in this function
These variables are useful when cpu_is_omap24xx(), and otherwise just
for useless cleanup.
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: Tony Lindgren <tony@atomide.com>
---
This has been in the OMAP tree since 24-Feb.
drivers/input/keyboard/omap-keypad.c | 3 +++
1 files changed, 3 insertions(+)
--- a/drivers/input/keyboard/omap-keypad.c 2008-02-24 18:50:32.000000000 -0800
+++ b/drivers/input/keyboard/omap-keypad.c 2008-02-24 19:10:17.000000000 -0800
@@ -352,6 +352,9 @@ static int __init omap_kp_probe(struct p
}
omap_set_gpio_direction(row_gpios[row_idx], 1);
}
+ } else {
+ col_idx = 0;
+ row_idx = 0;
}
setup_timer(&omap_kp->timer, omap_kp_timer, (unsigned long)omap_kp);
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [patch 2.6.25-rc8] omap-keypad: fix build warning
2008-04-11 8:09 [patch 2.6.25-rc8] omap-keypad: fix build warning David Brownell
@ 2008-04-15 5:36 ` Dmitry Torokhov
0 siblings, 0 replies; 2+ messages in thread
From: Dmitry Torokhov @ 2008-04-15 5:36 UTC (permalink / raw)
To: David Brownell; +Cc: linux-input, linux-input
On Fri, Apr 11, 2008 at 01:09:18AM -0700, David Brownell wrote:
> From: David Brownell <dbrownell@users.sourceforge.net>
>
> Build fixes:
>
> drivers/input/keyboard/omap-keypad.c: In function 'omap_kp_probe':
> drivers/input/keyboard/omap-keypad.c:418: warning: 'row_idx' is used uninitialized in this function
> drivers/input/keyboard/omap-keypad.c:421: warning: 'col_idx' is used uninitialized in this function
>
> These variables are useful when cpu_is_omap24xx(), and otherwise just
> for useless cleanup.
>
> Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
> Signed-off-by: Tony Lindgren <tony@atomide.com>
Applied, thank you David.
--
Dmitry
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2008-04-15 5:36 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-04-11 8:09 [patch 2.6.25-rc8] omap-keypad: fix build warning David Brownell
2008-04-15 5:36 ` Dmitry Torokhov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).