soc/intel/{apl,cnl,dnv,skl}: Skip ucode loading by FSP-T

It is a requirement for Firmware to have Firmware Interface Table (FIT),
which contains pointers to each microcode update.
The microcode update is loaded for all logical processors before reset vector.

FSPT_UPD.MicrocodeRegionBase and FSPT_UPD.MicrocodeRegionLength are
input parameters to TempRamInit API.
If these values are 0, FSP will not attempt to update microcode.

Since Gen-4 all IA-SoC has FIT loading ucode even before cpu reset in place
hence skipping FSP-T loading ucode after CPU reset options.

Also removed unused kconfig CONFIG_CPU_MICROCODE_CBFS_LOC and
CONFIG_CPU_MICROCODE_CBFS_LEN

Change-Id: I3a406fa0e2e62e3363c2960e173dc5f5f5ca0455
Signed-off-by: Subrata Banik <subrata.banik@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/37187
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: David Guckian
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>

5 files changed, 50 insertions, 20 deletions

diff --git a/src/soc/intel/apollolake/fspcar.c b/src/soc/intel/apollolake/fspcar.c
index 8b1089f397..a284116bac 100644
--- a/src/soc/intel/apollolake/fspcar.c
+++ b/src/soc/intel/apollolake/fspcar.c
@@ -25,6 +25,17 @@ const FSPT_UPD temp_ram_init_params = {
 	.FsptCommonUpd = {
 		.Revision = 0,
 		.Reserved = {0},
+		/*
+		 * It is a requirement for firmware to have Firmware Interface Table
+		 * (FIT), which contains pointers to each microcode update.
+		 * The microcode update is loaded for all logical processors before
+		 * cpu reset vector.
+		 *
+		 * All SoC since Gen-4 has above mechanism in place to load microcode
+		 * even before hitting CPU reset vector. Hence skipping FSP-T loading
+		 * microcode after CPU reset by passing '0' value to
+		 * FSPT_UPD.MicrocodeRegionBase and FSPT_UPD.MicrocodeRegionLength.
+		 */
 		.MicrocodeRegionBase = 0,
 		.MicrocodeRegionLength = 0,
 		.CodeRegionBase =
diff --git a/src/soc/intel/cannonlake/bootblock/bootblock.c b/src/soc/intel/cannonlake/bootblock/bootblock.c
index 653ba30563..9f8539766c 100644
--- a/src/soc/intel/cannonlake/bootblock/bootblock.c
+++ b/src/soc/intel/cannonlake/bootblock/bootblock.c
@@ -30,10 +30,19 @@ const FSPT_UPD temp_ram_init_params = {
 		.Reserved = {0},
 	},
 	.FsptCoreUpd = {
-		.MicrocodeRegionBase =
-			(uint32_t)CONFIG_CPU_MICROCODE_CBFS_LOC,
-		.MicrocodeRegionSize =
-			(uint32_t)CONFIG_CPU_MICROCODE_CBFS_LEN,
+		/*
+		 * It is a requirement for firmware to have Firmware Interface Table
+		 * (FIT), which contains pointers to each microcode update.
+		 * The microcode update is loaded for all logical processors before
+		 * cpu reset vector.
+		 *
+		 * All SoC since Gen-4 has above mechanism in place to load microcode
+		 * even before hitting CPU reset vector. Hence skipping FSP-T loading
+		 * microcode after CPU reset by passing '0' value to
+		 * FSPT_UPD.MicrocodeRegionBase and FSPT_UPD.MicrocodeRegionLength.
+		 */
+		.MicrocodeRegionBase = 0,
+		.MicrocodeRegionLength = 0,
 		.CodeRegionBase =
 			(uint32_t)(0x100000000ULL - CONFIG_ROM_SIZE),
 		.CodeRegionSize = (uint32_t)CONFIG_ROM_SIZE,
diff --git a/src/soc/intel/denverton_ns/Kconfig b/src/soc/intel/denverton_ns/Kconfig
index 0ce0d5bdd1..713aae60df 100644
--- a/src/soc/intel/denverton_ns/Kconfig
+++ b/src/soc/intel/denverton_ns/Kconfig
@@ -108,14 +108,6 @@ config DCACHE_BSP_STACK_SIZE
 	hex
 	default 0x10000
 
-config CPU_MICROCODE_CBFS_LOC
-	hex
-	default 0xfff20040
-
-config CPU_MICROCODE_CBFS_LEN
-	hex
-	default 0x0ff80
-
 config CPU_BCLK_MHZ
 	int
 	default 100
diff --git a/src/soc/intel/denverton_ns/bootblock/bootblock.c b/src/soc/intel/denverton_ns/bootblock/bootblock.c
index f75de1f2d0..47c76b5acd 100644
--- a/src/soc/intel/denverton_ns/bootblock/bootblock.c
+++ b/src/soc/intel/denverton_ns/bootblock/bootblock.c
@@ -31,10 +31,19 @@ const FSPT_UPD temp_ram_init_params = {
 			.Reserved = {0},
 	},
 	.FsptCoreUpd = {
-			.MicrocodeRegionBase =
-				(UINT32)CONFIG_CPU_MICROCODE_CBFS_LOC,
-			.MicrocodeRegionLength =
-				(UINT32)CONFIG_CPU_MICROCODE_CBFS_LEN,
+			/*
+			 * It is a requirement for firmware to have Firmware Interface Table
+			 * (FIT), which contains pointers to each microcode update.
+			 * The microcode update is loaded for all logical processors before
+			 * cpu reset vector.
+			 *
+			 * All SoC since Gen-4 has above mechanism in place to load microcode
+			 * even before hitting CPU reset vector. Hence skipping FSP-T loading
+			 * microcode after CPU reset by passing '0' value to
+			 * FSPT_UPD.MicrocodeRegionBase and FSPT_UPD.MicrocodeRegionLength.
+			 */
+			.MicrocodeRegionBase = 0,
+			.MicrocodeRegionLength = 0,
 			.CodeRegionBase =
 				(UINT32)(0x100000000ULL - CONFIG_ROM_SIZE),
 			.CodeRegionLength = (UINT32)CONFIG_ROM_SIZE,
diff --git a/src/soc/intel/skylake/fspcar.c b/src/soc/intel/skylake/fspcar.c
index a4c3726492..0d27f57698 100644
--- a/src/soc/intel/skylake/fspcar.c
+++ b/src/soc/intel/skylake/fspcar.c
@@ -23,10 +23,19 @@ const FSPT_UPD temp_ram_init_params = {
 		.Reserved = {0},
 	},
 	.FsptCoreUpd = {
-		.MicrocodeRegionBase =
-			(uint32_t)CONFIG_CPU_MICROCODE_CBFS_LOC,
-		.MicrocodeRegionSize =
-			(uint32_t)CONFIG_CPU_MICROCODE_CBFS_LEN,
+		/*
+		 * It is a requirement for firmware to have Firmware Interface Table
+		 * (FIT), which contains pointers to each microcode update.
+		 * The microcode update is loaded for all logical processors before
+		 * cpu reset vector.
+		 *
+		 * All SoC since Gen-4 has above mechanism in place to load microcode
+		 * even before hitting CPU reset vector. Hence skipping FSP-T loading
+		 * microcode after CPU reset by passing '0' value to
+		 * FSPT_UPD.MicrocodeRegionBase and FSPT_UPD.MicrocodeRegionLength.
+		 */
+		.MicrocodeRegionBase = 0,
+		.MicrocodeRegionLength = 0,
 		.CodeRegionBase =
 			(uint32_t)(0x100000000ULL - CONFIG_ROM_SIZE),
 		.CodeRegionSize = (uint32_t)CONFIG_ROM_SIZE,