Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32236 )
Change subject: Documentation: Add MSI MS-7707 ......................................................................
Patch Set 1:
(4 comments)
Do you have the copyright on the board photo? It would be more useful to provide a picture of the flash IC on the mainboard.
I don't see how the logs could be useful.
https://review.coreboot.org/#/c/32236/1/Documentation/mainboard/medion/ms_77... File Documentation/mainboard/medion/ms_7707/ms-7707.md:
https://review.coreboot.org/#/c/32236/1/Documentation/mainboard/medion/ms_77... PS1, Line 1: # MSI MS-7707 V1.1 file needs to be referenced in index.md
https://review.coreboot.org/#/c/32236/1/Documentation/mainboard/medion/ms_77... PS1, Line 12: How do you flash? Is it socketed? Is there a header? Position of flash IC? Can you flash in-circuit?
https://review.coreboot.org/#/c/32236/1/Documentation/mainboard/medion/ms_77... PS1, Line 13: ## Working whitespace
https://review.coreboot.org/#/c/32236/1/Documentation/mainboard/medion/ms_77... PS1, Line 24: - S0/S5 states work properly as long as the board gets powered. After power cut the system is dead. Needs battery removal and CMOSCLR to revive. SuperIO is supposed to be in an unknown state as current coreboot SuperIO initialization isn't able to set all registers (see cbmem.log - skipping PNP: 004e.a@e0 fixed resource, size=0!). line over 80 chars