Attention is currently required from: Simon Buhrow, Nico Huber, Aarya, Anastasia Klimchuk.
Edward O'Callaghan has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/66104 )
Change subject: flashrom.c: Add wrapper function to use the erase algorithm ......................................................................
Patch Set 76:
(1 comment)
File flashrom.c:
https://review.coreboot.org/c/flashrom/+/66104/comment/a245758d_fd02f647 PS72, Line 1349: static int erase_by_layout
I can do that, but then later do I have to again move back my implementation into flashrom. […]
It would be my view to more your whole implementation into its own separate file. It seems pretty modular so atomically adding everything in one commit is logical and subsequent commits deal with the integration challenge of switching from old to new, preferably with one liner that can be trivially reverted.
The `flashrom.c` kitchen sink should be broken up in any case and given your modular work this is a opportunity to advance that goal forwards at the same time rather than adding to the problem.
`erasure_layout.c`, `flashrom_erasure.c`, just `flash_erasure.c` something like that?