Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38016 )
Change subject: Documentation/superio: add generic PNP device documentation ......................................................................
Patch Set 4: Code-Review+1
(4 comments)
https://review.coreboot.org/c/coreboot/+/38016/4/Documentation/superio/commo... File Documentation/superio/common/pnp.md:
https://review.coreboot.org/c/coreboot/+/38016/4/Documentation/superio/commo... PS4, Line 3: super Super
https://review.coreboot.org/c/coreboot/+/38016/4/Documentation/superio/commo... PS4, Line 5: PNP devices are usually connected to the LPC bus of a system and shouldn't be confused with PCI(e) devices that use a completely different plug and play mechanism. I wouldn’t break the line after each sentence, end also adhere to the line length limit.
https://review.coreboot.org/c/coreboot/+/38016/4/Documentation/superio/commo... PS4, Line 6: ISA plug and play specification Add an URL?
https://review.coreboot.org/c/coreboot/+/38016/4/Documentation/superio/commo... PS4, Line 9: ## configuration access and config mode Start headings with capital letter?