[flashrom-gerrit] [M] Change in flashrom[master]: writeprotect.c: differentiate lack of WP from lack of implementation

[Sergii Dmytruk (Code Review)]

Attention is currently required from: Nico Huber, Nikolai Artemiev.

Sergii Dmytruk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/68221 )

Change subject: writeprotect.c: differentiate lack of WP from lack of implementation
......................................................................

Patch Set 5:

(1 comment)

Commit Message:

https://review.coreboot.org/c/flashrom/+/68221/comment/1d162bf8_be9ab615 
PS4, Line 10: preventing confusion
> We would still print the former in the NT case, right?

Yes.

I've added the change to handle `NA` value properly because reporting "unimplemented" for `NA` state looks inappropriate.  I guess I meant preventing potential confusion as this doesn't improve current situation, just makes better messages possible.  However, now that I think about multiple chip models being mapped onto the same entry and that this will only get worse with time, maybe messages shouldn't be precise or they will not be reliable.