phy: sun4i-usb: Determine USB OTG detection pin from devicetree
authorAndre Przywara <andre.przywara@arm.com>
Sun, 30 Mar 2025 16:13:50 +0000 (17:13 +0100)
committerTom Rini <trini@konsulko.com>
Mon, 28 Apr 2025 18:45:44 +0000 (12:45 -0600)
So far Allwinner boards controlled the USB OTG ID detection via the
respective GPIO pin specified in Kconfig, as a string. All boards should
have the same GPIO already specified in the devicetree, in the
usb0_id_det-gpios property.

Convert the usage of the Kconfig configured GPIO over to query that
information from the devicetree, then use the existing DM GPIO
infrastructure to request the GPIO.
Only PHY0 supports USB-OTG, so limit the GPIO request to that PHY, to
avoid claiming it multiple times.

This removes the need to name that GPIO in the defconfig file.

Signed-off-by: Andre Przywara <andre.przywara@arm.com>
Reviewed-by: Jernej Skrabec <jernej.skrabec@gmail.com>
drivers/phy/allwinner/phy-sun4i-usb.c

index 42e67b4..3acffb4 100644 (file)
@@ -85,23 +85,6 @@ struct sun4i_usb_phy_cfg {
        int missing_phys;
 };
 
-struct sun4i_usb_phy_info {
-       const char *gpio_id_det;
-} phy_info[] = {
-       {
-               .gpio_id_det = CONFIG_USB0_ID_DET,
-       },
-       {
-               .gpio_id_det = NULL,
-       },
-       {
-               .gpio_id_det = NULL,
-       },
-       {
-               .gpio_id_det = NULL,
-       },
-};
-
 struct sun4i_usb_phy_plat {
        void __iomem *pmu;
        struct gpio_desc gpio_vbus_det;
@@ -475,7 +458,6 @@ static int sun4i_usb_phy_probe(struct udevice *dev)
        data->usb_phy = plat;
        for (i = 0; i < data->cfg->num_phys; i++) {
                struct sun4i_usb_phy_plat *phy = &plat[i];
-               struct sun4i_usb_phy_info *info = &phy_info[i];
                char name[32];
 
                if (data->cfg->missing_phys & BIT(i))
@@ -499,17 +481,16 @@ static int sun4i_usb_phy_probe(struct udevice *dev)
                                        ret);
                                return ret;
                        }
-               }
 
-               ret = dm_gpio_lookup_name(info->gpio_id_det, &phy->gpio_id_det);
-               if (ret == 0) {
-                       ret = dm_gpio_request(&phy->gpio_id_det, "usb_id_det");
-                       if (ret)
-                               return ret;
-                       ret = dm_gpio_set_dir_flags(&phy->gpio_id_det,
-                                               GPIOD_IS_IN | GPIOD_PULL_UP);
-                       if (ret)
+                       ret = gpio_request_by_name(dev, "usb0_id_det-gpios", 0,
+                                                  &phy->gpio_id_det,
+                                                  GPIOD_IS_IN | GPIOD_PULL_UP);
+                       if (ret && ret != -ENOENT) {
+                               dev_err(dev,
+                                       "failed to get ID detect GPIO: %d\n",
+                                       ret);
                                return ret;
+                       }
                }
 
                if (data->cfg->dedicated_clocks)