openwrt/flashrom
1
0
Fork 0
mirror of https://review.coreboot.org/flashrom.git synced 2025-04-27 07:02:34 +02:00
Code Issues Releases Wiki Activity

writeprotect.c: refactor and fix wp_mode functions

Browse Source 
This is a follow up on commit 12dbc4e04508aecfff53ad95b6f68865da1b4f07.

Use a lookup table in get_wp_mode() and drop the srp_bit_present check,
since a chip without SRP is just FLASHROM_WP_MODE_DISABLED.

Add a srp_bit_present check to set_wp_mode() if the mode requires it.

BUG=b:182223106
BRANCH=none
TEST=flashrom --wp-{enable,disable,status} on AMD dut

Change-Id: Ib6c347453f9216e5816e4ed35bf9783fd3c720e0
Signed-off-by: Nikolai Artemiev <nartemiev@google.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/62643
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Anastasia Klimchuk <aklm@chromium.org>
Reviewed-by: Edward O'Callaghan <quasisec@chromium.org>
This commit is contained in:
Nikolai Artemiev 2022-03-08 01:07:01 +11:00 committed by Edward O'Callaghan
parent 9b20174fda
commit bd2d070f9f
1 changed files with 13 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
writeprotect.c
View File

@ -351,16 +351,17 @@ static int set_wp_range(struct wp_bits *bits, struct flashctx *flash, const stru
/** Get the mode selected by a WP configuration. */
static int get_wp_mode(enum flashrom_wp_mode *mode, const struct wp_bits *bits)
{
if (!bits->srp_bit_present)
return FLASHROM_WP_ERR_CHIP_UNSUPPORTED;
const enum flashrom_wp_mode wp_modes[2][2] = {
{
FLASHROM_WP_MODE_DISABLED, /* srl=0, srp=0 */
FLASHROM_WP_MODE_HARDWARE, /* srl=0, srp=1 */
}, {
FLASHROM_WP_MODE_POWER_CYCLE, /* srl=1, srp=0 */
FLASHROM_WP_MODE_PERMANENT, /* srl=1, srp=1 */
},
};
if (bits->srl_bit_present && bits->srl == 1) {
*mode = bits->srp ? FLASHROM_WP_MODE_PERMANENT :
FLASHROM_WP_MODE_POWER_CYCLE;
} else {
*mode = bits->srp ? FLASHROM_WP_MODE_HARDWARE :
FLASHROM_WP_MODE_DISABLED;
}
*mode = wp_modes[bits->srl][bits->srp];
return FLASHROM_WP_OK;
}
@ -375,6 +376,9 @@ static int set_wp_mode(struct wp_bits *bits, const enum flashrom_wp_mode mode)
return FLASHROM_WP_OK;
case FLASHROM_WP_MODE_HARDWARE:
if (!bits->srp_bit_present)
return FLASHROM_WP_ERR_CHIP_UNSUPPORTED;
bits->srl = 0;
bits->srp = 1;
return FLASHROM_WP_OK;