sravan/system76-coreboot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

soc/intel/apollolake: Make SMI_STS offset macro definition consistent

Browse Source 
This patch makes all bit field macro definition for SMI_STS register
(offset 0x44) be consistent i.e. ending with "_STS_BIT".

Also modified relevant files where those macros are getting used.

Change-Id: Ibe3fbb459c106a3a58cd9a8b6eb3d7ee92e6ed82
Signed-off-by: Subrata Banik <subrata.banik@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/39022
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
This commit is contained in:
Subrata Banik 2020-02-20 11:53:04 +05:30
parent f5529d9edc
commit 4ab7ef93ee
3 changed files with 52 additions and 51 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

43
src/soc/intel/apollolake/include/soc/pm.h
View File

@ -105,29 +105,30 @@
(ESPI_SMI_EN | APMC_EN | SLP_SMI_EN | GBL_SMI_EN | EOS | GPIO_EN)
#define SMI_STS 0x44
#define SMI_STS_BITS 32
/* Bits for SMI status */
#define ESPI_SMI_STS_BIT 28
#define PMC_OCP_SMI_STS 27
#define SPI_SMI_STS 26
#define SPI_SSMI_STS 25
#define SCC2_SMI_STS 21
#define PCIE_SMI_STS 20
#define SCS_SMI_STS 19
#define HSMBUS_SMI_STS 18
#define XHCI_SMI_STS 17
#define SMBUS_SMI_STS 16
#define SERIRQ_SMI_STS 15
#define PERIODIC_SMI_STS 14
#define TCO_SMI_STS 13
#define MC_SMI_STS 12
#define GPIO_UNLOCK_SMI_STS 11
#define GPIO_SMI_STS 10
#define FAKE_PM1_SMI_STS 8
#define SWSMI_TMR_SMI_STS 6
#define APM_SMI_STS 5
#define SLP_SMI_STS 4
#define LEGACY_USB_SMI_STS 3
#define BIOS_SMI_STS 2
#define PMC_OCP_SMI_STS_BIT 27
#define SPI_SMI_STS_BIT 26
#define SPI_SSMI_STS_BIT 25
#define SCC2_SMI_STS_BIT 21
#define PCI_EXP_SMI_STS_BIT 20
#define SCS_SMI_STS_BIT 19
#define HSMBUS_SMI_STS_BIT 18
#define XHCI_SMI_STS_BIT 17
#define SMBUS_SMI_STS_BIT 16
#define SERIRQ_SMI_STS_BIT 15
#define PERIODIC_STS_BIT 14
#define TCO_STS_BIT 13
#define MC_SMI_STS_BIT 12
#define GPIO_UNLOCK_SMI_STS_BIT 11
#define GPIO_STS_BIT 10
#define PM1_STS_BIT 8
#define SWSMI_TMR_STS_BIT 6
#define APM_STS_BIT 5
#define SMI_ON_SLP_EN_STS_BIT 4
#define LEGACY_USB_STS_BIT 3
#define BIOS_STS_BIT 2
#define GPE_CNTL 0x50
#define DEVACT_STS 0x4c

44
src/soc/intel/apollolake/pmutil.c
View File

