[PATCH] usb: gadget: Move USB_GADGET_DUALSPEED to be slightly more visible

Marek Vasut marek.vasut at mailbox.org
Sat Mar 21 19:50:24 CET 2026


On 3/21/26 6:34 PM, Tom Rini wrote:
> On Sat, Mar 21, 2026 at 02:53:13AM +0100, Marek Vasut wrote:
>> On 3/20/26 2:53 PM, Tom Rini wrote:
>>
>> [...]
>>
>>>> @@ -371,3 +367,11 @@ config SPL_SDP_USB_DEV
>>>>             so it can be used in compiled environment.
>>>>
>>>>    endif # SPL_USB_GADGET
>>>> +
>>>> +if USB_GADGET || SPL_USB_GADGET
>>>> +
>>>> +# Selected by UDC drivers that support high-speed operation.
>>>> +config USB_GADGET_DUALSPEED
>>>> +       bool
>>>> +
>>>> +endif
>>>> "
>>>
>>> You don't need to guard hidden symbols.
>>
>> We also do not want to expose this Kconfig symbol unless it makes sense to
>> expose it ? In fact, if the symbol is not available and someone attempts to
>> use it in Kconfig, they would get a warning, which should provide a clue
>> that they are doing something wrong ?
> 
> It's a hidden symbol, so it's not exposed to the user. The problem is
> that today someone is getting it wrong as you suggest but it's hard to
> trigger a warning over it. That said, your patch also fixes the problem,
> so:
> 
> Reviewed-by: Tom Rini <trini at konsulko.com>
Can you send that as a V2 please ?

Thank you


More information about the U-Boot mailing list