Change in coreboot[master]: soc/intel/common: Replace cse_bp with cse_lite in all console logs
Sridhar Siricilla has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/41972 )
Change subject: soc/intel/common: Replace cse_bp with cse_lite in all console logs ......................................................................
soc/intel/common: Replace cse_bp with cse_lite in all console logs
Replace cse_bp with cse_lite in all serial console logs in the cse_lite.c
TEST=Verified on hatch
Signed-off-by: Sridhar Siricilla sridhar.siricilla@intel.com Change-Id: I3fc677c9ec1962199c91cc310d7695dded4e0ba0 --- M src/soc/intel/common/block/cse/cse_lite.c 1 file changed, 14 insertions(+), 14 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/72/41972/1
diff --git a/src/soc/intel/common/block/cse/cse_lite.c b/src/soc/intel/common/block/cse/cse_lite.c index 1f9e2ce..6372248 100644 --- a/src/soc/intel/common/block/cse/cse_lite.c +++ b/src/soc/intel/common/block/cse/cse_lite.c @@ -222,19 +222,19 @@ };
if (!cse_is_bp_cmd_info_possible()) { - printk(BIOS_ERR, "cse_bp: CSE does not meet prerequisites\n"); + printk(BIOS_ERR, "cse_lite: CSE does not meet prerequisites\n"); return false; }
size_t resp_size = sizeof(struct get_bp_info_rsp);
if (!heci_send_receive(&info_req, sizeof(info_req), bp_info_rsp, &resp_size)) { - printk(BIOS_ERR, "cse_bp: Could not get partition info\n"); + printk(BIOS_ERR, "cse_lite: Could not get partition info\n"); return false; }
if (bp_info_rsp->hdr.result) { - printk(BIOS_ERR, "cse_bp: Get partition info resp failed: %d\n", + printk(BIOS_ERR, "cse_lite: Get partition info resp failed: %d\n", bp_info_rsp->hdr.result); return false; } @@ -266,14 +266,14 @@ };
if (bp != RO && bp != RW) { - printk(BIOS_ERR, "cse_bp: Incorrect partition id(%d) is provided", bp); + printk(BIOS_ERR, "cse_lite: Incorrect partition id(%d) is provided", bp); return false; }
- printk(BIOS_INFO, "cse_bp: Set Boot Partition Info Command (%s)\n", GET_BP_STR(bp)); + printk(BIOS_INFO, "cse_lite: Set Boot Partition Info Command (%s)\n", GET_BP_STR(bp));
if (!cse_is_bp_cmd_info_possible()) { - printk(BIOS_ERR, "cse_bp: CSE does not meet prerequisites\n"); + printk(BIOS_ERR, "cse_lite: CSE does not meet prerequisites\n"); return false; }
@@ -284,7 +284,7 @@ return false;
if (switch_resp.result) { - printk(BIOS_ERR, "cse_bp: Set Boot Partition Info Response Failed: %d\n", + printk(BIOS_ERR, "cse_lite: Set Boot Partition Info Response Failed: %d\n", switch_resp.result); return false; } @@ -302,7 +302,7 @@
do_global_reset();
- die("cse_bp: Failed to reset system\n"); + die("cse_lite: Failed to reset system\n");
/* Control never reaches here */ return false; @@ -317,7 +317,7 @@
if (rw_bp->status == BP_STATUS_PARTITION_NOT_PRESENT || rw_bp->status == BP_STATUS_GENERAL_FAILURE) { - printk(BIOS_ERR, "cse_bp: RW BP (status:%u) is not valid\n", rw_bp->status); + printk(BIOS_ERR, "cse_lite: RW BP (status:%u) is not valid\n", rw_bp->status); return false; } return true; @@ -333,28 +333,28 @@ static struct get_bp_info_rsp cse_bp_info;
if (vboot_recovery_mode_enabled()) { - printk(BIOS_DEBUG, "cse_bp: Skip switching to RW in the recovery path\n"); + printk(BIOS_DEBUG, "cse_lite: Skip switching to RW in the recovery path\n"); return; }
/* If CSE SKU type is not Lite, skip enabling CSE Lite SKU */ if (!cse_is_hfs3_fw_sku_lite()) { - printk(BIOS_ERR, "cse_bp: Not a CSE Lite SKU\n"); + printk(BIOS_ERR, "cse_lite: Not a CSE Lite SKU\n"); return; }
if (!cse_get_bp_info(&cse_bp_info)) { - printk(BIOS_ERR, "cse_bp: Failed to get CSE boot partition info\n"); + printk(BIOS_ERR, "cse_lite: Failed to get CSE boot partition info\n"); cse_trigger_recovery(CSE_LITE_SKU_COMMUNICATION_ERROR); }
if (!cse_is_rw_info_valid(&cse_bp_info.bp_info)) { - printk(BIOS_ERR, "cse_bp: CSE RW partition is not valid\n"); + printk(BIOS_ERR, "cse_lite: CSE RW partition is not valid\n"); cse_trigger_recovery(CSE_LITE_SKU_RW_JUMP_ERROR); }
if (!cse_boot_to_rw(&cse_bp_info.bp_info)) { - printk(BIOS_ERR, "cse_bp: Failed to switch to RW\n"); + printk(BIOS_ERR, "cse_lite: Failed to switch to RW\n"); cse_trigger_recovery(CSE_LITE_SKU_RW_SWITCH_ERROR); } }
V Sowmya has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41972 )
Change subject: soc/intel/common: Replace cse_bp with cse_lite in all console logs ......................................................................
Patch Set 1: Code-Review+2
Rizwan Qureshi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41972 )
Change subject: soc/intel/common: Replace cse_bp with cse_lite in all console logs ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/41972/1/src/soc/intel/common/block/... File src/soc/intel/common/block/cse/cse_lite.c:
https://review.coreboot.org/c/coreboot/+/41972/1/src/soc/intel/common/block/... PS1, Line 169: ME do we want to chnage these as well?
Hello V Sowmya, build bot (Jenkins), Furquan Shaikh, Rizwan Qureshi, Sowmya V, Patrick Rudolph,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/41972
to look at the new patch set (#2).
Change subject: soc/intel/common: Replace cse_bp with cse_lite in all console logs ......................................................................
soc/intel/common: Replace cse_bp with cse_lite in all console logs
Replace cse_bp and me with cse_lite in all serial console logs in the cse_lite.c.
TEST=Verified on hatch
Signed-off-by: Sridhar Siricilla sridhar.siricilla@intel.com Change-Id: I3fc677c9ec1962199c91cc310d7695dded4e0ba0 --- M src/soc/intel/common/block/cse/cse_lite.c 1 file changed, 20 insertions(+), 20 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/72/41972/2
Sridhar Siricilla has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41972 )
Change subject: soc/intel/common: Replace cse_bp with cse_lite in all console logs ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/41972/1/src/soc/intel/common/block/... File src/soc/intel/common/block/cse/cse_lite.c:
https://review.coreboot.org/c/coreboot/+/41972/1/src/soc/intel/common/block/... PS1, Line 169: ME
do we want to chnage these as well?
Ack
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41972 )
Change subject: soc/intel/common: Replace cse_bp with cse_lite in all console logs ......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/c/coreboot/+/41972/2/src/soc/intel/common/block/... File src/soc/intel/common/block/cse/cse_lite.c:
https://review.coreboot.org/c/coreboot/+/41972/2/src/soc/intel/common/block/... PS2, Line 169: printk(BIOS_DEBUG, "cse_lite: Number of partitions = %d\n", cse_bp_info->total_number_of_bp); line over 96 characters
https://review.coreboot.org/c/coreboot/+/41972/2/src/soc/intel/common/block/... PS2, Line 170: printk(BIOS_DEBUG, "cse_lite: Current partition = %s\n", GET_BP_STR(cse_bp_info->current_bp)); line over 96 characters
Hello V Sowmya, build bot (Jenkins), Furquan Shaikh, Rizwan Qureshi, Sowmya V, Patrick Rudolph,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/41972
to look at the new patch set (#3).
Change subject: soc/intel/common: Replace cse_bp with cse_lite in all console logs ......................................................................
soc/intel/common: Replace cse_bp with cse_lite in all console logs
Replace cse_bp and me with cse_lite in all serial console logs in the cse_lite.c.
TEST=Verified on hatch
Signed-off-by: Sridhar Siricilla sridhar.siricilla@intel.com Change-Id: I3fc677c9ec1962199c91cc310d7695dded4e0ba0 --- M src/soc/intel/common/block/cse/cse_lite.c 1 file changed, 22 insertions(+), 20 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/72/41972/3
V Sowmya has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41972 )
Change subject: soc/intel/common: Replace cse_bp with cse_lite in all console logs ......................................................................
Patch Set 3: Code-Review+2
Rizwan Qureshi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41972 )
Change subject: soc/intel/common: Replace cse_bp with cse_lite in all console logs ......................................................................
Patch Set 3: Code-Review+2
V Sowmya has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41972 )
Change subject: soc/intel/common: Replace cse_bp with cse_lite in all console logs ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/41972/3/src/soc/intel/common/block/... File src/soc/intel/common/block/cse/cse_lite.c:
https://review.coreboot.org/c/coreboot/+/41972/3/src/soc/intel/common/block/... PS3, Line 178: printk(BIOS_DEBUG, "cse_lite: %s version = %d.%d.%d.%d (Status=0x%x, Start=0x%x, End=0x%x)\n", Indentation is missing for this line.
https://review.coreboot.org/c/coreboot/+/41972/3/src/soc/intel/common/block/... PS3, Line 185: printk(BIOS_DEBUG, "cse_lite: %s version = %d.%d.%d.%d (Status=0x%x, Start=0x%x, End=0x%x)\n", same as above.
V Sowmya has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41972 )
Change subject: soc/intel/common: Replace cse_bp with cse_lite in all console logs ......................................................................
Patch Set 3: -Code-Review
V Sowmya has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41972 )
Change subject: soc/intel/common: Replace cse_bp with cse_lite in all console logs ......................................................................
Patch Set 3: Code-Review+2
(2 comments)
Sorry. it was my mistake. Gave +2 again.
https://review.coreboot.org/c/coreboot/+/41972/3/src/soc/intel/common/block/... File src/soc/intel/common/block/cse/cse_lite.c:
https://review.coreboot.org/c/coreboot/+/41972/3/src/soc/intel/common/block/... PS3, Line 178: printk(BIOS_DEBUG, "cse_lite: %s version = %d.%d.%d.%d (Status=0x%x, Start=0x%x, End=0x%x)\n",
Indentation is missing for this line.
Done
https://review.coreboot.org/c/coreboot/+/41972/3/src/soc/intel/common/block/... PS3, Line 185: printk(BIOS_DEBUG, "cse_lite: %s version = %d.%d.%d.%d (Status=0x%x, Start=0x%x, End=0x%x)\n",
same as above.
Done
Sridhar Siricilla has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41972 )
Change subject: soc/intel/common: Replace cse_bp with cse_lite in all console logs ......................................................................
Patch Set 3:
(4 comments)
https://review.coreboot.org/c/coreboot/+/41972/2/src/soc/intel/common/block/... File src/soc/intel/common/block/cse/cse_lite.c:
https://review.coreboot.org/c/coreboot/+/41972/2/src/soc/intel/common/block/... PS2, Line 169: printk(BIOS_DEBUG, "cse_lite: Number of partitions = %d\n", cse_bp_info->total_number_of_bp);
line over 96 characters
Done
https://review.coreboot.org/c/coreboot/+/41972/2/src/soc/intel/common/block/... PS2, Line 170: printk(BIOS_DEBUG, "cse_lite: Current partition = %s\n", GET_BP_STR(cse_bp_info->current_bp));
line over 96 characters
Done
https://review.coreboot.org/c/coreboot/+/41972/3/src/soc/intel/common/block/... File src/soc/intel/common/block/cse/cse_lite.c:
https://review.coreboot.org/c/coreboot/+/41972/3/src/soc/intel/common/block/... PS3, Line 178: printk(BIOS_DEBUG, "cse_lite: %s version = %d.%d.%d.%d (Status=0x%x, Start=0x%x, End=0x%x)\n",
Indentation is missing for this line.
it seems to be indentation is ok.
https://review.coreboot.org/c/coreboot/+/41972/3/src/soc/intel/common/block/... PS3, Line 185: printk(BIOS_DEBUG, "cse_lite: %s version = %d.%d.%d.%d (Status=0x%x, Start=0x%x, End=0x%x)\n",
same as above.
it seems to be indentation is ok.
V Sowmya has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41972 )
Change subject: soc/intel/common: Replace cse_bp with cse_lite in all console logs ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/41972/3/src/soc/intel/common/block/... File src/soc/intel/common/block/cse/cse_lite.c:
https://review.coreboot.org/c/coreboot/+/41972/3/src/soc/intel/common/block/... PS3, Line 178: printk(BIOS_DEBUG, "cse_lite: %s version = %d.%d.%d.%d (Status=0x%x, Start=0x%x, End=0x%x)\n",
it seems to be indentation is ok.
Done
https://review.coreboot.org/c/coreboot/+/41972/3/src/soc/intel/common/block/... PS3, Line 185: printk(BIOS_DEBUG, "cse_lite: %s version = %d.%d.%d.%d (Status=0x%x, Start=0x%x, End=0x%x)\n",
it seems to be indentation is ok.
Done
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41972 )
Change subject: soc/intel/common: Replace cse_bp with cse_lite in all console logs ......................................................................
Patch Set 3:
(3 comments)
https://review.coreboot.org/c/coreboot/+/41972/3//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/41972/3//COMMIT_MSG@7 PS3, Line 7: cse_bp and ME
https://review.coreboot.org/c/coreboot/+/41972/3//COMMIT_MSG@8 PS3, Line 8: For people, just looking at this commit, the reasoning is missing. Were ME and CSE bp, what is bp, renamed?
https://review.coreboot.org/c/coreboot/+/41972/3//COMMIT_MSG@9 PS3, Line 9: serial console logs It can also be transmitted/stored in other ways. ;-)
… in all log messages
Hello V Sowmya, build bot (Jenkins), Furquan Shaikh, Rizwan Qureshi, Sowmya V, Patrick Rudolph,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/41972
to look at the new patch set (#4).
Change subject: soc/intel/common: Replace cse_bp and ME with cse_lite in all console logs ......................................................................
soc/intel/common: Replace cse_bp and ME with cse_lite in all console logs
Replace 'cse_bp'(cse boot partition) and 'ME' with 'cse_lite' in all log messages in the cse_lite.c.
TEST=Verified on hatch
Signed-off-by: Sridhar Siricilla sridhar.siricilla@intel.com Change-Id: I3fc677c9ec1962199c91cc310d7695dded4e0ba0 --- M src/soc/intel/common/block/cse/cse_lite.c 1 file changed, 22 insertions(+), 20 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/72/41972/4
Sridhar Siricilla has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41972 )
Change subject: soc/intel/common: Replace cse_bp and ME with cse_lite in all console logs ......................................................................
Patch Set 4:
(3 comments)
https://review.coreboot.org/c/coreboot/+/41972/3//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/41972/3//COMMIT_MSG@7 PS3, Line 7: cse_bp
and ME
Ack
https://review.coreboot.org/c/coreboot/+/41972/3//COMMIT_MSG@8 PS3, Line 8:
For people, just looking at this commit, the reasoning is missing. […]
Ack
https://review.coreboot.org/c/coreboot/+/41972/3//COMMIT_MSG@9 PS3, Line 9: serial console logs
It can also be transmitted/stored in other ways. ;-) […]
Done
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41972 )
Change subject: soc/intel/common: Replace cse_bp and ME with cse_lite in all console logs ......................................................................
Patch Set 4: Code-Review+2
V Sowmya has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41972 )
Change subject: soc/intel/common: Replace cse_bp and ME with cse_lite in all console logs ......................................................................
Patch Set 4: Code-Review+2
Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/41972 )
Change subject: soc/intel/common: Replace cse_bp and ME with cse_lite in all console logs ......................................................................
soc/intel/common: Replace cse_bp and ME with cse_lite in all console logs
Replace 'cse_bp'(cse boot partition) and 'ME' with 'cse_lite' in all log messages in the cse_lite.c.
TEST=Verified on hatch
Signed-off-by: Sridhar Siricilla sridhar.siricilla@intel.com Change-Id: I3fc677c9ec1962199c91cc310d7695dded4e0ba0 Reviewed-on: https://review.coreboot.org/c/coreboot/+/41972 Reviewed-by: Furquan Shaikh furquan@google.com Reviewed-by: V Sowmya v.sowmya@intel.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/soc/intel/common/block/cse/cse_lite.c 1 file changed, 22 insertions(+), 20 deletions(-)
Approvals: build bot (Jenkins): Verified Furquan Shaikh: Looks good to me, approved V Sowmya: Looks good to me, approved
diff --git a/src/soc/intel/common/block/cse/cse_lite.c b/src/soc/intel/common/block/cse/cse_lite.c index 1f9e2ce..3077c7f 100644 --- a/src/soc/intel/common/block/cse/cse_lite.c +++ b/src/soc/intel/common/block/cse/cse_lite.c @@ -166,21 +166,23 @@ { const struct cse_bp_entry *cse_bp;
- printk(BIOS_DEBUG, "ME: Number of partitions = %d\n", cse_bp_info->total_number_of_bp); - printk(BIOS_DEBUG, "ME: Current partition = %s\n", GET_BP_STR(cse_bp_info->current_bp)); - printk(BIOS_DEBUG, "ME: Next partition = %s\n", GET_BP_STR(cse_bp_info->next_bp)); - printk(BIOS_DEBUG, "ME: Flags = 0x%x\n", cse_bp_info->flags); + printk(BIOS_DEBUG, "cse_lite: Number of partitions = %d\n", + cse_bp_info->total_number_of_bp); + printk(BIOS_DEBUG, "cse_lite: Current partition = %s\n", + GET_BP_STR(cse_bp_info->current_bp)); + printk(BIOS_DEBUG, "cse_lite: Next partition = %s\n", GET_BP_STR(cse_bp_info->next_bp)); + printk(BIOS_DEBUG, "cse_lite: Flags = 0x%x\n", cse_bp_info->flags);
/* Log version info of RO & RW partitions */ cse_bp = cse_get_bp_entry(RO, cse_bp_info); - printk(BIOS_DEBUG, "ME: %s version = %d.%d.%d.%d (Status=0x%x, Start=0x%x, End=0x%x)\n", + printk(BIOS_DEBUG, "cse_lite: %s version = %d.%d.%d.%d (Status=0x%x, Start=0x%x, End=0x%x)\n", GET_BP_STR(RO), cse_bp->fw_ver.major, cse_bp->fw_ver.minor, cse_bp->fw_ver.hotfix, cse_bp->fw_ver.build, cse_bp->status, cse_bp->start_offset, cse_bp->end_offset);
cse_bp = cse_get_bp_entry(RW, cse_bp_info); - printk(BIOS_DEBUG, "ME: %s version = %d.%d.%d.%d (Status=0x%x, Start=0x%x, End=0x%x)\n", + printk(BIOS_DEBUG, "cse_lite: %s version = %d.%d.%d.%d (Status=0x%x, Start=0x%x, End=0x%x)\n", GET_BP_STR(RW), cse_bp->fw_ver.major, cse_bp->fw_ver.minor, cse_bp->fw_ver.hotfix, cse_bp->fw_ver.build, cse_bp->status, cse_bp->start_offset, @@ -222,19 +224,19 @@ };
if (!cse_is_bp_cmd_info_possible()) { - printk(BIOS_ERR, "cse_bp: CSE does not meet prerequisites\n"); + printk(BIOS_ERR, "cse_lite: CSE does not meet prerequisites\n"); return false; }
size_t resp_size = sizeof(struct get_bp_info_rsp);
if (!heci_send_receive(&info_req, sizeof(info_req), bp_info_rsp, &resp_size)) { - printk(BIOS_ERR, "cse_bp: Could not get partition info\n"); + printk(BIOS_ERR, "cse_lite: Could not get partition info\n"); return false; }
if (bp_info_rsp->hdr.result) { - printk(BIOS_ERR, "cse_bp: Get partition info resp failed: %d\n", + printk(BIOS_ERR, "cse_lite: Get partition info resp failed: %d\n", bp_info_rsp->hdr.result); return false; } @@ -266,14 +268,14 @@ };
if (bp != RO && bp != RW) { - printk(BIOS_ERR, "cse_bp: Incorrect partition id(%d) is provided", bp); + printk(BIOS_ERR, "cse_lite: Incorrect partition id(%d) is provided", bp); return false; }
- printk(BIOS_INFO, "cse_bp: Set Boot Partition Info Command (%s)\n", GET_BP_STR(bp)); + printk(BIOS_INFO, "cse_lite: Set Boot Partition Info Command (%s)\n", GET_BP_STR(bp));
if (!cse_is_bp_cmd_info_possible()) { - printk(BIOS_ERR, "cse_bp: CSE does not meet prerequisites\n"); + printk(BIOS_ERR, "cse_lite: CSE does not meet prerequisites\n"); return false; }
@@ -284,7 +286,7 @@ return false;
if (switch_resp.result) { - printk(BIOS_ERR, "cse_bp: Set Boot Partition Info Response Failed: %d\n", + printk(BIOS_ERR, "cse_lite: Set Boot Partition Info Response Failed: %d\n", switch_resp.result); return false; } @@ -302,7 +304,7 @@
do_global_reset();
- die("cse_bp: Failed to reset system\n"); + die("cse_lite: Failed to reset system\n");
/* Control never reaches here */ return false; @@ -317,7 +319,7 @@
if (rw_bp->status == BP_STATUS_PARTITION_NOT_PRESENT || rw_bp->status == BP_STATUS_GENERAL_FAILURE) { - printk(BIOS_ERR, "cse_bp: RW BP (status:%u) is not valid\n", rw_bp->status); + printk(BIOS_ERR, "cse_lite: RW BP (status:%u) is not valid\n", rw_bp->status); return false; } return true; @@ -333,28 +335,28 @@ static struct get_bp_info_rsp cse_bp_info;
if (vboot_recovery_mode_enabled()) { - printk(BIOS_DEBUG, "cse_bp: Skip switching to RW in the recovery path\n"); + printk(BIOS_DEBUG, "cse_lite: Skip switching to RW in the recovery path\n"); return; }
/* If CSE SKU type is not Lite, skip enabling CSE Lite SKU */ if (!cse_is_hfs3_fw_sku_lite()) { - printk(BIOS_ERR, "cse_bp: Not a CSE Lite SKU\n"); + printk(BIOS_ERR, "cse_lite: Not a CSE Lite SKU\n"); return; }
if (!cse_get_bp_info(&cse_bp_info)) { - printk(BIOS_ERR, "cse_bp: Failed to get CSE boot partition info\n"); + printk(BIOS_ERR, "cse_lite: Failed to get CSE boot partition info\n"); cse_trigger_recovery(CSE_LITE_SKU_COMMUNICATION_ERROR); }
if (!cse_is_rw_info_valid(&cse_bp_info.bp_info)) { - printk(BIOS_ERR, "cse_bp: CSE RW partition is not valid\n"); + printk(BIOS_ERR, "cse_lite: CSE RW partition is not valid\n"); cse_trigger_recovery(CSE_LITE_SKU_RW_JUMP_ERROR); }
if (!cse_boot_to_rw(&cse_bp_info.bp_info)) { - printk(BIOS_ERR, "cse_bp: Failed to switch to RW\n"); + printk(BIOS_ERR, "cse_lite: Failed to switch to RW\n"); cse_trigger_recovery(CSE_LITE_SKU_RW_SWITCH_ERROR); } }
-
build bot (Jenkins) (Code Review) -
Furquan Shaikh (Code Review)
-
Patrick Georgi (Code Review)
-
Paul Menzel (Code Review)
-
Rizwan Qureshi (Code Review)
-
Sridhar Siricilla (Code Review)
-
V Sowmya (Code Review)