@ -57,27 +57,27 @@ uint32_t *soc_pmc_etr_addr(void)
const char *const *soc_smi_sts_array(size_t *a)
{
static const char *const smi_sts_bits[] = {
[BIOS_SMI_STS] = "BIOS",
[LEGACY_USB_SMI_STS] = "LEGACY USB",
[SLP_SMI_STS] = "SLP_SMI",
[APM_SMI_STS] = "APM",
[SWSMI_TMR_SMI_STS] = "SWSMI_TMR",
[FAKE_PM1_SMI_STS] = "PM1",
[GPIO_SMI_STS] = "GPIO_SMI",
[GPIO_UNLOCK_SMI_STS] = "GPIO_UNLOCK_SSMI",
[MC_SMI_STS] = "MCSMI",
[TCO_SMI_STS] = "TCO",
[PERIODIC_SMI_STS] = "PERIODIC",
[SERIRQ_SMI_STS] = "SERIRQ",
[SMBUS_SMI_STS] = "SMBUS_SMI",
[XHCI_SMI_STS] = "XHCI",
[HSMBUS_SMI_STS] = "HOST_SMBUS",
[SCS_SMI_STS] = "SCS",
[PCIE_SMI_STS] = "PCI_EXP_SMI",
[SCC2_SMI_STS] = "SCC2",
[SPI_SSMI_STS] = "SPI_SSMI",
[SPI_SMI_STS] = "SPI",
[PMC_OCP_SMI_STS] = "OCP_CSE",
[BIOS_STS_BIT] = "BIOS",
[LEGACY_USB_STS_BIT] = "LEGACY USB",
[SMI_ON_SLP_EN_STS_BIT] = "SLP_SMI",
[APM_STS_BIT] = "APM",
[SWSMI_TMR_STS_BIT] = "SWSMI_TMR",
[PM1_STS_BIT] = "PM1",
[GPIO_STS_BIT] = "GPIO_SMI",
[GPIO_UNLOCK_SMI_STS_BIT] = "GPIO_UNLOCK_SSMI",
[MC_SMI_STS_BIT] = "MCSMI",
[TCO_STS_BIT] = "TCO",
[PERIODIC_STS_BIT] = "PERIODIC",
[SERIRQ_SMI_STS_BIT] = "SERIRQ",
[SMBUS_SMI_STS_BIT] = "SMBUS_SMI",
[XHCI_SMI_STS_BIT] = "XHCI",
[SCS_SMI_STS_BIT] = "HOST_SMBUS",
[SCS_SMI_STS_BIT] = "SCS",
[PCI_EXP_SMI_STS_BIT] = "PCI_EXP_SMI",
[SCC2_SMI_STS_BIT] = "SCC2",
[SPI_SSMI_STS_BIT] = "SPI_SSMI",
[SPI_SMI_STS_BIT] = "SPI",
[PMC_OCP_SMI_STS_BIT] = "OCP_CSE",
};
*a = ARRAY_SIZE(smi_sts_bits);
@ -98,7 +98,7 @@ uint32_t soc_get_smi_status(uint32_t generic_sts)
/* Fake PM1 status bit if power button pressed. */
if (pm1_sts & PWRBTN_STS)
generic_sts |= (1 << FAKE_PM1_SMI_STS);
generic_sts |= (1 << PM1_STS_BIT);
}
return generic_sts;

16
src/soc/intel/apollolake/smihandler.c
View File

@ -38,19 +38,19 @@ const struct smm_save_state_ops *get_smm_save_state_ops(void)
uint32_t smihandler_soc_get_sci_mask(void)
{
uint32_t sci_mask =
SMI_HANDLER_SCI_EN(APM_SMI_STS) |
SMI_HANDLER_SCI_EN(SLP_SMI_STS);
SMI_HANDLER_SCI_EN(APM_STS_BIT) |
SMI_HANDLER_SCI_EN(SMI_ON_SLP_EN_STS_BIT);
return sci_mask;
}
const smi_handler_t southbridge_smi[32] = {
[SLP_SMI_STS] = smihandler_southbridge_sleep,
[APM_SMI_STS] = smihandler_southbridge_apmc,
[FAKE_PM1_SMI_STS] = smihandler_southbridge_pm1,
[GPIO_SMI_STS] = smihandler_southbridge_gpi,
[TCO_SMI_STS] = smihandler_southbridge_tco,
[PERIODIC_SMI_STS] = smihandler_southbridge_periodic,
[SMI_ON_SLP_EN_STS_BIT] = smihandler_southbridge_sleep,
[APM_STS_BIT] = smihandler_southbridge_apmc,
[PM1_STS_BIT] = smihandler_southbridge_pm1,
[GPIO_STS_BIT] = smihandler_southbridge_gpi,
[TCO_STS_BIT] = smihandler_southbridge_tco,
[PERIODIC_STS_BIT] = smihandler_southbridge_periodic,
#if CONFIG(SOC_ESPI)
[ESPI_SMI_STS_BIT] = smihandler_southbridge_espi,
#endif