Change in flashrom[master]: ichspi.c: Implement read_write_status for wp

[Sergii Dmytruk (Code Review)]

Attention is currently required from: Edward O'Callaghan, Nikolai Artemiev.
Sergii Dmytruk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/64540 )

Change subject: ichspi.c: Implement read_write_status for wp
......................................................................

Patch Set 3:

(2 comments)

File ichspi.c:

https://review.coreboot.org/c/flashrom/+/64540/comment/36789db4_05e0f16c 
PS2, Line 1349: static int ich_hwseq_read_status(const struct flashctx *flash, enum flash_reg reg_, uint8_t *value)
I think both functions should fail if reg_ is not SR1.

https://review.coreboot.org/c/flashrom/+/64540/comment/b1d90ebf_a7d242f8 
PS2, Line 1352:         int len = 1;
I'd make len a constant in both functions. It's used 15 lines below and one has to trace the code to understand that it's never updated.