[Buildroot] [PATCH] fs: cpio: uImage depends on U-Boot

Vivien Didelot vivien.didelot at savoirfairelinux.com
Tue Aug 11 14:27:54 UTC 2015


Hi Nicolas, Thomas,

On 15-08-11 10:17:31, Thomas Petazzoni wrote:
> Dear Nicolas Cavallari,
> 
> On Tue, 11 Aug 2015 09:26:26 +0200, Nicolas Cavallari wrote:
> > On 10/08/2015 22:05, Vivien Didelot wrote:
> > > Even though some bootloaders such as Barebox can boot uImage, this
> > > format is quite U-Boot specific. So make BR2_TARGET_ROOTFS_CPIO_UIMAGE
> > > depends on BR2_TARGET_UBOOT.
> > 
> > What if you already have a U-Boot binary compiled by your board vendor ?
> 
> Agreed. Even if uImage is indeed mainly U-Boot specific, I agree that
> you may not be building U-Boot with Buildroot, or it might be used in
> other contexts.

That is true.

> I'll mark Vivien's patch as Rejected in patchwork.

My initial concern was seeing the BR2_TARGET_ROOTFS_CPIO_UIMAGE entry
when enabling BR2_TARGET_ROOTFS_ISO9660 (this last one depends on
BR2_TARGET_GRUB, BR2_TARGET_GRUB2 or BR2_TARGET_SYSLINUX_ISOLINUX).

Should we make them exclusive?

    config BR2_TARGET_ROOTFS_CPIO_UIMAGE
        depends on !BR2_TARGET_ROOTFS_ISO9660

Or is that OK to display such options?

Thanks,
-v


More information about the buildroot mailing list