sparc: PCI: Fix incorrect address calculation of PCI Bridge windows on Simba-bridges
authoroftedal <oftedal@gmail.com>
Fri, 18 Oct 2013 20:28:29 +0000 (22:28 +0200)
committerDavid S. Miller <davem@davemloft.net>
Thu, 14 Nov 2013 22:56:38 +0000 (14:56 -0800)
The SIMBA APB Bridges lacks the 'ranges' of-property describing the
PCI I/O and memory areas located beneath the bridge. Faking this
information has been performed by reading range registers in the
APB bridge, and calculating the corresponding areas.

In commit 01f94c4a6ced476ce69b895426fc29bfc48c69bd
("Fix sabre pci controllers with new probing scheme.") a bug was
introduced into this calculation, causing the PCI memory areas
to be calculated incorrectly: The shift size was set to be
identical for I/O and MEM ranges, which is incorrect.

This patch set the shift size of the MEM range back to the
value used before 01f94c4a6ced476ce69b895426fc29bfc48c69bd.

Signed-off-by: Kjetil Oftedal <oftedal@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
arch/sparc/kernel/pci.c

index bc4d3f5d2e5d1683bdfe6d0b33a4a60adcd21916..cb021453de2aaac00323a6ea6e5612e7132ff41f 100644 (file)
@@ -398,8 +398,8 @@ static void apb_fake_ranges(struct pci_dev *dev,
        apb_calc_first_last(map, &first, &last);
        res = bus->resource[1];
        res->flags = IORESOURCE_MEM;
-       region.start = (first << 21);
-       region.end = (last << 21) + ((1 << 21) - 1);
+       region.start = (first << 29);
+       region.end = (last << 29) + ((1 << 29) - 1);
        pcibios_bus_to_resource(dev, res, &region);
 }