Anastasia Klimchuk has submitted this change. ( https://review.coreboot.org/c/flashrom/+/78492?usp=email )
Change subject: erasure_layout: Add region boundaries to printed info message ......................................................................
erasure_layout: Add region boundaries to printed info message
Change-Id: I511a79754cff15e7dba26f5313d7015830780f60 Signed-off-by: Anastasia Klimchuk aklm@flashrom.org Reviewed-on: https://review.coreboot.org/c/flashrom/+/78492 Reviewed-by: Alexander Goncharov chat@joursoir.net Reviewed-by: Aarya aarya.chaumal@gmail.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M erasure_layout.c 1 file changed, 8 insertions(+), 2 deletions(-)
Approvals: Alexander Goncharov: Looks good to me, approved build bot (Jenkins): Verified Aarya: Looks good to me, approved
diff --git a/erasure_layout.c b/erasure_layout.c index b43e52e..0c31911 100644 --- a/erasure_layout.c +++ b/erasure_layout.c @@ -174,11 +174,17 @@ }
if (start_diff) { - msg_cinfo("Region start not sector aligned! Extending start boundaries...\n"); + msg_cinfo("Region [0x%08x - 0x%08x] is not sector aligned! " + "Extending start boundaries by 0x%08x bytes, from 0x%08x -> 0x%08x\n", + *region_start, *region_end, + start_diff, *region_start, *region_start - start_diff); *region_start = *region_start - start_diff; } if (end_diff) { - msg_cinfo("Region end not sector aligned! Extending end boundaries...\n"); + msg_cinfo("Region [0x%08x - 0x%08x] is not sector aligned! " + "Extending end boundaries by 0x%08x bytes, from 0x%08x -> 0x%08x\n", + *region_start, *region_end, + end_diff, *region_end, *region_end + end_diff); *region_end = *region_end + end_diff; } }