Attention is currently required from: Bill XIE, Paul Menzel.
Keith Hui has posted comments on this change by Keith Hui. ( https://review.coreboot.org/c/coreboot/+/82556?usp=email )
Change subject: mb/asus/p8x7x-series: Mark variants using GPIO8 for power LED ......................................................................
Patch Set 2:
(1 comment)
File src/mainboard/asus/p8x7x-series/Kconfig:
https://review.coreboot.org/c/coreboot/+/82556/comment/197d6c0b_09a11bb9?usp... : PS2, Line 66: If power LED doesn't blink when entering S3 suspend, try setting this.
It’d be great if you elaborate and added the information from the commit message regarding the GPIOs […]
I really am not sure if this info can be deduced from running system or ACPI.
I checked the entire family of boards, at least those I can get a boardview for, and the power LED only goes to either GPIO 8 or 27 - I haven't seen a third GPIO used. And the only consequence of getting this wrong is that the power LED doesn't blink on suspend, so I'm content with "if it doesn't work, try this."
Honestly though, after seeing what went down with Malibal, I also think coreboot's documentation needs a major revamp, to the point that a CS undergrad who knows where to look can get started with working on coreboot. However that is a much larger undertaking than one patch for one board family.