diff options
| author | Subrata Banik <subratabanik@google.com> | 2022-01-03 19:00:00 +0000 |
|---|---|---|
| committer | Paul Fagerburg <pfagerburg@chromium.org> | 2022-01-14 00:33:23 +0000 |
| commit | f04e83abbf98d1d55ec2c4fea3fb74bf2f459139 (patch) | |
| tree | 913e22a7f68ea646b686ea151282a9fc07048078 /src/soc/intel/jasperlake/smihandler.c | |
| parent | ad50b40eed3f7f235e848a2382ffbee6a51d1755 (diff) | |
| download | coreboot-f04e83abbf98d1d55ec2c4fea3fb74bf2f459139.tar.gz coreboot-f04e83abbf98d1d55ec2c4fea3fb74bf2f459139.tar.bz2 coreboot-f04e83abbf98d1d55ec2c4fea3fb74bf2f459139.zip | |
soc/intel/jsl: Replace dt `HeciEnabled` by `HECI1 disable` config
List of changes:
1. Drop `HeciEnabled` from dt and dt chip configuration.
2. Replace all logic that disables HECI1 based on the `HeciEnabled`
chip config with `DISABLE_HECI1_AT_PRE_BOOT` config.
Mainboards that choose to make HECI1 enable during boot don't override
`heci1 disable` config.
Signed-off-by: Subrata Banik <subratabanik@google.com>
Change-Id: Ib9fb554c8f3cfd1e91bbcd1977905e1321db0802
Reviewed-on: https://review.coreboot.org/c/coreboot/+/60728
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Diffstat (limited to 'src/soc/intel/jasperlake/smihandler.c')
| -rw-r--r-- | src/soc/intel/jasperlake/smihandler.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/src/soc/intel/jasperlake/smihandler.c b/src/soc/intel/jasperlake/smihandler.c index 2e38b68768b2..463967686fa8 100644 --- a/src/soc/intel/jasperlake/smihandler.c +++ b/src/soc/intel/jasperlake/smihandler.c @@ -16,11 +16,7 @@ */ void smihandler_soc_at_finalize(void) { - const struct soc_intel_jasperlake_config *config; - - config = config_of_soc(); - - if (!config->HeciEnabled && CONFIG(HECI_DISABLE_USING_SMM)) + if (CONFIG(DISABLE_HECI1_AT_PRE_BOOT) && CONFIG(HECI_DISABLE_USING_SMM)) heci_disable(); } |