[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