[U-Boot] [PATCH v2 14/18] sunxi: Pine64: defconfig: enable SPL FIT support

[Icenowy Zheng]

2017年3月29日 14:57于 Maxime Ripard <maxime.ripard at free-electrons.com>写道：
>
> On Tue, Mar 28, 2017 at 01:45:22AM +0100, Andre Przywara wrote: 
> > The Pine64 (and all other 64-bit Allwinner boards) need to load an 
> > ARM Trusted Firmware image beside the actual U-Boot proper. 
> > This can now be easily achieved by using the just extended SPL FIT 
> > loading support, so enable it in the Pine64 defconfig. 
> > Also add the FIT image as a build target to 64-bit sunxi board to 
> > trigger the respective Makefile rules. 
> > 
> > Signed-off-by: Andre Przywara <andre.przywara at arm.com> 
> > --- 
> >  configs/pine64_plus_defconfig  | 6 ++++++ 
> >  include/configs/sunxi-common.h | 4 ++++ 
> >  2 files changed, 10 insertions(+) 
> > 
> > diff --git a/configs/pine64_plus_defconfig b/configs/pine64_plus_defconfig 
> > index 92bda60..8a90579 100644 
> > --- a/configs/pine64_plus_defconfig 
> > +++ b/configs/pine64_plus_defconfig 
> > @@ -3,9 +3,14 @@ CONFIG_ARCH_SUNXI=y 
> >  CONFIG_MACH_SUN50I=y 
> >  CONFIG_RESERVE_ALLWINNER_BOOT0_HEADER=y 
> >  CONFIG_DEFAULT_DEVICE_TREE="sun50i-a64-pine64-plus" 
> > +CONFIG_OF_LIST="sun50i-a64-pine64 sun50i-a64-pine64-plus" 
> >  # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set 
> >  CONFIG_CONSOLE_MUX=y 
> >  CONFIG_SPL=y 
> > +CONFIG_FIT=y 
> > +CONFIG_SPL_FIT=y 
> > +CONFIG_SPL_LOAD_FIT=y 
> > +CONFIG_SPL_OF_LIBFDT=y 
>
> Again, this doesn't make any sense to enable it in *all* our 
> defconfigs. If this is something that should be enabled by default for 
> the A64 support, then do so in Kconfig directly. 

Here it's "strongly recommended" to use SPL FIT support, but not "must", as
someone may use U-Boot to execute their own code that wants EL3. In that
situation SPL FIT support shouldn't be present. U-Boot is not Linux Loader, so
we should give people the permission to disable it; however, U-Boot is mostly
used to load and execute Linux, so this should be defaultly enabled -- is this
correctly the meaning of defconfig?

>
> Maxime 
>
> -- 
> Maxime Ripard, Free Electrons 
> Embedded Linux and Kernel engineering 
> http://free-electrons.com