[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)

Patchset:

PS22:
> I first thought the API is missing a function to create a new `config` […]

Thinking further about this, there are more reasons to tie
flash context and wp config together. The current internal
representation is actually only valid for a specific chip
model (it's storing the exact bits found on the chip; same
bit values could have a different meaning for another chip).

Completely different idea: we could pass an abstract (decoded)
config around instead of the current bit representation. Just
like one would deserialize data from a byte stream and use
a better, program-specific representation and only serialize
it again for writing.