Attention is currently required from: Anastasia Klimchuk, Alexander Goncharov.
Edward O'Callaghan has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/72807 )
Change subject: ch341a_spi: Refactor singleton states into reentrant pattern ......................................................................
Patch Set 5:
(1 comment)
File ch341a_spi.c:
https://review.coreboot.org/c/flashrom/+/72807/comment/012f1d28_321b3ba1 PS3, Line 80: struct libusb_transfer *transfer_out; : struct libusb_transfer *transfer_ins[USB_IN_TRANSFERS]; : : /* Accumulate delays to be plucked between CS deassertion and CS assertions. */ : unsigned int stored_delay_us; : : struct libusb_device_handle *handle;
Done
I recommend putting the handle at the start of the struct so that overflows in the above transfer buffers should they happen do not overflow into execution flow memory where function pointers exist.