[flashrom-gerrit] Change in flashrom[master]: libflashrom,writeprotect: add functions for reading/writing WP settings

[Nikolai Artemiev (Code Review)]

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

Change subject: libflashrom,writeprotect: add functions for reading/writing WP settings
......................................................................

Patch Set 25:

(4 comments)

File libflashrom.h:

https://review.coreboot.org/c/flashrom/+/58479/comment/d472e6ce_9db44993 
PS22, Line 129: };
> Please move to the commit that makes use of it.

Done

https://review.coreboot.org/c/flashrom/+/58479/comment/8c828b1e_eea17327 
PS22, Line 131: const struct flashrom_flashctx *, struct flashrom_wp_chip_config *
> Something about APIs in general. We are used to have the destination […]

Done - here and in other files/commits. I think I got all occurrences, let me know if I missed something :)

File writeprotect.c:

https://review.coreboot.org/c/flashrom/+/58479/comment/fa897570_4f3894a9 
PS22, Line 48: **
> Library functions should be in a respective documentation group, e.g. @defgroup or @addtogroup.

Done

https://review.coreboot.org/c/flashrom/+/58479/comment/abdb9bf5_109adc23 
PS22, Line 122: INVALID_REG
> IMO it is a bit confusing when loop starts with "invalid" value. […]

Done