Re: [PATCH v6 053/102] x86: Disable microcode section for FSP2

2019-12-07 Thread Bin Meng
On Sat, Dec 7, 2019 at 12:49 PM Simon Glass  wrote:
>
> At present we don't support loading microcode with FSP2. The correct way
> to do this is by adding it to the FIT. For now, disable including
> microcode in the image.
>
> Signed-off-by: Simon Glass 
> Reviewed-by: Bin Meng 
> ---
>
> Changes in v6: None
> Changes in v5: None
> Changes in v4: None
> Changes in v3:
> - Drop unnecessary #else part of CONFIG_HAVE_MICROCODE
>
> Changes in v2: None
>
>  arch/x86/Kconfig | 4 
>  arch/x86/dts/u-boot.dtsi | 7 +++
>  2 files changed, 11 insertions(+)
>

applied to u-boot-x86/next, thanks!


[PATCH v6 053/102] x86: Disable microcode section for FSP2

2019-12-06 Thread Simon Glass
At present we don't support loading microcode with FSP2. The correct way
to do this is by adding it to the FIT. For now, disable including
microcode in the image.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng 
---

Changes in v6: None
Changes in v5: None
Changes in v4: None
Changes in v3:
- Drop unnecessary #else part of CONFIG_HAVE_MICROCODE

Changes in v2: None

 arch/x86/Kconfig | 4 
 arch/x86/dts/u-boot.dtsi | 7 +++
 2 files changed, 11 insertions(+)

diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
index 44f7f0ab03..64f167306b 100644
--- a/arch/x86/Kconfig
+++ b/arch/x86/Kconfig
@@ -588,6 +588,10 @@ config HAVE_REFCODE
   broadwell) U-Boot will be missing some critical setup steps.
   Various peripherals may fail to work.
 
+config HAVE_MICROCODE
+   bool
+   default y if !FSP_VERSION2
+
 config SMP
bool "Enable Symmetric Multiprocessing"
default n
diff --git a/arch/x86/dts/u-boot.dtsi b/arch/x86/dts/u-boot.dtsi
index 33441c7c80..850fe3ac11 100644
--- a/arch/x86/dts/u-boot.dtsi
+++ b/arch/x86/dts/u-boot.dtsi
@@ -37,11 +37,13 @@
};
 #endif
 #ifdef CONFIG_TPL
+#ifdef CONFIG_HAVE_MICROCODE
u-boot-tpl-with-ucode-ptr {
offset = ;
};
u-boot-tpl-dtb {
};
+#endif
u-boot-spl {
offset = ;
};
@@ -77,11 +79,16 @@
offset = ;
};
 #endif
+#ifdef CONFIG_HAVE_MICROCODE
u-boot-dtb-with-ucode {
};
u-boot-ucode {
align = <16>;
};
+#else
+   u-boot-dtb {
+   };
+#endif
 #ifdef CONFIG_HAVE_X86_FIT
intel-fit {
};
-- 
2.24.0.393.g34dc348eaf-goog