Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/46299 )
Change subject: vendorcode/google/dram_part_num: Lower the severity of debug statement ......................................................................
vendorcode/google/dram_part_num: Lower the severity of debug statement
DRAM part number may not be provisioned in CBI during early stages of development. Logging the debug statement with error severity interferes with some of the test tools. Lower the severity of debug statement to BIOS_DEBUG.
BUG=b:170529094 TEST=Build and boot to ChromeOS in Drawlat.
Change-Id: Ib0c707ec6478060d6e18ea01cc467dfda00a6d42 Signed-off-by: Karthikeyan Ramasubramanian kramasub@google.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/46299 Reviewed-by: Angel Pons th3fanbus@gmail.com Reviewed-by: Justin TerAvest teravest@chromium.org Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/vendorcode/google/chromeos/dram_part_num_override.c 1 file changed, 2 insertions(+), 2 deletions(-)
Approvals: build bot (Jenkins): Verified Justin TerAvest: Looks good to me, approved Angel Pons: Looks good to me, approved
diff --git a/src/vendorcode/google/chromeos/dram_part_num_override.c b/src/vendorcode/google/chromeos/dram_part_num_override.c index d624e13..4b934fe 100644 --- a/src/vendorcode/google/chromeos/dram_part_num_override.c +++ b/src/vendorcode/google/chromeos/dram_part_num_override.c @@ -16,8 +16,8 @@ if (part_num_state == PART_NUM_NOT_READ) { if (google_chromeec_cbi_get_dram_part_num(part_num_store, sizeof(part_num_store)) < 0) { - printk(BIOS_ERR, - "ERROR: Couldn't obtain DRAM part number from CBI\n"); + printk(BIOS_DEBUG, + "Couldn't obtain DRAM part number from CBI\n"); part_num_state = PART_NUM_NOT_IN_CBI; } else { part_num_state = PART_NUM_AVAILABLE;