ARM: mach-meson: use new DWC3 glue for GXL & GXM
authorNeil Armstrong <narmstrong@baylibre.com>
Thu, 10 Sep 2020 08:48:14 +0000 (10:48 +0200)
committerNeil Armstrong <narmstrong@baylibre.com>
Mon, 5 Oct 2020 15:59:45 +0000 (17:59 +0200)
Use the new Amlogic GXL/GXM USB Glue instead of the set of USB3 PHY
and Simple DWC3 wrapper.

Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
arch/arm/mach-meson/board-gx.c
configs/khadas-vim2_defconfig
configs/khadas-vim_defconfig
configs/libretech-ac_defconfig
configs/libretech-cc_defconfig
configs/libretech-s905d-pc_defconfig
configs/libretech-s912-pc_defconfig
configs/p212_defconfig

index c4cc11f..7a0e703 100644 (file)
@@ -156,79 +156,70 @@ void meson_eth_init(phy_interface_t mode, unsigned int flags)
        clrbits_le32(GX_MEM_PD_REG_0, GX_MEM_PD_REG_0_ETH_MASK);
 }
 
-#if CONFIG_IS_ENABLED(USB_XHCI_DWC3_OF_SIMPLE) && \
+#if CONFIG_IS_ENABLED(USB_DWC3_MESON_GXL) && \
        CONFIG_IS_ENABLED(USB_GADGET_DWC2_OTG)
 static struct dwc2_plat_otg_data meson_gx_dwc2_data;
