From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Return-Path: Subject: Re: [PATCHv2 0/3] pci: remove msi_controller registration API To: Thomas Petazzoni , Rob Herring , Frank Rowand , Bjorn Helgaas , linux-pci@vger.kernel.org References: <1482484854-26418-1-git-send-email-thomas.petazzoni@free-electrons.com> From: Marc Zyngier Message-ID: Date: Mon, 30 Jan 2017 10:14:33 +0000 MIME-Version: 1.0 In-Reply-To: <1482484854-26418-1-git-send-email-thomas.petazzoni@free-electrons.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Gregory Clement , Andrew Lunn , Jason Cooper , linux-arm-kernel@lists.infradead.org, Sebastian Hesselbarth Content-Type: text/plain; charset="us-ascii" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+bjorn=helgaas.com@lists.infradead.org List-ID: On 23/12/16 09:20, Thomas Petazzoni wrote: > Hello, > > This small patch series aims at removing the small msi_controller > registration API implemented in drivers/of/of_pci.c. > > In order to do so, it moves the last two users of this API to the > generic MSI support: > > - The pci-aardvark driver is moved to use the generic MSI > support. This driver is both the provider of the MSI capability and > the consumer of this capability. > > - The pci-mvebu driver is cleaned up from using the msi_controller > registration API. This driver is only consuming MSI capabilies from > another driver (irq-armada-370-xp) which was migrated to the > generic MSI support some time ago. Therefore, this patch is simply > removing dead code. > > Finally, the last commit removed the now unused API. > > Changes between v1 to v2 are described in each individual patch. > > Thanks, > > Thomas > > Thomas Petazzoni (3): > pci: pci-aardvark: move to MSI handling using generic MSI support > pci: pci-mvebu: remove useless MSI enabling code > of: pci: remove unused MSI controller helpers > > drivers/of/of_pci.c | 45 ----------- > drivers/pci/host/pci-aardvark.c | 172 +++++++++++++++------------------------- > drivers/pci/host/pci-mvebu.c | 17 ---- > include/linux/of_pci.h | 11 --- > 4 files changed, 66 insertions(+), 179 deletions(-) > I somehow missed this series during the Xmas craze. It is definitely a good cleanup, so for the two patches I haven't Acked yet: Acked-by: Marc Zyngier Thanks, M. -- Jazz is not dead. It just smells funny... _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel From mboxrd@z Thu Jan 1 00:00:00 1970 From: marc.zyngier@arm.com (Marc Zyngier) Date: Mon, 30 Jan 2017 10:14:33 +0000 Subject: [PATCHv2 0/3] pci: remove msi_controller registration API In-Reply-To: <1482484854-26418-1-git-send-email-thomas.petazzoni@free-electrons.com> References: <1482484854-26418-1-git-send-email-thomas.petazzoni@free-electrons.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 23/12/16 09:20, Thomas Petazzoni wrote: > Hello, > > This small patch series aims at removing the small msi_controller > registration API implemented in drivers/of/of_pci.c. > > In order to do so, it moves the last two users of this API to the > generic MSI support: > > - The pci-aardvark driver is moved to use the generic MSI > support. This driver is both the provider of the MSI capability and > the consumer of this capability. > > - The pci-mvebu driver is cleaned up from using the msi_controller > registration API. This driver is only consuming MSI capabilies from > another driver (irq-armada-370-xp) which was migrated to the > generic MSI support some time ago. Therefore, this patch is simply > removing dead code. > > Finally, the last commit removed the now unused API. > > Changes between v1 to v2 are described in each individual patch. > > Thanks, > > Thomas > > Thomas Petazzoni (3): > pci: pci-aardvark: move to MSI handling using generic MSI support > pci: pci-mvebu: remove useless MSI enabling code > of: pci: remove unused MSI controller helpers > > drivers/of/of_pci.c | 45 ----------- > drivers/pci/host/pci-aardvark.c | 172 +++++++++++++++------------------------- > drivers/pci/host/pci-mvebu.c | 17 ---- > include/linux/of_pci.h | 11 --- > 4 files changed, 66 insertions(+), 179 deletions(-) > I somehow missed this series during the Xmas craze. It is definitely a good cleanup, so for the two patches I haven't Acked yet: Acked-by: Marc Zyngier Thanks, M. -- Jazz is not dead. It just smells funny...