[flashrom-gerrit] [S] 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 4:

(1 comment)

File util/flashrom_tester/src/tester.rs:

https://review.coreboot.org/c/flashrom/+/69402/comment/e09d18f8_01f80184 
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. The `cmd` also has `wp_range` function, which is a bit strange and maybe enables wp but doesnt disable wp? anyway it bypasses the stored state here.