[flashrom-gerrit] Change in flashrom[master]: libflashrom,writeprotect: add flashrom_wp_{read,write}_chip_config()

[Nico Huber (Code Review)]

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

Change subject: libflashrom,writeprotect: add flashrom_wp_{read,write}_chip_config()
......................................................................

Patch Set 22:

(1 comment)

File libflashrom.h:

https://review.coreboot.org/c/flashrom/+/58479/comment/a9b4adb1_f81b57e0 
PS22, Line 131: const struct flashrom_flashctx *, struct flashrom_wp_chip_config *
Something about APIs in general. We are used to have the destination
argument first. Just like in an assignment where the destination is on
the left-hand side. This also applies to functions added later. Also
to internal functions but we can make exceptions if there is a good
reason.