Felix Held has uploaded this change for review.

View Change

vc/amd/opensil/genoa_poc/opensil_console: fix host debug print function

Since we pass va_list list to the print function, we need to use vprintk
instead of printk. Earlier versions of this code used vsnprintf and a
local buffer, but when that code was reworked to not need the temporary
buffer, it was replaced by printk instead of the correct vprintk.

Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: Ia931cc80dea5b7eabb75cfb19f8baa9a09cd2dbf
---
M src/vendorcode/amd/opensil/genoa_poc/opensil_console.c
1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/03/80203/1
diff --git a/src/vendorcode/amd/opensil/genoa_poc/opensil_console.c b/src/vendorcode/amd/opensil/genoa_poc/opensil_console.c
index 55a3521..35e5eeb 100644
--- a/src/vendorcode/amd/opensil/genoa_poc/opensil_console.c
+++ b/src/vendorcode/amd/opensil/genoa_poc/opensil_console.c
@@ -39,6 +39,6 @@
/* print formatted message */
va_list args;
va_start(args, Line);
- printk(loglevel, Message, args);
+ vprintk(loglevel, Message, args);
va_end(args);
}

To view, visit change 80203. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ia931cc80dea5b7eabb75cfb19f8baa9a09cd2dbf
Gerrit-Change-Number: 80203
Gerrit-PatchSet: 1
Gerrit-Owner: Felix Held <felix-coreboot@felixheld.de>
Gerrit-MessageType: newchange