summaryrefslogtreecommitdiff
path: root/src/mainboard/intel/shadowmountain/variants
diff options
context:
space:
mode:
authorFurquan Shaikh <furquan@google.com>2021-04-20 16:57:59 -0700
committerTim Wawrzynczak <twawrzynczak@chromium.org>2021-04-22 15:59:16 +0000
commitc1c1ba5582fa0302476491c46f18cc73b69c88ac (patch)
tree5cbb5d7b5d3bb0c4b71e1665fd8ce7bba9edfa99 /src/mainboard/intel/shadowmountain/variants
parent7400b612041d2b139b743b00748d5e8f8dbb8b06 (diff)
soc/intel/alderlake and mb: Drop PchHdaAudioLink*Enable UPDs from chip.h
FSP uses PchHdaAudioLink{Hda|Dmic|Ssp|Sndw}Enable UPDs to configure GPIO pads for audio. However, mainboard is expected to perform all GPIO configration in coreboot and hence these UPDs must be set to 0. There is no need to expose these UPDs in chip.h and provide mainboard an option to set these in devicetree. This change drops PchHdaAudioLink{Hda|Dmic|Ssp|Sndw}Enable UPDs from chip.h and the corresponding devicetree in mainboards. Currently, shadowmountain already set these UPDs to 0, whereas adlrvp set these to 1. But all the ADL boards are correctly configuring the GPIO pads for audio, so this change should not impact audio for any of these boards. BUG=b:183482000 TEST=adlrvp and shadowmountain build successfully. Change-Id: I90e4eb5cc242a789800f4c9f8c71e9d8c8a2becf Signed-off-by: Furquan Shaikh <furquan@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/52559 Reviewed-by: EricR Lai <ericr_lai@compal.corp-partner.google.com> Reviewed-by: Angel Pons <th3fanbus@gmail.com> Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/mainboard/intel/shadowmountain/variants')
-rw-r--r--src/mainboard/intel/shadowmountain/variants/baseboard/devicetree.cb7
1 files changed, 0 insertions, 7 deletions
diff --git a/src/mainboard/intel/shadowmountain/variants/baseboard/devicetree.cb b/src/mainboard/intel/shadowmountain/variants/baseboard/devicetree.cb
index 9464f10fe0..b06e540b84 100644
--- a/src/mainboard/intel/shadowmountain/variants/baseboard/devicetree.cb
+++ b/src/mainboard/intel/shadowmountain/variants/baseboard/devicetree.cb
@@ -111,13 +111,6 @@ chip soc/intel/alderlake
# HD Audio
register "PchHdaDspEnable" = "1"
- register "PchHdaAudioLinkHdaEnable" = "0"
- register "PchHdaAudioLinkDmicEnable[0]" = "0"
- register "PchHdaAudioLinkDmicEnable[1]" = "0"
- register "PchHdaAudioLinkSspEnable[0]" = "0"
- register "PchHdaAudioLinkSspEnable[1]" = "0"
- register "PchHdaAudioLinkSndwEnable[0]" = "0"
- register "PchHdaAudioLinkSndwEnable[1]" = "0"
# iDisp-Link T-Mode 0: 2T, 2: 4T, 3: 8T, 4: 16T
register "PchHdaIDispLinkTmode" = "3"
# iDisp-Link Freq 4: 96MHz, 3: 48MHz.