From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 2F64BC3DA78 for ; Tue, 17 Jan 2023 09:44:07 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 411D9853BE; Tue, 17 Jan 2023 10:44:05 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=baylibre-com.20210112.gappssmtp.com header.i=@baylibre-com.20210112.gappssmtp.com header.b="735Ct0A2"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 288D0850D4; Tue, 17 Jan 2023 10:44:03 +0100 (CET) Received: from mail-wm1-x32d.google.com (mail-wm1-x32d.google.com [IPv6:2a00:1450:4864:20::32d]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 12E2B853BE for ; Tue, 17 Jan 2023 10:43:58 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=mkorpershoek@baylibre.com Received: by mail-wm1-x32d.google.com with SMTP id k22-20020a05600c1c9600b003d1ee3a6289so23957781wms.2 for ; Tue, 17 Jan 2023 01:43:58 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20210112.gappssmtp.com; s=20210112; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:from:to:cc:subject:date:message-id:reply-to; bh=AA+r2d2VvGC2iSgTK8AKD1qTgcdgGDpMaBnjMbAUTPM=; b=735Ct0A2HbWEzcwoZI2U3l1DX1uspvxxQQRYPA+ZAjjSm3u4tFveRQAcykrWXJZqJ+ CdaFB1XTWS2JQnOP4nfce8KkfEOCLFnS4tjEVcByvkERvAbOylawgs9BeDLgpTf6z7CS fkZ+TpWGX6Ewm39RDAfGgIeQWPMxNKfjunjEIDMwAWtF3D7Kp9+vJI6gwaDLz2/ImjcV uMpmngYfKZ6zW/Myp3m8JBEIPziLpg6eLdQ1fCVc0qfIOusnr2jH/C+h+WhJlumFxqcz wGwpPXivDRJwgMqX8WhKMwq0ftk8xUD7CqUrMcgmn/s63r9XnC7Z7djvER2EcZK2BCsk cdhQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=AA+r2d2VvGC2iSgTK8AKD1qTgcdgGDpMaBnjMbAUTPM=; b=uvb6MKtgkbClOAXOhUd3IjvZDekSIw+5Hib8RJLF1XvhdVq02zuQGiNCp9W1mgu+uq AnSakdSitzGhbv3yC4Rd/fqln8llSXe5WuhXGpQRJEihbbNJ2N99U0ECLVYjJFP0r/BR eXUt0WrLMjo6PETSOa0yXqKMa+SO2j5/fHLRfbiavJzq7h935oe5eyPFNFqsjlYeYKgW tPXniyu7/s6Y8jaBr8B7F6yCG6goWfZYDUieU1eLfchlLG9el8wrnM3nLEg/EQl8QyR7 4uaCkkrLsWi3KTfdMq0h8ugqywmrTBiruzJGvBNJkkBPt8lhz/W4SleLYrJozRtaV2TS BYpA== X-Gm-Message-State: AFqh2kpaYgPpV10/OLn4/6/p9T/guI9clZTJIwx11C2sJoUbrsyGCvKb g+BkMjOJXlWg0kp7wjmdpJAfHg== X-Google-Smtp-Source: AMrXdXvlneCtywweSD6QhHcyppuf/cv6WH5kFR7Fey9VhUdiAmo2V1D5RG0CPx5F0Wo2vsEu5P3E0g== X-Received: by 2002:a05:600c:3b82:b0:3d3:5d0f:6dfc with SMTP id n2-20020a05600c3b8200b003d35d0f6dfcmr2342659wms.30.1673948637511; Tue, 17 Jan 2023 01:43:57 -0800 (PST) Received: from localhost ([82.66.159.240]) by smtp.gmail.com with ESMTPSA id d6-20020a05600c3ac600b003da0dc39872sm18620467wms.6.2023.01.17.01.43.56 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 17 Jan 2023 01:43:57 -0800 (PST) From: Mattijs Korpershoek To: Neil Armstrong , Marek Vasut , Vyacheslav Bocharov Cc: u-boot-amlogic@groups.io, u-boot@lists.denx.de, Neil Armstrong Subject: Re: [PATCH 2/2] ARM: meson: switch AXG & GX dwc2 otg to DM In-Reply-To: <20230117-u-boot-usb-gxl-otg-dm-v1-2-2853f6d75b06@linaro.org> References: <20230117-u-boot-usb-gxl-otg-dm-v1-0-2853f6d75b06@linaro.org> <20230117-u-boot-usb-gxl-otg-dm-v1-2-2853f6d75b06@linaro.org> Date: Tue, 17 Jan 2023 10:43:56 +0100 Message-ID: <87r0vtmr6r.fsf@baylibre.com> MIME-Version: 1.0 Content-Type: text/plain X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.6 at phobos.denx.de X-Virus-Status: Clean On Tue, Jan 17, 2023 at 10:11, Neil Armstrong wrote: > With CONFIG_DM_USB_GADGET=y, we no longer need any board_usb_*() logic > because: > * the dwc2 driver is DM compatible, and handles its own clock enabling > * the dwc3-meson-gxl glue drivers handles "force mode switching" > > Remove all mach-meson gx & axg usb code and enable CONFIG_DM_USB_GADGET. > > Note: Only configs having both CONFIG_USB_DWC3_MESON_GXL=y *and* > USB_GADGET_DWC2_OTG=y have been updated. > > Thanks for the original work from Mattijs at [1] > > [1] https://lore.kernel.org/all/20221024-meson-dm-usb-v1-2-2ab077a503b9@baylibre.com/ > > Signed-off-by: Neil Armstrong Reviewed-by: Mattijs Korpershoek > --- > arch/arm/mach-meson/board-axg.c | 129 --------------------------------- > arch/arm/mach-meson/board-gx.c | 137 ----------------------------------- > configs/jethub_j100_defconfig | 1 + > configs/jethub_j80_defconfig | 1 + > configs/khadas-vim2_defconfig | 1 + > configs/khadas-vim_defconfig | 1 + > configs/libretech-ac_defconfig | 1 + > configs/libretech-cc_defconfig | 1 + > configs/libretech-s905d-pc_defconfig | 1 + > configs/libretech-s912-pc_defconfig | 1 + > configs/s400_defconfig | 1 + > configs/wetek-core2_defconfig | 1 + > 12 files changed, 10 insertions(+), 266 deletions(-) > > diff --git a/arch/arm/mach-meson/board-axg.c b/arch/arm/mach-meson/board-axg.c > index 236ec81579..fdf18752cd 100644 > --- a/arch/arm/mach-meson/board-axg.c > +++ b/arch/arm/mach-meson/board-axg.c > @@ -15,12 +15,6 @@ > #include > #include > #include > -#include > -#include > -#include > -#include > -#include > -#include > > DECLARE_GLOBAL_DATA_PTR; > > @@ -90,126 +84,3 @@ static struct mm_region axg_mem_map[] = { > }; > > struct mm_region *mem_map = axg_mem_map; > - > -#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; > - > -int board_usb_init(int index, enum usb_init_type init) > -{ > - 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; > - > - /* find the usb glue node */ > - node = fdt_node_offset_by_compatible(blob, -1, > - "amlogic,meson-axg-usb-ctrl"); > - if (node < 0) { > - debug("Not found usb-control node\n"); > - return -ENODEV; > - } > - > - if (!fdtdec_get_is_enabled(blob, node)) { > - debug("usb is disabled in the device tree\n"); > - return -ENODEV; > - } > - > - ret = uclass_get_device_by_of_offset(UCLASS_SIMPLE_BUS, node, &dev); > - if (ret) { > - debug("Not found usb-control device\n"); > - 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; > - } > - > - if (!fdtdec_get_is_enabled(blob, dwc2_node)) { > - debug("dwc2 is disabled in the device tree\n"); > - return -ENODEV; > - } > - > - 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 = 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_of_offset(UCLASS_CLK, args.node, &clk_dev); > - if (ret) > - return ret; > - > - if (args.args_count != 1) { > - debug("Can't find clock ID in the device tree\n"); > - return -ENODATA; > - } > - > - clk.dev = clk_dev; > - clk.id = args.args[0]; > - > - ret = clk_enable(&clk); > - if (ret) { > - debug("Failed to enable usbotg clock\n"); > - return ret; > - } > - > - 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) > -{ > - 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-axg-usb-ctrl"); > - if (node < 0) { > - debug("Not found usb-control node\n"); > - return -ENODEV; > - } > - > - if (!fdtdec_get_is_enabled(blob, node)) > - return -ENODEV; > - > - 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; > -} > -#endif > diff --git a/arch/arm/mach-meson/board-gx.c b/arch/arm/mach-meson/board-gx.c > index 01fafd81c4..f28f84d86f 100644 > --- a/arch/arm/mach-meson/board-gx.c > +++ b/arch/arm/mach-meson/board-gx.c > @@ -16,12 +16,6 @@ > #include > #include > #include > -#include > -#include > -#include > -#include > -#include > -#include > > DECLARE_GLOBAL_DATA_PTR; > > @@ -108,134 +102,3 @@ static struct mm_region gx_mem_map[] = { > }; > > struct mm_region *mem_map = gx_mem_map; > - > -#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; > - > -int board_usb_init(int index, enum usb_init_type init) > -{ > - 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; > - > - /* 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 (!fdtdec_get_is_enabled(blob, node)) { > - debug("usb is disabled in the device tree\n"); > - return -ENODEV; > - } > - > - ret = uclass_get_device_by_of_offset(UCLASS_SIMPLE_BUS, node, &dev); > - if (ret) { > - debug("Not found usb-control device\n"); > - 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; > - } > - > - if (!fdtdec_get_is_enabled(blob, dwc2_node)) { > - debug("dwc2 is disabled in the device tree\n"); > - return -ENODEV; > - } > - > - 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 = 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_of_offset(UCLASS_CLK, args.node, &clk_dev); > - if (ret) > - return ret; > - > - if (args.args_count != 1) { > - debug("Can't find clock ID in the device tree\n"); > - return -ENODATA; > - } > - > - clk.dev = clk_dev; > - clk.id = args.args[0]; > - > - ret = clk_enable(&clk); > - if (ret) { > - debug("Failed to enable usbotg clock\n"); > - return ret; > - } > - > - 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) > -{ > - 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; > - } > - } > - > - if (!fdtdec_get_is_enabled(blob, node)) > - return -ENODEV; > - > - 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; > -} > -#endif > diff --git a/configs/jethub_j100_defconfig b/configs/jethub_j100_defconfig > index c2651580f8..dac775bd3b 100644 > --- a/configs/jethub_j100_defconfig > +++ b/configs/jethub_j100_defconfig > @@ -55,6 +55,7 @@ CONFIG_DEBUG_UART_ANNOUNCE=y > CONFIG_DEBUG_UART_SKIP_INIT=y > CONFIG_MESON_SERIAL=y > CONFIG_USB=y > +CONFIG_DM_USB_GADGET=y > CONFIG_USB_XHCI_HCD=y > CONFIG_USB_XHCI_DWC3=y > CONFIG_USB_DWC2=y > diff --git a/configs/jethub_j80_defconfig b/configs/jethub_j80_defconfig > index f3c47a001a..1d11a961b9 100644 > --- a/configs/jethub_j80_defconfig > +++ b/configs/jethub_j80_defconfig > @@ -59,6 +59,7 @@ CONFIG_DEBUG_UART_ANNOUNCE=y > CONFIG_DEBUG_UART_SKIP_INIT=y > CONFIG_MESON_SERIAL=y > CONFIG_USB=y > +CONFIG_DM_USB_GADGET=y > CONFIG_USB_XHCI_HCD=y > CONFIG_USB_XHCI_DWC3=y > CONFIG_USB_DWC2=y > diff --git a/configs/khadas-vim2_defconfig b/configs/khadas-vim2_defconfig > index a8d11a83c9..3af6b446ec 100644 > --- a/configs/khadas-vim2_defconfig > +++ b/configs/khadas-vim2_defconfig > @@ -59,6 +59,7 @@ CONFIG_MESON_SPIFC=y > CONFIG_SYSINFO=y > CONFIG_SYSINFO_SMBIOS=y > CONFIG_USB=y > +CONFIG_DM_USB_GADGET=y > CONFIG_USB_XHCI_HCD=y > CONFIG_USB_XHCI_DWC3=y > CONFIG_USB_DWC3=y > diff --git a/configs/khadas-vim_defconfig b/configs/khadas-vim_defconfig > index d1ed45ff88..6bb2200a6b 100644 > --- a/configs/khadas-vim_defconfig > +++ b/configs/khadas-vim_defconfig > @@ -50,6 +50,7 @@ CONFIG_MESON_SERIAL=y > CONFIG_SYSINFO=y > CONFIG_SYSINFO_SMBIOS=y > CONFIG_USB=y > +CONFIG_DM_USB_GADGET=y > CONFIG_USB_XHCI_HCD=y > CONFIG_USB_XHCI_DWC3=y > CONFIG_USB_DWC3=y > diff --git a/configs/libretech-ac_defconfig b/configs/libretech-ac_defconfig > index a333106692..2f44771a84 100644 > --- a/configs/libretech-ac_defconfig > +++ b/configs/libretech-ac_defconfig > @@ -65,6 +65,7 @@ CONFIG_SPI=y > CONFIG_DM_SPI=y > CONFIG_MESON_SPIFC=y > CONFIG_USB=y > +CONFIG_DM_USB_GADGET=y > CONFIG_USB_XHCI_HCD=y > CONFIG_USB_XHCI_DWC3=y > CONFIG_USB_DWC3=y > diff --git a/configs/libretech-cc_defconfig b/configs/libretech-cc_defconfig > index 60b40e6270..05311d8e08 100644 > --- a/configs/libretech-cc_defconfig > +++ b/configs/libretech-cc_defconfig > @@ -49,6 +49,7 @@ CONFIG_DEBUG_UART_ANNOUNCE=y > CONFIG_DEBUG_UART_SKIP_INIT=y > CONFIG_MESON_SERIAL=y > CONFIG_USB=y > +CONFIG_DM_USB_GADGET=y > CONFIG_USB_XHCI_HCD=y > CONFIG_USB_XHCI_DWC3=y > CONFIG_USB_DWC3=y > diff --git a/configs/libretech-s905d-pc_defconfig b/configs/libretech-s905d-pc_defconfig > index d3e600ec02..4e3aee7542 100644 > --- a/configs/libretech-s905d-pc_defconfig > +++ b/configs/libretech-s905d-pc_defconfig > @@ -61,6 +61,7 @@ CONFIG_SPI=y > CONFIG_DM_SPI=y > CONFIG_MESON_SPIFC=y > CONFIG_USB=y > +CONFIG_DM_USB_GADGET=y > CONFIG_USB_XHCI_HCD=y > CONFIG_USB_XHCI_DWC3=y > CONFIG_USB_DWC3=y > diff --git a/configs/libretech-s912-pc_defconfig b/configs/libretech-s912-pc_defconfig > index 8b4f052d08..c9ceb35ff6 100644 > --- a/configs/libretech-s912-pc_defconfig > +++ b/configs/libretech-s912-pc_defconfig > @@ -60,6 +60,7 @@ CONFIG_SPI=y > CONFIG_DM_SPI=y > CONFIG_MESON_SPIFC=y > CONFIG_USB=y > +CONFIG_DM_USB_GADGET=y > CONFIG_USB_XHCI_HCD=y > CONFIG_USB_XHCI_DWC3=y > CONFIG_USB_DWC3=y > diff --git a/configs/s400_defconfig b/configs/s400_defconfig > index 827bd548c7..91930d7f48 100644 > --- a/configs/s400_defconfig > +++ b/configs/s400_defconfig > @@ -42,6 +42,7 @@ CONFIG_DEBUG_UART_ANNOUNCE=y > CONFIG_DEBUG_UART_SKIP_INIT=y > CONFIG_MESON_SERIAL=y > CONFIG_USB=y > +CONFIG_DM_USB_GADGET=y > CONFIG_USB_XHCI_HCD=y > CONFIG_USB_XHCI_DWC3=y > CONFIG_USB_DWC3=y > diff --git a/configs/wetek-core2_defconfig b/configs/wetek-core2_defconfig > index 13940dd56f..b730700630 100644 > --- a/configs/wetek-core2_defconfig > +++ b/configs/wetek-core2_defconfig > @@ -51,6 +51,7 @@ CONFIG_DEBUG_UART_ANNOUNCE=y > CONFIG_DEBUG_UART_SKIP_INIT=y > CONFIG_MESON_SERIAL=y > CONFIG_USB=y > +CONFIG_DM_USB_GADGET=y > CONFIG_USB_XHCI_HCD=y > CONFIG_USB_XHCI_DWC3=y > CONFIG_USB_DWC3=y > > -- > 2.34.1