Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/69623 )
(
1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: soc/amd/common/pi/def_callouts.c: Fix log messages ......................................................................
soc/amd/common/pi/def_callouts.c: Fix log messages
It is no longer necessary to explicitly add "Warning" in front of BIOS_WARNING message.
Change-Id: If1645180dd98ff5a1661fd568554de5831ef237e Signed-off-by: Elyes Haouas ehaouas@noos.fr Reviewed-on: https://review.coreboot.org/c/coreboot/+/69623 Reviewed-by: Felix Held felix-coreboot@felixheld.de Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/soc/amd/common/pi/def_callouts.c 1 file changed, 19 insertions(+), 3 deletions(-)
Approvals: build bot (Jenkins): Verified Felix Held: Looks good to me, approved
diff --git a/src/soc/amd/common/pi/def_callouts.c b/src/soc/amd/common/pi/def_callouts.c index 223abe3..ee70951 100644 --- a/src/soc/amd/common/pi/def_callouts.c +++ b/src/soc/amd/common/pi/def_callouts.c @@ -147,7 +147,7 @@ AGESA_STATUS __weak platform_PcieSlotResetControl(uint32_t Func, uintptr_t Data, void *ConfigPtr) { - printk(BIOS_WARNING, "Warning - AGESA callout: %s not supported\n", + printk(BIOS_WARNING, "AGESA callout: %s not supported\n", __func__); return AGESA_UNSUPPORTED; } @@ -231,7 +231,7 @@ AGESA_STATUS agesa_WaitForAllApsFinished(uint32_t Func, uintptr_t Data, void *ConfigPtr) { - printk(BIOS_WARNING, "Warning - AGESA callout: %s not supported\n", + printk(BIOS_WARNING, "AGESA callout: %s not supported\n", __func__); AGESA_STATUS Status = AGESA_UNSUPPORTED;
@@ -240,7 +240,7 @@
AGESA_STATUS agesa_IdleAnAp(uint32_t Func, uintptr_t Data, void *ConfigPtr) { - printk(BIOS_WARNING, "Warning - AGESA callout: %s no supported\n", + printk(BIOS_WARNING, "AGESA callout: %s not supported\n", __func__); AGESA_STATUS Status = AGESA_UNSUPPORTED;