diff options
| author | Elyes Haouas <ehaouas@noos.fr> | 2022-11-09 15:05:23 +0100 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2022-11-10 21:31:18 +0000 |
| commit | aba1c945cd44332521df49228b883c7081723812 (patch) | |
| tree | 4b0c2c61ba4f222ebe8b48e41c0b038ad0ffd873 /src/soc/amd/mendocino | |
| parent | 1d3c2e65724ed92eeef723c1acdc86ba91cbd682 (diff) | |
/: Remove "ERROR: "/"WARNING: " prefixes from log messages
It is no longer necessary to explicitly add "ERROR: "/"WARNING: " in
front of every BIOS_ERR/BIOS_WARN message.
Change-Id: I22ee6ae15c3d3a848853c5460b3b3c1795adf2f5
Signed-off-by: Elyes Haouas <ehaouas@noos.fr>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/69405
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Fred Reitberger <reitbergerfred@gmail.com>
Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
Diffstat (limited to 'src/soc/amd/mendocino')
| -rw-r--r-- | src/soc/amd/mendocino/i2c.c | 2 | ||||
| -rw-r--r-- | src/soc/amd/mendocino/psp_verstage/chipset.c | 6 | ||||
| -rw-r--r-- | src/soc/amd/mendocino/root_complex.c | 4 | ||||
| -rw-r--r-- | src/soc/amd/mendocino/smihandler.c | 2 |
4 files changed, 7 insertions, 7 deletions
diff --git a/src/soc/amd/mendocino/i2c.c b/src/soc/amd/mendocino/i2c.c index 75efe0c45f..92e46b9020 100644 --- a/src/soc/amd/mendocino/i2c.c +++ b/src/soc/amd/mendocino/i2c.c @@ -32,7 +32,7 @@ static struct soc_i2c_ctrlr_info i2c_ctrlr[I2C_CTRLR_COUNT] = { void i2c_set_bar(unsigned int bus, uintptr_t bar) { if (bus >= ARRAY_SIZE(i2c_ctrlr)) { - printk(BIOS_ERR, "Error: i2c index out of bounds: %u.", bus); + printk(BIOS_ERR, "i2c index out of bounds: %u.", bus); return; } diff --git a/src/soc/amd/mendocino/psp_verstage/chipset.c b/src/soc/amd/mendocino/psp_verstage/chipset.c index b89627981c..fe79979fbe 100644 --- a/src/soc/amd/mendocino/psp_verstage/chipset.c +++ b/src/soc/amd/mendocino/psp_verstage/chipset.c @@ -24,7 +24,7 @@ void update_psp_fw_hash_table(const char *fname) uint32_t len; if (!spi_ptr) { - printk(BIOS_ERR, "Error: AMD Firmware hash table %s not found\n", fname); + printk(BIOS_ERR, "AMD Firmware hash table %s not found\n", fname); /* * If we don't supply hash table, the PSP will refuse to boot. * So returning here is safe to do. @@ -36,7 +36,7 @@ void update_psp_fw_hash_table(const char *fname) if (hash_table.no_of_entries_256 > MAX_NUM_HASH_ENTRIES || hash_table.no_of_entries_384 > MAX_NUM_HASH_ENTRIES) { - printk(BIOS_ERR, "Error: Too many entries in AMD Firmware hash table" + printk(BIOS_ERR, "Too many entries in AMD Firmware hash table" " (SHA256:%d, SHA384:%d)\n", hash_table.no_of_entries_256, hash_table.no_of_entries_384); return; @@ -44,7 +44,7 @@ void update_psp_fw_hash_table(const char *fname) if (hash_table.no_of_entries_256 == 0 && hash_table.no_of_entries_384 == 0) { - printk(BIOS_ERR, "Error: No entries in AMD Firmware hash table" + printk(BIOS_ERR, "No entries in AMD Firmware hash table" " (SHA256:%d, SHA384:%d)\n", hash_table.no_of_entries_256, hash_table.no_of_entries_384); return; diff --git a/src/soc/amd/mendocino/root_complex.c b/src/soc/amd/mendocino/root_complex.c index 67686139b4..7feb7f7500 100644 --- a/src/soc/amd/mendocino/root_complex.c +++ b/src/soc/amd/mendocino/root_complex.c @@ -158,7 +158,7 @@ static void read_resources(struct device *dev) mmconf_resource(dev, idx++); if (!hob) { - printk(BIOS_ERR, "Error: %s incomplete because no HOB list was found\n", + printk(BIOS_ERR, "%s incomplete because no HOB list was found\n", __func__); return; } @@ -180,7 +180,7 @@ static void read_resources(struct device *dev) else if (res->type == EFI_RESOURCE_MEMORY_RESERVED) reserved_ram_resource_kb(dev, idx++, res->addr / KiB, res->length / KiB); else - printk(BIOS_ERR, "Error: failed to set resources for type %d\n", + printk(BIOS_ERR, "Failed to set resources for type %d\n", res->type); } diff --git a/src/soc/amd/mendocino/smihandler.c b/src/soc/amd/mendocino/smihandler.c index c05f3eea3e..8e32b5d543 100644 --- a/src/soc/amd/mendocino/smihandler.c +++ b/src/soc/amd/mendocino/smihandler.c @@ -100,7 +100,7 @@ static void fch_slp_typ_handler(void) psp_notify_sx_info(ACPI_S3); smu_sx_entry(); /* Leave SlpTypeEn clear, SMU will set */ - printk(BIOS_ERR, "Error: System did not go to sleep\n"); + printk(BIOS_ERR, "System did not go to sleep\n"); hlt(); } } |