Attention is currently required from: ChrisEric1 CECL.
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/71999 )
Change subject: src/mainboard/dell/inspiron_*: Add Support for Dell Inspiron 620 & 3847 ......................................................................
Patch Set 2:
(4 comments)
Commit Message:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-168533): https://review.coreboot.org/c/coreboot/+/71999/comment/28e02b6d_7f18779b PS2, Line 9: Add Initial Support for Dell Inspiron 620 and Dell Inspiron 3847. (using Autoport) Possible unwrapped commit description (prefer a maximum 72 chars per line)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-168533): https://review.coreboot.org/c/coreboot/+/71999/comment/e1f21d93_dcfd89f9 PS2, Line 10: On my 620, UEFI freezes up for ~5m then boots, my 2nd RAM slot doesn't get recognized (even in stock rom, although stock doesn't boot up and gives freeze or POST), else everything works from what I can tell (could mean my board is partially damaged) Possible unwrapped commit description (prefer a maximum 72 chars per line)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-168533): https://review.coreboot.org/c/coreboot/+/71999/comment/3bdbca19_6ab2afdf PS2, Line 11: On my 3847, everything works, UEFI doesn't freeze up, but my external GPU doesn't work (internal does), it is a NVIDIA GEFORCE GT 710 and cannot figure out how to default to it, though the linux driver works fine with it (windows was also untested on both machines). Possible unwrapped commit description (prefer a maximum 72 chars per line)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-168533): https://review.coreboot.org/c/coreboot/+/71999/comment/3a4b1186_c0bd4f71 PS2, Line 12: NOTE: As of 2022, I need to get a replacement 620 MB, due to the fact that the BIOS chip fried. My 3847 Works fine though. Someone also had the same freezing issue on a SNB in coreboot discord, which later upgraded to IVY Bridge, could also need a SATA Patch, not known yet until a get a new MB. Possible unwrapped commit description (prefer a maximum 72 chars per line)