Frans Hendriks has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/46659 )
Change subject: security/tpm/tspi/crtm: Add line break to debug messages ......................................................................
security/tpm/tspi/crtm: Add line break to debug messages
Add line break at debug messages.
Tested on Facebook FBG1701
Change-Id: Idbfcd6ce7139efcb79e2980b366937e9fdcb3a4e Signed-off-by: Frans Hendriks fhendriks@eltan.com --- M src/security/tpm/tspi/crtm.c 1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/59/46659/1
diff --git a/src/security/tpm/tspi/crtm.c b/src/security/tpm/tspi/crtm.c index d9c62e1..eb07442 100644 --- a/src/security/tpm/tspi/crtm.c +++ b/src/security/tpm/tspi/crtm.c @@ -112,10 +112,10 @@ if (!tcpa_log_available()) { if (tspi_init_crtm() != VB2_SUCCESS) { printk(BIOS_WARNING, - "Initializing CRTM failed!"); + "Initializing CRTM failed!\n"); return 0; } - printk(BIOS_DEBUG, "CRTM initialized."); + printk(BIOS_DEBUG, "CRTM initialized.\n"); }
cbfsf_file_type(fh, &cbfs_type);
Christian Walter has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46659 )
Change subject: security/tpm/tspi/crtm: Add line break to debug messages ......................................................................
Patch Set 1: Code-Review+2
Wim Vervoorn has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46659 )
Change subject: security/tpm/tspi/crtm: Add line break to debug messages ......................................................................
Patch Set 1: Code-Review+2
Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/46659 )
Change subject: security/tpm/tspi/crtm: Add line break to debug messages ......................................................................
security/tpm/tspi/crtm: Add line break to debug messages
Add line break at debug messages.
Tested on Facebook FBG1701
Change-Id: Idbfcd6ce7139efcb79e2980b366937e9fdcb3a4e Signed-off-by: Frans Hendriks fhendriks@eltan.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/46659 Reviewed-by: Christian Walter christian.walter@9elements.com Reviewed-by: Wim Vervoorn wvervoorn@eltan.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/security/tpm/tspi/crtm.c 1 file changed, 2 insertions(+), 2 deletions(-)
Approvals: build bot (Jenkins): Verified Wim Vervoorn: Looks good to me, approved Christian Walter: Looks good to me, approved
diff --git a/src/security/tpm/tspi/crtm.c b/src/security/tpm/tspi/crtm.c index d9c62e1..eb07442 100644 --- a/src/security/tpm/tspi/crtm.c +++ b/src/security/tpm/tspi/crtm.c @@ -112,10 +112,10 @@ if (!tcpa_log_available()) { if (tspi_init_crtm() != VB2_SUCCESS) { printk(BIOS_WARNING, - "Initializing CRTM failed!"); + "Initializing CRTM failed!\n"); return 0; } - printk(BIOS_DEBUG, "CRTM initialized."); + printk(BIOS_DEBUG, "CRTM initialized.\n"); }
cbfsf_file_type(fh, &cbfs_type);