Attention is currently required from: Felix Singer, Thomas Heijligen, Alexander Goncharov.
Edward O'Callaghan 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/65faa565_2347592b PS7, Line 14: TOPIC=register_master_api
That topic was actual as long as this patch changed programmer delay API. […]
I can add that into the commit message of my patch sure and will push once this lands so its rebased on top but I would just drop the topic here and keep it simple. This is a pretty simple patch it's just numerically big on the number of files front but otherwise is a NOP.
File pony_spi.c:
https://review.coreboot.org/c/flashrom/+/66373/comment/5cc3138e_f9a42130 PS7, Line 247: flashctx not needed (NULL) because pony_spi does not : * use it in its delay function.
It might be done in CB:67393 because that patch determines how the parameter will be used.
I would say this is resolvable, what do you think Felix?