usb: ehci-mx6: Move fdtdec_get_alias_seq() inside the CONFIG_MX6
authorFabio Estevam <festevam@denx.de>
Sun, 20 Jun 2021 15:00:51 +0000 (12:00 -0300)
committerMarek Vasut <marex@denx.de>
Thu, 24 Jun 2021 18:23:23 +0000 (20:23 +0200)
On a imx7s-warp board the fdtdec_get_alias_seq() function
always fails.

As priv->portnr is only used on i.MX6, move fdtdec_get_alias_seq()
inside the CONFIG_MX6 block.

Signed-off-by: Fabio Estevam <festevam@denx.de>
drivers/usb/host/ehci-mx6.c

index 238c931..48b5c8b 100644 (file)
@@ -569,7 +569,6 @@ static int mx6_parse_dt_addrs(struct udevice *dev)
        const void *blob = gd->fdt_blob;
        int offset = dev_of_offset(dev);
        void *__iomem addr;
-       int ret, devnump;
 
        phy_off = fdtdec_lookup_phandle(blob, offset, "fsl,usbphy");
        if (phy_off < 0) {
@@ -578,11 +577,6 @@ static int mx6_parse_dt_addrs(struct udevice *dev)
                        return -EINVAL;
        }
 
-       ret = fdtdec_get_alias_seq(blob, dev->uclass->uc_drv->name,
-                                  phy_off, &devnump);
-       if (ret < 0)
-               return ret;
-
        misc_off = fdtdec_lookup_phandle(blob, offset, "fsl,usbmisc");
        if (misc_off < 0)
                return -EINVAL;
@@ -592,7 +586,6 @@ static int mx6_parse_dt_addrs(struct udevice *dev)
                return -EINVAL;
 
        priv->phy_addr = addr;
-       priv->portnr = devnump;
 
        addr = (void __iomem *)fdtdec_get_addr(blob, misc_off, "reg");
        if ((fdt_addr_t)addr == FDT_ADDR_T_NONE)
@@ -601,7 +594,13 @@ static int mx6_parse_dt_addrs(struct udevice *dev)
        priv->misc_addr = addr;
 
 #if defined(CONFIG_MX6)
-       int anatop_off;
+       int anatop_off, ret, devnump;
+
+       ret = fdtdec_get_alias_seq(blob, dev->uclass->uc_drv->name,
+                                  phy_off, &devnump);
+       if (ret < 0)
+               return ret;
+       priv->portnr = devnump;
 
        /* Resolve ANATOP offset through USB PHY node */
        anatop_off = fdtdec_lookup_phandle(blob, phy_off, "fsl,anatop");