> Am 26.04.2018 um 19:34 schrieb H. Nikolaus Schaller <h...@goldelico.com>:
> 
> which makes it easier to match them with the data sheets.
> 
> Signed-off-by: H. Nikolaus Schaller <h...@goldelico.com>
> ---
> drivers/gpio/gpio-pca953x.c | 32 ++++++++++++++++----------------
> 1 file changed, 16 insertions(+), 16 deletions(-)
> 
> diff --git a/drivers/gpio/gpio-pca953x.c b/drivers/gpio/gpio-pca953x.c
> index d02964983b5b..bd0593afdae3 100644
> --- a/drivers/gpio/gpio-pca953x.c
> +++ b/drivers/gpio/gpio-pca953x.c
> @@ -25,25 +25,25 @@
> 
> #include <asm/unaligned.h>
> 
> -#define PCA953X_INPUT                0
> -#define PCA953X_OUTPUT               1
> -#define PCA953X_INVERT               2
> -#define PCA953X_DIRECTION    3
> +#define PCA953X_INPUT                0x00
> +#define PCA953X_OUTPUT               0x01
> +#define PCA953X_INVERT               0x02
> +#define PCA953X_DIRECTION    0x03
> 
> #define REG_ADDR_AI           0x80
> 
> -#define PCA957X_IN           0
> -#define PCA957X_INVRT                1
> -#define PCA957X_BKEN         2
> -#define PCA957X_PUPD         3
> -#define PCA957X_CFG          4
> -#define PCA957X_OUT          5
> -#define PCA957X_MSK          6
> -#define PCA957X_INTS         7
> -
> -#define PCAL953X_IN_LATCH    34
> -#define PCAL953X_INT_MASK    37
> -#define PCAL953X_INT_STAT    38
> +#define PCA957X_IN           0x00
> +#define PCA957X_INVRT                0x01
> +#define PCA957X_BKEN         0x02
> +#define PCA957X_PUPD         0x03
> +#define PCA957X_CFG          0x04
> +#define PCA957X_OUT          0x05
> +#define PCA957X_MSK          0x06
> +#define PCA957X_INTS         0x07
> +
> +#define PCAL953X_IN_LATCH    0x22
> +#define PCAL953X_PULL_EN     0x23
> +#define PCAL953X_PULL_SEL    0x24

oops, just spotted a bug above.

Here, we should update the registers PCAL953X_INT_MASK and
PCAL953X_INT_STAT. And add PCAL953X_PULL_EN and PCAL953X_PULL_SEL
in the next patch.

Sorry. I will send an update for the patch set.

> 
> #define PCA_GPIO_MASK         0x00FF
> #define PCA_INT                       0x0100
> -- 
> 2.12.2
> 

BR,
Nikolaus

Reply via email to