On Thursday 09 January 2014, Mohit KUMAR DCG wrote: > - To just clarify this dw PCIe driver consists 3 parts: > 1. Common part: Common PCIe code (host_init() etc..) this should > be common to all hosts. As already discussed it should be ideally > moved out from this driver. It is in todo list and will move to separate > file in due course of time. > > 2. Viewport handling: ATU translation code > > 3. MSI specific code > > Synopsys controllers version 3.70+have been standardized for viewport > handling and msi implementation. Current dw driver covers these standard > implementation for viewport and MSI. > > Now I can understand that you are using only common part of dw driver > And have specific viewport and msi implementation. So its getting a bit > messy if you modify the driver for your specific implementation as > discussed in this thread. > > So ideally I will suggest that you should move the common code (point 1) > to outside this driver, and then use this along with your specific viewport > and msi implementation. > > Arnd, Jingoo, > > Do you agree with above comments? I think it depends a bit on how much code ends up being shared in the first category, and whether it's easily separable from the second one. I assume the MSI implementation is fairly isolated already. I definitely agree with the general principle of layering drivers in this way, specific version of the driver basically uses exported functions of the more general driver like st-pcie -> dw-pcie-3.7 -> dw-pcie -> pci-common I have two comments about the specifics here: * On most architectures, the MSI controller is integrated into the primary irqchip. If a dw-pcie-3.7+ is used on such a machine, I guess it will just ignore the MSI part * On server systems, the setup of the viewport would typically be handled by the firmware before the kernel is started, and the kernel does not actually need to know about it. This means we might actually want to split out the MSI and viewport parts into two separate pieces of code that can optionally be used by any of the SoC specific host drivers. 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
