From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932820AbcHVTod (ORCPT ); Mon, 22 Aug 2016 15:44:33 -0400 Received: from mail.kernel.org ([198.145.29.136]:45794 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932791AbcHVTob (ORCPT ); Mon, 22 Aug 2016 15:44:31 -0400 Date: Mon, 22 Aug 2016 14:44:27 -0500 From: Bjorn Helgaas To: Paul Gortmaker Cc: Keith Busch , linux-kernel@vger.kernel.org, Bjorn Helgaas , Mika Westerberg , linux-pci@vger.kernel.org Subject: Re: [PATCH -next] PCI/PCIe: make DPC explicitly non-modular Message-ID: <20160822194427.GM18628@localhost> References: <20160728193303.12945-1-paul.gortmaker@windriver.com> <20160728202859.GA2420@localhost.localdomain> <20160728212801.GY11410@windriver.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160728212801.GY11410@windriver.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Paul, On Thu, Jul 28, 2016 at 05:28:02PM -0400, Paul Gortmaker wrote: > [Re: [PATCH -next] PCI/PCIe: make DPC explicitly non-modular] On 28/07/2016 (Thu 16:29) Keith Busch wrote: > > > On Thu, Jul 28, 2016 at 03:33:03PM -0400, Paul Gortmaker wrote: > > > The Kconfig currently controlling compilation of this code is: > > > > > > drivers/pci/pcie/Kconfig:config PCIE_DPC > > > drivers/pci/pcie/Kconfig: bool "PCIe Downstream Port Containment support" > > > > > > ...meaning that it currently is not being built as a module by anyone. > > > > > > Lets remove the modular code that is essentially orphaned, so that > > > when reading the driver there is no doubt it is builtin-only. > > > > > > Since module_init translates to device_initcall in the non-modular > > > case, the init ordering remains unchanged with this commit. > > > > > > 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. > > > > Thanks for cleaning this up. > > > > FWIW, all of the other pcie service drivers look like they could use > > the same cleanup. > > Yep, not to worry, I've still got the following patches pending for-4.9: > > $ grep PCI series > # PCI > PCI-aerdrv-make-it-explicitly-non-modular.patch > PCI-altera-make-it-explicitly-non-modular.patch > PCI-dra7xx-make-it-explicitly-non-modular.patch > PCI-imx6-make-it-explicitly-non-modular.patch > PCI-pme-make-it-explicitly-non-modular.patch > PCI-qcom-make-it-explicitly-non-modular.patch > PCI-spear13xx-make-it-explicitly-non-modular.patch > PCI-xilinx-make-it-explicitly-non-modular.patch > PCI-xilinx-nwl-make-it-eplicitly-non-modular.patch > PCI-portdrv-make-it-explicitly-non-modular.patch > PCI-PCIe-make-dpc-explicitly-non-modular.patch > PCI-generic-make-host-common-explicitly-non-modular.patch > PCI-hotplug_core-make-it-explicitly-non-modular.patch > PCI-hotplug-make-core-code-explicitly-non-modular.patch Were you planning to post the patches above? If so, I'll bundle them all up along with this DPC one. Bjorn