[coreboot-gerrit] Change in coreboot[master]: soc/intel/cannonlake: Add ability to disable Heci1

[Subrata Banik (Code Review)]

Subrata Banik has submitted this change. ( https://review.coreboot.org/c/coreboot/+/32992 )

Change subject: soc/intel/cannonlake: Add ability to disable Heci1
......................................................................

soc/intel/cannonlake: Add ability to disable Heci1

Decide if HECI1 should be hidden prior to boot to OS.

BUG=none
TEST=Boot to OS, verify if Heci1 is disabled on hatch system
     using FSP 1344.

Change-Id: I7c63316c8b04fb101d34064daac5ba4fdc05a63c
Signed-off-by: Bora Guvendik 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/32992
Tested-by: build bot (Jenkins) 
Reviewed-by: Subrata Banik 
---
M src/soc/intel/cannonlake/fsp_params.c
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Subrata Banik: Looks good to me, approved

diff --git a/src/soc/intel/cannonlake/fsp_params.c b/src/soc/intel/cannonlake/fsp_params.c
index 5cdddc6..06c556c 100644
--- a/src/soc/intel/cannonlake/fsp_params.c
+++ b/src/soc/intel/cannonlake/fsp_params.c
@@ -349,6 +349,9 @@
    	params->ScsUfsEnabled = dev->enabled;

params->Heci3Enabled = config->Heci3Enabled;
+#if !CONFIG(HECI_DISABLE_USING_SMM)
+	params->Heci1Disabled = !config->HeciEnabled;
+#endif
    params->Device4Enable = config->Device4Enable;

/* VrConfig Settings for 5 domains