[U-Boot] [RESEND PATCH v3 13/18] usb: dwc3: Kconfig: get rid of obsolete mode selection

Marek Vasut marex at denx.de
Thu Jun 27 15:09:37 UTC 2019


On 6/27/19 4:49 PM, Jean-Jacques Hiblot wrote:
> 
> On 27/06/2019 16:33, Patrice CHOTARD wrote:
>> Hi Jean-Jacques
>>
>> On 6/27/19 3:06 PM, Jean-Jacques Hiblot wrote:
>>> 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 at ti.com>
>>>
>>> ---
>>>
>>> Changes in v3: None
>>> Changes in v2:
>>> Select USB_GADGET_DUALSPEED if USB_DWC3_GADGET is selected
>>>
>>>   configs/evb-rk3328_defconfig |  1 +
>>>   drivers/usb/dwc3/Kconfig     | 18 ++----------------
>>>   drivers/usb/dwc3/core.c      |  4 ++--
>>>   3 files changed, 5 insertions(+), 18 deletions(-)
>>>
>>> diff --git a/configs/evb-rk3328_defconfig b/configs/evb-rk3328_defconfig
>>> index aff9c32362..98929f220a 100644
>>> --- a/configs/evb-rk3328_defconfig
>>> +++ b/configs/evb-rk3328_defconfig
>>> @@ -58,6 +58,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 25e1a38aee..c302486291 100644
>>> --- a/drivers/usb/dwc3/Kconfig
>>> +++ b/drivers/usb/dwc3/Kconfig
>>> @@ -7,25 +7,11 @@ 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"
>>>   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))
>>
>> Testing your series on 96board STiH410-B2260, this patch is breaking
>> fastboot feature.
>>
>> Should be
>>
>> +    if (!IS_ENABLED(CONFIG_USB_DWC3_GADGET))
> 
> Thanks for testing and catching this.
> 
> It'll be fixed in the next version of the series

Just send me an incremental patch on top of usb/next.

-- 
Best regards,
Marek Vasut


More information about the U-Boot mailing list