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

1 Dec 2018


      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 3:
Is this just setting subsystem ids of a PCI device? These should be
set in the devicetree and no blob needed.
If yes, this seems backwards. Shouldn't we set the id of the PCI
device first and when configuring the codec take the id of the PCI
device?
-- 
To view, visit https://review.coreboot.org/c/coreboot/+/29651
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I25e22313fd99479f1a2f68636a2eab83126ca488
Gerrit-Change-Number: 29651
Gerrit-PatchSet: 3
Gerrit-Owner: Lijian Zhao lijian.zhao@intel.com
Gerrit-Reviewer: Jairaj Arava jairaj.arava@intel.com
Gerrit-Reviewer: Krzysztof M Sywula krzysztof.m.sywula@intel.com
Gerrit-Reviewer: Lijian Zhao lijian.zhao@intel.com
Gerrit-Reviewer: Patrick Rudolph siro@das-labor.org
Gerrit-Reviewer: Pratikkumar V Prajapati pratikkumar.v.prajapati@intel.com
Gerrit-Reviewer: Sathyanarayana Nujella sathyanarayana.nujella@intel.com
Gerrit-Reviewer: Subrata Banik subrata.banik@intel.com
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-CC: Duncan Laurie dlaurie@chromium.org
Gerrit-CC: Nico Huber nico.h@gmx.de
Gerrit-Comment-Date: Fri, 30 Nov 2018 23:49:29 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment