[U-Boot] [PATCH] usb: dwc3: fix Kconfig dependency to accept host driver in drivers/usb/dwc3
Masahiro Yamada
yamada.masahiro at socionext.com
Fri Aug 25 03:56:05 UTC 2017
2017-08-25 6:57 GMT+09:00 Marek Vasut <marex at denx.de>:
> On 08/24/2017 06:30 PM, Masahiro Yamada wrote:
>> We see the choice of USB_DWC3_HOST / _GADGET in drivers/usb/dwc3/Kconfig,
>> but we can not choose USB_DWC3_HOST unless USB_GADGET is defined.
>> This is strange.
>>
>> Loosen the "depends on" and also move "select USB_GADGET_DUALSPEED" to
>> the correct place.
>>
>> Signed-off-by: Masahiro Yamada <yamada.masahiro at socionext.com>
>
> Applied, thanks
>
>> ---
>>
>> drivers/usb/dwc3/Kconfig | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/usb/dwc3/Kconfig b/drivers/usb/dwc3/Kconfig
>> index e93398fe7c1c..7f2f400c09ce 100644
>> --- a/drivers/usb/dwc3/Kconfig
>> +++ b/drivers/usb/dwc3/Kconfig
>> @@ -1,7 +1,6 @@
>> config USB_DWC3
>> bool "DesignWare USB3 DRD Core Support"
>> - depends on (USB && USB_GADGET)
>> - select USB_GADGET_DUALSPEED
>> + depends on (USB_HOST || USB_GADGET)
>> help
>> Say Y here if your system has a Dual Role SuperSpeed
>> USB controller based on the DesignWare USB3 IP Core.
>> @@ -21,6 +20,7 @@ config USB_DWC3_HOST
>> config USB_DWC3_GADGET
>> bool "Gadget only mode"
>> 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.
>>
>
Marek,
I noticed redundant parentheses for depends on.
Could you fix it up locally like follows?
depends on (USB_HOST || USB_GADGET)
->
depends on USB_HOST || USB_GADGET
Thanks.
--
Best Regards
Masahiro Yamada
More information about the U-Boot
mailing list