-static struct phy usb_phys[2];
 
 int board_usb_init(int index, enum usb_init_type init)
 {
-       struct ofnode_phandle_args args;
-       struct udevice *clk_dev;
-       ofnode dwc2_node;
+       struct fdtdec_phandle_args args;
+       const void *blob = gd->fdt_blob;
+       int node, dwc2_node;
+       struct udevice *dev, *clk_dev;
        struct clk clk;
-       int ret, i;
-       u32 val;
-
-       /* find the dwc2 node */
-       dwc2_node = ofnode_by_compatible(ofnode_null(), "snps,dwc2");
-       if (!ofnode_valid(dwc2_node)) {
-               debug("Not found dwc2 node\n");
-               return -ENODEV;
+       int ret;
+
+       /* find the usb glue node */
+       node = fdt_node_offset_by_compatible(blob, -1,
+                                            "amlogic,meson-gxl-usb-ctrl");
+       if (node < 0) {
+               node = fdt_node_offset_by_compatible(blob, -1,
+                                       "amlogic,meson-gxm-usb-ctrl");
+               if (node < 0) {
+                       debug("Not found usb-control node\n");
+                       return -ENODEV;
+               }
        }
 
-       if (!ofnode_is_available(dwc2_node)) {
-               debug("dwc2 is disabled in the device tree\n");
+       if (!fdtdec_get_is_enabled(blob, node)) {
+               debug("usb is disabled in the device tree\n");
                return -ENODEV;
        }
 
-       /* get the PHYs */
-       for (i = 0; i < 2; i++) {
-               ret = generic_phy_get_by_index_nodev(dwc2_node, i,
-                                                    &usb_phys[i]);
-               if (ret && ret != -ENOENT) {
-                       pr_err("Failed to get USB PHY%d for %s\n",
-                              i, ofnode_get_name(dwc2_node));
-                       return ret;
-               }
+       ret = uclass_get_device_by_of_offset(UCLASS_SIMPLE_BUS, node, &dev);
+       if (ret) {
+               debug("Not found usb-control device\n");
+               return ret;
        }
 
-       for (i = 0; i < 2; i++) {
-               ret = generic_phy_init(&usb_phys[i]);
-               if (ret) {
-                       pr_err("Can't init USB PHY%d for %s\n",
-                              i, ofnode_get_name(dwc2_node));
-                       return ret;
-               }
+       /* find the dwc2 node */
+       dwc2_node = fdt_node_offset_by_compatible(blob, node,
+                                                 "amlogic,meson-g12a-usb");
+       if (dwc2_node < 0) {
+               debug("Not found dwc2 node\n");
+               return -ENODEV;
        }
 
-       for (i = 0; i < 2; i++) {
-               ret = generic_phy_power_on(&usb_phys[i]);
-               if (ret) {
-                       pr_err("Can't power USB PHY%d for %s\n",
-                              i, ofnode_get_name(dwc2_node));
-                       return ret;
-               }
+       if (!fdtdec_get_is_enabled(blob, dwc2_node)) {
+               debug("dwc2 is disabled in the device tree\n");
+               return -ENODEV;
        }
 
-       phy_meson_gxl_usb3_set_mode(&usb_phys[0], USB_DR_MODE_PERIPHERAL);
-       phy_meson_gxl_usb2_set_mode(&usb_phys[1], USB_DR_MODE_PERIPHERAL);
-
-       meson_gx_dwc2_data.regs_otg = ofnode_get_addr(dwc2_node);
+       meson_gx_dwc2_data.regs_otg = fdtdec_get_addr(blob, dwc2_node, "reg");
        if (meson_gx_dwc2_data.regs_otg == FDT_ADDR_T_NONE) {
                debug("usbotg: can't get base address\n");
                return -ENODATA;
        }
 
        /* Enable clock */
-       ret = ofnode_parse_phandle_with_args(dwc2_node, "clocks",
+       ret = fdtdec_parse_phandle_with_args(blob, dwc2_node, "clocks",
                                             "#clock-cells", 0, 0, &args);
        if (ret) {
                debug("usbotg has no clocks defined in the device tree\n");
                return ret;
        }
 
-       ret = uclass_get_device_by_ofnode(UCLASS_CLK, args.node, &clk_dev);
+       ret = uclass_get_device_by_of_offset(UCLASS_CLK, args.node, &clk_dev);
        if (ret)
                return ret;
 
@@ -246,25 +237,51 @@ int board_usb_init(int index, enum usb_init_type init)
                return ret;
        }
 
-       ofnode_read_u32(dwc2_node, "g-rx-fifo-size", &val);
-       meson_gx_dwc2_data.rx_fifo_sz = val;
-       ofnode_read_u32(dwc2_node, "g-np-tx-fifo-size", &val);
-       meson_gx_dwc2_data.np_tx_fifo_sz = val;
-       ofnode_read_u32(dwc2_node, "g-tx-fifo-size", &val);
-       meson_gx_dwc2_data.tx_fifo_sz = val;
+       meson_gx_dwc2_data.rx_fifo_sz = fdtdec_get_int(blob, dwc2_node,
+                                                    "g-rx-fifo-size", 0);
+       meson_gx_dwc2_data.np_tx_fifo_sz = fdtdec_get_int(blob, dwc2_node,
+                                                       "g-np-tx-fifo-size", 0);
+       meson_gx_dwc2_data.tx_fifo_sz = fdtdec_get_int(blob, dwc2_node,
+                                                    "g-tx-fifo-size", 0);
+
+       /* Switch to peripheral mode */
+       ret = dwc3_meson_gxl_force_mode(dev, USB_DR_MODE_PERIPHERAL);
+       if (ret)
+               return ret;
 
        return dwc2_udc_probe(&meson_gx_dwc2_data);
 }
 
 int board_usb_cleanup(int index, enum usb_init_type init)
 {
-       int i;
+       const void *blob = gd->fdt_blob;
+       struct udevice *dev;
+       int node;
+       int ret;
+
+       /* find the usb glue node */
+       node = fdt_node_offset_by_compatible(blob, -1,
+                                            "amlogic,meson-gxl-usb-ctrl");
+       if (node < 0) {
+               node = fdt_node_offset_by_compatible(blob, -1,
+                                       "amlogic,meson-gxm-usb-ctrl");
+               if (node < 0) {
+                       debug("Not found usb-control node\n");
+                       return -ENODEV;
+               }
+       }
 
-       phy_meson_gxl_usb3_set_mode(&usb_phys[0], USB_DR_MODE_HOST);
-       phy_meson_gxl_usb2_set_mode(&usb_phys[1], USB_DR_MODE_HOST);
+       if (!fdtdec_get_is_enabled(blob, node))
+               return -ENODEV;
 
-       for (i = 0; i < 2; i++)
-               usb_phys[i].dev = NULL;
+       ret = uclass_get_device_by_of_offset(UCLASS_SIMPLE_BUS, node, &dev);
+       if (ret)
+               return ret;
+
+       /* Switch to OTG mode */
+       ret = dwc3_meson_gxl_force_mode(dev, USB_DR_MODE_HOST);
+       if (ret)
+               return ret;
 
        return 0;
 }
index 1435781..31db02e 100644 (file)
@@ -41,6 +41,7 @@ CONFIG_DM_ETH=y
 CONFIG_ETH_DESIGNWARE=y
 CONFIG_PHY=y
 CONFIG_MESON_GXL_USB_PHY=y
+CONFIG_USB_DWC3_MESON_GXL=y
 CONFIG_PINCTRL=y
 CONFIG_PINCTRL_MESON_GXL=y
 CONFIG_DM_REGULATOR=y
@@ -56,7 +57,6 @@ CONFIG_USB=y
 CONFIG_DM_USB=y
 CONFIG_USB_XHCI_HCD=y
 CONFIG_USB_XHCI_DWC3=y
-CONFIG_USB_XHCI_DWC3_OF_SIMPLE=y
 CONFIG_USB_DWC3=y
 # CONFIG_USB_DWC3_GADGET is not set
 CONFIG_USB_GADGET=y
index d075dd1..65bd7f5 100644 (file)
@@ -37,6 +37,7 @@ CONFIG_DM_ETH=y
 CONFIG_ETH_DESIGNWARE=y
 CONFIG_PHY=y
 CONFIG_MESON_GXL_USB_PHY=y
+CONFIG_USB_DWC3_MESON_GXL=y
 CONFIG_PINCTRL=y
 CONFIG_PINCTRL_MESON_GXL=y
 CONFIG_DM_REGULATOR=y
@@ -49,7 +50,6 @@ CONFIG_USB=y
 CONFIG_DM_USB=y
 CONFIG_USB_XHCI_HCD=y
 CONFIG_USB_XHCI_DWC3=y
-CONFIG_USB_XHCI_DWC3_OF_SIMPLE=y
 CONFIG_USB_DWC3=y
 # CONFIG_USB_DWC3_GADGET is not set
 CONFIG_USB_GADGET=y
index e898fb1..5e8ea0b 100644 (file)
@@ -49,6 +49,7 @@ CONFIG_DM_ETH=y
 CONFIG_ETH_DESIGNWARE=y
 CONFIG_PHY=y
 CONFIG_MESON_GXL_USB_PHY=y
+CONFIG_USB_DWC3_MESON_GXL=y
 CONFIG_PINCTRL=y
 CONFIG_PINCTRL_MESON_GXL=y
 CONFIG_POWER_DOMAIN=y
@@ -66,7 +67,6 @@ CONFIG_USB=y
 CONFIG_DM_USB=y
 CONFIG_USB_XHCI_HCD=y
 CONFIG_USB_XHCI_DWC3=y
-CONFIG_USB_XHCI_DWC3_OF_SIMPLE=y
 CONFIG_USB_DWC3=y
 # CONFIG_USB_DWC3_GADGET is not set
 CONFIG_USB_KEYBOARD=y
index 65ad7f5..559d674 100644 (file)
@@ -36,6 +36,7 @@ CONFIG_DM_ETH=y
 CONFIG_ETH_DESIGNWARE=y
 CONFIG_PHY=y
 CONFIG_MESON_GXL_USB_PHY=y
+CONFIG_USB_DWC3_MESON_GXL=y
 CONFIG_PINCTRL=y
 CONFIG_PINCTRL_MESON_GXL=y
 CONFIG_POWER_DOMAIN=y
@@ -50,7 +51,6 @@ CONFIG_USB=y
 CONFIG_DM_USB=y
 CONFIG_USB_XHCI_HCD=y
 CONFIG_USB_XHCI_DWC3=y
-CONFIG_USB_XHCI_DWC3_OF_SIMPLE=y
 CONFIG_USB_DWC3=y
 # CONFIG_USB_DWC3_GADGET is not set
 CONFIG_USB_GADGET=y
index c99a6c5..5f8074a 100644 (file)
@@ -44,6 +44,7 @@ CONFIG_DM_ETH=y
 CONFIG_ETH_DESIGNWARE=y
 CONFIG_PHY=y
 CONFIG_MESON_GXL_USB_PHY=y
+CONFIG_USB_DWC3_MESON_GXL=y
 CONFIG_PINCTRL=y
 CONFIG_PINCTRL_MESON_GXL=y
 CONFIG_POWER_DOMAIN=y
@@ -61,7 +62,6 @@ CONFIG_USB=y
 CONFIG_DM_USB=y
 CONFIG_USB_XHCI_HCD=y
 CONFIG_USB_XHCI_DWC3=y
-CONFIG_USB_XHCI_DWC3_OF_SIMPLE=y
 CONFIG_USB_DWC3=y
 # CONFIG_USB_DWC3_GADGET is not set
 CONFIG_USB_KEYBOARD=y
index 7975c59..cbd42b4 100644 (file)
@@ -43,6 +43,7 @@ CONFIG_DM_ETH=y
 CONFIG_ETH_DESIGNWARE=y
 CONFIG_PHY=y
 CONFIG_MESON_GXL_USB_PHY=y
+CONFIG_USB_DWC3_MESON_GXL=y
 CONFIG_PINCTRL=y
 CONFIG_PINCTRL_MESON_GXL=y
 CONFIG_POWER_DOMAIN=y
@@ -60,7 +61,6 @@ CONFIG_USB=y
 CONFIG_DM_USB=y
 CONFIG_USB_XHCI_HCD=y
 CONFIG_USB_XHCI_DWC3=y
-CONFIG_USB_XHCI_DWC3_OF_SIMPLE=y
 CONFIG_USB_DWC3=y
 # CONFIG_USB_DWC3_GADGET is not set
 CONFIG_USB_KEYBOARD=y
index 108b608..1823211 100644 (file)
@@ -34,6 +34,7 @@ CONFIG_DM_ETH=y
 CONFIG_ETH_DESIGNWARE=y
 CONFIG_PHY=y
 CONFIG_MESON_GXL_USB_PHY=y
+CONFIG_USB_DWC3_MESON_GXL=y
 CONFIG_PINCTRL=y
 CONFIG_PINCTRL_MESON_GXL=y
 CONFIG_DM_REGULATOR=y
@@ -46,6 +47,5 @@ CONFIG_USB=y
 CONFIG_DM_USB=y
 CONFIG_USB_XHCI_HCD=y
 CONFIG_USB_XHCI_DWC3=y
-CONFIG_USB_XHCI_DWC3_OF_SIMPLE=y
 CONFIG_USB_DWC3=y
 CONFIG_OF_LIBFDT_OVERLAY=y