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 1:
(4 comments)
Commit Message:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-168531): https://review.coreboot.org/c/coreboot/+/71999/comment/9e304468_2f535a14 PS1, 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-168531): https://review.coreboot.org/c/coreboot/+/71999/comment/d54e1dcf_e3f44548 PS1, 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-168531): https://review.coreboot.org/c/coreboot/+/71999/comment/64a9b046_22652ae1 PS1, 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-168531): https://review.coreboot.org/c/coreboot/+/71999/comment/b99e2bed_8b7c3be5 PS1, 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)