[coreboot-gerrit] [M] Change in coreboot[main]: vendorcode/amd/opensil/genoa: Implement console callback

[Felix Held (Code Review)]

Attention is currently required from: Arthur Heymans, Varshit Pandya.

Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/76512?usp=email )

Change subject: vendorcode/amd/opensil/genoa: Implement console callback
......................................................................

Patch Set 10:

(4 comments)

Patchset:

PS10: 
i'll rework this patch

File src/vendorcode/amd/opensil/genoa_poc/opensil_console.c:

https://review.coreboot.org/c/coreboot/+/76512/comment/13c2306b_440b52f7 :
PS10, Line 34: 	char prefix[60];
             : 	snprintf(prefix, sizeof(prefix), "%s%s:%d:", SilPrefix, (uintptr_t)Function, Line);
this doesn't need to be formatted to a buffer and we can use printk directly here

https://review.coreboot.org/c/coreboot/+/76512/comment/a1580856_883b625c :
PS10, Line 35: (uintptr_t)
this cast looks wrong

https://review.coreboot.org/c/coreboot/+/76512/comment/5c16cf90_aab8c75c :
PS10, Line 35: %d
%zu