summaryrefslogtreecommitdiff
path: root/resources/coreboot
diff options
context:
space:
mode:
authorLeah Rowe <vimuser@noreply.codeberg.org>2023-06-13 01:09:03 +0000
committerLeah Rowe <vimuser@noreply.codeberg.org>2023-06-13 01:09:03 +0000
commit81bf2293df0fc93513a5c6ae9f4028a877c6f7d9 (patch)
tree3179335d3f08d0ea4579d1c6731b1de7fd2b2f0c /resources/coreboot
parentd617135d38243d82c6eacf856450318522123c0d (diff)
parent4ecd289fa19560a0b12715705222b893312a81a3 (diff)
Merge pull request 'resources/coreboot/default/patches: Add patch for E6400 SD card' (#78) from nic3-14159/lbmk:e6400-sd-card-patch into master
Reviewed-on: https://codeberg.org/libreboot/lbmk/pulls/78
Diffstat (limited to 'resources/coreboot')
-rw-r--r--resources/coreboot/default/patches/0023-mb-dell-e6400-acpi-Route-Ricoh-R5C847-PCI-IRQ-lines-.patch125
1 files changed, 125 insertions, 0 deletions
diff --git a/resources/coreboot/default/patches/0023-mb-dell-e6400-acpi-Route-Ricoh-R5C847-PCI-IRQ-lines-.patch b/resources/coreboot/default/patches/0023-mb-dell-e6400-acpi-Route-Ricoh-R5C847-PCI-IRQ-lines-.patch
new file mode 100644
index 00000000..483ea0e1
--- /dev/null
+++ b/resources/coreboot/default/patches/0023-mb-dell-e6400-acpi-Route-Ricoh-R5C847-PCI-IRQ-lines-.patch
@@ -0,0 +1,125 @@
+From 5c1455495e8d2030473d8194fcf2e1d1111696b7 Mon Sep 17 00:00:00 2001
+From: Nicholas Chin <nic.c3.14@gmail.com>
+Date: Tue, 23 May 2023 20:59:56 -0600
+Subject: [PATCH] mb/dell/e6400/acpi: Route Ricoh R5C847 PCI IRQ lines as DBC
+
+Based on the schematic and vendor ASL code, PCI interrupt lines ABC of
+the Ricoh R5C847 PC Card/Media Card/FireWire controller are routed DBC.
+From lspci and the schematic this chip is PCI device 1. The original
+config copied from the T400 was routed ABCD->BCDA, causing Linux to
+issue an "irq 18: nobody cared" message when inserting an SD card.
+This is fixed by this patch and the SD card now works properly.
+
+Change-Id: Iede1de72d5369f1aebbac170792733739add3431
+Signed-off-by: Nicholas Chin <nic.c3.14@gmail.com>
+Reviewed-on: https://review.coreboot.org/c/coreboot/+/75411
+Reviewed-by: Paul Menzel <paulepanter@mailbox.org>
+Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
+Reviewed-by: Angel Pons <th3fanbus@gmail.com>
+---
+ .../dell/e6400/acpi/ich9_pci_irqs.asl | 85 ++-----------------
+ 1 file changed, 8 insertions(+), 77 deletions(-)
+
+diff --git a/src/mainboard/dell/e6400/acpi/ich9_pci_irqs.asl b/src/mainboard/dell/e6400/acpi/ich9_pci_irqs.asl
+index 21066fbf3b..9a4cdfb75b 100644
+--- a/src/mainboard/dell/e6400/acpi/ich9_pci_irqs.asl
++++ b/src/mainboard/dell/e6400/acpi/ich9_pci_irqs.asl
+@@ -4,87 +4,18 @@
+ * 0:1e.0 PCI bridge of the ICH9
+ */
+
+-/* TODO: which slots are actually relevant? */
+ If (PICM) {
+ Return (Package() {
+- // PCI Slot 1 routes ABCD
+- Package() { 0x0000ffff, 0, 0, 16},
+- Package() { 0x0000ffff, 1, 0, 17},
+- Package() { 0x0000ffff, 2, 0, 18},
+- Package() { 0x0000ffff, 3, 0, 19},
+-
+- // PCI Slot 2 routes BCDA
+- Package() { 0x0001ffff, 0, 0, 17},
+- Package() { 0x0001ffff, 1, 0, 18},
+- Package() { 0x0001ffff, 2, 0, 19},
+- Package() { 0x0001ffff, 3, 0, 16},
+-
+- // PCI Slot 3 routes CDAB
+- Package() { 0x0002ffff, 0, 0, 18},
+- Package() { 0x0002ffff, 1, 0, 19},
+- Package() { 0x0002ffff, 2, 0, 16},
+- Package() { 0x0002ffff, 3, 0, 17},
+-
+- // PCI Slot 4 routes ABCD
+- Package() { 0x0003ffff, 0, 0, 16},
+- Package() { 0x0003ffff, 1, 0, 17},
+- Package() { 0x0003ffff, 2, 0, 18},
+- Package() { 0x0003ffff, 3, 0, 19},
+-
+- // PCI Slot 5 routes ABCD
+- Package() { 0x0004ffff, 0, 0, 16},
+- Package() { 0x0004ffff, 1, 0, 17},
+- Package() { 0x0004ffff, 2, 0, 18},
+- Package() { 0x0004ffff, 3, 0, 19},
+-
+- // PCI Slot 6 routes BCDA
+- Package() { 0x0005ffff, 0, 0, 17},
+- Package() { 0x0005ffff, 1, 0, 18},
+- Package() { 0x0005ffff, 2, 0, 19},
+- Package() { 0x0005ffff, 3, 0, 16},
+-
+- // FIXME: what's this supposed to mean? (adopted from ich7)
+- //Package() { 0x0008ffff, 0, 0, 20},
++ // PCI Device 1, Ricoh R5C847 routes DBC
++ Package() { 0x0001ffff, 0, 0, 19},
++ Package() { 0x0001ffff, 1, 0, 17},
++ Package() { 0x0001ffff, 2, 0, 18},
+ })
+ } Else {
+ Return (Package() {
+- // PCI Slot 1 routes ABCD
+- Package() { 0x0000ffff, 0, \_SB.PCI0.LPCB.LNKA, 0},
+- Package() { 0x0000ffff, 1, \_SB.PCI0.LPCB.LNKB, 0},
+- Package() { 0x0000ffff, 2, \_SB.PCI0.LPCB.LNKC, 0},
+- Package() { 0x0000ffff, 3, \_SB.PCI0.LPCB.LNKD, 0},
+-
+- // PCI Slot 2 routes BCDA
+- Package() { 0x0001ffff, 0, \_SB.PCI0.LPCB.LNKB, 0},
+- Package() { 0x0001ffff, 1, \_SB.PCI0.LPCB.LNKC, 0},
+- Package() { 0x0001ffff, 2, \_SB.PCI0.LPCB.LNKD, 0},
+- Package() { 0x0001ffff, 3, \_SB.PCI0.LPCB.LNKA, 0},
+-
+- // PCI Slot 3 routes CDAB
+- Package() { 0x0002ffff, 0, \_SB.PCI0.LPCB.LNKC, 0},
+- Package() { 0x0002ffff, 1, \_SB.PCI0.LPCB.LNKD, 0},
+- Package() { 0x0002ffff, 2, \_SB.PCI0.LPCB.LNKA, 0},
+- Package() { 0x0002ffff, 3, \_SB.PCI0.LPCB.LNKB, 0},
+-
+- // PCI Slot 4 routes ABCD
+- Package() { 0x0003ffff, 0, \_SB.PCI0.LPCB.LNKA, 0},
+- Package() { 0x0003ffff, 1, \_SB.PCI0.LPCB.LNKB, 0},
+- Package() { 0x0003ffff, 2, \_SB.PCI0.LPCB.LNKC, 0},
+- Package() { 0x0003ffff, 3, \_SB.PCI0.LPCB.LNKD, 0},
+-
+- // PCI Slot 5 routes ABCD
+- Package() { 0x0004ffff, 0, \_SB.PCI0.LPCB.LNKA, 0},
+- Package() { 0x0004ffff, 1, \_SB.PCI0.LPCB.LNKB, 0},
+- Package() { 0x0004ffff, 2, \_SB.PCI0.LPCB.LNKC, 0},
+- Package() { 0x0004ffff, 3, \_SB.PCI0.LPCB.LNKD, 0},
+-
+- // PCI Slot 6 routes BCDA
+- Package() { 0x0005ffff, 0, \_SB.PCI0.LPCB.LNKB, 0},
+- Package() { 0x0005ffff, 1, \_SB.PCI0.LPCB.LNKC, 0},
+- Package() { 0x0005ffff, 2, \_SB.PCI0.LPCB.LNKD, 0},
+- Package() { 0x0005ffff, 3, \_SB.PCI0.LPCB.LNKA, 0},
+-
+- // FIXME
+- // Package() { 0x0008ffff, 0, \_SB.PCI0.LPCB.LNKE, 0},
++ // PCI Device 1, Ricoh R5C847 routes DBC
++ Package() { 0x0001ffff, 0, \_SB.PCI0.LPCB.LNKD, 0},
++ Package() { 0x0001ffff, 1, \_SB.PCI0.LPCB.LNKB, 0},
++ Package() { 0x0001ffff, 2, \_SB.PCI0.LPCB.LNKC, 0},
+ })
+ }
+--
+2.41.0
+