Attention is currently required from: Name of user not set #1004406.
Paul Menzel has posted comments on this change by Name of user not set #1004406. ( https://review.coreboot.org/c/coreboot/+/65225?usp=email )
Change subject: added bare Asrock H370m-ITX support ......................................................................
Patch Set 6:
(6 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/65225/comment/9b3baea1_19c841ff?usp... : PS6, Line 7: added bare Asrock H370m-ITX support Please follow the commit message style of other commits, for example, please add a prefix.
https://review.coreboot.org/c/coreboot/+/65225/comment/4ea6f177_ff5c7ac2?usp... : PS6, Line 9: the following things are working flawlessly: What payloads and operating system did you use?
https://review.coreboot.org/c/coreboot/+/65225/comment/10fda224_5d2eef73?usp... : PS6, Line 12: seabios SeaBIOS
https://review.coreboot.org/c/coreboot/+/65225/comment/c8900b83_ab909d47?usp... : PS6, Line 12: VGA How did you get the VGA BIOS Option ROM? From what vendor firmware version?
https://review.coreboot.org/c/coreboot/+/65225/comment/28d6682d_1de2a2fb?usp... : PS6, Line 10: *all PCIe slots : *all USB ports : *VGA init in seabios : *WiFi and onboard sound : *both Ethernet ports : *all SATA ports Please add a space after the bullets.
https://review.coreboot.org/c/coreboot/+/65225/comment/3763825c_edc04131?usp... : PS6, Line 22: *corrected resistor cfg (see 573387) : *removed unused dq/dqs mappings : : *added default config for Asrock H370M-ITX I’d use imperative mood.