Re: [patch 2.6.25-rc3] omap-keypad buildfix

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



* David Brownell <david-b@xxxxxxxxxxx> [080224 19:14]:
> 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 probe() cleanup.
> 
> Signed-off-by: David Brownell <dbrownell@xxxxxxxxxxxxxxxxxxxxx>
> ---
> The OMAP tree has the same bug.
> 
>  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);

Pushing today.

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux