gpio: axp: Remove virtual VBUS enable GPIO

Now that this functionality is modeled using the device tree and
regulator uclass, the named GPIO is not referenced anywhere. Remove
it, along with the rest of the support for AXP virtual GPIOs.

Signed-off-by: Samuel Holland <samuel@sholland.org>
Signed-off-by: Andre Przywara <andre.przywara@arm.com>
Reviewed-by: Jernej Skrabec <jernej.skrabec@gmail.com>

diff --git a/drivers/gpio/axp_gpio.c b/drivers/gpio/axp_gpio.c
index 6e632c8fc7308da348adeedccd347e880410253d..181c53bfe7221989de9d89609d27ce3c3f56a070 100644 (file)
--- a/drivers/gpio/axp_gpio.c
+++ b/drivers/gpio/axp_gpio.c
@@ -15,6 +15,9 @@
 #include <errno.h>
 #include <sunxi_gpio.h>
 
+#define AXP_GPIO_PREFIX                        "AXP0-"
+#define AXP_GPIO_COUNT                 4
+
 static int axp_gpio_set_value(struct udevice *dev, unsigned pin, int val);
 
 static u8 axp_get_gpio_ctrl_reg(unsigned pin)
@@ -46,28 +49,14 @@ static int axp_gpio_direction_input(struct udevice *dev, unsigned pin)
 static int axp_gpio_direction_output(struct udevice *dev, unsigned pin,
                                     int val)
 {
-       __maybe_unused int ret;
        u8 reg;
 
-       switch (pin) {
-#ifdef AXP_MISC_CTRL_N_VBUSEN_FUNC
-       /* Only available on later PMICs */
-       case SUNXI_GPIO_AXP0_VBUS_ENABLE:
-               ret = pmic_bus_clrbits(AXP_MISC_CTRL,
-                                      AXP_MISC_CTRL_N_VBUSEN_FUNC);
-               if (ret)
-                       return ret;
-
-               return axp_gpio_set_value(dev, pin, val);
-#endif
-       default:
-               reg = axp_get_gpio_ctrl_reg(pin);
-               if (reg == 0)
-                       return -EINVAL;
+       reg = axp_get_gpio_ctrl_reg(pin);
+       if (reg == 0)
+               return -EINVAL;
 
-               return pmic_bus_write(reg, val ? AXP_GPIO_CTRL_OUTPUT_HIGH :
-                                                AXP_GPIO_CTRL_OUTPUT_LOW);
-       }
+       return pmic_bus_write(reg, val ? AXP_GPIO_CTRL_OUTPUT_HIGH :
+                                        AXP_GPIO_CTRL_OUTPUT_LOW);
 }
 
 static int axp_gpio_get_value(struct udevice *dev, unsigned pin)
@@ -75,25 +64,16 @@ static int axp_gpio_get_value(struct udevice *dev, unsigned pin)
        u8 reg, val, mask;
        int ret;
 
-       switch (pin) {
-#ifdef AXP_MISC_CTRL_N_VBUSEN_FUNC
-       /* Only available on later PMICs */
-       case SUNXI_GPIO_AXP0_VBUS_ENABLE:
-               ret = pmic_bus_read(AXP_VBUS_IPSOUT, &val);
-               mask = AXP_VBUS_IPSOUT_DRIVEBUS;
-               break;
-#endif
-       default:
-               reg = axp_get_gpio_ctrl_reg(pin);
-               if (reg == 0)
-                       return -EINVAL;
+       reg = axp_get_gpio_ctrl_reg(pin);
+       if (reg == 0)
+               return -EINVAL;
 
-               ret = pmic_bus_read(AXP_GPIO_STATE, &val);
-               mask = 1 << (pin + AXP_GPIO_STATE_OFFSET);
-       }
+       ret = pmic_bus_read(AXP_GPIO_STATE, &val);
        if (ret)
                return ret;
 
+       mask = 1 << (pin + AXP_GPIO_STATE_OFFSET);
+
        return (val & mask) ? 1 : 0;
 }
 
