diff options
| author | Gustavo Pimentel <gustavo.pimentel@synopsys.com> | 2018-11-23 18:00:21 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2018-12-05 19:32:07 +0100 |
| commit | 1ce69ec307fd540d13355f3a3c6c83dcd87e6f9f (patch) | |
| tree | 10962de0fac06be5c3541e549373e80ab5c55065 /drivers/pci | |
| parent | b391ed731b393a857067de658d826b059652ca02 (diff) | |
| download | linux-rpi3-1ce69ec307fd540d13355f3a3c6c83dcd87e6f9f.tar.gz linux-rpi3-1ce69ec307fd540d13355f3a3c6c83dcd87e6f9f.tar.bz2 linux-rpi3-1ce69ec307fd540d13355f3a3c6c83dcd87e6f9f.zip | |
PCI: dwc: Fix MSI-X EP framework address calculation bug
commit 15cb127e3c8f6232096d5dba6a5b4046bc292d70 upstream.
Fix an error caused by 3-bit right rotation on offset address
calculation of MSI-X table in dw_pcie_ep_raise_msix_irq().
The initial testing code was setting by default the offset address of
MSI-X table to zero, so that even with a 3-bit right rotation the
computed result would still be zero and valid, therefore this bug went
unnoticed.
Fixes: beb4641a787d ("PCI: dwc: Add MSI-X callbacks handler")
Signed-off-by: Gustavo Pimentel <gustavo.pimentel@synopsys.com>
[lorenzo.pieralisi@arm.com: updated commit log]
Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Cc: stable@vger.kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/pci')
| -rw-r--r-- | drivers/pci/controller/dwc/pcie-designware-ep.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/pci/controller/dwc/pcie-designware-ep.c b/drivers/pci/controller/dwc/pcie-designware-ep.c index 1e7b02221eac..de8635af4cde 100644 --- a/drivers/pci/controller/dwc/pcie-designware-ep.c +++ b/drivers/pci/controller/dwc/pcie-designware-ep.c @@ -440,7 +440,6 @@ int dw_pcie_ep_raise_msix_irq(struct dw_pcie_ep *ep, u8 func_no, tbl_offset = dw_pcie_readl_dbi(pci, reg); bir = (tbl_offset & PCI_MSIX_TABLE_BIR); tbl_offset &= PCI_MSIX_TABLE_OFFSET; - tbl_offset >>= 3; reg = PCI_BASE_ADDRESS_0 + (4 * bir); bar_addr_upper = 0; |