Hi Mario,

On Mon, May 23, 2016 at 09:54:56AM +0200, Mario Six wrote:
>A bug in the pca953x driver prevents correct reading of GPIO input
>values beyond the 8th GPIO; all values are reported as zero. Setting of
>GPIO output values is not affected.
>
>This patch fixes the reading behavior.
>
>Signed-off-by: Mario Six <mario....@gdsys.cc>

Reviewed-by: Peng Fan <van.free...@gmail.com>

Regards,
Peng.

>---
> drivers/gpio/pca953x_gpio.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
>diff --git a/drivers/gpio/pca953x_gpio.c b/drivers/gpio/pca953x_gpio.c
>index 987d10e..3cbfa09 100644
>--- a/drivers/gpio/pca953x_gpio.c
>+++ b/drivers/gpio/pca953x_gpio.c
>@@ -145,11 +145,13 @@ static int pca953x_get_value(struct udevice *dev, 
>unsigned offset)
>       int ret;
>       u8 val = 0;
> 
>+      int off = offset % BANK_SZ;
>+
>       ret = pca953x_read_single(dev, PCA953X_INPUT, &val, offset);
>       if (ret)
>               return ret;
> 
>-      return (val >> offset) & 0x1;
>+      return (val >> off) & 0x1;
> }
> 
> static int pca953x_set_value(struct udevice *dev, unsigned offset,
>-- 
>2.7.0.GIT
>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to