> Am 26.04.2018 um 19:34 schrieb H. Nikolaus Schaller <hns@xxxxxxxxxxxxx>: > > which makes it easier to match them with the data sheets. > > Signed-off-by: H. Nikolaus Schaller <hns@xxxxxxxxxxxxx> > --- > 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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
