All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jean-Jacques Hiblot <jjhiblot@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v1 13/18] usb: dwc3: Kconfig: get rid of obsolete mode selection
Date: Fri, 5 Apr 2019 14:55:49 +0200	[thread overview]
Message-ID: <20190405125554.18070-14-jjhiblot@ti.com> (raw)
In-Reply-To: <20190405125554.18070-1-jjhiblot@ti.com>

The mode selection for the DWC3 is kind of obsolete. The driver does not
have to be host only or gadget only. This choice is confusing.
All the remaining callers of dwc3_uboot_init() explicitly set dr_mode
before calling the function, so none rely on a default behavior.

Remove the choice menu and keep only the USB_DWC3_GADGET option. Enable it
by default if USB_GADGET and USB_DWC3 are enabled.
It must be disabled for the evb-rk3328 as it uses DWC2 for the gadget and
DWC3 for the host.

Signed-off-by: Jean-Jacques Hiblot <jjhiblot@ti.com>
---

 configs/evb-rk3328_defconfig |  1 +
 drivers/usb/dwc3/Kconfig     | 20 ++------------------
 drivers/usb/dwc3/core.c      |  4 ++--
 3 files changed, 5 insertions(+), 20 deletions(-)

diff --git a/configs/evb-rk3328_defconfig b/configs/evb-rk3328_defconfig
index 6783d5a29e..52df5bc46c 100644
--- a/configs/evb-rk3328_defconfig
+++ b/configs/evb-rk3328_defconfig
@@ -57,6 +57,7 @@ CONFIG_USB_OHCI_HCD=y
 CONFIG_USB_OHCI_GENERIC=y
 CONFIG_USB_DWC2=y
 CONFIG_USB_DWC3=y
+# CONFIG_USB_DWC3_GADGET is not set
 CONFIG_USB_GADGET=y
 CONFIG_USB_GADGET_MANUFACTURER="Rockchip"
 CONFIG_USB_GADGET_VENDOR_NUM=0x2207
diff --git a/drivers/usb/dwc3/Kconfig b/drivers/usb/dwc3/Kconfig
index bbd8105c06..58147c3d53 100644
--- a/drivers/usb/dwc3/Kconfig
+++ b/drivers/usb/dwc3/Kconfig
@@ -7,26 +7,10 @@ config USB_DWC3
 
 if USB_DWC3
 
-choice
-	bool "DWC3 Mode Selection"
-
-config USB_DWC3_HOST
-	bool "Host only mode"
-	depends on USB
-	help
-	  Select this when you want to use DWC3 in host mode only,
-	  thereby the gadget feature will be regressed.
-
 config USB_DWC3_GADGET
-	bool "Gadget only mode"
+	bool "USB Gadget support for DWC3"
+	default y
 	depends on USB_GADGET
-	select USB_GADGET_DUALSPEED
-	help
-	  Select this when you want to use DWC3 in gadget mode only,
-	  thereby the host feature will be regressed.
-
-endchoice
-
 comment "Platform Glue Driver Support"
 
 config USB_DWC3_OMAP
diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c
index 1baad39796..9f7f053265 100644
--- a/drivers/usb/dwc3/core.c
+++ b/drivers/usb/dwc3/core.c
@@ -707,9 +707,9 @@ int dwc3_uboot_init(struct dwc3_device *dwc3_dev)
 		return -ENOMEM;
 	}
 
-	if (IS_ENABLED(CONFIG_USB_DWC3_HOST))
+	if (!IS_ENABLED(USB_DWC3_GADGET))
 		dwc->dr_mode = USB_DR_MODE_HOST;
-	else if (IS_ENABLED(CONFIG_USB_DWC3_GADGET))
+	else if (!IS_ENABLED(CONFIG_USB_HOST))
 		dwc->dr_mode = USB_DR_MODE_PERIPHERAL;
 
 	if (dwc->dr_mode == USB_DR_MODE_UNKNOWN)
