On Monday 03 February 2014 17:07:48 Rob Herring wrote: > On Mon, Feb 3, 2014 at 2:05 PM, Arnd Bergmann <arnd@xxxxxxxx> wrote: > > You might want to re-read the SBSA. Unless ARM provides an IP block or > there is some other standard such as EHCI or AHCI, there is no generic > implementation. You only have to go look at the Linux EHCI or AHCI > drivers and see how meaningless and inadequate "use EHCI" is. For PCI, > the text is so brief in the SBSA there will be no way PCI is going to > just work given all the variations of root complexes, bridges, address > windows, etc. we typically see on ARM platforms. I could be wrong and > some AML magic will solve all the problems. I don't think you need any AML, and SBSA seems to cover the PCI case just fine, though I have to agree that the EHCI/AHCI/xHCI part is rather ambiguous. What the existing PCI host controller drivers do is essentially: 1. provide a config space access method 2. set up the I/O and memory address windows 3. set up clocks, PHYs etc 4. work around any deviations from the PCI standard 5. provide an MSI/MSI-X controller For all I can tell, any SBSA compliant system should handle those four like this: 1. config space is ECAM compliant, we only need to pass the location in DT. (SBSA 8.1) 2. all address windows are set up by the boot loader, we only need to know the location (IMHO this should be the preferred way to do things regardless of SBSA). 3. any external hardware dependencies are set up statically by the boot loader and are operational as we enter the kernel. 4. deviations from PCI are not allowed (SBSA 8.8) 5. MSI has to be handled by GICv3 (SBSA 8.3.2) So I definitely expect SBSA compliant systems to work fine with a very simple generic PCI host bridge driver (which is likely what Liviu has implemented and waiting for approval to submit). The more important question is what systems will actually be compliant with the above. X-Gene manages to get all five wrong, for instance, and so would any system that reuses the existing PCI hardware (alphabetically: exynos, imx6, mvebu, rcar-gen2, tegra, designware), although points 2 and 3 are a question of the boot loader, and it's possible that the designware based ones get point 4 right. Arnd -- 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
