|
Mian Yousaf Kaukab |
1e5d17 |
From: Russell King <rmk+kernel@armlinux.org.uk>
|
|
Mian Yousaf Kaukab |
1e5d17 |
Date: Thu, 25 Mar 2021 10:26:21 +0000
|
|
Mian Yousaf Kaukab |
1e5d17 |
Subject: ARM: footbridge: fix PCI interrupt mapping
|
|
Mian Yousaf Kaukab |
1e5d17 |
|
|
Mian Yousaf Kaukab |
1e5d17 |
Git-commit: 30e3b4f256b4e366a61658c294f6a21b8626dda7
|
|
Mian Yousaf Kaukab |
1e5d17 |
Patch-mainline: v5.12-rc8
|
|
Mian Yousaf Kaukab |
1e5d17 |
References: git-fixes
|
|
Mian Yousaf Kaukab |
1e5d17 |
|
|
Mian Yousaf Kaukab |
1e5d17 |
Since commit 30fdfb929e82 ("PCI: Add a call to pci_assign_irq() in
|
|
Mian Yousaf Kaukab |
1e5d17 |
pci_device_probe()"), the PCI code will call the IRQ mapping function
|
|
Mian Yousaf Kaukab |
1e5d17 |
whenever a PCI driver is probed. If these are marked as __init, this
|
|
Mian Yousaf Kaukab |
1e5d17 |
causes an oops if a PCI driver is loaded or bound after the kernel has
|
|
Mian Yousaf Kaukab |
1e5d17 |
initialised.
|
|
Mian Yousaf Kaukab |
1e5d17 |
|
|
Mian Yousaf Kaukab |
1e5d17 |
Fixes: 30fdfb929e82 ("PCI: Add a call to pci_assign_irq() in pci_device_probe()")
|
|
Mian Yousaf Kaukab |
1e5d17 |
Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
|
|
Mian Yousaf Kaukab |
1e5d17 |
Signed-off-by: Mian Yousaf Kaukab <ykaukab@suse.de>
|
|
Mian Yousaf Kaukab |
1e5d17 |
---
|
|
Mian Yousaf Kaukab |
1e5d17 |
arch/arm/mach-footbridge/cats-pci.c | 4 ++--
|
|
Mian Yousaf Kaukab |
1e5d17 |
arch/arm/mach-footbridge/ebsa285-pci.c | 4 ++--
|
|
Mian Yousaf Kaukab |
1e5d17 |
arch/arm/mach-footbridge/netwinder-pci.c | 2 +-
|
|
Mian Yousaf Kaukab |
1e5d17 |
arch/arm/mach-footbridge/personal-pci.c | 5 ++---
|
|
Mian Yousaf Kaukab |
1e5d17 |
4 files changed, 7 insertions(+), 8 deletions(-)
|
|
Mian Yousaf Kaukab |
1e5d17 |
|
|
Mian Yousaf Kaukab |
1e5d17 |
diff --git a/arch/arm/mach-footbridge/cats-pci.c b/arch/arm/mach-footbridge/cats-pci.c
|
|
Mian Yousaf Kaukab |
1e5d17 |
index 0b2fd7e2e9b4..90b1e9be430e 100644
|
|
Mian Yousaf Kaukab |
1e5d17 |
--- a/arch/arm/mach-footbridge/cats-pci.c
|
|
Mian Yousaf Kaukab |
1e5d17 |
+++ b/arch/arm/mach-footbridge/cats-pci.c
|
|
Mian Yousaf Kaukab |
1e5d17 |
@@ -15,14 +15,14 @@
|
|
Mian Yousaf Kaukab |
1e5d17 |
#include <asm/mach-types.h>
|
|
Mian Yousaf Kaukab |
1e5d17 |
|
|
Mian Yousaf Kaukab |
1e5d17 |
/* cats host-specific stuff */
|
|
Mian Yousaf Kaukab |
1e5d17 |
-static int irqmap_cats[] __initdata = { IRQ_PCI, IRQ_IN0, IRQ_IN1, IRQ_IN3 };
|
|
Mian Yousaf Kaukab |
1e5d17 |
+static int irqmap_cats[] = { IRQ_PCI, IRQ_IN0, IRQ_IN1, IRQ_IN3 };
|
|
Mian Yousaf Kaukab |
1e5d17 |
|
|
Mian Yousaf Kaukab |
1e5d17 |
static u8 cats_no_swizzle(struct pci_dev *dev, u8 *pin)
|
|
Mian Yousaf Kaukab |
1e5d17 |
{
|
|
Mian Yousaf Kaukab |
1e5d17 |
return 0;
|
|
Mian Yousaf Kaukab |
1e5d17 |
}
|
|
Mian Yousaf Kaukab |
1e5d17 |
|
|
Mian Yousaf Kaukab |
1e5d17 |
-static int __init cats_map_irq(const struct pci_dev *dev, u8 slot, u8 pin)
|
|
Mian Yousaf Kaukab |
1e5d17 |
+static int cats_map_irq(const struct pci_dev *dev, u8 slot, u8 pin)
|
|
Mian Yousaf Kaukab |
1e5d17 |
{
|
|
Mian Yousaf Kaukab |
1e5d17 |
if (dev->irq >= 255)
|
|
Mian Yousaf Kaukab |
1e5d17 |
return -1; /* not a valid interrupt. */
|
|
Mian Yousaf Kaukab |
1e5d17 |
diff --git a/arch/arm/mach-footbridge/ebsa285-pci.c b/arch/arm/mach-footbridge/ebsa285-pci.c
|
|
Mian Yousaf Kaukab |
1e5d17 |
index 6f28aaa9ca79..c3f280d08fa7 100644
|
|
Mian Yousaf Kaukab |
1e5d17 |
--- a/arch/arm/mach-footbridge/ebsa285-pci.c
|
|
Mian Yousaf Kaukab |
1e5d17 |
+++ b/arch/arm/mach-footbridge/ebsa285-pci.c
|
|
Mian Yousaf Kaukab |
1e5d17 |
@@ -14,9 +14,9 @@
|
|
Mian Yousaf Kaukab |
1e5d17 |
#include <asm/mach/pci.h>
|
|
Mian Yousaf Kaukab |
1e5d17 |
#include <asm/mach-types.h>
|
|
Mian Yousaf Kaukab |
1e5d17 |
|
|
Mian Yousaf Kaukab |
1e5d17 |
-static int irqmap_ebsa285[] __initdata = { IRQ_IN3, IRQ_IN1, IRQ_IN0, IRQ_PCI };
|
|
Mian Yousaf Kaukab |
1e5d17 |
+static int irqmap_ebsa285[] = { IRQ_IN3, IRQ_IN1, IRQ_IN0, IRQ_PCI };
|
|
Mian Yousaf Kaukab |
1e5d17 |
|
|
Mian Yousaf Kaukab |
1e5d17 |
-static int __init ebsa285_map_irq(const struct pci_dev *dev, u8 slot, u8 pin)
|
|
Mian Yousaf Kaukab |
1e5d17 |
+static int ebsa285_map_irq(const struct pci_dev *dev, u8 slot, u8 pin)
|
|
Mian Yousaf Kaukab |
1e5d17 |
{
|
|
Mian Yousaf Kaukab |
1e5d17 |
if (dev->vendor == PCI_VENDOR_ID_CONTAQ &&
|
|
Mian Yousaf Kaukab |
1e5d17 |
dev->device == PCI_DEVICE_ID_CONTAQ_82C693)
|
|
Mian Yousaf Kaukab |
1e5d17 |
diff --git a/arch/arm/mach-footbridge/netwinder-pci.c b/arch/arm/mach-footbridge/netwinder-pci.c
|
|
Mian Yousaf Kaukab |
1e5d17 |
index 9473aa0305e5..e8304392074b 100644
|
|
Mian Yousaf Kaukab |
1e5d17 |
--- a/arch/arm/mach-footbridge/netwinder-pci.c
|
|
Mian Yousaf Kaukab |
1e5d17 |
+++ b/arch/arm/mach-footbridge/netwinder-pci.c
|
|
Mian Yousaf Kaukab |
1e5d17 |
@@ -18,7 +18,7 @@
|
|
Mian Yousaf Kaukab |
1e5d17 |
* We now use the slot ID instead of the device identifiers to select
|
|
Mian Yousaf Kaukab |
1e5d17 |
* which interrupt is routed where.
|
|
Mian Yousaf Kaukab |
1e5d17 |
*/
|
|
Mian Yousaf Kaukab |
1e5d17 |
-static int __init netwinder_map_irq(const struct pci_dev *dev, u8 slot, u8 pin)
|
|
Mian Yousaf Kaukab |
1e5d17 |
+static int netwinder_map_irq(const struct pci_dev *dev, u8 slot, u8 pin)
|
|
Mian Yousaf Kaukab |
1e5d17 |
{
|
|
Mian Yousaf Kaukab |
1e5d17 |
switch (slot) {
|
|
Mian Yousaf Kaukab |
1e5d17 |
case 0: /* host bridge */
|
|
Mian Yousaf Kaukab |
1e5d17 |
diff --git a/arch/arm/mach-footbridge/personal-pci.c b/arch/arm/mach-footbridge/personal-pci.c
|
|
Mian Yousaf Kaukab |
1e5d17 |
index 4391e433a4b2..9d19aa98a663 100644
|
|
Mian Yousaf Kaukab |
1e5d17 |
--- a/arch/arm/mach-footbridge/personal-pci.c
|
|
Mian Yousaf Kaukab |
1e5d17 |
+++ b/arch/arm/mach-footbridge/personal-pci.c
|
|
Mian Yousaf Kaukab |
1e5d17 |
@@ -14,13 +14,12 @@
|
|
Mian Yousaf Kaukab |
1e5d17 |
#include <asm/mach/pci.h>
|
|
Mian Yousaf Kaukab |
1e5d17 |
#include <asm/mach-types.h>
|
|
Mian Yousaf Kaukab |
1e5d17 |
|
|
Mian Yousaf Kaukab |
1e5d17 |
-static int irqmap_personal_server[] __initdata = {
|
|
Mian Yousaf Kaukab |
1e5d17 |
+static int irqmap_personal_server[] = {
|
|
Mian Yousaf Kaukab |
1e5d17 |
IRQ_IN0, IRQ_IN1, IRQ_IN2, IRQ_IN3, 0, 0, 0,
|
|
Mian Yousaf Kaukab |
1e5d17 |
IRQ_DOORBELLHOST, IRQ_DMA1, IRQ_DMA2, IRQ_PCI
|
|
Mian Yousaf Kaukab |
1e5d17 |
};
|
|
Mian Yousaf Kaukab |
1e5d17 |
|
|
Mian Yousaf Kaukab |
1e5d17 |
-static int __init personal_server_map_irq(const struct pci_dev *dev, u8 slot,
|
|
Mian Yousaf Kaukab |
1e5d17 |
- u8 pin)
|
|
Mian Yousaf Kaukab |
1e5d17 |
+static int personal_server_map_irq(const struct pci_dev *dev, u8 slot, u8 pin)
|
|
Mian Yousaf Kaukab |
1e5d17 |
{
|
|
Mian Yousaf Kaukab |
1e5d17 |
unsigned char line;
|
|
Mian Yousaf Kaukab |
1e5d17 |
|
|
Mian Yousaf Kaukab |
1e5d17 |
--
|
|
Mian Yousaf Kaukab |
1e5d17 |
2.26.2
|
|
Mian Yousaf Kaukab |
1e5d17 |
|