From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lorenzo Pieralisi Subject: Re: [PATCH] PCI: altera-msi: Allow building as module Date: Thu, 30 May 2019 16:08:41 +0100 Message-ID: <20190530150841.GC13993@redmoon> References: <1556081835-12921-1-git-send-email-ley.foon.tan@intel.com> <1556081835-12921-2-git-send-email-ley.foon.tan@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1556081835-12921-2-git-send-email-ley.foon.tan@intel.com> Sender: linux-kernel-owner@vger.kernel.org To: Ley Foon Tan Cc: Bjorn Helgaas , linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org, devicetree@vger.kernel.org, lftan.linux@gmail.com List-Id: devicetree@vger.kernel.org On Wed, Apr 24, 2019 at 12:57:15PM +0800, Ley Foon Tan wrote: > Altera MSI IP is a soft IP and is only available after > FPGA image is programmed. > > Make driver modulable to support use case FPGA image is programmed > after kernel is booted. User proram FPGA image in kernel then only load > MSI driver module. > > Signed-off-by: Ley Foon Tan > --- > drivers/pci/controller/Kconfig | 2 +- > drivers/pci/controller/pcie-altera-msi.c | 10 ++++++++++ > 2 files changed, 11 insertions(+), 1 deletion(-) Applied to pci/altera for v5.3, thanks. Lorenzo > diff --git a/drivers/pci/controller/Kconfig b/drivers/pci/controller/Kconfig > index 4b550f9cdd56..920546cb84e2 100644 > --- a/drivers/pci/controller/Kconfig > +++ b/drivers/pci/controller/Kconfig > @@ -181,7 +181,7 @@ config PCIE_ALTERA > FPGA. > > config PCIE_ALTERA_MSI > - bool "Altera PCIe MSI feature" > + tristate "Altera PCIe MSI feature" > depends on PCIE_ALTERA > depends on PCI_MSI_IRQ_DOMAIN > help > diff --git a/drivers/pci/controller/pcie-altera-msi.c b/drivers/pci/controller/pcie-altera-msi.c > index 025ef7d9a046..16d938920ca5 100644 > --- a/drivers/pci/controller/pcie-altera-msi.c > +++ b/drivers/pci/controller/pcie-altera-msi.c > @@ -10,6 +10,7 @@ > #include > #include > #include > +#include > #include > #include > #include > @@ -288,4 +289,13 @@ static int __init altera_msi_init(void) > { > return platform_driver_register(&altera_msi_driver); > } > + > +static void __exit altera_msi_exit(void) > +{ > + platform_driver_unregister(&altera_msi_driver); > +} > + > subsys_initcall(altera_msi_init); > +MODULE_DEVICE_TABLE(of, altera_msi_of_match); > +module_exit(altera_msi_exit); > +MODULE_LICENSE("GPL v2"); > -- > 2.19.0 >