soc/intel/alderlake: Add (and fix) devices in IRQ table

Some devices were missing from the IRQ table, and this lack of
IRQ programming for the devices (although unused), was causing S0ix
entry to fail.

BUG=b:176858827
TEST=suspend_stress_test -c10 passes, EC observes SLP_S0IX# toggle
correctly upon entry/exit from S0ix

Signed-off-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Change-Id: Ia7612ee008842ba2b8dcd36deb201f4f26130660
Reviewed-on: https://review.coreboot.org/c/coreboot/+/56175
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
Reviewed-by: Maulik V Vaghela <maulik.v.vaghela@intel.com>
This commit is contained in:
Tim Wawrzynczak 2021-07-09 10:23:10 -06:00
parent 989c7c4f8b
commit 82225b81f8
1 changed files with 51 additions and 6 deletions

View File

@ -48,7 +48,8 @@ static const struct slot_irq_constraints irq_constraints[] = {
{ {
.slot = SA_DEV_SLOT_IGD, .slot = SA_DEV_SLOT_IGD,
.fns = { .fns = {
ANY_PIRQ(SA_DEVFN_IGD), /* INTERRUPT_PIN is RO/0x01 */
FIXED_INT_ANY_PIRQ(SA_DEVFN_IGD, PCI_INT_A),
}, },
}, },
{ {
@ -60,14 +61,16 @@ static const struct slot_irq_constraints irq_constraints[] = {
{ {
.slot = SA_DEV_SLOT_IPU, .slot = SA_DEV_SLOT_IPU,
.fns = { .fns = {
ANY_PIRQ(SA_DEVFN_IPU), /* INTERRUPT_PIN is RO/0x01, and INTERRUPT_LINE is RW,
but S0ix fails when not set to 16 (b/193434192) */
FIXED_INT_PIRQ(SA_DEVFN_IPU, PCI_INT_A, PIRQ_A),
}, },
}, },
{ {
.slot = SA_DEV_SLOT_CPU_6, .slot = SA_DEV_SLOT_CPU_6,
.fns = { .fns = {
ANY_PIRQ(SA_DEVFN_CPU_PCIE6_0), FIXED_INT_PIRQ(SA_DEVFN_CPU_PCIE6_0, PCI_INT_A, PIRQ_A),
ANY_PIRQ(SA_DEVFN_CPU_PCIE6_2), FIXED_INT_PIRQ(SA_DEVFN_CPU_PCIE6_2, PCI_INT_C, PIRQ_C),
}, },
}, },
{ {
@ -79,10 +82,36 @@ static const struct slot_irq_constraints irq_constraints[] = {
ANY_PIRQ(SA_DEVFN_TBT3), ANY_PIRQ(SA_DEVFN_TBT3),
}, },
}, },
{
.slot = SA_DEV_SLOT_GNA,
.fns = {
/* INTERRUPT_PIN is RO/0x01 */
FIXED_INT_ANY_PIRQ(SA_DEVFN_GNA, PCI_INT_A),
},
},
{ {
.slot = SA_DEV_SLOT_TCSS, .slot = SA_DEV_SLOT_TCSS,
.fns = { .fns = {
ANY_PIRQ(SA_DEVFN_TCSS_XHCI), ANY_PIRQ(SA_DEVFN_TCSS_XHCI),
ANY_PIRQ(SA_DEVFN_TCSS_XDCI),
},
},
{
.slot = PCH_DEV_SLOT_SIO0,
.fns = {
DIRECT_IRQ(PCH_DEVFN_I2C6),
DIRECT_IRQ(PCH_DEVFN_I2C7),
ANY_PIRQ(PCH_DEVFN_THC0),
ANY_PIRQ(PCH_DEVFN_THC1),
},
},
{
.slot = PCH_DEV_SLOT_SIO6,
.fns = {
DIRECT_IRQ(PCH_DEVFN_UART3),
DIRECT_IRQ(PCH_DEVFN_UART4),
DIRECT_IRQ(PCH_DEVFN_UART5),
DIRECT_IRQ(PCH_DEVFN_UART6),
}, },
}, },
{ {
@ -90,12 +119,23 @@ static const struct slot_irq_constraints irq_constraints[] = {
.fns = { .fns = {
DIRECT_IRQ(PCH_DEVFN_ISH), DIRECT_IRQ(PCH_DEVFN_ISH),
DIRECT_IRQ(PCH_DEVFN_GSPI2), DIRECT_IRQ(PCH_DEVFN_GSPI2),
ANY_PIRQ(PCH_DEVFN_UFS),
},
},
{
.slot = PCH_DEV_SLOT_SIO2,
.fns = {
DIRECT_IRQ(PCH_DEVFN_GSPI3),
DIRECT_IRQ(PCH_DEVFN_GSPI4),
DIRECT_IRQ(PCH_DEVFN_GSPI5),
DIRECT_IRQ(PCH_DEVFN_GSPI6),
}, },
}, },
{ {
.slot = PCH_DEV_SLOT_XHCI, .slot = PCH_DEV_SLOT_XHCI,
.fns = { .fns = {
ANY_PIRQ(PCH_DEVFN_XHCI), ANY_PIRQ(PCH_DEVFN_XHCI),
DIRECT_IRQ(PCH_DEVFN_USBOTG),
ANY_PIRQ(PCH_DEVFN_CNVI_WIFI), ANY_PIRQ(PCH_DEVFN_CNVI_WIFI),
}, },
}, },
@ -158,10 +198,14 @@ static const struct slot_irq_constraints irq_constraints[] = {
{ {
.slot = PCH_DEV_SLOT_SIO5, .slot = PCH_DEV_SLOT_SIO5,
.fns = { .fns = {
/* UART0 shares an interrupt line with TSN0, so must use
a PIRQ */
FIXED_INT_ANY_PIRQ(PCH_DEVFN_UART0, PCI_INT_A), FIXED_INT_ANY_PIRQ(PCH_DEVFN_UART0, PCI_INT_A),
/* UART1 shares an interrupt line with TSN1, so must use
a PIRQ */
FIXED_INT_ANY_PIRQ(PCH_DEVFN_UART1, PCI_INT_B), FIXED_INT_ANY_PIRQ(PCH_DEVFN_UART1, PCI_INT_B),
ANY_PIRQ(PCH_DEVFN_GSPI0), DIRECT_IRQ(PCH_DEVFN_GSPI0),
ANY_PIRQ(PCH_DEVFN_GSPI1), DIRECT_IRQ(PCH_DEVFN_GSPI1),
}, },
}, },
{ {
@ -170,6 +214,7 @@ static const struct slot_irq_constraints irq_constraints[] = {
ANY_PIRQ(PCH_DEVFN_HDA), ANY_PIRQ(PCH_DEVFN_HDA),
ANY_PIRQ(PCH_DEVFN_SMBUS), ANY_PIRQ(PCH_DEVFN_SMBUS),
ANY_PIRQ(PCH_DEVFN_GBE), ANY_PIRQ(PCH_DEVFN_GBE),
/* INTERRUPT_PIN is RO/0x01 */
FIXED_INT_ANY_PIRQ(PCH_DEVFN_TRACEHUB, PCI_INT_A), FIXED_INT_ANY_PIRQ(PCH_DEVFN_TRACEHUB, PCI_INT_A),
}, },
}, },