@@ -101,25 +81,12 @@ static int axp_gpio_set_value(struct udevice *dev, unsigned pin, int val)
 {
        u8 reg;
 
-       switch (pin) {
-#ifdef AXP_MISC_CTRL_N_VBUSEN_FUNC
-       /* Only available on later PMICs */
-       case SUNXI_GPIO_AXP0_VBUS_ENABLE:
-               if (val)
-                       return pmic_bus_setbits(AXP_VBUS_IPSOUT,
-                                               AXP_VBUS_IPSOUT_DRIVEBUS);
-               else
-                       return pmic_bus_clrbits(AXP_VBUS_IPSOUT,
-                                               AXP_VBUS_IPSOUT_DRIVEBUS);
-#endif
-       default:
-               reg = axp_get_gpio_ctrl_reg(pin);
-               if (reg == 0)
-                       return -EINVAL;
+       reg = axp_get_gpio_ctrl_reg(pin);
+       if (reg == 0)
+               return -EINVAL;
 
-               return pmic_bus_write(reg, val ? AXP_GPIO_CTRL_OUTPUT_HIGH :
-                                                AXP_GPIO_CTRL_OUTPUT_LOW);
-       }
+       return pmic_bus_write(reg, val ? AXP_GPIO_CTRL_OUTPUT_HIGH :
+                                        AXP_GPIO_CTRL_OUTPUT_LOW);
 }
 
 static const struct dm_gpio_ops gpio_axp_ops = {
@@ -134,8 +101,8 @@ static int gpio_axp_probe(struct udevice *dev)
        struct gpio_dev_priv *uc_priv = dev_get_uclass_priv(dev);
 
        /* Tell the uclass how many GPIOs we have */
-       uc_priv->bank_name = strdup(SUNXI_GPIO_AXP0_PREFIX);
-       uc_priv->gpio_count = SUNXI_GPIO_AXP0_GPIO_COUNT;
+       uc_priv->bank_name = AXP_GPIO_PREFIX;
+       uc_priv->gpio_count = AXP_GPIO_COUNT;
 
        return 0;
 }

diff --git a/drivers/gpio/sunxi_gpio.c b/drivers/gpio/sunxi_gpio.c
index 2ca4960f17ac406988db35b68481534b436eb161..094c45a69273ff7baca603aca51e249ebeb21cf9 100644 (file)
--- a/drivers/gpio/sunxi_gpio.c
+++ b/drivers/gpio/sunxi_gpio.c
@@ -244,17 +244,7 @@ int sunxi_name_to_gpio(const char *name)
 int sunxi_name_to_gpio(const char *name)
 {
        unsigned int gpio;
-       int ret;
-#if !defined CONFIG_XPL_BUILD && defined CONFIG_AXP_GPIO
-       char lookup[8];
-
-       if (strcasecmp(name, "AXP0-VBUS-ENABLE") == 0) {
-               sprintf(lookup, SUNXI_GPIO_AXP0_PREFIX "%d",
-                       SUNXI_GPIO_AXP0_VBUS_ENABLE);
-               name = lookup;
-       }
-#endif
-       ret = gpio_lookup_name(name, NULL, NULL, &gpio);
+       int ret = gpio_lookup_name(name, NULL, NULL, &gpio);
 
        return ret ? ret : gpio;
 }

diff --git a/include/axp221.h b/include/axp221.h
index 32b988f3a9cd775ee265543c2b4cd30b70a553b9..8a4a3cca82f735b7b3f7e8277a8212d5b16789d7 100644 (file)
--- a/include/axp221.h
+++ b/include/axp221.h
@@ -53,10 +53,6 @@
 #ifdef CONFIG_AXP221_POWER
 #define AXP_POWER_STATUS               0x00
 #define AXP_POWER_STATUS_ALDO_IN               BIT(0)
-#define AXP_VBUS_IPSOUT                        0x30
-#define AXP_VBUS_IPSOUT_DRIVEBUS               (1 << 2)
-#define AXP_MISC_CTRL                  0x8f
-#define AXP_MISC_CTRL_N_VBUSEN_FUNC            (1 << 4)
 #define AXP_GPIO0_CTRL                 0x90
 #define AXP_GPIO1_CTRL                 0x92
 #define AXP_GPIO_CTRL_OUTPUT_LOW               0x00 /* Drive pin low */

