From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755469AbcDDNwI (ORCPT ); Mon, 4 Apr 2016 09:52:08 -0400 Received: from mail1.windriver.com ([147.11.146.13]:61697 "EHLO mail1.windriver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752017AbcDDNwG (ORCPT ); Mon, 4 Apr 2016 09:52:06 -0400 Date: Mon, 4 Apr 2016 09:51:46 -0400 From: Paul Gortmaker To: Lee Jones CC: , Chanwoo Choi , Haojian Zhuang , Krzysztof Kozlowski , Kyungmin Park , MyungJoo Ham , Samuel Ortiz Subject: Re: [PATCH 0/7] drivers/mfd: make max drivers explicitly non-modular Message-ID: <20160404135146.GF1778@windriver.com> References: <1459718659-28316-1-git-send-email-paul.gortmaker@windriver.com> <20160404081149.GU3323@x1> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20160404081149.GU3323@x1> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org [Re: [PATCH 0/7] drivers/mfd: make max drivers explicitly non-modular] On 04/04/2016 (Mon 09:11) Lee Jones wrote: > How did you come up with the subject lines for these patches? I just used the general default of: : rest of subject > > Please do `git log --oneline -- `, as is normal when > submitting to mainline. Just to be clear, looking at other mfd history, you want me to just drop the "drivers/" prefix ; i.e. change from: drivers/mfd: text text text driver.c text ...to: mfd: text text text driver.c text If so, that is no problem and I can do that for the next revision when I drop the patches pending tristate conversion that I wasn't aware of. Thanks, Paul. -- > > > For anyone new to the underlying goal of this cleanup, we are trying to > > make driver code consistent with the Makefiles/Kconfigs that control them. > > > > This means not using modular functions/macros for drivers that can never > > be built as a module. Some of the other downfalls this leads to are: > > > > (1) it is easy to accidentally write unused module_exit and remove code > > (2) it can be misleading when reading the source, thinking it can be > > modular when the Makefile and/or Kconfig prohibit it > > (3) it requires the include of the module.h header file which in turn > > includes nearly everything else, thus adding to CPP overhead. > > (4) it gets copied/replicated into other drivers and spreads like weeds. > > > > There are quite a few in the mfd space, so rather than send a large > > series, I'll try and batch them up into reasonable sized queues. To that > > end, here we fix up all the maxim drivers as the 1st mfd batch. > > > > As always, the option exists for someone with the hardware and the desire > > to extend the functionality to make any given driver tristate. But given > > the number of these tree wide and the fact that I can't test that new > > extended functionality in all cases, I just make the code consistent with > > the existing Kconfig/Makefile settings that restrict them to "bool". > > > > Build tested on linux-next for arm, arm64 and x86-64 to ensure no typos > > or similar issues crept in. > > > > Paul. > > --- > > > > Cc: Chanwoo Choi > > Cc: Haojian Zhuang > > Cc: Krzysztof Kozlowski > > Cc: Kyungmin Park > > Cc: Lee Jones > > Cc: MyungJoo Ham > > Cc: Samuel Ortiz > > > > Paul Gortmaker (7): > > drivers/mfd: make max8925-i2c.c explicitly non-modular > > drivers/mfd: make max8997.c explicitly non-modular > > drivers/mfd: make max8998.c explicitly non-modular > > drivers/mfd: make max14577.c explicitly non-modular > > drivers/mfd: make max77686.c explicitly non-modular > > drivers/mfd: make max77693.c explicitly non-modular > > drivers/mfd: make max77843.c explicitly non-modular > > > > drivers/mfd/max14577.c | 13 +------------ > > drivers/mfd/max77686.c | 26 ++------------------------ > > drivers/mfd/max77693.c | 34 +++------------------------------- > > drivers/mfd/max77843.c | 24 ++---------------------- > > drivers/mfd/max8925-i2c.c | 14 +------------- > > drivers/mfd/max8997.c | 30 +++--------------------------- > > drivers/mfd/max8998.c | 27 +-------------------------- > > 7 files changed, 13 insertions(+), 155 deletions(-) > > > > -- > Lee Jones > Linaro STMicroelectronics Landing Team Lead > Linaro.org │ Open source software for ARM SoCs > Follow Linaro: Facebook | Twitter | Blog