Patrick Georgi merged this change.

View Change

Approvals: build bot (Jenkins): Verified Furquan Shaikh: Looks good to me, approved Sathyanarayana Nujella: Looks good to me, but someone else must approve
mb/google/drallion: Enable HDA for drallion platform

Enable PchHdaIDispCodecDisconnect and
PchHdaAudioLinkHda for drallion variants.
This is needed with FSP 1263.

Signed-off-by: Selma BENSAID <selma.bensaid@intel.com>
Change-Id: I13d3dd832c6fbdc2aad5ba578695edb8470806e8
Reviewed-on: https://review.coreboot.org/c/coreboot/+/35079
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Sathyanarayana Nujella <sathyanarayana.nujella@intel.com>
Reviewed-by: Furquan Shaikh <furquan@google.com>
---
M src/mainboard/google/drallion/variants/arcada_cml/devicetree.cb
M src/mainboard/google/drallion/variants/drallion/devicetree.cb
M src/mainboard/google/drallion/variants/sarien_cml/devicetree.cb
3 files changed, 12 insertions(+), 0 deletions(-)

diff --git a/src/mainboard/google/drallion/variants/arcada_cml/devicetree.cb b/src/mainboard/google/drallion/variants/arcada_cml/devicetree.cb
index fe3b6c9..3628264 100644
--- a/src/mainboard/google/drallion/variants/arcada_cml/devicetree.cb
+++ b/src/mainboard/google/drallion/variants/arcada_cml/devicetree.cb
@@ -50,6 +50,10 @@
# Enable DDC for DDI port B
register "DdiPortBDdc" = "1"

+ # Disable iDisplay codec enumeration
+ register "PchHdaIDispCodecDisconnect" = "1"
+ register "PchHdaAudioLinkHda" = "1"
+
# VR Settings Configuration for 4 Domains
#+----------------+-------+-------+-------+-------+
#| Domain/Setting | SA | IA | GTUS | GTS |
diff --git a/src/mainboard/google/drallion/variants/drallion/devicetree.cb b/src/mainboard/google/drallion/variants/drallion/devicetree.cb
index 54184f0..8870126 100644
--- a/src/mainboard/google/drallion/variants/drallion/devicetree.cb
+++ b/src/mainboard/google/drallion/variants/drallion/devicetree.cb
@@ -50,6 +50,10 @@
# Enable DDC for DDI port B
register "DdiPortBDdc" = "1"

+ # Disable iDisplay codec enumeration
+ register "PchHdaIDispCodecDisconnect" = "1"
+ register "PchHdaAudioLinkHda" = "1"
+
# VR Settings Configuration for 4 Domains
#+----------------+-------+-------+-------+-------+
#| Domain/Setting | SA | IA | GTUS | GTS |
diff --git a/src/mainboard/google/drallion/variants/sarien_cml/devicetree.cb b/src/mainboard/google/drallion/variants/sarien_cml/devicetree.cb
index d3aab62..84aacd5 100644
--- a/src/mainboard/google/drallion/variants/sarien_cml/devicetree.cb
+++ b/src/mainboard/google/drallion/variants/sarien_cml/devicetree.cb
@@ -56,6 +56,10 @@
register "LanWakeFromDeepSx" = "0"
register "WolEnableOverride" = "0"

+ # Disable iDisplay codec enumeration
+ register "PchHdaIDispCodecDisconnect" = "1"
+ register "PchHdaAudioLinkHda" = "1"
+
# VR Settings Configuration for 4 Domains
#+----------------+-------+-------+-------+-------+
#| Domain/Setting | SA | IA | GTUS | GTS |

To view, visit change 35079. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I13d3dd832c6fbdc2aad5ba578695edb8470806e8
Gerrit-Change-Number: 35079
Gerrit-PatchSet: 5
Gerrit-Owner: Selma Bensaid <selma.bensaid@intel.com>
Gerrit-Reviewer: Aamir Bohra <aamir.bohra@intel.com>
Gerrit-Reviewer: Bernardo Perez Priego <bernardo.perez.priego@intel.com>
Gerrit-Reviewer: Furquan Shaikh <furquan@google.com>
Gerrit-Reviewer: Joyce Toh <joyce.toh@intel.com>
Gerrit-Reviewer: Mathew King <mathewk@chromium.org>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Sathyanarayana Nujella <sathyanarayana.nujella@intel.com>
Gerrit-Reviewer: Selma Bensaid <selma.bensaid@intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Martin Roth <martinroth@google.com>
Gerrit-MessageType: merged