mirror of
https://review.coreboot.org/flashrom.git
synced 2025-07-01 22:21:16 +02:00
writeprotect: Add some debug logging if wp_verify fails
Change-Id: I5fcaf767570418f90ae44826a1135d9b49653033 Signed-off-by: Evan Benn <evanbenn@chromium.org> Reviewed-on: https://review.coreboot.org/c/flashrom/+/67720 Reviewed-by: Nikolai Artemiev <nartemiev@google.com> Reviewed-by: Edward O'Callaghan <quasisec@chromium.org> Reviewed-by: Sergii Dmytruk <sergii.dmytruk@3mdeb.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:

committed by
Anastasia Klimchuk

parent
b9c03286b6
commit
74c3e56ec2
@ -155,6 +155,7 @@ static enum flashrom_wp_result write_wp_bits(struct flashctx *flash, struct wp_b
|
||||
return FLASHROM_WP_ERR_WRITE_FAILED;
|
||||
}
|
||||
|
||||
enum flashrom_wp_result ret = FLASHROM_WP_OK;
|
||||
/* Verify each register */
|
||||
for (enum flash_reg reg = STATUS1; reg < MAX_REGISTERS; reg++) {
|
||||
if (!write_masks[reg])
|
||||
@ -164,14 +165,18 @@ static enum flashrom_wp_result write_wp_bits(struct flashctx *flash, struct wp_b
|
||||
if (wp_read_register(flash, reg, &value))
|
||||
return FLASHROM_WP_ERR_READ_FAILED;
|
||||
|
||||
msg_cdbg2("%s: wp_verify reg:%u value:0x%x\n", __func__, reg, value);
|
||||
uint8_t actual = value & write_masks[reg];
|
||||
uint8_t expected = reg_values[reg] & write_masks[reg];
|
||||
|
||||
if (actual != expected)
|
||||
return FLASHROM_WP_ERR_VERIFY_FAILED;
|
||||
if (actual != expected) {
|
||||
msg_cdbg("%s: wp_verify failed: reg:%u actual:0x%x expected:0x%x\n",
|
||||
__func__, reg, actual, expected);
|
||||
ret = FLASHROM_WP_ERR_VERIFY_FAILED;
|
||||
}
|
||||
}
|
||||
|
||||
return FLASHROM_WP_OK;
|
||||
return ret;
|
||||
}
|
||||
|
||||
static decode_range_func_t *lookup_decode_range_func_ptr(const struct flashchip *chip)
|
||||
|
Reference in New Issue
Block a user