Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/74305 )
Change subject: soc/intel/common: Update cpu_apic_info_type struct ......................................................................
soc/intel/common: Update cpu_apic_info_type struct
The patch updates total cpu count variable and total P-core count in cpu_apic_info_type structure to `unsigned short int` to address more cores.
TEST=Verify the build on Rex
Signed-off-by: Sridhar Siricilla sridhar.siricilla@intel.com Change-Id: I46239cc7ad9870e7134955af56b9f6625be2b002 Reviewed-on: https://review.coreboot.org/c/coreboot/+/74305 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Kyösti Mälkki kyosti.malkki@gmail.com Reviewed-by: Subrata Banik subratabanik@google.com --- M src/soc/intel/common/block/acpi/cpu_hybrid.c 1 file changed, 22 insertions(+), 2 deletions(-)
Approvals: build bot (Jenkins): Verified Kyösti Mälkki: Looks good to me, approved Subrata Banik: Looks good to me, approved
diff --git a/src/soc/intel/common/block/acpi/cpu_hybrid.c b/src/soc/intel/common/block/acpi/cpu_hybrid.c index e18c288..6e8b641 100644 --- a/src/soc/intel/common/block/acpi/cpu_hybrid.c +++ b/src/soc/intel/common/block/acpi/cpu_hybrid.c @@ -28,14 +28,14 @@ int32_t apic_ids[CONFIG_MAX_CPUS];
/* Total CPU count */ - uint8_t total_cpu_cnt; + uint16_t total_cpu_cnt;
/* * Total Performance core count. This will be used * to identify the start of Efficient Cores's * APIC ID list */ - uint8_t perf_cpu_cnt; + uint16_t perf_cpu_cnt; };
static struct cpu_apic_info_type cpu_apic_info;