diff options
author | Subrata Banik <subrata.banik@intel.com> | 2021-06-09 03:57:01 +0530 |
---|---|---|
committer | Subrata Banik <subrata.banik@intel.com> | 2021-06-16 03:48:29 +0000 |
commit | 6f910e24b6650da46bc45ea6bd25d426b3361752 (patch) | |
tree | 5ef621cb237c208803b980dc8380020b311920c6 /src/soc/intel/cannonlake | |
parent | 54a3417254858fda03476f6bbbad37d7f8d251e6 (diff) |
soc/intel/cannonlake: Make use of is_devfn_enabled() function
1. Replace all pcidev_path_on_root() and is_dev_enabled() functions
combination with is_devfn_enabled().
2. Remove unused local variable of device structure type (struct device *).
3. Replace pcidev_path_on_root() and dev->enabled check with is_devfn_enabled()
call.
TEST=Able to build and boot without any regression seen on CML.
Signed-off-by: Subrata Banik <subrata.banik@intel.com>
Change-Id: Ib5df5fd32e2e2742d349754a942bf81ca505dd39
Reviewed-on: https://review.coreboot.org/c/coreboot/+/55328
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Diffstat (limited to 'src/soc/intel/cannonlake')
-rw-r--r-- | src/soc/intel/cannonlake/fsp_params.c | 67 | ||||
-rw-r--r-- | src/soc/intel/cannonlake/smihandler.c | 3 |
2 files changed, 18 insertions, 52 deletions
diff --git a/src/soc/intel/cannonlake/fsp_params.c b/src/soc/intel/cannonlake/fsp_params.c index 49b26696af..7f1476c39c 100644 --- a/src/soc/intel/cannonlake/fsp_params.c +++ b/src/soc/intel/cannonlake/fsp_params.c @@ -42,10 +42,7 @@ static const pci_devfn_t serial_io_dev[] = { static uint8_t get_param_value(const config_t *config, uint32_t dev_offset) { - struct device *dev; - - dev = pcidev_path_on_root(serial_io_dev[dev_offset]); - if (!dev || !dev->enabled) + if (!is_devfn_enabled(serial_io_dev[dev_offset])) return PCH_SERIAL_IO_INDEX(PchSerialIoDisabled); if ((config->SerialIoDevMode[dev_offset] >= PchSerialIoMax) || @@ -149,11 +146,8 @@ void platform_fsp_silicon_init_params_cb(FSPS_UPD *supd) params->PchLockDownRtcMemoryLock = 0; /* SATA */ - dev = pcidev_path_on_root(PCH_DEVFN_SATA); - if (!dev) - params->SataEnable = 0; - else { - params->SataEnable = dev->enabled; + params->SataEnable = is_devfn_enabled(PCH_DEVFN_SATA); + if (params->SataEnable) { params->SataMode = config->SataMode; params->SataPwrOptEnable = config->satapwroptimize; params->SataSalpSupport = config->SataSalpSupport; @@ -175,12 +169,9 @@ void platform_fsp_silicon_init_params_cb(FSPS_UPD *supd) params->PchPmSlpS0Vm075VSupport = config->PchPmSlpS0Vm075VSupport; /* Lan */ - dev = pcidev_path_on_root(PCH_DEVFN_GBE); - if (!dev) - params->PchLanEnable = 0; - else { - params->PchLanEnable = dev->enabled; - if (config->s0ix_enable && params->PchLanEnable) { + params->PchLanEnable = is_devfn_enabled(PCH_DEVFN_GBE); + if (params->PchLanEnable) { + if (config->s0ix_enable) { /* * The VmControl UPDs need to be set as per board * design to allow voltage margining in S0ix to lower @@ -324,17 +315,10 @@ void platform_fsp_silicon_init_params_cb(FSPS_UPD *supd) #endif /* Enable CNVi Wifi if enabled in device tree */ - dev = pcidev_path_on_root(PCH_DEVFN_CNViWIFI); #if CONFIG(SOC_INTEL_COMETLAKE) - if (dev) - params->CnviMode = dev->enabled; - else - params->CnviMode = 0; + params->CnviMode = is_devfn_enabled(PCH_DEVFN_CNViWIFI); #else - if (dev) - params->PchCnviMode = dev->enabled; - else - params->PchCnviMode = 0; + params->PchCnviMode = is_devfn_enabled(PCH_DEVFN_CNViWIFI); #endif /* PCI Express */ for (i = 0; i < ARRAY_SIZE(config->PcieClkSrcUsage); i++) { @@ -366,11 +350,8 @@ void platform_fsp_silicon_init_params_cb(FSPS_UPD *supd) }; /* eMMC and SD */ - dev = pcidev_path_on_root(PCH_DEVFN_EMMC); - if (!dev) - params->ScsEmmcEnabled = 0; - else { - params->ScsEmmcEnabled = dev->enabled; + params->ScsEmmcEnabled = is_devfn_enabled(PCH_DEVFN_EMMC); + if (params->ScsEmmcEnabled) { params->ScsEmmcHs400Enabled = config->ScsEmmcHs400Enabled; params->PchScsEmmcHs400DllDataValid = config->EmmcHs400DllNeed; if (config->EmmcHs400DllNeed == 1) { @@ -381,29 +362,19 @@ void platform_fsp_silicon_init_params_cb(FSPS_UPD *supd) } } - dev = pcidev_path_on_root(PCH_DEVFN_SDCARD); - if (!dev) { - params->ScsSdCardEnabled = 0; - } else { - params->ScsSdCardEnabled = dev->enabled; - params->SdCardPowerEnableActiveHigh = - CONFIG(MB_HAS_ACTIVE_HIGH_SD_PWR_ENABLE); + params->ScsSdCardEnabled = is_devfn_enabled(PCH_DEVFN_SDCARD); + if (params->ScsSdCardEnabled) { + params->SdCardPowerEnableActiveHigh = CONFIG(MB_HAS_ACTIVE_HIGH_SD_PWR_ENABLE); #if CONFIG(SOC_INTEL_COMETLAKE) params->ScsSdCardWpPinEnabled = config->ScsSdCardWpPinEnabled; #endif } - dev = pcidev_path_on_root(PCH_DEVFN_UFS); - if (!dev) - params->ScsUfsEnabled = 0; - else - params->ScsUfsEnabled = dev->enabled; + params->ScsUfsEnabled = is_devfn_enabled(PCH_DEVFN_UFS); - dev = pcidev_path_on_root(PCH_DEVFN_CSE_3); - params->Heci3Enabled = is_dev_enabled(dev); + params->Heci3Enabled = is_devfn_enabled(PCH_DEVFN_CSE_3); #if !CONFIG(HECI_DISABLE_USING_SMM) - dev = pcidev_path_on_root(PCH_DEVFN_CSE); - params->Heci1Disabled = !is_dev_enabled(dev); + params->Heci1Disabled = !is_devfn_enabled(PCH_DEVFN_CSE); #endif params->Device4Enable = config->Device4Enable; @@ -510,11 +481,7 @@ void platform_fsp_silicon_init_params_cb(FSPS_UPD *supd) params->VrPowerDeliveryDesign = config->VrPowerDeliveryDesign; #endif - dev = pcidev_path_on_root(SA_DEVFN_IGD); - if (CONFIG(RUN_FSP_GOP) && dev && dev->enabled) - params->PeiGraphicsPeimInit = 1; - else - params->PeiGraphicsPeimInit = 0; + params->PeiGraphicsPeimInit = CONFIG(RUN_FSP_GOP) && is_devfn_enabled(SA_DEVFN_IGD); params->PavpEnable = CONFIG(PAVP); diff --git a/src/soc/intel/cannonlake/smihandler.c b/src/soc/intel/cannonlake/smihandler.c index bfee1c0904..a287a7bc0a 100644 --- a/src/soc/intel/cannonlake/smihandler.c +++ b/src/soc/intel/cannonlake/smihandler.c @@ -19,8 +19,7 @@ void smihandler_soc_at_finalize(void) if (!CONFIG(HECI_DISABLE_USING_SMM)) return; - const struct device *dev = pcidev_path_on_root(PCH_DEVFN_CSE); - if (!is_dev_enabled(dev)) + if (!is_devfn_enabled(PCH_DEVFN_CSE)) heci_disable(); } |