Skip to content

L
linux
Commit e2577d22 authored by Linus Torvalds's avatar Linus Torvalds
Browse files
Options

Merge tag 'pci-v4.14-fixes-2' of git://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci 

Pull PCI fixes from Bjorn Helgaas:

 - fix endpoint "end of test" interrupt issue (introduced in v4.14-rc1)
   (John Keeping)

 - fix MIPS use-after-free map_irq() issue (introduced in v4.14-rc1)
   (Lorenzo Pieralisi)

* tag 'pci-v4.14-fixes-2' of git://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci:
  PCI: endpoint: Use correct "end of test" interrupt
  MIPS: PCI: Move map_irq() hooks out of initdata
parents 9ae0b37e 749aaf33
Show whitespace changes
Inline Side-by-side
Showing with 44 additions and 43 deletions
arch/mips/ath79/pci.c
View file @ e2577d22
...@@ -22,10 +22,10 @@ ...@@ -22,10 +22,10 @@
#include "pci.h" #include "pci.h"
static int (*ath79_pci_plat_dev_init)(struct pci_dev *dev); static int (*ath79_pci_plat_dev_init)(struct pci_dev *dev);
static const struct ath79_pci_irq *ath79_pci_irq_map __initdata; static const struct ath79_pci_irq *ath79_pci_irq_map;
static unsigned ath79_pci_nr_irqs __initdata; static unsigned ath79_pci_nr_irqs;
static const struct ath79_pci_irq ar71xx_pci_irq_map[] __initconst = { static const struct ath79_pci_irq ar71xx_pci_irq_map[] = {
{ {
.slot = 17, .slot = 17,
.pin = 1, .pin = 1,
...@@ -41,7 +41,7 @@ static const struct ath79_pci_irq ar71xx_pci_irq_map[] __initconst = { ...@@ -41,7 +41,7 @@ static const struct ath79_pci_irq ar71xx_pci_irq_map[] __initconst = {
} }
}; };
static const struct ath79_pci_irq ar724x_pci_irq_map[] __initconst = { static const struct ath79_pci_irq ar724x_pci_irq_map[] = {
{ {
.slot = 0, .slot = 0,
.pin = 1, .pin = 1,
...@@ -49,7 +49,7 @@ static const struct ath79_pci_irq ar724x_pci_irq_map[] __initconst = { ...@@ -49,7 +49,7 @@ static const struct ath79_pci_irq ar724x_pci_irq_map[] __initconst = {
} }
}; };
static const struct ath79_pci_irq qca955x_pci_irq_map[] __initconst = { static const struct ath79_pci_irq qca955x_pci_irq_map[] = {
{ {
.bus = 0, .bus = 0,
.slot = 0, .slot = 0,
...@@ -64,7 +64,7 @@ static const struct ath79_pci_irq qca955x_pci_irq_map[] __initconst = { ...@@ -64,7 +64,7 @@ static const struct ath79_pci_irq qca955x_pci_irq_map[] __initconst = {
}, },
}; };
int __init pcibios_map_irq(const struct pci_dev *dev, uint8_t slot, uint8_t pin) int pcibios_map_irq(const struct pci_dev *dev, uint8_t slot, uint8_t pin)
{ {
int irq = -1; int irq = -1;
int i; int i;
......
arch/mips/pci/fixup-capcella.c
View file @ e2577d22
...@@ -32,7 +32,7 @@ ...@@ -32,7 +32,7 @@
#define INTC PC104PLUS_INTC_IRQ #define INTC PC104PLUS_INTC_IRQ
#define INTD PC104PLUS_INTD_IRQ #define INTD PC104PLUS_INTD_IRQ
static char irq_tab_capcella[][5] __initdata = { static char irq_tab_capcella[][5] = {
[11] = { -1, INT1, INT1, INT1, INT1 }, [11] = { -1, INT1, INT1, INT1, INT1 },
[12] = { -1, INT2, INT2, INT2, INT2 }, [12] = { -1, INT2, INT2, INT2, INT2 },
[14] = { -1, INTA, INTB, INTC, INTD } [14] = { -1, INTA, INTB, INTC, INTD }
......
arch/mips/pci/fixup-cobalt.c
View file @ e2577d22
...@@ -147,7 +147,7 @@ static void qube_raq_via_board_id_fixup(struct pci_dev *dev) ...@@ -147,7 +147,7 @@ static void qube_raq_via_board_id_fixup(struct pci_dev *dev)
DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_VIA, PCI_DEVICE_ID_VIA_82C586_0, DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_VIA, PCI_DEVICE_ID_VIA_82C586_0,
qube_raq_via_board_id_fixup); qube_raq_via_board_id_fixup);
static char irq_tab_qube1[] __initdata = { static char irq_tab_qube1[] = {
[COBALT_PCICONF_CPU] = 0, [COBALT_PCICONF_CPU] = 0,
[COBALT_PCICONF_ETH0] = QUBE1_ETH0_IRQ, [COBALT_PCICONF_ETH0] = QUBE1_ETH0_IRQ,
[COBALT_PCICONF_RAQSCSI] = SCSI_IRQ, [COBALT_PCICONF_RAQSCSI] = SCSI_IRQ,
...@@ -156,7 +156,7 @@ static char irq_tab_qube1[] __initdata = { ...@@ -156,7 +156,7 @@ static char irq_tab_qube1[] __initdata = {
[COBALT_PCICONF_ETH1] = 0 [COBALT_PCICONF_ETH1] = 0
}; };
static char irq_tab_cobalt[] __initdata = { static char irq_tab_cobalt[] = {
[COBALT_PCICONF_CPU] = 0, [COBALT_PCICONF_CPU] = 0,
[COBALT_PCICONF_ETH0] = ETH0_IRQ, [COBALT_PCICONF_ETH0] = ETH0_IRQ,
[COBALT_PCICONF_RAQSCSI] = SCSI_IRQ, [COBALT_PCICONF_RAQSCSI] = SCSI_IRQ,
...@@ -165,7 +165,7 @@ static char irq_tab_cobalt[] __initdata = { ...@@ -165,7 +165,7 @@ static char irq_tab_cobalt[] __initdata = {
[COBALT_PCICONF_ETH1] = ETH1_IRQ [COBALT_PCICONF_ETH1] = ETH1_IRQ
}; };
static char irq_tab_raq2[] __initdata = { static char irq_tab_raq2[] = {
[COBALT_PCICONF_CPU] = 0, [COBALT_PCICONF_CPU] = 0,
[COBALT_PCICONF_ETH0] = ETH0_IRQ, [COBALT_PCICONF_ETH0] = ETH0_IRQ,
[COBALT_PCICONF_RAQSCSI] = RAQ2_SCSI_IRQ, [COBALT_PCICONF_RAQSCSI] = RAQ2_SCSI_IRQ,
......
arch/mips/pci/fixup-emma2rh.c
View file @ e2577d22
...@@ -43,7 +43,7 @@ ...@@ -43,7 +43,7 @@
*/ */
#define MAX_SLOT_NUM 10 #define MAX_SLOT_NUM 10
static unsigned char irq_map[][5] __initdata = { static unsigned char irq_map[][5] = {
[3] = {0, MARKEINS_PCI_IRQ_INTB, MARKEINS_PCI_IRQ_INTC, [3] = {0, MARKEINS_PCI_IRQ_INTB, MARKEINS_PCI_IRQ_INTC,
MARKEINS_PCI_IRQ_INTD, 0,}, MARKEINS_PCI_IRQ_INTD, 0,},
[4] = {0, MARKEINS_PCI_IRQ_INTA, 0, 0, 0,}, [4] = {0, MARKEINS_PCI_IRQ_INTA, 0, 0, 0,},
......
arch/mips/pci/fixup-ip32.c
View file @ e2577d22
...@@ -21,7 +21,7 @@ ...@@ -21,7 +21,7 @@
#define INTB MACEPCI_SHARED0_IRQ #define INTB MACEPCI_SHARED0_IRQ
#define INTC MACEPCI_SHARED1_IRQ #define INTC MACEPCI_SHARED1_IRQ
#define INTD MACEPCI_SHARED2_IRQ #define INTD MACEPCI_SHARED2_IRQ
static char irq_tab_mace[][5] __initdata = { static char irq_tab_mace[][5] = {
/* Dummy INT#A INT#B INT#C INT#D */ /* Dummy INT#A INT#B INT#C INT#D */
{0, 0, 0, 0, 0}, /* This is placeholder row - never used */ {0, 0, 0, 0, 0}, /* This is placeholder row - never used */
{0, SCSI0, SCSI0, SCSI0, SCSI0}, {0, SCSI0, SCSI0, SCSI0, SCSI0},
......
arch/mips/pci/fixup-jmr3927.c
View file @ e2577d22
...@@ -31,7 +31,7 @@ ...@@ -31,7 +31,7 @@
#include <asm/txx9/pci.h> #include <asm/txx9/pci.h>
#include <asm/txx9/jmr3927.h> #include <asm/txx9/jmr3927.h>
int __init jmr3927_pci_map_irq(const struct pci_dev *dev, u8 slot, u8 pin) int jmr3927_pci_map_irq(const struct pci_dev *dev, u8 slot, u8 pin)
{ {
unsigned char irq = pin; unsigned char irq = pin;
......
arch/mips/pci/fixup-lemote2f.c
View file @ e2577d22
...@@ -30,7 +30,7 @@ ...@@ -30,7 +30,7 @@
#define PCID 7 #define PCID 7
/* all the pci device has the PCIA pin, check the datasheet. */ /* all the pci device has the PCIA pin, check the datasheet. */
static char irq_tab[][5] __initdata = { static char irq_tab[][5] = {
/* INTA INTB INTC INTD */ /* INTA INTB INTC INTD */
{0, 0, 0, 0, 0}, /* 11: Unused */ {0, 0, 0, 0, 0}, /* 11: Unused */
{0, 0, 0, 0, 0}, /* 12: Unused */ {0, 0, 0, 0, 0}, /* 12: Unused */
......
arch/mips/pci/fixup-malta.c
View file @ e2577d22
...@@ -12,7 +12,7 @@ ...@@ -12,7 +12,7 @@
static char pci_irq[5] = { static char pci_irq[5] = {
}; };
static char irq_tab[][5] __initdata = { static char irq_tab[][5] = {
/* INTA INTB INTC INTD */ /* INTA INTB INTC INTD */
{0, 0, 0, 0, 0 }, /* 0: GT64120 PCI bridge */ {0, 0, 0, 0, 0 }, /* 0: GT64120 PCI bridge */
{0, 0, 0, 0, 0 }, /* 1: Unused */ {0, 0, 0, 0, 0 }, /* 1: Unused */
......
arch/mips/pci/fixup-mpc30x.c
View file @ e2577d22
...@@ -22,13 +22,13 @@ ...@@ -22,13 +22,13 @@
#include <asm/vr41xx/mpc30x.h> #include <asm/vr41xx/mpc30x.h>
static const int internal_func_irqs[] __initconst = { static const int internal_func_irqs[] = {
VRC4173_CASCADE_IRQ, VRC4173_CASCADE_IRQ,
VRC4173_AC97_IRQ, VRC4173_AC97_IRQ,
VRC4173_USB_IRQ, VRC4173_USB_IRQ,
}; };
static const int irq_tab_mpc30x[] __initconst = { static const int irq_tab_mpc30x[] = {
[12] = VRC4173_PCMCIA1_IRQ, [12] = VRC4173_PCMCIA1_IRQ,
[13] = VRC4173_PCMCIA2_IRQ, [13] = VRC4173_PCMCIA2_IRQ,
[29] = MQ200_IRQ, [29] = MQ200_IRQ,
......
arch/mips/pci/fixup-pmcmsp.c
View file @ e2577d22
...@@ -47,7 +47,7 @@ ...@@ -47,7 +47,7 @@
#if defined(CONFIG_PMC_MSP7120_GW) #if defined(CONFIG_PMC_MSP7120_GW)
/* Garibaldi Board IRQ wiring to PCI slots */ /* Garibaldi Board IRQ wiring to PCI slots */
static char irq_tab[][5] __initdata = { static char irq_tab[][5] = {
/* INTA INTB INTC INTD */ /* INTA INTB INTC INTD */
{0, 0, 0, 0, 0 }, /* (AD[0]): Unused */ {0, 0, 0, 0, 0 }, /* (AD[0]): Unused */
{0, 0, 0, 0, 0 }, /* (AD[1]): Unused */ {0, 0, 0, 0, 0 }, /* (AD[1]): Unused */
...@@ -86,7 +86,7 @@ static char irq_tab[][5] __initdata = { ...@@ -86,7 +86,7 @@ static char irq_tab[][5] __initdata = {
#elif defined(CONFIG_PMC_MSP7120_EVAL) #elif defined(CONFIG_PMC_MSP7120_EVAL)
/* MSP7120 Eval Board IRQ wiring to PCI slots */ /* MSP7120 Eval Board IRQ wiring to PCI slots */
static char irq_tab[][5] __initdata = { static char irq_tab[][5] = {
/* INTA INTB INTC INTD */ /* INTA INTB INTC INTD */
{0, 0, 0, 0, 0 }, /* (AD[0]): Unused */ {0, 0, 0, 0, 0 }, /* (AD[0]): Unused */
{0, 0, 0, 0, 0 }, /* (AD[1]): Unused */ {0, 0, 0, 0, 0 }, /* (AD[1]): Unused */
...@@ -125,7 +125,7 @@ static char irq_tab[][5] __initdata = { ...@@ -125,7 +125,7 @@ static char irq_tab[][5] __initdata = {
#else #else
/* Unknown board -- don't assign any IRQs */ /* Unknown board -- don't assign any IRQs */
static char irq_tab[][5] __initdata = { static char irq_tab[][5] = {
/* INTA INTB INTC INTD */ /* INTA INTB INTC INTD */
{0, 0, 0, 0, 0 }, /* (AD[0]): Unused */ {0, 0, 0, 0, 0 }, /* (AD[0]): Unused */
{0, 0, 0, 0, 0 }, /* (AD[1]): Unused */ {0, 0, 0, 0, 0 }, /* (AD[1]): Unused */
......
arch/mips/pci/fixup-rbtx4927.c
View file @ e2577d22
...@@ -36,7 +36,7 @@ ...@@ -36,7 +36,7 @@
#include <asm/txx9/pci.h> #include <asm/txx9/pci.h>
#include <asm/txx9/rbtx4927.h> #include <asm/txx9/rbtx4927.h>
int __init rbtx4927_pci_map_irq(const struct pci_dev *dev, u8 slot, u8 pin) int rbtx4927_pci_map_irq(const struct pci_dev *dev, u8 slot, u8 pin)
{ {
unsigned char irq = pin; unsigned char irq = pin;
......
arch/mips/pci/fixup-rbtx4938.c
View file @ e2577d22
...@@ -13,7 +13,7 @@ ...@@ -13,7 +13,7 @@
#include <asm/txx9/pci.h> #include <asm/txx9/pci.h>
#include <asm/txx9/rbtx4938.h> #include <asm/txx9/rbtx4938.h>
int __init rbtx4938_pci_map_irq(const struct pci_dev *dev, u8 slot, u8 pin) int rbtx4938_pci_map_irq(const struct pci_dev *dev, u8 slot, u8 pin)
{ {
int irq = tx4938_pcic1_map_irq(dev, slot); int irq = tx4938_pcic1_map_irq(dev, slot);
......
arch/mips/pci/fixup-sni.c
View file @ e2577d22
...@@ -40,7 +40,7 @@ ...@@ -40,7 +40,7 @@
* seem to be a documentation error. At least on my RM200C the Cirrus * seem to be a documentation error. At least on my RM200C the Cirrus
* Logic CL-GD5434 VGA is device 3. * Logic CL-GD5434 VGA is device 3.
*/ */
static char irq_tab_rm200[8][5] __initdata = { static char irq_tab_rm200[8][5] = {
/* INTA INTB INTC INTD */ /* INTA INTB INTC INTD */
{ 0, 0, 0, 0, 0 }, /* EISA bridge */ { 0, 0, 0, 0, 0 }, /* EISA bridge */
{ SCSI, SCSI, SCSI, SCSI, SCSI }, /* SCSI */ { SCSI, SCSI, SCSI, SCSI, SCSI }, /* SCSI */
...@@ -57,7 +57,7 @@ static char irq_tab_rm200[8][5] __initdata = { ...@@ -57,7 +57,7 @@ static char irq_tab_rm200[8][5] __initdata = {
* *
* The VGA card is optional for RM300 systems. * The VGA card is optional for RM300 systems.
*/ */
static char irq_tab_rm300d[8][5] __initdata = { static char irq_tab_rm300d[8][5] = {
/* INTA INTB INTC INTD */ /* INTA INTB INTC INTD */
{ 0, 0, 0, 0, 0 }, /* EISA bridge */ { 0, 0, 0, 0, 0 }, /* EISA bridge */
{ SCSI, SCSI, SCSI, SCSI, SCSI }, /* SCSI */ { SCSI, SCSI, SCSI, SCSI, SCSI }, /* SCSI */
...@@ -69,7 +69,7 @@ static char irq_tab_rm300d[8][5] __initdata = { ...@@ -69,7 +69,7 @@ static char irq_tab_rm300d[8][5] __initdata = {
{ 0, INTD, INTA, INTB, INTC }, /* Slot 4 */ { 0, INTD, INTA, INTB, INTC }, /* Slot 4 */
}; };
static char irq_tab_rm300e[5][5] __initdata = { static char irq_tab_rm300e[5][5] = {
/* INTA INTB INTC INTD */ /* INTA INTB INTC INTD */
{ 0, 0, 0, 0, 0 }, /* HOST bridge */ { 0, 0, 0, 0, 0 }, /* HOST bridge */
{ SCSI, SCSI, SCSI, SCSI, SCSI }, /* SCSI */ { SCSI, SCSI, SCSI, SCSI, SCSI }, /* SCSI */
...@@ -96,7 +96,7 @@ static char irq_tab_rm300e[5][5] __initdata = { ...@@ -96,7 +96,7 @@ static char irq_tab_rm300e[5][5] __initdata = {
#define INTC PCIT_IRQ_INTC #define INTC PCIT_IRQ_INTC
#define INTD PCIT_IRQ_INTD #define INTD PCIT_IRQ_INTD
static char irq_tab_pcit[13][5] __initdata = { static char irq_tab_pcit[13][5] = {
/* INTA INTB INTC INTD */ /* INTA INTB INTC INTD */
{ 0, 0, 0, 0, 0 }, /* HOST bridge */ { 0, 0, 0, 0, 0 }, /* HOST bridge */
{ SCSI0, SCSI0, SCSI0, SCSI0, SCSI0 }, /* SCSI */ { SCSI0, SCSI0, SCSI0, SCSI0, SCSI0 }, /* SCSI */
...@@ -113,7 +113,7 @@ static char irq_tab_pcit[13][5] __initdata = { ...@@ -113,7 +113,7 @@ static char irq_tab_pcit[13][5] __initdata = {
{ 0, INTA, INTB, INTC, INTD }, /* Slot 5 */ { 0, INTA, INTB, INTC, INTD }, /* Slot 5 */
}; };
static char irq_tab_pcit_cplus[13][5] __initdata = { static char irq_tab_pcit_cplus[13][5] = {
/* INTA INTB INTC INTD */ /* INTA INTB INTC INTD */
{ 0, 0, 0, 0, 0 }, /* HOST bridge */ { 0, 0, 0, 0, 0 }, /* HOST bridge */
{ 0, INTB, INTC, INTD, INTA }, /* PCI Slot 9 */ { 0, INTB, INTC, INTD, INTA }, /* PCI Slot 9 */
......
arch/mips/pci/pci-octeon.c
View file @ e2577d22
...@@ -59,8 +59,7 @@ union octeon_pci_address { ...@@ -59,8 +59,7 @@ union octeon_pci_address {
} s; } s;
}; };
int (*octeon_pcibios_map_irq)(const struct pci_dev *dev, int (*octeon_pcibios_map_irq)(const struct pci_dev *dev, u8 slot, u8 pin);
u8 slot, u8 pin);
enum octeon_dma_bar_type octeon_dma_bar_type = OCTEON_DMA_BAR_TYPE_INVALID; enum octeon_dma_bar_type octeon_dma_bar_type = OCTEON_DMA_BAR_TYPE_INVALID;
/** /**
......
arch/mips/pci/pci-tx4938.c
View file @ e2577d22
...@@ -112,7 +112,7 @@ int __init tx4938_pciclk66_setup(void) ...@@ -112,7 +112,7 @@ int __init tx4938_pciclk66_setup(void)
return pciclk; return pciclk;
} }
int __init tx4938_pcic1_map_irq(const struct pci_dev *dev, u8 slot) int tx4938_pcic1_map_irq(const struct pci_dev *dev, u8 slot)
{ {
if (get_tx4927_pcicptr(dev->bus->sysdata) == tx4938_pcic1ptr) { if (get_tx4927_pcicptr(dev->bus->sysdata) == tx4938_pcic1ptr) {
switch (slot) { switch (slot) {
......
arch/mips/pci/pci-tx4939.c
View file @ e2577d22
...@@ -48,7 +48,7 @@ void __init tx4939_report_pci1clk(void) ...@@ -48,7 +48,7 @@ void __init tx4939_report_pci1clk(void)
((pciclk + 50000) / 100000) % 10); ((pciclk + 50000) / 100000) % 10);
} }
int __init tx4939_pcic1_map_irq(const struct pci_dev *dev, u8 slot) int tx4939_pcic1_map_irq(const struct pci_dev *dev, u8 slot)
{ {
if (get_tx4927_pcicptr(dev->bus->sysdata) == tx4939_pcic1ptr) { if (get_tx4927_pcicptr(dev->bus->sysdata) == tx4939_pcic1ptr) {
switch (slot) { switch (slot) {
...@@ -68,7 +68,7 @@ int __init tx4939_pcic1_map_irq(const struct pci_dev *dev, u8 slot) ...@@ -68,7 +68,7 @@ int __init tx4939_pcic1_map_irq(const struct pci_dev *dev, u8 slot)
return -1; return -1;
} }
int __init tx4939_pci_map_irq(const struct pci_dev *dev, u8 slot, u8 pin) int tx4939_pci_map_irq(const struct pci_dev *dev, u8 slot, u8 pin)
{ {
int irq = tx4939_pcic1_map_irq(dev, slot); int irq = tx4939_pcic1_map_irq(dev, slot);
......
arch/mips/pci/pcie-octeon.c
View file @ e2577d22
...@@ -1464,8 +1464,7 @@ static int cvmx_pcie_rc_initialize(int pcie_port) ...@@ -1464,8 +1464,7 @@ static int cvmx_pcie_rc_initialize(int pcie_port)
* as it goes through each bridge. * as it goes through each bridge.
* Returns Interrupt number for the device * Returns Interrupt number for the device
*/ */
int __init octeon_pcie_pcibios_map_irq(const struct pci_dev *dev, int octeon_pcie_pcibios_map_irq(const struct pci_dev *dev, u8 slot, u8 pin)
u8 slot, u8 pin)
{ {
/* /*
* The EBH5600 board with the PCI to PCIe bridge mistakenly * The EBH5600 board with the PCI to PCIe bridge mistakenly
......
arch/mips/txx9/generic/pci.c
View file @ e2577d22
...@@ -386,9 +386,10 @@ int pcibios_plat_dev_init(struct pci_dev *dev) ...@@ -386,9 +386,10 @@ int pcibios_plat_dev_init(struct pci_dev *dev)
return 0; return 0;
} }
int __init pcibios_map_irq(const struct pci_dev *dev, u8 slot, u8 pin) static int (*txx9_pci_map_irq)(const struct pci_dev *dev, u8 slot, u8 pin);
int pcibios_map_irq(const struct pci_dev *dev, u8 slot, u8 pin)
{ {
return txx9_board_vec->pci_map_irq(dev, slot, pin); return txx9_pci_map_irq(dev, slot, pin);
} }
char * (*txx9_board_pcibios_setup)(char *str) __initdata; char * (*txx9_board_pcibios_setup)(char *str) __initdata;
...@@ -424,5 +425,8 @@ char *__init txx9_pcibios_setup(char *str) ...@@ -424,5 +425,8 @@ char *__init txx9_pcibios_setup(char *str)
txx9_pci_err_action = TXX9_PCI_ERR_IGNORE; txx9_pci_err_action = TXX9_PCI_ERR_IGNORE;
return NULL; return NULL;
} }
txx9_pci_map_irq = txx9_board_vec->pci_map_irq;
return str; return str;
} }
drivers/pci/endpoint/functions/pci-epf-test.c
View file @ e2577d22
...@@ -251,9 +251,8 @@ static int pci_epf_test_write(struct pci_epf_test *epf_test) ...@@ -251,9 +251,8 @@ static int pci_epf_test_write(struct pci_epf_test *epf_test)
return ret; return ret;
} }
static void pci_epf_test_raise_irq(struct pci_epf_test *epf_test) static void pci_epf_test_raise_irq(struct pci_epf_test *epf_test, u8 irq)
{ {
u8 irq;
u8 msi_count; u8 msi_count;
struct pci_epf *epf = epf_test->epf; struct pci_epf *epf = epf_test->epf;
struct pci_epc *epc = epf->epc; struct pci_epc *epc = epf->epc;
...@@ -262,7 +261,6 @@ static void pci_epf_test_raise_irq(struct pci_epf_test *epf_test) ...@@ -262,7 +261,6 @@ static void pci_epf_test_raise_irq(struct pci_epf_test *epf_test)
reg->status |= STATUS_IRQ_RAISED; reg->status |= STATUS_IRQ_RAISED;
msi_count = pci_epc_get_msi(epc); msi_count = pci_epc_get_msi(epc);
irq = (reg->command & MSI_NUMBER_MASK) >> MSI_NUMBER_SHIFT;
if (irq > msi_count || msi_count <= 0) if (irq > msi_count || msi_count <= 0)
pci_epc_raise_irq(epc, PCI_EPC_IRQ_LEGACY, 0); pci_epc_raise_irq(epc, PCI_EPC_IRQ_LEGACY, 0);
else else
...@@ -289,6 +287,8 @@ static void pci_epf_test_cmd_handler(struct work_struct *work) ...@@ -289,6 +287,8 @@ static void pci_epf_test_cmd_handler(struct work_struct *work)
reg->command = 0; reg->command = 0;
reg->status = 0; reg->status = 0;
irq = (command & MSI_NUMBER_MASK) >> MSI_NUMBER_SHIFT;
if (command & COMMAND_RAISE_LEGACY_IRQ) { if (command & COMMAND_RAISE_LEGACY_IRQ) {
reg->status = STATUS_IRQ_RAISED; reg->status = STATUS_IRQ_RAISED;
pci_epc_raise_irq(epc, PCI_EPC_IRQ_LEGACY, 0); pci_epc_raise_irq(epc, PCI_EPC_IRQ_LEGACY, 0);
...@@ -301,7 +301,7 @@ static void pci_epf_test_cmd_handler(struct work_struct *work) ...@@ -301,7 +301,7 @@ static void pci_epf_test_cmd_handler(struct work_struct *work)
reg->status |= STATUS_WRITE_FAIL; reg->status |= STATUS_WRITE_FAIL;
else else
reg->status |= STATUS_WRITE_SUCCESS; reg->status |= STATUS_WRITE_SUCCESS;
pci_epf_test_raise_irq(epf_test); pci_epf_test_raise_irq(epf_test, irq);
goto reset_handler; goto reset_handler;
} }
...@@ -311,7 +311,7 @@ static void pci_epf_test_cmd_handler(struct work_struct *work) ...@@ -311,7 +311,7 @@ static void pci_epf_test_cmd_handler(struct work_struct *work)
reg->status |= STATUS_READ_SUCCESS; reg->status |= STATUS_READ_SUCCESS;
else else
reg->status |= STATUS_READ_FAIL; reg->status |= STATUS_READ_FAIL;
pci_epf_test_raise_irq(epf_test); pci_epf_test_raise_irq(epf_test, irq);
goto reset_handler; goto reset_handler;
} }
...@@ -321,13 +321,12 @@ static void pci_epf_test_cmd_handler(struct work_struct *work) ...@@ -321,13 +321,12 @@ static void pci_epf_test_cmd_handler(struct work_struct *work)
reg->status |= STATUS_COPY_SUCCESS; reg->status |= STATUS_COPY_SUCCESS;
else else
reg->status |= STATUS_COPY_FAIL; reg->status |= STATUS_COPY_FAIL;
pci_epf_test_raise_irq(epf_test); pci_epf_test_raise_irq(epf_test, irq);
goto reset_handler; goto reset_handler;
} }
if (command & COMMAND_RAISE_MSI_IRQ) { if (command & COMMAND_RAISE_MSI_IRQ) {
msi_count = pci_epc_get_msi(epc); msi_count = pci_epc_get_msi(epc);
irq = (command & MSI_NUMBER_MASK) >> MSI_NUMBER_SHIFT;
if (irq > msi_count || msi_count <= 0) if (irq > msi_count || msi_count <= 0)
goto reset_handler; goto reset_handler;
reg->status = STATUS_IRQ_RAISED; reg->status = STATUS_IRQ_RAISED;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7