From e9718ff79d3fcff1d01e43ff0d8f4a138f08c2f1 Mon Sep 17 00:00:00 2001 From: Elyes Haouas Date: Tue, 28 Jan 2025 16:07:15 +0100 Subject: [PATCH] tree: Use boolean for PchHdaDspEnable Change-Id: I47852c9b023cc4839000019b8a932b6e471fa839 Signed-off-by: Elyes Haouas Reviewed-on: https://review.coreboot.org/c/coreboot/+/86190 Tested-by: build bot (Jenkins) Reviewed-by: Maxim Polyakov --- src/mainboard/google/dedede/variants/baseboard/devicetree.cb | 2 +- src/mainboard/google/hatch/variants/baseboard/devicetree.cb | 2 +- src/mainboard/google/puff/variants/baseboard/devicetree.cb | 2 +- src/mainboard/google/volteer/variants/baseboard/devicetree.cb | 2 +- .../intel/coffeelake_rvp/variants/baseboard/devicetree.cb | 2 +- .../intel/coffeelake_rvp/variants/cfl_h/overridetree.cb | 2 +- .../intel/coffeelake_rvp/variants/cfl_s/overridetree.cb | 2 +- .../intel/coffeelake_rvp/variants/cfl_u/overridetree.cb | 2 +- .../intel/coffeelake_rvp/variants/cml_u/overridetree.cb | 2 +- .../intel/coffeelake_rvp/variants/whl_u/overridetree.cb | 2 +- .../intel/jasperlake_rvp/variants/jslrvp/devicetree.cb | 2 +- src/mainboard/intel/tglrvp/variants/tglrvp_up3/devicetree.cb | 2 +- src/mainboard/intel/tglrvp/variants/tglrvp_up4/devicetree.cb | 2 +- src/mainboard/prodrive/hermes/devicetree.cb | 2 +- src/mainboard/purism/librem_jsl/devicetree.cb | 2 +- src/mainboard/siemens/chili/variants/base/devicetree.cb | 2 +- src/mainboard/siemens/chili/variants/chili/devicetree.cb | 2 +- src/soc/intel/elkhartlake/chip.h | 2 +- src/soc/intel/tigerlake/chip.h | 2 +- 19 files changed, 19 insertions(+), 19 deletions(-) diff --git a/src/mainboard/google/dedede/variants/baseboard/devicetree.cb b/src/mainboard/google/dedede/variants/baseboard/devicetree.cb index 2b55e7c9eb..879f4d07cb 100644 --- a/src/mainboard/google/dedede/variants/baseboard/devicetree.cb +++ b/src/mainboard/google/dedede/variants/baseboard/devicetree.cb @@ -129,7 +129,7 @@ chip soc/intel/jasperlake register "PcieClkSrcClkReq[5]" = "5" # Audio related configurations - register "PchHdaDspEnable" = "1" + register "PchHdaDspEnable" = "true" register "PchHdaAudioLinkHdaEnable" = "1" register "PchHdaAudioLinkSspEnable[0]" = "1" register "PchHdaAudioLinkSspEnable[1]" = "1" diff --git a/src/mainboard/google/hatch/variants/baseboard/devicetree.cb b/src/mainboard/google/hatch/variants/baseboard/devicetree.cb index 6b3eff6676..da1c45f1a5 100644 --- a/src/mainboard/google/hatch/variants/baseboard/devicetree.cb +++ b/src/mainboard/google/hatch/variants/baseboard/devicetree.cb @@ -177,7 +177,7 @@ chip soc/intel/cannonlake register "PcieClkSrcClkReq[3]" = "3" #Enable I2S Audio, SSP0, SSP1 and DMIC0, default DMIC1 N/A (by variants override) - register "PchHdaDspEnable" = "1" + register "PchHdaDspEnable" = "true" register "PchHdaAudioLinkSsp0" = "1" register "PchHdaAudioLinkSsp1" = "1" register "PchHdaAudioLinkDmic0" = "1" diff --git a/src/mainboard/google/puff/variants/baseboard/devicetree.cb b/src/mainboard/google/puff/variants/baseboard/devicetree.cb index de4631aa52..c5b01215a5 100644 --- a/src/mainboard/google/puff/variants/baseboard/devicetree.cb +++ b/src/mainboard/google/puff/variants/baseboard/devicetree.cb @@ -177,7 +177,7 @@ chip soc/intel/cannonlake register "PcieClkSrcClkReq[3]" = "3" #Enable I2S Audio, SSP0, SSP1 and DMIC0, default DMIC1 N/A (by variants override) - register "PchHdaDspEnable" = "1" + register "PchHdaDspEnable" = "true" register "PchHdaAudioLinkSsp0" = "1" register "PchHdaAudioLinkSsp1" = "1" register "PchHdaAudioLinkDmic0" = "1" diff --git a/src/mainboard/google/volteer/variants/baseboard/devicetree.cb b/src/mainboard/google/volteer/variants/baseboard/devicetree.cb index 8110bb7e07..9c7b592162 100644 --- a/src/mainboard/google/volteer/variants/baseboard/devicetree.cb +++ b/src/mainboard/google/volteer/variants/baseboard/devicetree.cb @@ -506,7 +506,7 @@ chip soc/intel/tigerlake end end device ref hda on - register "PchHdaDspEnable" = "1" + register "PchHdaDspEnable" = "true" chip drivers/sof register "spkr_tplg" = "max98373" diff --git a/src/mainboard/intel/coffeelake_rvp/variants/baseboard/devicetree.cb b/src/mainboard/intel/coffeelake_rvp/variants/baseboard/devicetree.cb index 012afc41f8..16c68cd1a3 100644 --- a/src/mainboard/intel/coffeelake_rvp/variants/baseboard/devicetree.cb +++ b/src/mainboard/intel/coffeelake_rvp/variants/baseboard/devicetree.cb @@ -22,7 +22,7 @@ chip soc/intel/cannonlake register "usb3_ports[4]" = "USB3_PORT_DEFAULT(OC0)" register "usb3_ports[5]" = "USB3_PORT_DEFAULT(OC0)" - register "PchHdaDspEnable" = "1" + register "PchHdaDspEnable" = "true" register "PchHdaAudioLinkHda" = "1" register "PcieClkSrcUsage[0]" = "1" diff --git a/src/mainboard/intel/coffeelake_rvp/variants/cfl_h/overridetree.cb b/src/mainboard/intel/coffeelake_rvp/variants/cfl_h/overridetree.cb index 049859a1ba..031974fdca 100644 --- a/src/mainboard/intel/coffeelake_rvp/variants/cfl_h/overridetree.cb +++ b/src/mainboard/intel/coffeelake_rvp/variants/cfl_h/overridetree.cb @@ -39,7 +39,7 @@ chip soc/intel/cannonlake register "SataPortsEnable[6]" = "1" register "SataPortsEnable[7]" = "1" - register "PchHdaDspEnable" = "0" + register "PchHdaDspEnable" = "false" register "PchHdaAudioLinkHda" = "1" register "PcieRpEnable[0]" = "true" diff --git a/src/mainboard/intel/coffeelake_rvp/variants/cfl_s/overridetree.cb b/src/mainboard/intel/coffeelake_rvp/variants/cfl_s/overridetree.cb index 266c698afc..3f1820e3e0 100644 --- a/src/mainboard/intel/coffeelake_rvp/variants/cfl_s/overridetree.cb +++ b/src/mainboard/intel/coffeelake_rvp/variants/cfl_s/overridetree.cb @@ -37,7 +37,7 @@ chip soc/intel/cannonlake register "SataPortsEnable[6]" = "1" register "SataPortsEnable[7]" = "1" - register "PchHdaDspEnable" = "0" + register "PchHdaDspEnable" = "false" register "PchHdaAudioLinkHda" = "1" register "PcieRpEnable[0]" = "true" diff --git a/src/mainboard/intel/coffeelake_rvp/variants/cfl_u/overridetree.cb b/src/mainboard/intel/coffeelake_rvp/variants/cfl_u/overridetree.cb index 0133a1154e..84ebda58fc 100644 --- a/src/mainboard/intel/coffeelake_rvp/variants/cfl_u/overridetree.cb +++ b/src/mainboard/intel/coffeelake_rvp/variants/cfl_u/overridetree.cb @@ -1,6 +1,6 @@ chip soc/intel/cannonlake - register "PchHdaDspEnable" = "1" + register "PchHdaDspEnable" = "true" register "PchHdaAudioLinkHda" = "1" register "PcieRpEnable[0]" = "true" diff --git a/src/mainboard/intel/coffeelake_rvp/variants/cml_u/overridetree.cb b/src/mainboard/intel/coffeelake_rvp/variants/cml_u/overridetree.cb index 157a465f68..846fea9c3d 100644 --- a/src/mainboard/intel/coffeelake_rvp/variants/cml_u/overridetree.cb +++ b/src/mainboard/intel/coffeelake_rvp/variants/cml_u/overridetree.cb @@ -38,7 +38,7 @@ chip soc/intel/cannonlake register "SataPortsEnable[6]" = "1" register "SataPortsEnable[7]" = "1" - register "PchHdaDspEnable" = "1" + register "PchHdaDspEnable" = "true" register "PchHdaAudioLinkHda" = "1" register "PcieRpEnable[0]" = "true" diff --git a/src/mainboard/intel/coffeelake_rvp/variants/whl_u/overridetree.cb b/src/mainboard/intel/coffeelake_rvp/variants/whl_u/overridetree.cb index 8df985164f..89e59a469c 100644 --- a/src/mainboard/intel/coffeelake_rvp/variants/whl_u/overridetree.cb +++ b/src/mainboard/intel/coffeelake_rvp/variants/whl_u/overridetree.cb @@ -23,7 +23,7 @@ chip soc/intel/cannonlake register "SataPortsEnable[6]" = "1" register "SataPortsEnable[7]" = "1" - register "PchHdaDspEnable" = "1" + register "PchHdaDspEnable" = "true" register "PchHdaAudioLinkHda" = "1" register "PcieRpEnable[0]" = "true" diff --git a/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb b/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb index 56c7bd6a0b..b4a9261188 100644 --- a/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb +++ b/src/mainboard/intel/jasperlake_rvp/variants/jslrvp/devicetree.cb @@ -56,7 +56,7 @@ chip soc/intel/jasperlake # Skip the CPU replacement check register "SkipCpuReplacementCheck" = "1" - register "PchHdaDspEnable" = "1" + register "PchHdaDspEnable" = "true" register "PchHdaAudioLinkHdaEnable" = "0" register "PchHdaAudioLinkSspEnable[0]" = "1" register "PchHdaAudioLinkSspEnable[1]" = "1" diff --git a/src/mainboard/intel/tglrvp/variants/tglrvp_up3/devicetree.cb b/src/mainboard/intel/tglrvp/variants/tglrvp_up3/devicetree.cb index b9af2c573c..032e80efc6 100644 --- a/src/mainboard/intel/tglrvp/variants/tglrvp_up3/devicetree.cb +++ b/src/mainboard/intel/tglrvp/variants/tglrvp_up3/devicetree.cb @@ -322,7 +322,7 @@ chip soc/intel/tigerlake end end device ref hda on - register "PchHdaDspEnable" = "1" + register "PchHdaDspEnable" = "true" register "PchHdaAudioLinkDmicEnable" = "{ [0] = 1, [1] = 1, diff --git a/src/mainboard/intel/tglrvp/variants/tglrvp_up4/devicetree.cb b/src/mainboard/intel/tglrvp/variants/tglrvp_up4/devicetree.cb index 7e00917a12..720e40f438 100644 --- a/src/mainboard/intel/tglrvp/variants/tglrvp_up4/devicetree.cb +++ b/src/mainboard/intel/tglrvp/variants/tglrvp_up4/devicetree.cb @@ -325,7 +325,7 @@ chip soc/intel/tigerlake end end device ref hda on - register "PchHdaDspEnable" = "1" + register "PchHdaDspEnable" = "true" register "PchHdaAudioLinkDmicEnable" = "{ [0] = 1, [1] = 1, diff --git a/src/mainboard/prodrive/hermes/devicetree.cb b/src/mainboard/prodrive/hermes/devicetree.cb index cb5f211b29..e9bd24a072 100644 --- a/src/mainboard/prodrive/hermes/devicetree.cb +++ b/src/mainboard/prodrive/hermes/devicetree.cb @@ -23,7 +23,7 @@ chip soc/intel/cannonlake register "SataPortsHotPlug[6]" = "1" register "SataPortsHotPlug[7]" = "1" - register "PchHdaDspEnable" = "0" + register "PchHdaDspEnable" = "false" register "PchHdaAudioLinkHda" = "1" register "PcieClkSrcUsage[0]" = "20" # PCIe Slot1 diff --git a/src/mainboard/purism/librem_jsl/devicetree.cb b/src/mainboard/purism/librem_jsl/devicetree.cb index c00404dff5..82cac1239a 100644 --- a/src/mainboard/purism/librem_jsl/devicetree.cb +++ b/src/mainboard/purism/librem_jsl/devicetree.cb @@ -39,7 +39,7 @@ chip soc/intel/jasperlake register "SerialIoUartMode[2]" = "PchSerialIoSkipInit" # Audio related configurations - register "PchHdaDspEnable" = "1" + register "PchHdaDspEnable" = "true" register "PchHdaAudioLinkHdaEnable" = "1" device domain 0 on diff --git a/src/mainboard/siemens/chili/variants/base/devicetree.cb b/src/mainboard/siemens/chili/variants/base/devicetree.cb index 851a8bfa58..f217461645 100644 --- a/src/mainboard/siemens/chili/variants/base/devicetree.cb +++ b/src/mainboard/siemens/chili/variants/base/devicetree.cb @@ -4,7 +4,7 @@ chip soc/intel/cannonlake # FSP configuration register "RMT" = "0" - register "PchHdaDspEnable" = "0" + register "PchHdaDspEnable" = "false" register "PchHdaAudioLinkHda" = "1" device domain 0 on diff --git a/src/mainboard/siemens/chili/variants/chili/devicetree.cb b/src/mainboard/siemens/chili/variants/chili/devicetree.cb index c5121f9a8a..38358d824f 100644 --- a/src/mainboard/siemens/chili/variants/chili/devicetree.cb +++ b/src/mainboard/siemens/chili/variants/chili/devicetree.cb @@ -4,7 +4,7 @@ chip soc/intel/cannonlake # FSP configuration register "RMT" = "0" - register "PchHdaDspEnable" = "0" + register "PchHdaDspEnable" = "false" register "PchHdaAudioLinkHda" = "1" device domain 0 on diff --git a/src/soc/intel/elkhartlake/chip.h b/src/soc/intel/elkhartlake/chip.h index 397372d98d..d4bb04ff75 100644 --- a/src/soc/intel/elkhartlake/chip.h +++ b/src/soc/intel/elkhartlake/chip.h @@ -203,7 +203,7 @@ struct soc_intel_elkhartlake_config { uint16_t SataPortsDitoVal[CONFIG_MAX_SATA_PORTS]; /* Audio related */ - uint8_t PchHdaDspEnable; + bool PchHdaDspEnable; uint8_t PchHdaAudioLinkHdaEnable; uint8_t PchHdaSdiEnable[MAX_HD_AUDIO_SDI_LINKS]; uint8_t PchHdaAudioLinkDmicEnable[MAX_HD_AUDIO_DMIC_LINKS]; diff --git a/src/soc/intel/tigerlake/chip.h b/src/soc/intel/tigerlake/chip.h index 41c7dd3c5d..2c0a9b4073 100644 --- a/src/soc/intel/tigerlake/chip.h +++ b/src/soc/intel/tigerlake/chip.h @@ -249,7 +249,7 @@ struct soc_intel_tigerlake_config { uint16_t SataPortsDitoVal[8]; /* Audio related */ - uint8_t PchHdaDspEnable; + bool PchHdaDspEnable; uint8_t PchHdaAudioLinkHdaEnable; uint8_t PchHdaAudioLinkDmicEnable[MAX_HD_AUDIO_DMIC_LINKS]; uint8_t PchHdaAudioLinkSspEnable[MAX_HD_AUDIO_SSP_LINKS];