summaryrefslogtreecommitdiff
path: root/src/soc/amd
diff options
context:
space:
mode:
authorFelix Held <felix-coreboot@felixheld.de>2021-02-23 12:36:16 +0100
committerFelix Held <felix-coreboot@felixheld.de>2021-02-25 19:58:32 +0000
commitec098b5e7f3546356bdd3a215711745c336bd3d8 (patch)
treee2a8f05a7a401672eb3a2ddc337f88d94ff7e515 /src/soc/amd
parent28e2353e07801e122da879a5848e5ce00af42824 (diff)
soc/amd: only print CPU family and model in bootblock
Printing this in both bootblock and romstage is redundant, since the CPU family and model aren't expected to change between bootblock and romstage entry. Signed-off-by: Felix Held <felix-coreboot@felixheld.de> Reported-by: Paul Menzel <pmenzel@molgen.mpg.de> Change-Id: Id7c6aea0d8a6dac39114593584e534661faea89d Reviewed-on: https://review.coreboot.org/c/coreboot/+/51062 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Raul Rangel <rrangel@chromium.org>
Diffstat (limited to 'src/soc/amd')
-rw-r--r--src/soc/amd/cezanne/romstage.c3
-rw-r--r--src/soc/amd/picasso/romstage.c2
2 files changed, 0 insertions, 5 deletions
diff --git a/src/soc/amd/cezanne/romstage.c b/src/soc/amd/cezanne/romstage.c
index 773c6a97d5..2740d8c74c 100644
--- a/src/soc/amd/cezanne/romstage.c
+++ b/src/soc/amd/cezanne/romstage.c
@@ -29,9 +29,6 @@ asmlinkage void car_stage_entry(void)
post_code(0x41);
- u32 val = cpuid_eax(1);
- printk(BIOS_DEBUG, "Family_Model: %08x\n", val);
-
fsp_memory_init(acpi_is_wakeup_s3());
memmap_stash_early_dram_usage();
diff --git a/src/soc/amd/picasso/romstage.c b/src/soc/amd/picasso/romstage.c
index aa8da1ea69..c085a5355f 100644
--- a/src/soc/amd/picasso/romstage.c
+++ b/src/soc/amd/picasso/romstage.c
@@ -147,8 +147,6 @@ asmlinkage void car_stage_entry(void)
console_init();
post_code(0x42);
- u32 val = cpuid_eax(1);
- printk(BIOS_DEBUG, "Family_Model: %08x\n", val);
/* Snapshot chipset state prior to any FSP call. */
fill_chipset_state();