[coreboot-gerrit] [M] Change in coreboot[master]: mb/google/skyrim: override Markarth PCIe config

[Paul Menzel (Code Review)]

Attention is currently required from: John Su, Jason Nien, Chris Wang, Martin Roth, Karthik Ramasubramanian, Patrick Huang.

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74133 )

Change subject: mb/google/skyrim: override Markarth PCIe config
......................................................................

Patch Set 3:

(5 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/74133/comment/bba478e1_d06efa32 
PS3, Line 7: mb/google/skyrim
Shouldn’t variant names be mentioned in the prefix?

https://review.coreboot.org/c/coreboot/+/74133/comment/09b14e21_bdf2f689 
PS3, Line 8: 
I’d start by describing the problem. Do you have a non-working test device? Or did you just check the schematics?

https://review.coreboot.org/c/coreboot/+/74133/comment/3285033b_399edf22 
PS3, Line 9: We
lowercase: we

https://review.coreboot.org/c/coreboot/+/74133/comment/30bd1920_adadac36 
PS3, Line 12: allows
allowing

https://review.coreboot.org/c/coreboot/+/74133/comment/8cce2c22_83961b59 
PS3, Line 17: TEST=emerge-skyrim coreboot chromeos-bootimage
Did you test this on a real device?