-- 
2.17.1

  parent reply	other threads:[~2019-04-05 12:55 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-04-05 12:55 [U-Boot] [PATCH v1 00/18] Improvement for the DWC3 USB generic driver and fixes for the K2 platforms Jean-Jacques Hiblot
2019-04-05 12:55 ` [U-Boot] [PATCH v1 01/18] usb: dwc3-generic: remove dm_scan_fdt_dev() from the remove() callback Jean-Jacques Hiblot
2019-04-05 12:55 ` [U-Boot] [PATCH v1 02/18] usb: host: remove the xhci-zynqmp driver Jean-Jacques Hiblot
2019-04-29  9:55   ` Marek Vasut
2019-04-29 15:16     ` Michal Simek
2019-04-05 12:55 ` [U-Boot] [PATCH v1 03/18] dm: Add a No-op uclass Jean-Jacques Hiblot
2019-04-29  9:56   ` Marek Vasut
2019-05-03  9:30     ` Jean-Jacques Hiblot
2019-05-03  9:57       ` Marek Vasut
2019-05-07  3:52   ` Simon Glass
2019-05-13 10:28     ` Jean-Jacques Hiblot
2019-04-05 12:55 ` [U-Boot] [PATCH v1 04/18] usb: dwc3: Use UCLASS_NOP instead of UCLASS_MISC for the DWC3 generic glue Jean-Jacques Hiblot
2019-04-05 12:55 ` [U-Boot] [PATCH v1 05/18] usb: dwc3: switch to peripheral mode when exiting Jean-Jacques Hiblot
2019-04-29  9:56   ` Marek Vasut
2019-05-03  9:26     ` Jean-Jacques Hiblot
2019-05-03  9:57       ` Marek Vasut
2019-04-05 12:55 ` [U-Boot] [PATCH v1 06/18] usb: xhci: move xhci.h to include usb Jean-Jacques Hiblot
2019-04-05 12:55 ` [U-Boot] [PATCH v1 07/18] usb: dwc3: always use the inlined version of dwc3_host_init/dwc3_host_exit Jean-Jacques Hiblot
2019-04-05 12:55 ` [U-Boot] [PATCH v1 08/18] usb: dwc3-generic: use platdata Jean-Jacques Hiblot
2019-04-05 12:55 ` [U-Boot] [PATCH v1 09/18] usb: dwc3-generic: factorize code Jean-Jacques Hiblot
2019-04-05 12:55 ` [U-Boot] [PATCH v1 10/18] usb: dwc3-generic: add a new host driver that uses the dwc3 core Jean-Jacques Hiblot
2019-04-05 12:55 ` [U-Boot] [PATCH v1 11/18] usb: dwc3-generic: if no max speed is specified in DT, assume super speed Jean-Jacques Hiblot
2019-04-05 12:55 ` [U-Boot] [PATCH v1 12/18] usb: dwc3: Add dwc3_of_parse() to get quirks information from DT Jean-Jacques Hiblot
2019-04-29  9:58   ` Marek Vasut
2019-05-03  9:38     ` Jean-Jacques Hiblot
2019-05-03  9:58       ` Marek Vasut
2019-04-05 12:55 ` Jean-Jacques Hiblot [this message]
2019-04-05 12:55 ` [U-Boot] [PATCH v1 14/18] ARM: keystone: increase PSC timeout Jean-Jacques Hiblot
2019-04-05 12:55 ` [U-Boot] [PATCH v1 15/18] ARM: keystone: Do not enable the USB power domains at the board level Jean-Jacques Hiblot
2019-04-05 12:55 ` [U-Boot] [PATCH v1 16/18] phy: keystone-usb: handle the transition of the USB power domain Jean-Jacques Hiblot
2019-04-05 12:55 ` [U-Boot] [PATCH v1 17/18] configs: k2g_evm_defconfig: disable XHCI_DWC3 and enable KEYSTONE_USB_PHY Jean-Jacques Hiblot
2019-04-05 12:55 ` [U-Boot] [PATCH v1 18/18] ARM: DTS: keystone: complete the description of the USB PHY devices Jean-Jacques Hiblot
2019-04-29  9:09 ` [U-Boot] [PATCH v1 00/18] Improvement for the DWC3 USB generic driver and fixes for the K2 platforms Jean-Jacques Hiblot
2019-04-29  9:52   ` Marek Vasut
2019-04-29  9:54     ` Marek Vasut
2019-05-03  9:39     ` Jean-Jacques Hiblot

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190405125554.18070-14-jjhiblot@ti.com \
    --to=jjhiblot@ti.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.