Attention is currently required from: Riku Viitanen, Sydney, Thomas Heijligen.
J. Neuschäfer has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/82229?usp=email )
Change subject: Add documentation for pico-serprog ......................................................................
Patch Set 3:
(4 comments)
Patchset:
PS3: Some comments on wording
File doc/supported_hw/supported_prog/serprog/overview.rst:
https://review.coreboot.org/c/flashrom/+/82229/comment/98f6d4c2_c15fc6de : PS2, Line 95: etter performance
I am not aware of any exiting benchmarks. […]
I did the performance improvements and I did some very rudimentary benchmarking. It's in this commit message:
https://github.com/stacksmashing/pico-serprog/commit/ed2520ccb39965deb20d6be...
Total payload throughput gets to about 5 Mbit/s, and I think I'm limited by USB (12 Mbit/s nominal for Full Speed, and USB protocol overhead that I don't fully understand).
I haven't benchmarked your implementation.
File doc/supported_hw/supported_prog/serprog/overview.rst:
https://review.coreboot.org/c/flashrom/+/82229/comment/2d6095bc_5d967f5c : PS3, Line 94: over PIO "over" can be misunderstood. "instead of" is less ambiguous
https://review.coreboot.org/c/flashrom/+/82229/comment/cacc5838_eee2ce2d : PS3, Line 95: pinouts but implements USB descriptors allowing custom udev-rules. : Nothing wrong with what this _says_, but what it _implies_ is a bit confusing. Because these two things are in the same sentence, it seems to imply that arbitrary pinouts are sacrified for gaining custom USB descriptors, but really these are two unrelated features, AFAIUI:
1. Hardware SPI is used, therefore the pins can't be moved around arbitrarily
2. Tinyusb used used directly instead of going through the pico-stdio library's USB code. That is what allows having custom (libreboot/pico-serprog) instead of generic (raspberry pi pico) descriptors. The USB device has descriptors in any case.