Hello Murali, > -----Original Message----- > From: Murali Karicheri [mailto:m-kariche...@ti.com] > Sent: Wednesday, June 11, 2014 12:21 AM > To: linux-arm-ker...@lists.infradead.org; linux-kernel@vger.kernel.org; > linux-...@vger.kernel.org; devicet...@vger.kernel.org; linux- > d...@vger.kernel.org > Cc: Murali Karicheri; Santosh Shilimkar; Russell King; Grant Likely; Rob > Herring; > Mohit KUMAR DCG; Jingoo Han; Bjorn Helgaas; Pratyush ANAND; Richard > Zhu; Kishon Vijay Abraham I; Marek Vasut; Arnd Bergmann; Pawel Moll; > Mark Rutland; Ian Campbell; Kumar Gala; Randy Dunlap > Subject: [PATCH v2 0/8] Add Keystone PCIe controller driver > > This patch adds a PCIe controller driver for Keystone SoCs. This is based on > v1 > of the series posted to the mailing list. >
1. I think your first patch is OK which handles platform specific ATU implementation. 2. For MSI part, I think you just need to add two new callbacks with pp-ops, something similar to: pp->ops->msi_set pp->ops->msi_clear With these two platform specific callbacks you should be able to manage MSI handling. So idea is that dw_msi code uses pp->ops->msi_set/clear if platform define these, otherwise use dw_msi_set/clear (which you need to refactor from existing code) So other than your keystone changes we expect 3 patches: -- 1st same as you sent 1/8: for ATU handeling -- 2nd to refactor dw_msi_set/clear: refactor from existing code -- 3rd to use pp->ops->msi_set/clear if defined. Pls let us know for any issue or have different opinion. Regards Mohit > > -- > 1.7.9.5 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/