Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755188AbbDIMnk (ORCPT ); Thu, 9 Apr 2015 08:43:40 -0400 Received: from mail-ig0-f173.google.com ([209.85.213.173]:33029 "EHLO mail-ig0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755107AbbDIMng (ORCPT ); Thu, 9 Apr 2015 08:43:36 -0400 Date: Thu, 9 Apr 2015 07:43:30 -0500 From: Bjorn Helgaas To: Gabriel Fernandez Cc: Fabrice Gasnier , Gabriel FERNANDEZ , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Srinivas Kandagatla , Maxime Coquelin , Patrice Chotard , Russell King , Mohit Kumar , Jingoo Han , Lucas Stach , Kishon Vijay Abraham I , Andrew Morton , "David S. Miller" , Greg KH , Mauro Carvalho Chehab , Joe Perches , Tejun Heo , Arnd Bergmann , Viresh Kumar , Thierry Reding , Phil Edworthy , Minghuan Lian , Tanmay Inamdar , Muralidharan Karicheri , Sachin Kamat , Andrew Lunn , Liviu Dudau , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "kernel@stlinux.com" , "linux-pci@vger.kernel.org" , Lee Jones , Paul Bolle Subject: Re: [PATCH v2 3/5] PCI: st: Provide support for the sti PCIe controller Message-ID: <20150409124330.GG30967@google.com> References: <1426515635-9466-1-git-send-email-gabriel.fernandez@linaro.org> <1426515635-9466-4-git-send-email-gabriel.fernandez@linaro.org> <1426518695.26437.55.camel@x220> <55093C12.7070004@st.com> <1426674930.2649.50.camel@tiscali.nl> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: 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 Content-Length: 2338 Lines: 62 On Tue, Mar 31, 2015 at 11:11:44AM +0200, Gabriel Fernandez wrote: > Hi Bjorn, > > pci-st.c driver could be modular with modification of pcie-designware > core driver. But as Fabrice said it should be another patchset. > > What do you prefer ? > > drop all the module related macros as mentioned by Paul ? > or > keep macros like other vendors do ? I don't think this is a big deal either way. You made it consistent with the style of the existing PCI host drivers, which is exactly what you should do. If somebody wants to do the work of making them all modular, or leaving them non-modular and removing all the MODULE_*() annotations, that can be done later. > On 18 March 2015 at 11:35, Paul Bolle wrote: > > Hi Fabrice, > > > > Fabrice Gasnier schreef op wo 18-03-2015 om 09:49 [+0100]: > >> On 03/16/2015 04:11 PM, Paul Bolle wrote: > >> >> +config PCI_ST > >> >> + bool "ST PCIe controller" > >> > You add a bool Kconfig symbol. A week or two ago I saw some patches fly > >> > by that - I think - allowed PCIe controllers to be built modular. > >> > >> Thanks for your review. > >> > >> Are you talking about "PCI: Export symbols of PCI functions" patch, that > >> is part of a series > >> named "pci: iproc: Add Broadcom iProc PCIe support" ? > > > > Yes, that is the series I was thinking about. (I made you search lkml, > > and that was a bit rude. But you found the patch anyhow.) > > > >> This controller doesn't look like to be based on pcie-designware core > >> driver. > >> Other vendors that are using "pcie-designware" core driver are also make > >> it bool. > >> The current core driver doesn't support module loading/unloading as I > >> see it. > >> If this is required, I also think this should be part of another patchset. > >> > >> What do you think ? > > > > I wouldn't know whether your driver might work as a loadable module, but > > other people reading this surely will. But if it can't work as a module > > you should drop all the module related macros etc. I spotted. Because > > then they serve no purpose. > > > > > > Paul Bolle > > -- 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/