Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35912 )
Change subject: [WIP] Documentation/basics: Add Super I/O section ......................................................................
Patch Set 4:
(5 comments)
Maybe link to wiki.osdev.org and Wikipedia(?)?
https://review.coreboot.org/c/coreboot/+/35912/4/Documentation/basics/superi... File Documentation/basics/superio.md:
https://review.coreboot.org/c/coreboot/+/35912/4/Documentation/basics/superi... PS4, Line 4: and are usually Fits on the line above?
https://review.coreboot.org/c/coreboot/+/35912/4/Documentation/basics/superi... PS4, Line 18: BMCs (Board Management Controller) - mostly found on server boards - are also connected via LPC and can manage Super I/O tasks next to their usual ones like: Wrap line?
https://review.coreboot.org/c/coreboot/+/35912/4/Documentation/basics/superi... PS4, Line 23: Very most x86 mainboards have at least one Super I/O chip. Most x86 …
https://review.coreboot.org/c/coreboot/+/35912/4/Documentation/basics/superi... PS4, Line 27: does the Supermicro X11SSH-TF *does* at the end
https://review.coreboot.org/c/coreboot/+/35912/4/Documentation/basics/superi... PS4, Line 33: superio Super I/O