diff options
author | Angel Pons <th3fanbus@gmail.com> | 2021-04-17 12:55:59 +0200 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2021-04-21 09:17:17 +0000 |
commit | f643b63c4dcd74c3de0c1d54de8b740d831d6d73 (patch) | |
tree | 96e1dd61d48d07146eef48b8203e010550631382 | |
parent | e4844ce7c93b97adad543dc868a302d34f9e0282 (diff) |
soc/intel/skylake: Move pmc_set_disb() to pmutil.c
To drop bad __SIMPLE_DEVICE__ usage and for consistency with newer
platforms, move pmc_set_disb() to pmutil.c and adapt it accordingly.
Change-Id: I1a137b5b3120c350a04273567b9cb18c9a42a543
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/52463
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Reviewed-by: Michael Niewöhner <foss@mniewoehner.de>
-rw-r--r-- | src/soc/intel/skylake/pmc.c | 18 | ||||
-rw-r--r-- | src/soc/intel/skylake/pmutil.c | 14 |
2 files changed, 14 insertions, 18 deletions
diff --git a/src/soc/intel/skylake/pmc.c b/src/soc/intel/skylake/pmc.c index 14e9d4ce81..c70710f9c2 100644 --- a/src/soc/intel/skylake/pmc.c +++ b/src/soc/intel/skylake/pmc.c @@ -14,24 +14,6 @@ #include "chip.h" -void pmc_set_disb(void) -{ - /* Set the DISB after DRAM init */ - u32 disb_val; -#if defined(__SIMPLE_DEVICE__) - pci_devfn_t dev = PCH_DEV_PMC; -#else - struct device *dev = PCH_DEV_PMC; -#endif - - disb_val = pci_read_config32(dev, GEN_PMCON_A); - disb_val |= DISB; - - /* Don't clear bits that are write-1-to-clear */ - disb_val &= ~(GBL_RST_STS | MS4V); - pci_write_config32(dev, GEN_PMCON_A, disb_val); -} - /* * Set which power state system will be after reapplying * the power (from G3 State) diff --git a/src/soc/intel/skylake/pmutil.c b/src/soc/intel/skylake/pmutil.c index 115a9e58ec..35862c278b 100644 --- a/src/soc/intel/skylake/pmutil.c +++ b/src/soc/intel/skylake/pmutil.c @@ -113,6 +113,20 @@ const char *const *soc_std_gpe_sts_array(size_t *gpe_arr) return gpe_sts_bits; } +void pmc_set_disb(void) +{ + /* Set the DISB after DRAM init */ + u32 disb_val; + const pci_devfn_t dev = PCH_DEV_PMC; + + disb_val = pci_read_config32(dev, GEN_PMCON_A); + disb_val |= DISB; + + /* Don't clear bits that are write-1-to-clear */ + disb_val &= ~(GBL_RST_STS | MS4V); + pci_write_config32(dev, GEN_PMCON_A, disb_val); +} + uint8_t *pmc_mmio_regs(void) { uint32_t reg32; |