MIPS: Loongson 3: Keep CPU physical (not virtual) addresses in shadow ROM resource
authorBjorn Helgaas <bhelgaas@google.com>
Thu, 3 Mar 2016 03:42:04 +0000 (21:42 -0600)
committerBjorn Helgaas <bhelgaas@google.com>
Sat, 12 Mar 2016 12:00:29 +0000 (06:00 -0600)
commit97f47e73c4a0dc2519276cd63274696c850e0613
tree16905743f6759d80a8c3386d8860b9f28bc9d0e7
parent53f0a509775dae77dab64d0b858635a70e3d6f4f
MIPS: Loongson 3: Keep CPU physical (not virtual) addresses in shadow ROM resource

Loongson 3 used the IORESOURCE_ROM_COPY flag for its ROM resource.  There
are two problems with this:

  - When IORESOURCE_ROM_COPY is set, pci_map_rom() assumes the resource
    contains virtual addresses, so it doesn't ioremap the resource.  This
    implies loongson_sysconf.vgabios_addr is a virtual address.  That's a
    problem because resources should contain CPU *physical* addresses not
    virtual addresses.

  - When IORESOURCE_ROM_COPY is set, pci_cleanup_rom() calls kfree() on the
    resource.  We did not kmalloc() the loongson_sysconf.vgabios_addr area,
    so it is incorrect to kfree() it.

If we're using a shadow copy in RAM for the Loongson 3 VGA BIOS area,
disable the ROM BAR and release the address space it was consuming.

Use IORESOURCE_ROM_SHADOW instead of IORESOURCE_ROM_COPY.  This means the
struct resource contains CPU physical addresses, and pci_map_rom() will
ioremap() it as needed.

Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
arch/mips/pci/fixup-loongson3.c