[U-Boot] [PATCH 1/7] ARM: sunxi: Fix build break when CONFIG_USB_EHCI is not defined
Hans de Goede
hdegoede at redhat.com
Thu Sep 11 19:19:19 CEST 2014
Hi Chen,
On 09/11/2014 07:07 PM, Chen-Yu Tsai wrote:
> Hi Ian, Hans,
>
> On Mon, Sep 8, 2014 at 9:28 PM, Chen-Yu Tsai <wens at csie.org> wrote:
>> BOOT_TARGET_DEVICES includes USB unconditionally. This breaks when
>> CONFIG_CMD_USB is not defined. Use a secondary macro to conditionally
>> include it when CONFIG_EHCI is enabled, as we do for CONFIG_AHCI.
>
> The other patches are for the next release, but maybe this fix could
> go into 2014.10?
I agree that this is a benign bug fix, but since we don't have any
boards not setting CONFIG_EHCI atm I don't really see the value
for getting it into 2014.10.
Regards,
Hans
>
> Thanks
>
> ChenYu
>
>>
>> Signed-off-by: Chen-Yu Tsai <wens at csie.org>
>> ---
>> include/configs/sunxi-common.h | 8 +++++++-
>> 1 file changed, 7 insertions(+), 1 deletion(-)
>>
>> diff --git a/include/configs/sunxi-common.h b/include/configs/sunxi-common.h
>> index 1d947d7..a31656e 100644
>> --- a/include/configs/sunxi-common.h
>> +++ b/include/configs/sunxi-common.h
>> @@ -233,10 +233,16 @@
>> #define BOOT_TARGET_DEVICES_SCSI(func)
>> #endif
>>
>> +#ifdef CONFIG_USB_EHCI
>> +#define BOOT_TARGET_DEVICES_USB(func) func(USB, usb, 0)
>> +#else
>> +#define BOOT_TARGET_DEVICES_USB(func)
>> +#endif
>> +
>> #define BOOT_TARGET_DEVICES(func) \
>> func(MMC, mmc, 0) \
>> BOOT_TARGET_DEVICES_SCSI(func) \
>> - func(USB, usb, 0) \
>> + BOOT_TARGET_DEVICES_USB(func) \
>> func(PXE, pxe, na) \
>> func(DHCP, dhcp, na)
>>
>> --
>> 2.1.0
>>
More information about the U-Boot
mailing list