linux-toradex.git/arch/tile/kernel/pci.c, branch v4.5

PCI: Assign resources before drivers claim devices (pci_scan_root_bus())

2015-03-19T15:17:13+00:00

Previously, pci_scan_root_bus() created a root PCI bus, enumerated the
devices on it, and called pci_bus_add_devices(), which made the devices
available for drivers to claim them.

Most callers assigned resources to devices after pci_scan_root_bus()
returns, which may be after drivers have claimed the devices.  This is
incorrect; the PCI core should not change device resources while a driver
is managing the device.

Remove pci_bus_add_devices() from pci_scan_root_bus() and do it after any
resource assignment in the callers.

Note that ARM's pci_common_init_dev() already called pci_bus_add_devices()
after pci_scan_root_bus(), so we only need to remove the first call:

  pci_common_init_dev
    pcibios_init_hw
      pci_scan_root_bus
        pci_bus_add_devices        # first call
    pci_bus_assign_resources
    pci_bus_add_devices            # second call

[bhelgaas: changelog, drop "root_bus" var in alpha common_init_pci(),
return failure earlier in mn10300, add "return" in x86 pcibios_scan_root(),
return early if xtensa platform_pcibios_fixup() fails]
Signed-off-by: Yijing Wang 
Signed-off-by: Bjorn Helgaas 
CC: Richard Henderson 
CC: Ivan Kokshaysky 
CC: Matt Turner 
CC: David Howells 
CC: Tony Luck 
CC: Michal Simek 
CC: Ralf Baechle 
CC: Koichi Yasutake 
CC: Sebastian Ott 
CC: "David S. Miller" 
CC: Chris Metcalf 
CC: Chris Zankel 
CC: Max Filippov 
CC: Thomas Gleixner

tile: use PCI define for Max_Read_Request_Size

2015-01-27T14:14:26+00:00

Replace a magic number with a PCI #define symbol.

Signed-off-by: Rafał Miłecki 
Signed-off-by: Bjorn Helgaas 
Acked-by: Chris Metcalf

tile: Use the more common pr_warn instead of pr_warning

2014-11-11T20:51:42+00:00

And other message logging neatening.

Other miscellanea:

o coalesce formats
o realign arguments
o standardize a couple of macros
o use __func__ instead of embedding the function name

Signed-off-by: Joe Perches 
Signed-off-by: Chris Metcalf

arch/tile: remove unused variable 'devcap'

2014-04-04T17:11:40+00:00

Commit 503275bf37 removed the use of the variable but not
the variable itself.

Signed-off-by: Chris Metcalf

tile/PCI: use cached pci_dev->pcie_mpss to simplify code

2013-09-24T18:10:04+00:00

The PCI core caches the "PCIe Max Payload Size Supported" in
pci_dev->pcie_mpss, so use that instead of pcie_capability_read_dword().

Signed-off-by: Yijing Wang 
Signed-off-by: Bjorn Helgaas

tile: mark pcibios_init() as __init

2013-09-03T18:53:15+00:00

It was bombed away because it was previously marked as __devinit,
but it should be an __init function.

Signed-off-by: Chris Metcalf

tile PCI RC: support pci=off boot arg for tilepro

2013-08-05T20:12:57+00:00

Signed-off-by: Chris Metcalf

tile PCI RC: cleanups for tilepro PCI RC

2013-08-05T20:12:38+00:00

- remove unneeded  include in pci.c
- eliminate unused pci_controller.first_busno field
- prefer msleep to mdelay
- remove stale comment about pci_scan_bus_parented()

Signed-off-by: Chris Metcalf

ARCH: drivers remove __dev* attributes.

2013-01-03T23:57:13+00:00

This fixes up all of the smaller arches that had __dev* markings for
their platform-specific drivers.

CONFIG_HOTPLUG is going away as an option.  As a result, the __dev*
markings need to be removed.

This change removes the use of __devinit, __devexit_p, __devinitdata,
__devinitconst, and __devexit from these drivers.

Based on patches originally written by Bill Pemberton, but redone by me
in order to handle some of the coding style issues better, by hand.

Cc: Bill Pemberton 
Cc: Peter Zijlstra 
Cc: Paul Mackerras 
Cc: Ingo Molnar 
Cc: Arnaldo Carvalho de Melo 
Cc: Catalin Marinas 
Cc: Will Deacon 
Cc: Haavard Skinnemoen 
Cc: Hans-Christian Egtvedt 
Cc: Mike Frysinger 
Cc: Mikael Starvik 
Cc: Jesper Nilsson 
Cc: David Howells 
Cc: Hirokazu Takata 
Cc: Geert Uytterhoeven 
Cc: Michal Simek 
Cc: Koichi Yasutake 
Cc: Jonas Bonn 
Cc: "James E.J. Bottomley" 
Cc: Helge Deller 
Cc: Martin Schwidefsky 
Cc: Heiko Carstens 
Cc: Chen Liqin 
Cc: Lennox Wu 
Cc: Paul Mundt 
Cc: Chris Metcalf 
Cc: Guan Xuetao 
Cc: Bob Liu 
Cc: Srinivas Kandagatla 
Cc: Bjorn Helgaas 
Cc: Myron Stowe 
Cc: Thomas Gleixner 
Cc: Andrew Morton 
Cc: Andi Kleen 
Cc: Jesse Barnes 
Cc: Sebastian Andrzej Siewior 
Cc: Yinghai Lu 
Cc: Thierry Reding 
Cc: Greg Ungerer 
Cc: Grant Likely 
Cc: "Srivatsa S. Bhat" 
Cc: Mark Salter 
Cc: Yong Zhang 
Cc: Michael Holzheu 
Cc: Cornelia Huck 
Cc: Jan Glauber 
Cc: Wei Yongjun 
Cc: Nobuhiro Iwamatsu 
Signed-off-by: Greg Kroah-Hartman

tile/PCI: use for_each_pci_dev to simplify the code

2012-12-12T22:22:18+00:00

Use for_each_pci_dev to simplify the code.

Signed-off-by: Wei Yongjun 
Signed-off-by: Chris Metcalf