[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/cannonlake: Program HD Audio SVID/SSID

[Duncan Laurie (Code Review)]

Duncan Laurie has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29651 )

Change subject: soc/intel/cannonlake: Program HD Audio SVID/SSID
......................................................................

Patch Set 5:
> Patch Set 3:
>
> Is this just setting subsystem ids of a PCI device? These should be
> set in the devicetree and no blob needed.

FWIW the reason we set subsystem IDs in Kconfig and not devicetree is so we (google) can easily override them when building firmware images for unreleased devices.