[flashrom-gerrit] [M] Change in flashrom[master]: tree: provide flashrom context into programmer_delay()

[Felix Singer (Code Review)]

Attention is currently required from: Thomas Heijligen, Edward O'Callaghan, Alexander Goncharov.

Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/66373 )

Change subject: tree: provide flashrom context into programmer_delay()
......................................................................

Patch Set 9:

(2 comments)

Commit Message:

https://review.coreboot.org/c/flashrom/+/66373/comment/cd4c2d81_43964403 
PS9, Line 11: state. Programmers uses internal delay can provide NULL as a context.
The commit message should mention that only the function signature is changed and that this commit has no functional effect. Also, refer to the commit that makes use of the parameter using `CB:123456`.

File pony_spi.c:

https://review.coreboot.org/c/flashrom/+/66373/comment/ea6a6115_0871463e 
PS7, Line 247: flashctx not needed (NULL) because pony_spi does not
             : 			 * use it in its delay function.
> I would say this is resolvable, what do you think Felix?

A comment at the function declaration that you can pass flashctx but also NULL would be good.