From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans de Goede Date: Thu, 11 Sep 2014 19:19:19 +0200 Subject: [U-Boot] [PATCH 1/7] ARM: sunxi: Fix build break when CONFIG_USB_EHCI is not defined In-Reply-To: References: <1410182892-18647-1-git-send-email-wens@csie.org> <1410182892-18647-2-git-send-email-wens@csie.org> Message-ID: <5411D997.8040801@redhat.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 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 >> --- >> 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 >>