diff --git a/include/axp809.h b/include/axp809.h
index 71a7cb2aaa12559fdb75247cbade192eecd25568..3bd71b3d1a31c9d9eff1725ff4cce43d5f9cec23 100644 (file)
--- a/include/axp809.h
+++ b/include/axp809.h
@@ -47,10 +47,6 @@
 #ifdef CONFIG_AXP809_POWER
 #define AXP_POWER_STATUS               0x00
 #define AXP_POWER_STATUS_ALDO_IN               BIT(0)
-#define AXP_VBUS_IPSOUT                        0x30
-#define AXP_VBUS_IPSOUT_DRIVEBUS               (1 << 2)
-#define AXP_MISC_CTRL                  0x8f
-#define AXP_MISC_CTRL_N_VBUSEN_FUNC            (1 << 4)
 #define AXP_GPIO0_CTRL                 0x90
 #define AXP_GPIO1_CTRL                 0x92
 #define AXP_GPIO_CTRL_OUTPUT_LOW       0x00 /* Drive pin low */

diff --git a/include/axp818.h b/include/axp818.h
index 08ac35d15fa7a42b89428e87ae5b7540123c7a89..b3a9686e0e59b771a7216a9442199f5f50adc393 100644 (file)
--- a/include/axp818.h
+++ b/include/axp818.h
@@ -61,10 +61,6 @@
 #ifdef CONFIG_AXP818_POWER
 #define AXP_POWER_STATUS               0x00
 #define AXP_POWER_STATUS_ALDO_IN               BIT(0)
-#define AXP_VBUS_IPSOUT                        0x30
-#define AXP_VBUS_IPSOUT_DRIVEBUS               (1 << 2)
-#define AXP_MISC_CTRL                  0x8f
-#define AXP_MISC_CTRL_N_VBUSEN_FUNC            (1 << 4)
 #define AXP_GPIO0_CTRL                 0x90
 #define AXP_GPIO1_CTRL                 0x92
 #define AXP_GPIO_CTRL_OUTPUT_LOW       0x00 /* Drive pin low */

diff --git a/include/sunxi_gpio.h b/include/sunxi_gpio.h
index 122987c395e008e8d2e6595e92317f7baa77deb5..12b54c8dda4de3b292801bc23a241b9fc703bb00 100644 (file)
--- a/include/sunxi_gpio.h
+++ b/include/sunxi_gpio.h
@@ -82,7 +82,6 @@ enum sunxi_gpio_number {
        SUNXI_GPIO_L_START = 352,
        SUNXI_GPIO_M_START = SUNXI_GPIO_NEXT(SUNXI_GPIO_L),
        SUNXI_GPIO_N_START = SUNXI_GPIO_NEXT(SUNXI_GPIO_M),
-       SUNXI_GPIO_AXP0_START = 1024,
 };
 
 /* SUNXI GPIO number definitions */
@@ -99,8 +98,6 @@ enum sunxi_gpio_number {
 #define SUNXI_GPM(_nr) (SUNXI_GPIO_M_START + (_nr))
 #define SUNXI_GPN(_nr) (SUNXI_GPIO_N_START + (_nr))
 
-#define SUNXI_GPAXP0(_nr)      (SUNXI_GPIO_AXP0_START + (_nr))
-
 /* GPIO pin function config */
 #define SUNXI_GPIO_INPUT       0
 #define SUNXI_GPIO_OUTPUT      1
@@ -185,11 +182,6 @@ enum sunxi_gpio_number {
 #define SUNXI_GPIO_PULL_UP     1
 #define SUNXI_GPIO_PULL_DOWN   2
 
-/* Virtual AXP0 GPIOs */
-#define SUNXI_GPIO_AXP0_PREFIX "AXP0-"
-#define SUNXI_GPIO_AXP0_VBUS_ENABLE    5
-#define SUNXI_GPIO_AXP0_GPIO_COUNT     6
-
 struct sunxi_gpio_plat {
        void                    *regs;
        char                    bank_name[3];