Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-09-18 Thread Bjorn Helgaas
On Tue, Sep 18, 2012 at 12:33 AM, Thierry Reding wrote: > On Sat, Sep 08, 2012 at 11:51:00AM -0600, Bjorn Helgaas wrote: >> On Fri, Sep 7, 2012 at 6:04 PM, Russell King - ARM Linux >> wrote: >> > On Fri, Sep 07, 2012 at 05:34:35PM -0600, Stephen Warren wrote: >> >> I guess it's a pretty basic pre

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-09-17 Thread Thierry Reding
On Sat, Sep 08, 2012 at 11:51:00AM -0600, Bjorn Helgaas wrote: > On Fri, Sep 7, 2012 at 6:04 PM, Russell King - ARM Linux > wrote: > > On Fri, Sep 07, 2012 at 05:34:35PM -0600, Stephen Warren wrote: > >> I guess it's a pretty basic premise of the current PCI code that all the > >> PCI scanning hap

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-09-08 Thread Bjorn Helgaas
On Fri, Sep 7, 2012 at 6:04 PM, Russell King - ARM Linux wrote: > On Fri, Sep 07, 2012 at 05:34:35PM -0600, Stephen Warren wrote: >> I guess it's a pretty basic premise of the current PCI code that all the >> PCI scanning happens well before any device drivers are registered, >> which in turn mean

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-09-07 Thread Stephen Warren
On 09/07/2012 06:04 PM, Russell King - ARM Linux wrote: > On Fri, Sep 07, 2012 at 05:34:35PM -0600, Stephen Warren wrote: >> I guess it's a pretty basic premise of the current PCI code that all the >> PCI scanning happens well before any device drivers are registered, >> which in turn means that de

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-09-07 Thread Russell King - ARM Linux
On Fri, Sep 07, 2012 at 05:34:35PM -0600, Stephen Warren wrote: > I guess it's a pretty basic premise of the current PCI code that all the > PCI scanning happens well before any device drivers are registered, > which in turn means that device_add() doesn't trigger the device's > probe() until much

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-09-07 Thread Stephen Warren
On 08/14/2012 05:51 PM, Stephen Warren wrote: > On 08/14/2012 04:58 PM, Stephen Warren wrote: >> On 08/14/2012 03:55 PM, Bjorn Helgaas wrote: >>> On Tue, Aug 14, 2012 at 12:58 PM, Thierry Reding >>> wrote: On Tue, Aug 14, 2012 at 01:39:23PM -0600, Stephen Warren wrote: > On 08/13/2012 05:

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-15 Thread Thierry Reding
On Wed, Aug 15, 2012 at 02:11:10PM -0600, Stephen Warren wrote: > On 08/15/2012 02:09 PM, Thierry Reding wrote: > > On Wed, Aug 15, 2012 at 01:04:20PM -0600, Stephen Warren wrote: > >> On 08/14/2012 05:51 PM, Stephen Warren wrote: > >>> On 08/14/2012 04:58 PM, Stephen Warren wrote: > >> ... >

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-15 Thread Stephen Warren
On 08/15/2012 02:09 PM, Thierry Reding wrote: > On Wed, Aug 15, 2012 at 01:04:20PM -0600, Stephen Warren wrote: >> On 08/14/2012 05:51 PM, Stephen Warren wrote: >>> On 08/14/2012 04:58 PM, Stephen Warren wrote: >> ... Can't we make the call to pci_bus_add_devices() optional in pci_scan_r

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-15 Thread Thierry Reding
On Wed, Aug 15, 2012 at 01:04:20PM -0600, Stephen Warren wrote: > On 08/14/2012 05:51 PM, Stephen Warren wrote: > > On 08/14/2012 04:58 PM, Stephen Warren wrote: > ... > >> Can't we make the call to pci_bus_add_devices() optional in > >> pci_scan_root_bus() somehow; one of: > > > > Sigh, that turn

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-15 Thread Stephen Warren
On 08/14/2012 05:51 PM, Stephen Warren wrote: > On 08/14/2012 04:58 PM, Stephen Warren wrote: ... >> Can't we make the call to pci_bus_add_devices() optional in >> pci_scan_root_bus() somehow; one of: > > Sigh, that turns out not to work correctly; it solves at least this part > of the problem whe

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-14 Thread Bjorn Helgaas
On Tue, Aug 14, 2012 at 3:58 PM, Stephen Warren wrote: > On 08/14/2012 03:55 PM, Bjorn Helgaas wrote: >> On Tue, Aug 14, 2012 at 12:58 PM, Thierry Reding >> wrote: >>> On Tue, Aug 14, 2012 at 01:39:23PM -0600, Stephen Warren wrote: On 08/13/2012 05:18 PM, Bjorn Helgaas wrote: > On Mon, A

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-14 Thread Stephen Warren
On 08/14/2012 04:58 PM, Stephen Warren wrote: > On 08/14/2012 03:55 PM, Bjorn Helgaas wrote: >> On Tue, Aug 14, 2012 at 12:58 PM, Thierry Reding >> wrote: >>> On Tue, Aug 14, 2012 at 01:39:23PM -0600, Stephen Warren wrote: On 08/13/2012 05:18 PM, Bjorn Helgaas wrote: > On Mon, Aug 13, 201

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-14 Thread Stephen Warren
On 08/14/2012 03:55 PM, Bjorn Helgaas wrote: > On Tue, Aug 14, 2012 at 12:58 PM, Thierry Reding > wrote: >> On Tue, Aug 14, 2012 at 01:39:23PM -0600, Stephen Warren wrote: >>> On 08/13/2012 05:18 PM, Bjorn Helgaas wrote: On Mon, Aug 13, 2012 at 11:47 AM, Stephen Warren wrote: >>> ... >

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-14 Thread Bjorn Helgaas
On Tue, Aug 14, 2012 at 12:58 PM, Thierry Reding wrote: > On Tue, Aug 14, 2012 at 01:39:23PM -0600, Stephen Warren wrote: >> On 08/13/2012 05:18 PM, Bjorn Helgaas wrote: >> > On Mon, Aug 13, 2012 at 11:47 AM, Stephen Warren >> > wrote: >> ... >> >> whereas for a device tree boot: >> >> >> >> (sa

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-14 Thread Thierry Reding
On Tue, Aug 14, 2012 at 01:39:23PM -0600, Stephen Warren wrote: > On 08/13/2012 05:18 PM, Bjorn Helgaas wrote: > > On Mon, Aug 13, 2012 at 11:47 AM, Stephen Warren > > wrote: > ... > >> whereas for a device tree boot: > >> > >> (same): > >>> [2.112217] pci :01:00.0: reg 10: [io 0x-0x

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-14 Thread Stephen Warren
On 08/13/2012 05:18 PM, Bjorn Helgaas wrote: > On Mon, Aug 13, 2012 at 11:47 AM, Stephen Warren > wrote: ... >> whereas for a device tree boot: >> >> (same): >>> [2.112217] pci :01:00.0: reg 10: [io 0x-0x00ff] >>> [2.117635] pci :01:00.0: reg 18: [mem 0x-0x0fff 64

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-13 Thread Thierry Reding
On Mon, Aug 13, 2012 at 04:18:16PM -0700, Bjorn Helgaas wrote: > On Mon, Aug 13, 2012 at 11:47 AM, Stephen Warren > wrote: > > On 08/13/2012 11:40 AM, Thierry Reding wrote: > >> On Mon, Aug 06, 2012 at 01:42:21PM -0600, Stephen Warren wrote: > >>> On 07/26/2012 01:55 PM, Thierry Reding wrote: > >

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-13 Thread Thierry Reding
On Mon, Aug 13, 2012 at 04:38:45PM -0500, Rob Herring wrote: > On 08/13/2012 03:33 PM, Thierry Reding wrote: > > On Mon, Aug 13, 2012 at 12:47:38PM -0600, Stephen Warren wrote: > >> On 08/13/2012 11:40 AM, Thierry Reding wrote: > >>> On Mon, Aug 06, 2012 at 01:42:21PM -0600, Stephen Warren wrote: >

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-13 Thread Bjorn Helgaas
On Mon, Aug 13, 2012 at 11:47 AM, Stephen Warren wrote: > On 08/13/2012 11:40 AM, Thierry Reding wrote: >> On Mon, Aug 06, 2012 at 01:42:21PM -0600, Stephen Warren wrote: >>> On 07/26/2012 01:55 PM, Thierry Reding wrote: This patch series adds support for device tree based probing of the

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-13 Thread Rob Herring
On 08/13/2012 03:33 PM, Thierry Reding wrote: > On Mon, Aug 13, 2012 at 12:47:38PM -0600, Stephen Warren wrote: >> On 08/13/2012 11:40 AM, Thierry Reding wrote: >>> On Mon, Aug 06, 2012 at 01:42:21PM -0600, Stephen Warren wrote: On 07/26/2012 01:55 PM, Thierry Reding wrote: > This patch se

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-13 Thread Thierry Reding
On Mon, Aug 13, 2012 at 12:47:38PM -0600, Stephen Warren wrote: > On 08/13/2012 11:40 AM, Thierry Reding wrote: > > On Mon, Aug 06, 2012 at 01:42:21PM -0600, Stephen Warren wrote: > >> On 07/26/2012 01:55 PM, Thierry Reding wrote: > >>> This patch series adds support for device tree based probing o

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-13 Thread Stephen Warren
On 08/13/2012 11:40 AM, Thierry Reding wrote: > On Mon, Aug 06, 2012 at 01:42:21PM -0600, Stephen Warren wrote: >> On 07/26/2012 01:55 PM, Thierry Reding wrote: >>> This patch series adds support for device tree based probing of >>> the PCIe controller found on Tegra SoCs. >> >> Thierry, I just te

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-13 Thread Thierry Reding
On Mon, Aug 06, 2012 at 01:42:21PM -0600, Stephen Warren wrote: > On 07/26/2012 01:55 PM, Thierry Reding wrote: > > This patch series adds support for device tree based probing of the PCIe > > controller found on Tegra SoCs. > > Thierry, I just tested all Tegra boards in v3.6-rc1, and noticed that

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-07 Thread Thierry Reding
On Mon, Aug 06, 2012 at 01:42:21PM -0600, Stephen Warren wrote: > On 07/26/2012 01:55 PM, Thierry Reding wrote: > > This patch series adds support for device tree based probing of the PCIe > > controller found on Tegra SoCs. > > Thierry, I just tested all Tegra boards in v3.6-rc1, and noticed that

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-06 Thread Stephen Warren
On 07/26/2012 01:55 PM, Thierry Reding wrote: > This patch series adds support for device tree based probing of the PCIe > controller found on Tegra SoCs. Thierry, I just tested all Tegra boards in v3.6-rc1, and noticed that PCIe doesn't work on TrimSlice when booting use device tree. I think I fo

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-01 Thread Thierry Reding
On Wed, Aug 01, 2012 at 11:02:18AM -0600, Stephen Warren wrote: > On 08/01/2012 12:35 AM, Thierry Reding wrote: > > On Tue, Jul 31, 2012 at 10:18:15AM -0600, Stephen Warren wrote: > >> On 07/26/2012 01:55 PM, Thierry Reding wrote: > >>> This patch series adds support for device tree based probing o

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-08-01 Thread Stephen Warren
On 08/01/2012 12:35 AM, Thierry Reding wrote: > On Tue, Jul 31, 2012 at 10:18:15AM -0600, Stephen Warren wrote: >> On 07/26/2012 01:55 PM, Thierry Reding wrote: >>> This patch series adds support for device tree based probing of >>> the PCIe controller found on Tegra SoCs. >> >> Thierry, >> >> I

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-07-31 Thread Thierry Reding
On Tue, Jul 31, 2012 at 10:18:15AM -0600, Stephen Warren wrote: > On 07/26/2012 01:55 PM, Thierry Reding wrote: > > This patch series adds support for device tree based probing of the PCIe > > controller found on Tegra SoCs. > > Thierry, > > I think one thing that would help here would be to spli

Re: [PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-07-31 Thread Stephen Warren
On 07/26/2012 01:55 PM, Thierry Reding wrote: > This patch series adds support for device tree based probing of the PCIe > controller found on Tegra SoCs. Thierry, I think one thing that would help here would be to split up this series into one per subsystem, and to get all the dependencies merge

[PATCH v3 00/10] ARM: tegra: Add PCIe device tree support

2012-07-26 Thread Thierry Reding
This patch series adds support for device tree based probing of the PCIe controller found on Tegra SoCs. Patches 1 and 2 keep the pci_fixup_irqs() and ARM-specific pci_common_init() functions around after init. This is required to support driver probe deferral, which may cause built-in drivers to