[coreboot-gerrit] Change in coreboot[master]: mb/google/zork/var/shuboz: Add fw_config probe for ALC5682-VD & VS

[Reka Norman (Code Review)]

Attention is currently required from: Kangheui Won, Isaac Lee, Kane Chen, chris wang, Felix Held.
Reka Norman has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59558 )

Change subject: mb/google/zork/var/shuboz: Add fw_config probe for ALC5682-VD & VS
......................................................................

Patch Set 6:

(1 comment)

File src/mainboard/google/zork/variants/shuboz/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/59558/comment/ba823fff_38974985 
PS5, Line 2: fw_config
> Thanks Kane. I've added a comment to https://crrev. […]

Ok, so IIUC based on your comment in https://crrev.com/c/3226182, the issue isn't with probe not enabling the device as we initially thought. I've tested that the correct device is enabled when SSFC is not set (as long as the two devices have different IDs). But rather, having two codec devices in overridetree.cb means that the codec doesn't show up in the ACPI tables for some other reason? I'll have a look at the ACPI tables myself and see if I can reproduce this.