[U-Boot] [PATCH] M28: Fix OB1 bug in GPIO driver
Marek Vasut
marek.vasut at gmail.com
Tue Nov 22 15:21:22 CET 2011
> This patch fixes a small off-by-one bug in the GPIO driver for the mxs
> platform that allowed the selection gpio pins of one bank more than the
> SoC actually has.
>
> Signed-off-by: Robert Deliën <robert at delien.nl>
>
> diff --git a/drivers/gpio/mxs_gpio.c b/drivers/gpio/mxs_gpio.c
> index b7e9591..539738b 100644
> --- a/drivers/gpio/mxs_gpio.c
> +++ b/drivers/gpio/mxs_gpio.c
> @@ -120,7 +120,7 @@ int gpio_direction_output(int gp, int value)
>
> int gpio_request(int gp, const char *label)
> {
> - if (PAD_BANK(gp) > PINCTRL_BANKS)
> + if (PAD_BANK(gp) >= PINCTRL_BANKS)
> return -EINVAL;
>
> return 0;
Good catch, I'll verify and ack afterwards.
Thanks
M
More information about the U-Boot
mailing list