Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753164AbbLNIPy (ORCPT ); Mon, 14 Dec 2015 03:15:54 -0500 Received: from mail-pf0-f172.google.com ([209.85.192.172]:33874 "EHLO mail-pf0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752864AbbLNIPu (ORCPT ); Mon, 14 Dec 2015 03:15:50 -0500 Date: Mon, 14 Dec 2015 09:15:45 +0100 From: Thierry Reding To: Paul Gortmaker Cc: linux-kernel@vger.kernel.org, Bjorn Helgaas , Stephen Warren , Alexandre Courbot , linux-tegra@vger.kernel.org, linux-pci@vger.kernel.org Subject: Re: [PATCH 06/10] drivers/pci: make host/pci-tegra.c explicitly non-modular Message-ID: <20151214081545.GB10055@ulmo.nvidia.com> References: <1449970917-12633-1-git-send-email-paul.gortmaker@windriver.com> <1449970917-12633-7-git-send-email-paul.gortmaker@windriver.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="4SFOXa2GPu3tIq4H" Content-Disposition: inline In-Reply-To: <1449970917-12633-7-git-send-email-paul.gortmaker@windriver.com> User-Agent: Mutt/1.5.23+102 (2ca89bed6448) (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2944 Lines: 74 --4SFOXa2GPu3tIq4H Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sat, Dec 12, 2015 at 08:41:53PM -0500, Paul Gortmaker wrote: > The Kconfig currently controlling compilation of this code is: >=20 > drivers/pci/host/Kconfig:config PCI_TEGRA > drivers/pci/host/Kconfig: bool "NVIDIA Tegra PCIe controller" >=20 > ...meaning that it currently is not being built as a module by anyone. >=20 > Lets remove the modular code that is essentially orphaned, so that > when reading the driver there is no doubt it is builtin-only. >=20 > We don't have to disallow a driver unbind, since that is already > done for us in this driver. >=20 > Since module_platform_driver() uses the same init level priority as > builtin_platform_driver() the init ordering remains unchanged with > this commit. >=20 > Also note that MODULE_DEVICE_TABLE is a no-op for non-modular code. >=20 > We also delete the MODULE_LICENSE tag etc. since all that information > was (or is now) contained at the top of the file in the comments. >=20 > Cc: Thierry Reding > Cc: Bjorn Helgaas > Cc: Stephen Warren > Cc: Alexandre Courbot > Cc: linux-tegra@vger.kernel.org > Cc: linux-pci@vger.kernel.org > Signed-off-by: Paul Gortmaker > --- > drivers/pci/host/pci-tegra.c | 11 ++++------- > 1 file changed, 4 insertions(+), 7 deletions(-) I'd rather not do this. There is work underway to make it possible to turn PCI host drivers into modules. I have a local tree with the necessary changes and, though I suspect it might take a few releases to get everything merged, I think we'll get there eventually. Thierry --4SFOXa2GPu3tIq4H Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAABCAAGBQJWbnqxAAoJEN0jrNd/PrOhRQQP/jJM5GR+zJ2XfllEyito5A1H NH4ZkqIkEe5EQSgxWBw+Ijh4I9wOsvsJ6I2tc2AkypYtqhzW3gYO235+dY5J70+H YhK5Xw0HhMGadAZBO9N1nnV76hCGGdev4rU2PE7c/l6+DsgrEfVNW996mhUtHMG9 sByIDu41+yK0dKaIFLEAJQ6TkYCi7wnN5miIgvO5eshUXlMBxk8o0PwUTWP/mPFx bOF9EXVmtJcprumuvdTUYrsD7/YWv3f8hlLpJ1IdKyyVxsW2g05F+dRgeUrhibkh 8WFIn+JC3sQIa2rFo+kEAkzSp+zbp93pHYFkPGs0hwqQoLFFAhDmcd3y0XESbPNa JTGWFxWkQzIgVG1NvEKj+ZVoWUXwP+Oid7YvdHuMoLq0TW6wjpV22M2FKPNS4NHE Jh6jrF1MHloLL4TJQutTuMiRRx2LguUA5tlXGo2rdFhtY80ivmXfAUGMmyRR2gYQ QMeAX8Qi2q2176sIs5o+8G3J7uI6pVzBdyVU2hKLUnpuvSxVWxC5MrY9EPnq2aWm 7Ox1zrrdwMzwPQTbPeZ8LUbWKbSK6Y/ZKWg8Teo77YowzrnwvlbkSm9NuaKhwQ7s 3oOioG1iwKzlb02eFg9RFDToKRfJAcXlfj05zEeGLzpnUkcQ4ozU+YT6xfWZmr4L 93tc7AF6CQj2MY3wsLHy =/vf4 -----END PGP SIGNATURE----- --4SFOXa2GPu3tIq4H-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/