diff options
Diffstat (limited to 'src/soc/intel/common/block/p2sb/p2sblib.c')
| -rw-r--r-- | src/soc/intel/common/block/p2sb/p2sblib.c | 31 |
1 files changed, 27 insertions, 4 deletions
diff --git a/src/soc/intel/common/block/p2sb/p2sblib.c b/src/soc/intel/common/block/p2sb/p2sblib.c index e1a5bc2ef8..537f388cb2 100644 --- a/src/soc/intel/common/block/p2sb/p2sblib.c +++ b/src/soc/intel/common/block/p2sb/p2sblib.c @@ -63,7 +63,7 @@ void p2sb_dev_hide(pci_devfn_t dev) "Unable to hide the P2SB device!\n"); } -static void p2sb_execute_sideband_access(pci_devfn_t dev, uint8_t cmd, uint8_t pid, +static void p2sb_send_sideband_msg(pci_devfn_t dev, uint8_t cmd, uint8_t pid, uint16_t reg, uint32_t *data) { struct pcr_sbi_msg msg = { @@ -78,17 +78,40 @@ static void p2sb_execute_sideband_access(pci_devfn_t dev, uint8_t cmd, uint8_t p uint8_t response; int status; - /* Unhide the P2SB device */ - p2sb_dev_unhide(dev); - status = pcr_execute_sideband_msg(dev, &msg, data, &response); if (status || response) printk(BIOS_ERR, "Fail to execute p2sb sideband access\n"); +} + +static void p2sb_execute_sbi_in_smm(pci_devfn_t dev, uint8_t cmd, uint8_t pid, + uint16_t reg, uint32_t *data) +{ + /* Unhide the P2SB device */ + p2sb_dev_unhide(dev); + + p2sb_send_sideband_msg(dev, cmd, pid, reg, data); /* Hide the P2SB device */ p2sb_dev_hide(dev); } +static void p2sb_execute_sideband_access(pci_devfn_t dev, uint8_t cmd, uint8_t pid, + uint16_t reg, uint32_t *data) +{ + if (ENV_SMM) + /* + * FSP-S will hide the P2SB device. With the device hidden, we will not be + * able to send the sideband interface message. Therefore, we need to unhide + * the P2SB device which can only be done in SMM requiring that this + * function is called from SMM. + */ + p2sb_execute_sbi_in_smm(dev, cmd, pid, reg, data); + else if (!p2sb_dev_is_hidden(dev)) + p2sb_send_sideband_msg(dev, cmd, pid, reg, data); + else + printk(BIOS_WARNING, "Error: P2SB must be hidden, try calling from SMM!\n"); +} + uint32_t p2sb_dev_sbi_read(pci_devfn_t dev, uint8_t pid, uint16_t reg) { uint32_t val = 0; |