Change in flashrom[master]: writeprotect: add {get,set}_wp_mode()

[Nico Huber (Code Review)]

Attention is currently required from: Angel Pons, Nikolai Artemiev, Sergii Dmytruk.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/58483 )

Change subject: writeprotect: add {get,set}_wp_mode()
......................................................................

Patch Set 58: Code-Review+2

(1 comment)

File writeprotect.c:

https://review.coreboot.org/c/flashrom/+/58483/comment/87b8ca29_3a27f0c4 
PS58, Line 366: 	}
General thought: That the positive path comes first in such an if and ?:
can confuse the logical mind, because when enumerating cases usually
(logical) 0 comes first. A table might help, e.g.

static const int wp_mode[][] = {
                /*           srp=0                         srp=1           */
    /* srl=0 */ { FLASHROM_WP_MODE_DISABLED,    FLASHROM_WP_MODE_HARDWARE  },
    /* srl=1 */ { FLASHROM_WP_MODE_POWER_CYCLE, FLASHROM_WP_MODE_PERMANENT },
    };