Bjorn, Jason, Andrew, Gregory, Here is an updated version of the PCIe improvements. Changes since v1: * Fix some typos in the comments of PATCH 2, noticed by Bjorn Helgaas. * Rework the two last patches (first one was fixing the capability list, the other one the emulation of the devsel bits in the status register), after the comments of Bjorn. In fact, it turns out that the status register is composed of read-only bits for which we want to emulate a value of zero, and the other bits are write-1-to-clear bits that expose error conditions that we do not handle in the emulated bridge. So, in summary, emulating a 0 status register on reads, and ignoring writes to the status register is the simplest solution. If we later on implement a capability list, we can use a more clever emulation of the status register, but for the moment, this simple solution is enough. Therefore, the two last patches are now one single, simpler patch. I'd like those improvements to be kept separated from the original PCIe driver itself: while the PCIe driver has been around and reviewed for a long time, those improvements are newer. And I clearly do not want the PCIe driver to miss 3.11 because of any problem that could be found in those additional improvements. The main improvement being brought here is that the PCI-to-PCI bridge logic is fixed/extended to properly support physical PCIe bridges that are connected on a PCIe interface of a Marvell board. Without this improvement, only the devices connected directly to the PCIe interfaces of the board are properly enumerated. Any device that would sit beyond a physical bridge is not visible. Bjorn, with your Acked-by, could the Marvell maintainers include those patches in their branch, merged through arm-soc? They already have the Marvell PCIe driver itself, so I believe it makes sense to merge those improvements through the same path. Jason, those patches have been prepared on top of my marvell-pcie-v10 branch, I hope that's ok for you. If you want me to rebase them on some other branch in which you have integrated the PCIe driver, don't hesitate to tell me to do so. That said, since those patches are only touching the driver itself and no other file, they should not cause any conflict with other changes. Thanks, Thomas Thomas Petazzoni (3): pci: mvebu: no longer fake the slot location of downstream devices pci: mvebu: allow the enumeration of devices beyond physical bridges pci: mvebu: fix the emulation of the status register drivers/pci/host/pci-mvebu.c | 55 +++++++++++++++++++++++++++++++++--------- 1 file changed, 44 insertions(+), 11 deletions(-) -- 1.7.9.5 -- 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
