[coreboot-gerrit] Change in coreboot[master]: mb/system76/oryp5: Add System76 Oryx Pro 5

[Angel Pons (Code Review)]

Attention is currently required from: Tim Crawford.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47892 )

Change subject: mb/system76/oryp5: Add System76 Oryx Pro 5
......................................................................

Patch Set 10:

(2 comments)

File src/mainboard/system76/oryp5/devicetree.cb:

https://review.coreboot.org/c/coreboot/+/47892/comment/e9cd152b_c6966f53 
PS2, Line 26: register "SaGv" = "SaGv_Enabled"
> SaGv is enabled on the RVP. […]

Probably. Did you get to test with SaGv disabled?

File src/mainboard/system76/oryp5/hda_verb.c:

https://review.coreboot.org/c/coreboot/+/47892/comment/ed1cecd8_e8a2ce52 
PS2, Line 25: // Enable DMIC microphone on ALC1220
            : 	0x02050036,
            : 	0x02042a6a,
> The ALC1220 spec we got doesn't have the supported verbs, so I can't even confirm what these actuall […]

Ack