[U-Boot] [PATCH v2 1/4] usb: Introduce CONFIG_SPL_DM_USB
Sven Schwermer
sven at svenschwermer.de
Sun Nov 18 19:08:21 UTC 2018
Hi!
>> -ifdef CONFIG_CMD_USB
>> +ifdef CONFIG_USB
>
> Shouldn't this be a separate change ?
Do you mean a separate patch set or its own patch? I was actually not 100% sure about this one but ran into link problems with the status quo. It seemed odd that this code would depend on the USB command. Maybe you can elaborate a little?
>> obj-y += usb.o usb_hub.o
>> obj-$(CONFIG_USB_STORAGE) += usb_storage.o
>> endif
>> diff --git a/drivers/usb/Kconfig b/drivers/usb/Kconfig
>> index 4fbe172e05..03746dd12f 100644
>> --- a/drivers/usb/Kconfig
>> +++ b/drivers/usb/Kconfig
>> @@ -47,6 +47,11 @@ config DM_USB
>> declared with the U_BOOT_USB_DEVICE() macro and will be
>> automatically probed when found on the bus.
>>
>> +config SPL_DM_USB
>
> I think this should be introduced last, after all the core changes,
> since this patch won't compile cleanly on it's own, will it ?
Not sure, I understand. Just introducing a new Kconfig variable won’t change the “compilibility” of existing code?!
Best,
Sven
More information about the U-Boot
mailing list