litmus-rt.git/arch/sh/drivers/pci, branch master

PCI: Cleanup control flow

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

Return errors immediately so the straightline path is the normal,
no-error path.  No functional change.

Signed-off-by: Bjorn Helgaas

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

sh/PCI: Pass GAPSPCI_DMA_BASE CPU & bus address to dma_declare_coherent_memory()

2014-05-20T22:55:39+00:00

dma_declare_coherent_memory() needs both the CPU physical address and the
bus address of the device memory.  They are the same on this platform, but
in general we should use pcibios_resource_to_bus() to account for any
address translation done by the PCI host bridge.

This makes no difference on Dreamcast, but is safer if the usage is copied
to future drivers.

Signed-off-by: Bjorn Helgaas 
Acked-by: Arnd Bergmann 
CC: Magnus Damm 
CC: linux-sh@vger.kernel.org

arch/sh/drivers/pci/pcie-sh7786.h: remove duplicate SH4A_PCIEPHYCTLR

2014-04-03T23:20:53+00:00

Signed-off-by: Geert Uytterhoeven 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

sh/PCI: Use default pcibios_enable_device()

2014-03-19T21:00:15+00:00

We don't need anything arch-specific in pcibios_enable_device(), so drop
the arch implementation and use the default generic one.

Signed-off-by: Bjorn Helgaas 
CC: Paul Mundt 
CC: linux-sh@vger.kernel.org

PCI: Delay enabling bridges until they're needed

2013-07-25T18:35:03+00:00

We currently enable PCI bridges after scanning a bus and assigning
resources.  This is often done in arch code.

This patch changes this so we don't enable a bridge until necessary, i.e.,
until we enable a PCI device behind the bridge.  We do this in the generic
pci_enable_device() path, so this also removes the arch-specific code to
enable bridges.

[bhelgaas: changelog]
Signed-off-by: Yinghai Lu 
Signed-off-by: Bjorn Helgaas

async: rename and redefine async_func_ptr

2013-03-12T20:59:14+00:00

A function type is typically defined as
typedef ret_type (*func)(args..)

but async_func_ptr is not.  Redefine it.

Also rename async_func_ptr to async_func_t for _func_t suffix is more generic.

Signed-off-by: Lai Jiangshan 
Signed-off-by: Tejun Heo 
Cc: Arjan van de Ven

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

sh/PCI: Remove CONFIG_HOTPLUG ifdefs

2012-11-28T20:50:22+00:00

Remove conditional code based on CONFIG_HOTPLUG being false.  It's
always on now in preparation of it going away as an option.

Signed-off-by: Bill Pemberton 
Cc: Paul Mundt 
Acked-by: Bjorn Helgaas 
Signed-off-by: Greg Kroah-Hartman

PCI: Provide a default pcibios_update_irq()

2012-09-18T23:28:21+00:00

Most architectures implement this in exactly the same way. Instead of
having each architecture duplicate this function, provide a single
implementation in the core and make it a weak symbol so that it can be
overridden on architectures where it is required.

Signed-off-by: Thierry Reding 
Signed-off-by: Bjorn Helgaas