[flashrom-gerrit] [M] Change in flashrom[master]: flashrom_tester: Check the WP state when setting

[Evan Benn (Code Review)]

Attention is currently required from: Peter Marheine.

Evan Benn has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/69402 )

Change subject: flashrom_tester: Check the WP state when setting
......................................................................

Patch Set 5:

(2 comments)

File util/flashrom_tester/src/tester.rs:

https://review.coreboot.org/c/flashrom/+/69402/comment/ffb54d18_2ae36eec 
PS4, Line 238:         let actual_state = Self::get_sw(self.cmd).map_err(|e| e.to_string())?;
> Aha I have discovered a bug with this. […]

Done

https://review.coreboot.org/c/flashrom/+/69402/comment/f575a8ac_c1a22208 
PS4, Line 240:             Err(format!("set_sw request={}, real={}", enable, actual_state))
> This error could be more descriptive. […]

Done