Message ID | 20220906093747.12328-1-alice.guo@oss.nxp.com |
---|---|
State | Superseded |
Delegated to: | Tom Rini |
Headers | show |
Series | [v1] gpio: adp5585: add gpio driver for ADP5585 I/O Expander Controller | expand |
A friendly ping... Best Regards, Alice Guo > -----Original Message----- > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Alice Guo (OSS) > Sent: Tuesday, September 6, 2022 5:38 PM > To: sbabic@denx.de; festevam@gmail.com; sjg@chromium.org > Cc: dl-uboot-imx <uboot-imx@nxp.com>; u-boot@lists.denx.de > Subject: [PATCH v1] gpio: adp5585: add gpio driver for ADP5585 I/O Expander > Controller > > From: Alice Guo <alice.guo@nxp.com> > > Signed-off-by: Alice Guo <alice.guo@nxp.com> > --- > arch/arm/include/asm/arch-imx9/gpio.h | 2 + > drivers/gpio/Kconfig | 6 + > drivers/gpio/Makefile | 1 + > drivers/gpio/adp5585_gpio.c | 238 > ++++++++++++++++++++++++++ > 4 files changed, 247 insertions(+) > create mode 100644 drivers/gpio/adp5585_gpio.c > > diff --git a/arch/arm/include/asm/arch-imx9/gpio.h > b/arch/arm/include/asm/arch-imx9/gpio.h > index 40732022e7..599f7511c3 100644 > --- a/arch/arm/include/asm/arch-imx9/gpio.h > +++ b/arch/arm/include/asm/arch-imx9/gpio.h > @@ -6,6 +6,8 @@ > #ifndef __ASM_ARCH_IMX9_GPIO_H > #define __ASM_ARCH_IMX9_GPIO_H > > +#include <common.h> > + > struct gpio_regs { > u32 gpio_pdor; > u32 gpio_psor; > diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig index > c949f9d2f7..550ff9c586 100644 > --- a/drivers/gpio/Kconfig > +++ b/drivers/gpio/Kconfig > @@ -605,4 +605,10 @@ config TURRIS_OMNIA_MCU > help > Support for GPIOs on MCU connected to Turris Omnia via i2c. > > +config ADP5585_GPIO > + bool "ADP5585 GPIO driver" > + depends on DM_GPIO && DM_I2C > + help > + Support ADP5585 GPIO expander on i.MX platform. > + > endif > diff --git a/drivers/gpio/Makefile b/drivers/gpio/Makefile index > 9d718a554e..2f60b98384 100644 > --- a/drivers/gpio/Makefile > +++ b/drivers/gpio/Makefile > @@ -75,3 +75,4 @@ obj-$(CONFIG_SL28CPLD_GPIO) += sl28cpld-gpio.o > obj-$(CONFIG_ZYNQMP_GPIO_MODEPIN) += zynqmp_gpio_modepin.o > obj-$(CONFIG_SLG7XL45106_I2C_GPO) += gpio_slg7xl45106.o > obj-$(CONFIG_$(SPL_TPL_)TURRIS_OMNIA_MCU) += turris_omnia_mcu.o > +obj-$(CONFIG_ADP5585_GPIO) += adp5585_gpio.o > diff --git a/drivers/gpio/adp5585_gpio.c b/drivers/gpio/adp5585_gpio.c new > file mode 100644 index 0000000000..ea0cb75459 > --- /dev/null > +++ b/drivers/gpio/adp5585_gpio.c > @@ -0,0 +1,238 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * Copyright 2022 NXP > + * > + * ADP5585 I/O Expander Controller > + * > + * Author: Alice Guo <alice.guo@nxp.com> */ > + > +#include <asm/gpio.h> > +#include <dm.h> > +#include <dt-bindings/gpio/gpio.h> > +#include <i2c.h> > + > +#define ADP5585_ID 0x00 > +#define ADP5585_INT_STATUS 0x01 > +#define ADP5585_STATUS 0x02 > +#define ADP5585_FIFO_1 0x03 > +#define ADP5585_FIFO_2 0x04 > +#define ADP5585_FIFO_3 0x05 > +#define ADP5585_FIFO_4 0x06 > +#define ADP5585_FIFO_5 0x07 > +#define ADP5585_FIFO_6 0x08 > +#define ADP5585_FIFO_7 0x09 > +#define ADP5585_FIFO_8 0x0A > +#define ADP5585_FIFO_9 0x0B > +#define ADP5585_FIFO_10 0x0C > +#define ADP5585_FIFO_11 0x0D > +#define ADP5585_FIFO_12 0x0E > +#define ADP5585_FIFO_13 0x0F > +#define ADP5585_FIFO_14 0x10 > +#define ADP5585_FIFO_15 0x11 > +#define ADP5585_FIFO_16 0x12 > +#define ADP5585_GPI_INT_STAT_A 0x13 > +#define ADP5585_GPI_INT_STAT_B 0x14 > +#define ADP5585_GPI_STATUS_A 0x15 > +#define ADP5585_GPI_STATUS_B 0x16 > +#define ADP5585_RPULL_CONFIG_A 0x17 > +#define ADP5585_RPULL_CONFIG_B 0x18 > +#define ADP5585_RPULL_CONFIG_C 0x19 > +#define ADP5585_RPULL_CONFIG_D 0x1A > +#define ADP5585_GPI_INT_LEVEL_A 0x1B > +#define ADP5585_GPI_INT_LEVEL_B 0x1C > +#define ADP5585_GPI_EVENT_EN_A 0x1D > +#define ADP5585_GPI_EVENT_EN_B 0x1E > +#define ADP5585_GPI_INTERRUPT_EN_A 0x1F > +#define ADP5585_GPI_INTERRUPT_EN_B 0x20 > +#define ADP5585_DEBOUNCE_DIS_A 0x21 > +#define ADP5585_DEBOUNCE_DIS_B 0x22 > +#define ADP5585_GPO_DATA_OUT_A 0x23 > +#define ADP5585_GPO_DATA_OUT_B 0x24 > +#define ADP5585_GPO_OUT_MODE_A 0x25 > +#define ADP5585_GPO_OUT_MODE_B 0x26 > +#define ADP5585_GPIO_DIRECTION_A 0x27 > +#define ADP5585_GPIO_DIRECTION_B 0x28 > +#define ADP5585_RESET1_EVENT_A 0x29 > +#define ADP5585_RESET1_EVENT_B 0x2A > +#define ADP5585_RESET1_EVENT_C 0x2B > +#define ADP5585_RESET2_EVENT_A 0x2C > +#define ADP5585_RESET2_EVENT_B 0x2D > +#define ADP5585_RESET_CFG 0x2E > +#define ADP5585_PWM_OFFT_LOW 0x2F > +#define ADP5585_PWM_OFFT_HIGH 0x30 > +#define ADP5585_PWM_ONT_LOW 0x31 > +#define ADP5585_PWM_ONT_HIGH 0x32 > +#define ADP5585_PWM_CFG 0x33 > +#define ADP5585_LOGIC_CFG 0x34 > +#define ADP5585_LOGIC_FF_CFG 0x35 > +#define ADP5585_LOGIC_INT_EVENT_EN 0x36 > +#define ADP5585_POLL_PTIME_CFG 0x37 > +#define ADP5585_PIN_CONFIG_A 0x38 > +#define ADP5585_PIN_CONFIG_B 0x39 > +#define ADP5585_PIN_CONFIG_D 0x3A > +#define ADP5585_GENERAL_CFG 0x3B > +#define ADP5585_INT_EN 0x3C > + > +#define ADP5585_MAXGPIO 10 > +#define ADP5585_BANK(offs) ((offs) > 4) > +#define ADP5585_BIT(offs) ((offs) > 4 ? \ > + 1u << ((offs) - 5) : 1u << (offs)) > + > +struct adp5585_plat { > + fdt_addr_t addr; > + u8 id; > + u8 dat_out[2]; > + u8 dir[2]; > +}; > + > +static int adp5585_direction_input(struct udevice *dev, unsigned int > +offset) { > + int ret; > + unsigned int bank; > + struct adp5585_plat *plat = dev_get_plat(dev); > + > + bank = ADP5585_BANK(offset); > + > + plat->dir[bank] &= ~ADP5585_BIT(offset); > + ret = dm_i2c_write(dev, ADP5585_GPIO_DIRECTION_A + bank, > +&plat->dir[bank], 1); > + > + return ret; > +} > + > +static int adp5585_direction_output(struct udevice *dev, unsigned int offset, > + int value) > +{ > + int ret; > + unsigned int bank, bit; > + struct adp5585_plat *plat = dev_get_plat(dev); > + > + bank = ADP5585_BANK(offset); > + bit = ADP5585_BIT(offset); > + > + plat->dir[bank] |= bit; > + > + if (value) > + plat->dat_out[bank] |= bit; > + else > + plat->dat_out[bank] &= ~bit; > + > + ret = dm_i2c_write(dev, ADP5585_GPO_DATA_OUT_A + bank, > &plat->dat_out[bank], 1); > + ret |= dm_i2c_write(dev, ADP5585_GPIO_DIRECTION_A + bank, > +&plat->dir[bank], 1); > + > + return ret; > +} > + > +static int adp5585_get_value(struct udevice *dev, unsigned int offset) > +{ > + struct adp5585_plat *plat = dev_get_plat(dev); > + unsigned int bank = ADP5585_BANK(offset); > + unsigned int bit = ADP5585_BIT(offset); > + u8 val; > + > + if (plat->dir[bank] & bit) > + val = plat->dat_out[bank]; > + else > + dm_i2c_read(dev, ADP5585_GPI_STATUS_A + bank, &val, 1); > + > + return !!(val & bit); > +} > + > +static int adp5585_set_value(struct udevice *dev, unsigned int offset, > +int value) { > + int ret; > + unsigned int bank, bit; > + struct adp5585_plat *plat = dev_get_plat(dev); > + > + bank = ADP5585_BANK(offset); > + bit = ADP5585_BIT(offset); > + > + if (value) > + plat->dat_out[bank] |= bit; > + else > + plat->dat_out[bank] &= ~bit; > + > + ret = dm_i2c_write(dev, ADP5585_GPO_DATA_OUT_A + bank, > +&plat->dat_out[bank], 1); > + > + return ret; > +} > + > +static int adp5585_get_function(struct udevice *dev, unsigned int > +offset) { > + unsigned int bank, bit, dir; > + struct adp5585_plat *plat = dev_get_plat(dev); > + > + bank = ADP5585_BANK(offset); > + bit = ADP5585_BIT(offset); > + dir = plat->dir[bank] & bit; > + > + if (!dir) > + return GPIOF_INPUT; > + else > + return GPIOF_OUTPUT; > +} > + > +static int adp5585_xlate(struct udevice *dev, struct gpio_desc *desc, > + struct ofnode_phandle_args *args) > +{ > + desc->offset = args->args[0]; > + desc->flags = args->args[1] & GPIO_ACTIVE_LOW ? GPIOD_ACTIVE_LOW : > 0; > + > + return 0; > +} > + > +static const struct dm_gpio_ops adp5585_ops = { > + .direction_input = adp5585_direction_input, > + .direction_output = adp5585_direction_output, > + .get_value = adp5585_get_value, > + .set_value = adp5585_set_value, > + .get_function = adp5585_get_function, > + .xlate = adp5585_xlate, > +}; > + > +static int adp5585_probe(struct udevice *dev) { > + struct adp5585_plat *plat = dev_get_plat(dev); > + struct gpio_dev_priv *uc_priv = dev_get_uclass_priv(dev); > + int ret; > + > + if (!plat) > + return 0; > + > + plat->addr = dev_read_addr(dev); > + if (plat->addr == FDT_ADDR_T_NONE) > + return -EINVAL; > + > + ret = dm_i2c_read(dev, ADP5585_ID, &plat->id, 1); > + if (ret < 0) > + return ret; > + > + uc_priv->gpio_count = ADP5585_MAXGPIO; > + uc_priv->bank_name = "adp5585-gpio"; > + > + for (int i = 0; i < 2; i++) { > + ret = dm_i2c_read(dev, ADP5585_GPO_DATA_OUT_A + i, > &plat->dat_out[i], 1); > + if (ret) > + return ret; > + > + ret = dm_i2c_read(dev, ADP5585_GPIO_DIRECTION_A + i, > &plat->dir[i], 1); > + if (ret) > + return ret; > + } > + > + return 0; > +} > + > +static const struct udevice_id adp5585_ids[] = { > + { .compatible = "adp5585" }, > + { } > +}; > + > +U_BOOT_DRIVER(adp5585) = { > + .name = "adp5585", > + .id = UCLASS_GPIO, > + .of_match = adp5585_ids, > + .probe = adp5585_probe, > + .ops = &adp5585_ops, > + .plat_auto = sizeof(struct adp5585_plat), > +}; > -- > 2.17.1
Hi Alice, On Tue, Sep 6, 2022 at 6:37 AM Alice Guo (OSS) <alice.guo@oss.nxp.com> wrote: > > From: Alice Guo <alice.guo@nxp.com> Please add a commit log. > Signed-off-by: Alice Guo <alice.guo@nxp.com> > --- a/arch/arm/include/asm/arch-imx9/gpio.h > +++ b/arch/arm/include/asm/arch-imx9/gpio.h > @@ -6,6 +6,8 @@ > #ifndef __ASM_ARCH_IMX9_GPIO_H > #define __ASM_ARCH_IMX9_GPIO_H > > +#include <common.h> This looks like an unrelated change. > +config ADP5585_GPIO > + bool "ADP5585 GPIO driver" > + depends on DM_GPIO && DM_I2C > + help > + Support ADP5585 GPIO expander on i.MX platform. Please remove "on i.MX platform" as the GPIO expander can be used on any platform.
> -----Original Message----- > From: Fabio Estevam <festevam@gmail.com> > Sent: Tuesday, September 27, 2022 8:16 PM > To: Alice Guo (OSS) <alice.guo@oss.nxp.com> > Cc: sbabic@denx.de; sjg@chromium.org; dl-uboot-imx <uboot-imx@nxp.com>; > u-boot@lists.denx.de > Subject: Re: [PATCH v1] gpio: adp5585: add gpio driver for ADP5585 I/O > Expander Controller > > Hi Alice, > > On Tue, Sep 6, 2022 at 6:37 AM Alice Guo (OSS) <alice.guo@oss.nxp.com> > wrote: > > > > From: Alice Guo <alice.guo@nxp.com> > > Please add a commit log. > > > Signed-off-by: Alice Guo <alice.guo@nxp.com> > > > --- a/arch/arm/include/asm/arch-imx9/gpio.h > > +++ b/arch/arm/include/asm/arch-imx9/gpio.h > > @@ -6,6 +6,8 @@ > > #ifndef __ASM_ARCH_IMX9_GPIO_H > > #define __ASM_ARCH_IMX9_GPIO_H > > > > +#include <common.h> > > This looks like an unrelated change. Hi Fabio, Thanks for your reply. When compiling without "#include <common.h>", the following error will appear: In file included from ./arch/arm/include/asm/gpio.h:2, from drivers/gpio/adp5585_gpio.c:10: ./arch/arm/include/asm/arch/gpio.h:12:9: error: unknown type name ‘u32’ 12 | u32 gpio_pdor; | ^~~ ./arch/arm/include/asm/arch/gpio.h:13:9: error: unknown type name ‘u32’ 13 | u32 gpio_psor; | ^~~ ./arch/arm/include/asm/arch/gpio.h:14:9: error: unknown type name ‘u32’ 14 | u32 gpio_pcor; | ^~~ ./arch/arm/include/asm/arch/gpio.h:15:9: error: unknown type name ‘u32’ 15 | u32 gpio_ptor; | ^~~ ./arch/arm/include/asm/arch/gpio.h:16:9: error: unknown type name ‘u32’ 16 | u32 gpio_pdir; | ^~~ ./arch/arm/include/asm/arch/gpio.h:17:9: error: unknown type name ‘u32’ 17 | u32 gpio_pddr; | ^~~ ./arch/arm/include/asm/arch/gpio.h:18:9: error: unknown type name ‘u32’ 18 | u32 gpio_pidr; | ^~~ ./arch/arm/include/asm/arch/gpio.h:19:9: error: unknown type name ‘u8’ 19 | u8 gpio_pxdr[32]; Best Regards, Alice Guo > > +config ADP5585_GPIO > > + bool "ADP5585 GPIO driver" > > + depends on DM_GPIO && DM_I2C > > + help > > + Support ADP5585 GPIO expander on i.MX platform. > > Please remove "on i.MX platform" as the GPIO expander can be used on any > platform.
Hi Alice, On Wed, Sep 28, 2022 at 4:52 AM Alice Guo (OSS) <alice.guo@oss.nxp.com> wrote: > Hi Fabio, > > Thanks for your reply. When compiling without "#include <common.h>", the following error will appear: > In file included from ./arch/arm/include/asm/gpio.h:2, > from drivers/gpio/adp5585_gpio.c:10: > ./arch/arm/include/asm/arch/gpio.h:12:9: error: unknown type name ‘u32’ > 12 | u32 gpio_pdor; Whatever change you need to make in arch/arm/include/asm/arch-imx9/gpio.h should be addressed by a separate patch. The patch that adds support for a new GPIO expander chip should not touch any i.MX-specific header.
diff --git a/arch/arm/include/asm/arch-imx9/gpio.h b/arch/arm/include/asm/arch-imx9/gpio.h index 40732022e7..599f7511c3 100644 --- a/arch/arm/include/asm/arch-imx9/gpio.h +++ b/arch/arm/include/asm/arch-imx9/gpio.h @@ -6,6 +6,8 @@ #ifndef __ASM_ARCH_IMX9_GPIO_H #define __ASM_ARCH_IMX9_GPIO_H +#include <common.h> + struct gpio_regs { u32 gpio_pdor; u32 gpio_psor; diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig index c949f9d2f7..550ff9c586 100644 --- a/drivers/gpio/Kconfig +++ b/drivers/gpio/Kconfig @@ -605,4 +605,10 @@ config TURRIS_OMNIA_MCU help Support for GPIOs on MCU connected to Turris Omnia via i2c. +config ADP5585_GPIO + bool "ADP5585 GPIO driver" + depends on DM_GPIO && DM_I2C + help + Support ADP5585 GPIO expander on i.MX platform. + endif diff --git a/drivers/gpio/Makefile b/drivers/gpio/Makefile index 9d718a554e..2f60b98384 100644 --- a/drivers/gpio/Makefile +++ b/drivers/gpio/Makefile @@ -75,3 +75,4 @@ obj-$(CONFIG_SL28CPLD_GPIO) += sl28cpld-gpio.o obj-$(CONFIG_ZYNQMP_GPIO_MODEPIN) += zynqmp_gpio_modepin.o obj-$(CONFIG_SLG7XL45106_I2C_GPO) += gpio_slg7xl45106.o obj-$(CONFIG_$(SPL_TPL_)TURRIS_OMNIA_MCU) += turris_omnia_mcu.o +obj-$(CONFIG_ADP5585_GPIO) += adp5585_gpio.o diff --git a/drivers/gpio/adp5585_gpio.c b/drivers/gpio/adp5585_gpio.c new file mode 100644 index 0000000000..ea0cb75459 --- /dev/null +++ b/drivers/gpio/adp5585_gpio.c @@ -0,0 +1,238 @@ +// SPDX-License-Identifier: GPL-2.0+ +/* + * Copyright 2022 NXP + * + * ADP5585 I/O Expander Controller + * + * Author: Alice Guo <alice.guo@nxp.com> + */ + +#include <asm/gpio.h> +#include <dm.h> +#include <dt-bindings/gpio/gpio.h> +#include <i2c.h> + +#define ADP5585_ID 0x00 +#define ADP5585_INT_STATUS 0x01 +#define ADP5585_STATUS 0x02 +#define ADP5585_FIFO_1 0x03 +#define ADP5585_FIFO_2 0x04 +#define ADP5585_FIFO_3 0x05 +#define ADP5585_FIFO_4 0x06 +#define ADP5585_FIFO_5 0x07 +#define ADP5585_FIFO_6 0x08 +#define ADP5585_FIFO_7 0x09 +#define ADP5585_FIFO_8 0x0A +#define ADP5585_FIFO_9 0x0B +#define ADP5585_FIFO_10 0x0C +#define ADP5585_FIFO_11 0x0D +#define ADP5585_FIFO_12 0x0E +#define ADP5585_FIFO_13 0x0F +#define ADP5585_FIFO_14 0x10 +#define ADP5585_FIFO_15 0x11 +#define ADP5585_FIFO_16 0x12 +#define ADP5585_GPI_INT_STAT_A 0x13 +#define ADP5585_GPI_INT_STAT_B 0x14 +#define ADP5585_GPI_STATUS_A 0x15 +#define ADP5585_GPI_STATUS_B 0x16 +#define ADP5585_RPULL_CONFIG_A 0x17 +#define ADP5585_RPULL_CONFIG_B 0x18 +#define ADP5585_RPULL_CONFIG_C 0x19 +#define ADP5585_RPULL_CONFIG_D 0x1A +#define ADP5585_GPI_INT_LEVEL_A 0x1B +#define ADP5585_GPI_INT_LEVEL_B 0x1C +#define ADP5585_GPI_EVENT_EN_A 0x1D +#define ADP5585_GPI_EVENT_EN_B 0x1E +#define ADP5585_GPI_INTERRUPT_EN_A 0x1F +#define ADP5585_GPI_INTERRUPT_EN_B 0x20 +#define ADP5585_DEBOUNCE_DIS_A 0x21 +#define ADP5585_DEBOUNCE_DIS_B 0x22 +#define ADP5585_GPO_DATA_OUT_A 0x23 +#define ADP5585_GPO_DATA_OUT_B 0x24 +#define ADP5585_GPO_OUT_MODE_A 0x25 +#define ADP5585_GPO_OUT_MODE_B 0x26 +#define ADP5585_GPIO_DIRECTION_A 0x27 +#define ADP5585_GPIO_DIRECTION_B 0x28 +#define ADP5585_RESET1_EVENT_A 0x29 +#define ADP5585_RESET1_EVENT_B 0x2A +#define ADP5585_RESET1_EVENT_C 0x2B +#define ADP5585_RESET2_EVENT_A 0x2C +#define ADP5585_RESET2_EVENT_B 0x2D +#define ADP5585_RESET_CFG 0x2E +#define ADP5585_PWM_OFFT_LOW 0x2F +#define ADP5585_PWM_OFFT_HIGH 0x30 +#define ADP5585_PWM_ONT_LOW 0x31 +#define ADP5585_PWM_ONT_HIGH 0x32 +#define ADP5585_PWM_CFG 0x33 +#define ADP5585_LOGIC_CFG 0x34 +#define ADP5585_LOGIC_FF_CFG 0x35 +#define ADP5585_LOGIC_INT_EVENT_EN 0x36 +#define ADP5585_POLL_PTIME_CFG 0x37 +#define ADP5585_PIN_CONFIG_A 0x38 +#define ADP5585_PIN_CONFIG_B 0x39 +#define ADP5585_PIN_CONFIG_D 0x3A +#define ADP5585_GENERAL_CFG 0x3B +#define ADP5585_INT_EN 0x3C + +#define ADP5585_MAXGPIO 10 +#define ADP5585_BANK(offs) ((offs) > 4) +#define ADP5585_BIT(offs) ((offs) > 4 ? \ + 1u << ((offs) - 5) : 1u << (offs)) + +struct adp5585_plat { + fdt_addr_t addr; + u8 id; + u8 dat_out[2]; + u8 dir[2]; +}; + +static int adp5585_direction_input(struct udevice *dev, unsigned int offset) +{ + int ret; + unsigned int bank; + struct adp5585_plat *plat = dev_get_plat(dev); + + bank = ADP5585_BANK(offset); + + plat->dir[bank] &= ~ADP5585_BIT(offset); + ret = dm_i2c_write(dev, ADP5585_GPIO_DIRECTION_A + bank, &plat->dir[bank], 1); + + return ret; +} + +static int adp5585_direction_output(struct udevice *dev, unsigned int offset, + int value) +{ + int ret; + unsigned int bank, bit; + struct adp5585_plat *plat = dev_get_plat(dev); + + bank = ADP5585_BANK(offset); + bit = ADP5585_BIT(offset); + + plat->dir[bank] |= bit; + + if (value) + plat->dat_out[bank] |= bit; + else + plat->dat_out[bank] &= ~bit; + + ret = dm_i2c_write(dev, ADP5585_GPO_DATA_OUT_A + bank, &plat->dat_out[bank], 1); + ret |= dm_i2c_write(dev, ADP5585_GPIO_DIRECTION_A + bank, &plat->dir[bank], 1); + + return ret; +} + +static int adp5585_get_value(struct udevice *dev, unsigned int offset) +{ + struct adp5585_plat *plat = dev_get_plat(dev); + unsigned int bank = ADP5585_BANK(offset); + unsigned int bit = ADP5585_BIT(offset); + u8 val; + + if (plat->dir[bank] & bit) + val = plat->dat_out[bank]; + else + dm_i2c_read(dev, ADP5585_GPI_STATUS_A + bank, &val, 1); + + return !!(val & bit); +} + +static int adp5585_set_value(struct udevice *dev, unsigned int offset, int value) +{ + int ret; + unsigned int bank, bit; + struct adp5585_plat *plat = dev_get_plat(dev); + + bank = ADP5585_BANK(offset); + bit = ADP5585_BIT(offset); + + if (value) + plat->dat_out[bank] |= bit; + else + plat->dat_out[bank] &= ~bit; + + ret = dm_i2c_write(dev, ADP5585_GPO_DATA_OUT_A + bank, &plat->dat_out[bank], 1); + + return ret; +} + +static int adp5585_get_function(struct udevice *dev, unsigned int offset) +{ + unsigned int bank, bit, dir; + struct adp5585_plat *plat = dev_get_plat(dev); + + bank = ADP5585_BANK(offset); + bit = ADP5585_BIT(offset); + dir = plat->dir[bank] & bit; + + if (!dir) + return GPIOF_INPUT; + else + return GPIOF_OUTPUT; +} + +static int adp5585_xlate(struct udevice *dev, struct gpio_desc *desc, + struct ofnode_phandle_args *args) +{ + desc->offset = args->args[0]; + desc->flags = args->args[1] & GPIO_ACTIVE_LOW ? GPIOD_ACTIVE_LOW : 0; + + return 0; +} + +static const struct dm_gpio_ops adp5585_ops = { + .direction_input = adp5585_direction_input, + .direction_output = adp5585_direction_output, + .get_value = adp5585_get_value, + .set_value = adp5585_set_value, + .get_function = adp5585_get_function, + .xlate = adp5585_xlate, +}; + +static int adp5585_probe(struct udevice *dev) +{ + struct adp5585_plat *plat = dev_get_plat(dev); + struct gpio_dev_priv *uc_priv = dev_get_uclass_priv(dev); + int ret; + + if (!plat) + return 0; + + plat->addr = dev_read_addr(dev); + if (plat->addr == FDT_ADDR_T_NONE) + return -EINVAL; + + ret = dm_i2c_read(dev, ADP5585_ID, &plat->id, 1); + if (ret < 0) + return ret; + + uc_priv->gpio_count = ADP5585_MAXGPIO; + uc_priv->bank_name = "adp5585-gpio"; + + for (int i = 0; i < 2; i++) { + ret = dm_i2c_read(dev, ADP5585_GPO_DATA_OUT_A + i, &plat->dat_out[i], 1); + if (ret) + return ret; + + ret = dm_i2c_read(dev, ADP5585_GPIO_DIRECTION_A + i, &plat->dir[i], 1); + if (ret) + return ret; + } + + return 0; +} + +static const struct udevice_id adp5585_ids[] = { + { .compatible = "adp5585" }, + { } +}; + +U_BOOT_DRIVER(adp5585) = { + .name = "adp5585", + .id = UCLASS_GPIO, + .of_match = adp5585_ids, + .probe = adp5585_probe, + .ops = &adp5585_ops, + .plat_auto = sizeof(struct adp5585_plat), +};