On Mon, 2013-11-25 at 16:01 -0700, Bjorn Helgaas wrote: > On Wed, Oct 23, 2013 at 06:41:23PM +0800, Minghuan Lian wrote: > > The Freescale's Layerscape series processors will use ARM cores. > > The LS1's PCIe controllers is the same as T4240's. So it's better > > the PCIe controller driver can support PowerPC and ARM > > simultaneously. This patch is for this purpose. It derives > > the common functions from arch/powerpc/sysdev/fsl_pci.c to > > drivers/pci/host/pci-fsl-common.c and leaves the architecture > > specific functions which should be implemented in arch related files. > > > > Signed-off-by: Minghuan Lian <Minghuan.Lian@xxxxxxxxxxxxx> > > It doesn't look like we have a consensus on how to proceed here, so I'm > ignoring this series for now. Let me know if there really *is* agreement, > or if there's some subset of this work that everybody can agree on. It'd > be good to make forward progress, even if it's not completely perfect yet. I'd like to see what this would look like with actual ARM support, rather than just some theoretical attempts to make it less PPC-dependent that involve copying PPC stuff that may or may not work well with what ARM code will expect. When I replied to Kumar's comment saying that this seemed like a reasonable interim approach, that was before I saw how much was copied from places other than fsl_pci files. -Scott -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
