Hi Simon, On Wed, Sep 25, 2019 at 10:59 PM Simon Glass <s...@chromium.org> wrote: > > Many Intel SoCs require a FIT in order to boot properly. Add an option to > include this and enable it by default. > > Signed-off-by: Simon Glass <s...@chromium.org> > --- > > arch/x86/Kconfig | 9 +++++++++ > arch/x86/dts/u-boot.dtsi | 6 ++++++ > 2 files changed, 15 insertions(+) > > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig > index 49b5f0f4ebd..556e26080de 100644 > --- a/arch/x86/Kconfig > +++ b/arch/x86/Kconfig > @@ -222,6 +222,15 @@ config SYS_X86_START16 > depends on X86_RESET_VECTOR > default 0xfffff800 > > +config X86_HAS_FIT > + bool
Please include a string to describe the option. > + default y This should not be y. Instead let's imply it in the SoC Kconfig file. > + help > + Enable inclusion of an Intel Firmware Interface Table (FIT) into the > + image. This table is supposed to point to microcode and the like. So > + far it is just a fixed table with the minimum set of headers, so > that > + it is actually present. > + > config X86_LOAD_FROM_32_BIT > bool "Boot from a 32-bit program" > help > diff --git a/arch/x86/dts/u-boot.dtsi b/arch/x86/dts/u-boot.dtsi > index 0e87b88e105..049f47c9ffd 100644 > --- a/arch/x86/dts/u-boot.dtsi > +++ b/arch/x86/dts/u-boot.dtsi > @@ -82,6 +82,12 @@ > u-boot-ucode { > align = <16>; > }; > +#ifdef CONFIG_X86_HAS_FIT > + intel-fit { > + }; > + intel-fit-ptr { > + }; > +#endif > #ifdef CONFIG_HAVE_MRC > intel-mrc { > offset = <CONFIG_X86_MRC_ADDR>; > -- Regards, Bin _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot