[S] Change in flashrom[main]: libflashrom: Allow getting the progress_state from the flashctx

[Peter Marheine (Code Review)]

Attention is currently required from: Anastasia Klimchuk, Daniel Campello, Edward O'Callaghan, Nico Huber, Richard Hughes.

Peter Marheine has posted comments on this change by Richard Hughes. ( https://review.coreboot.org/c/flashrom/+/64663?usp=email )

The change is no longer submittable: All-Comments-Resolved is unsatisfied now.

Change subject: libflashrom: Allow getting the progress_state from the flashctx
......................................................................

Patch Set 5:

(1 comment)

File include/libflashrom.h:

https://review.coreboot.org/c/flashrom/+/64663/comment/a9829e26_9bbb11df?usp... :
PS5, Line 108: struct flashrom_progress *flashrom_get_progress_state(struct flashrom_flashctx *flashctx);
I wonder if this should be a `const struct flashrom_progress*` to indicate that we don't expect the caller to modify the progress information? It wouldn't be harmful if they did (at least with the current implementation), but I think it would be better to exclude mutability completely.