diff options
| author | Felix Held <felix-coreboot@felixheld.de> | 2023-02-27 21:02:48 +0100 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2023-02-28 20:25:46 +0000 |
| commit | c0ae0ba4cccb455da07ea7057881c294b5d1fc72 (patch) | |
| tree | 6b1d64ef3a76fb4ad4eb8eafe48754eb2da0fde3 | |
| parent | 3c74a5107de47aa4d4307336793b7210330d8639 (diff) | |
soc/amd/picasso/acpi: use ACPI_SCI_IRQ definition
Since there's a define for the ACPI_SCI_IRQ 9, use the define instead of
a magic number in the code.
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I23c8f62929f3f66192698e10826d10329ef3d8cc
Reviewed-on: https://review.coreboot.org/c/coreboot/+/73319
Reviewed-by: Matt DeVillier <matt.devillier@amd.corp-partner.google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
| -rw-r--r-- | src/soc/amd/picasso/acpi.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/soc/amd/picasso/acpi.c b/src/soc/amd/picasso/acpi.c index 78f7c2f8bf..1d5a275d42 100644 --- a/src/soc/amd/picasso/acpi.c +++ b/src/soc/amd/picasso/acpi.c @@ -44,7 +44,7 @@ unsigned long acpi_fill_madt(unsigned long current) MP_IRQ_TRIGGER_DEFAULT | MP_IRQ_POLARITY_DEFAULT); /* SCI IRQ type override */ current += acpi_create_madt_irqoverride((acpi_madt_irqoverride_t *)current, - MP_BUS_ISA, 9, 9, + MP_BUS_ISA, ACPI_SCI_IRQ, ACPI_SCI_IRQ, MP_IRQ_TRIGGER_LEVEL | MP_IRQ_POLARITY_LOW); current = acpi_fill_madt_irqoverride(current); @@ -62,7 +62,7 @@ void acpi_fill_fadt(acpi_fadt_t *fadt) printk(BIOS_DEBUG, "pm_base: 0x%04x\n", ACPI_IO_BASE); - fadt->sci_int = 9; /* IRQ 09 - ACPI SCI */ + fadt->sci_int = ACPI_SCI_IRQ; if (permanent_smi_handler()) { fadt->smi_cmd = APM_CNT; |