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

[Nico Huber (Code Review)]

Nico Huber 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:
> > 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.

That's a valid case. But why jump through hoops here instead of
updating the whole of coreboot to be more flexible about subsystem
IDs?