]> git.baikalelectronics.ru Git - kernel.git/commitdiff
arch/sh: pcie-sh7786: mark unavailable PCI resource as disabled
authorThomas Petazzoni <thomas.petazzoni@free-electrons.com>
Mon, 4 Dec 2017 15:09:03 +0000 (16:09 +0100)
committerRich Felker <dalias@libc.org>
Thu, 12 Apr 2018 23:47:55 +0000 (19:47 -0400)
Some PCI MEM resources are marked as IORESOURCE_MEM_32BIT, which means
they are only usable when the SH core runs in 32-bit mode. In 29-bit
mode, such memory regions are not usable.

The existing code for SH7786 properly skips such regions when
configuring the PCIe controller registers. However, because such
regions are still described in the resource array, the
pcibios_scanbus() function in the SuperH pci.c will register them to
the PCI core. Due to this, the PCI core will allocate MEM areas from
this resource, and assign BARs pointing to this area, even though it's
unusable.

In order to prevent this from happening, we mark such regions as
IORESOURCE_DISABLED, which tells the SuperH pci.c pcibios_scanbus()
function to skip them.

Note that we separate marking the region as disabled from skipping it,
because other regions will be marked as disabled in follow-up patches.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Signed-off-by: Rich Felker <dalias@libc.org>
arch/sh/drivers/pci/pcie-sh7786.c

index 0167a735271942a04580d527c844ce6dd0628673..e713c398a6b98b20043b08734bc98228a315fb93 100644 (file)
@@ -438,6 +438,9 @@ static int __init pcie_init(struct sh7786_pcie_port *port)
                 * mode, so just skip them entirely.
                 */
                if ((res->flags & IORESOURCE_MEM_32BIT) && __in_29bit_mode())
+                       res->flags |= IORESOURCE_DISABLED;
+
+               if (res->flags & IORESOURCE_DISABLED)
                        continue;
 
                pci_write_reg(chan, 0x00000000, SH4A_PCIEPTCTLR(win));