Attention is currently required from: Intel coreboot Reviewers, Jayvik Desai, Kapil Porwal, Pranava Y N.
Subrata Banik has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/87109?usp=email )
Change subject: soc/intel/pantherlake: Directly assign HDA SDI enable ......................................................................
soc/intel/pantherlake: Directly assign HDA SDI enable
The double negation (`!!`) was unnecessarily used when assigning the `pch_hda_sdi_enable` type boolean from the SOC config to the FSP M config.
This commit removes the redundant `!!` operator, directly assigning the boolean value of `config->pch_hda_sdi_enable[i]` to `m_cfg->PchHdaSdiEnable[i]`.
TEST=Able to build and boot google/fatcat.
Change-Id: I9233116ca2bfaeac2f685d464a1cb261f067db6a Signed-off-by: Subrata Banik subratabanik@google.com --- M src/soc/intel/pantherlake/romstage/fsp_params.c 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/09/87109/1
diff --git a/src/soc/intel/pantherlake/romstage/fsp_params.c b/src/soc/intel/pantherlake/romstage/fsp_params.c index b5fc93b..521d2f2 100644 --- a/src/soc/intel/pantherlake/romstage/fsp_params.c +++ b/src/soc/intel/pantherlake/romstage/fsp_params.c @@ -162,7 +162,7 @@ m_cfg->PchHdaIDispCodecDisconnect = !config->pch_hda_idisp_codec_enable;
for (int i = 0; i < MAX_HD_AUDIO_SDI_LINKS; i++) - m_cfg->PchHdaSdiEnable[i] = !!config->pch_hda_sdi_enable[i]; + m_cfg->PchHdaSdiEnable[i] = config->pch_hda_sdi_enable[i];
/* * All the PchHdaAudioLink{Hda|Dmic|Ssp|Sndw}Enable UPDs are used by FSP