Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37643 )
Change subject: Documentation: Add proposal for firmware testing ......................................................................
Patch Set 3:
(5 comments)
https://review.coreboot.org/c/coreboot/+/37643/3/Documentation/technotes/201... File Documentation/technotes/2019-12-firmware-testing.md:
https://review.coreboot.org/c/coreboot/+/37643/3/Documentation/technotes/201... PS3, Line 9: Lava This lava? https://minecraft.gamepedia.com/Lava
https://review.coreboot.org/c/coreboot/+/37643/3/Documentation/technotes/201... PS3, Line 20: test host Would be nice if the test host could be in control of more than one DUT. This is rather easy to do if the interfaces with the DUT are via USB devices. After booting, the network is also useful to have.
https://review.coreboot.org/c/coreboot/+/37643/3/Documentation/technotes/201... PS3, Line 32: It may also be augmented by additional circuitry that : allows automating these steps. That's exactly the only thing I've done of my automated test setup. With just a few transistors and resistors, I managed to make my Raspberry Pi able to control my Asrock G41M-S3. I used three GPIOs: one of them controls the power button, another monitors the power LED state (to know if the board is on or off) and the last one switches between two flash chips using the HOLD# pin.
https://review.coreboot.org/c/coreboot/+/37643/3/Documentation/technotes/201... PS3, Line 39: So these are Seems like something went missing here?
https://review.coreboot.org/c/coreboot/+/37643/3/Documentation/technotes/201... PS3, Line 45: and ?