Andrey Petrov has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40334 )
Change subject: configs: Add builder config to create a working Sonora Pass ......................................................................
Patch Set 8:
(2 comments)
https://review.coreboot.org/c/coreboot/+/40334/8/configs/builder/config.ocp.... File configs/builder/config.ocp.sonorapass:
https://review.coreboot.org/c/coreboot/+/40334/8/configs/builder/config.ocp.... PS8, Line 21: uart8250,io,0x3f8 you really do not want to hard the port here. I think it should be: console=ttyS0,57600n1.
Also: on SonoraPass we have 2 useful SIO ports: com1 that is wired to debug header and com2 that is wired to SOL. this should be noted int the comment below.
https://review.coreboot.org/c/coreboot/+/40334/8/configs/builder/config.ocp.... PS8, Line 21: earlyprintk=uart8250,io,0x3f8 please note: this syntax is not correct. This is extremely confusing, and must be fixed. Since earlyprintk is not specified correctly and under certain conditions kernel might crash before regular console becomes operational, this is really ugly problem.
what we want instead is: earlyprintk=serial,0x3f8,57600n1 Another option: earlyprintk=serial,ttyS0,57600n1 which is better.
https://www.kernel.org/doc/html/v4.14/admin-guide/kernel-parameters.html