Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1457087 > unrolled thread
| Started by | Lee Jones <lee.jones@linaro.org> |
|---|---|
| First post | 2016-08-05 15:00 +0200 |
| Last post | 2016-08-08 16:10 +0200 |
| Articles | 4 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH 2/6] mfd: as3722: Make it explicitly non-modular Lee Jones <lee.jones@linaro.org> - 2016-08-05 15:00 +0200
Re: [PATCH 2/6] mfd: as3722: Make it explicitly non-modular Paul Gortmaker <paul.gortmaker@windriver.com> - 2016-08-05 16:00 +0200
Re: [PATCH 2/6] mfd: as3722: Make it explicitly non-modular Lee Jones <lee.jones@linaro.org> - 2016-08-08 11:20 +0200
Re: [PATCH 2/6] mfd: as3722: Make it explicitly non-modular Paul Gortmaker <paul.gortmaker@windriver.com> - 2016-08-08 16:10 +0200
| From | Lee Jones <lee.jones@linaro.org> |
|---|---|
| Date | 2016-08-05 15:00 +0200 |
| Subject | Re: [PATCH 2/6] mfd: as3722: Make it explicitly non-modular |
| Message-ID | <s2MTf-7El-17@gated-at.bofh.it> |
On Wed, 06 Jul 2016, Paul Gortmaker wrote: > [Re: [PATCH 2/6] mfd: as3722: Make it explicitly non-modular] On 05/07/2016 (Tue 11:18) Laxman Dewangan wrote: > > > > > On Tuesday 05 July 2016 06:55 AM, Paul Gortmaker wrote: > > >The Kconfig currently controlling compilation of this code is: > > > > > >drivers/mfd/Kconfig:config MFD_AS3722 > > >drivers/mfd/Kconfig: bool "ams AS3722 Power Management IC" > > > > > > > > > Should we convert this to as "tristate" so that it can be built as module > > also. > > Seems to compile and modpost w/o missing symbols as tristate: > > paul@yow-builder:~/git/linux-head$ ls -l ../arm-build/drivers/mfd/as3722* > -rw-rw-r-- 1 paul paul 13010 Jul 6 12:30 ../arm-build/drivers/mfd/as3722.ko > -rw-rw-r-- 1 paul paul 1937 Jul 6 12:28 ../arm-build/drivers/mfd/as3722.mod.c > -rw-rw-r-- 1 paul paul 3820 Jul 6 12:29 ../arm-build/drivers/mfd/as3722.mod.o > -rw-rw-r-- 1 paul paul 11292 Jul 6 12:13 ../arm-build/drivers/mfd/as3722.o > paul@yow-builder:~/git/linux-head$ > > ...so I'll drop this patch in favour of the one line tristate conversion in v2. Did you ever get around to submitting a v2? -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog
[toc] | [next] | [standalone]
| From | Paul Gortmaker <paul.gortmaker@windriver.com> |
|---|---|
| Date | 2016-08-05 16:00 +0200 |
| Message-ID | <s2NPk-8h7-13@gated-at.bofh.it> |
| In reply to | #1457087 |
[Re: [PATCH 2/6] mfd: as3722: Make it explicitly non-modular] On 05/08/2016 (Fri 13:56) Lee Jones wrote: > On Wed, 06 Jul 2016, Paul Gortmaker wrote: > > > [Re: [PATCH 2/6] mfd: as3722: Make it explicitly non-modular] On 05/07/2016 (Tue 11:18) Laxman Dewangan wrote: > > > > > > > > On Tuesday 05 July 2016 06:55 AM, Paul Gortmaker wrote: > > > >The Kconfig currently controlling compilation of this code is: > > > > > > > >drivers/mfd/Kconfig:config MFD_AS3722 > > > >drivers/mfd/Kconfig: bool "ams AS3722 Power Management IC" > > > > > > > > > > > > > Should we convert this to as "tristate" so that it can be built as module > > > also. > > > > Seems to compile and modpost w/o missing symbols as tristate: > > > > paul@yow-builder:~/git/linux-head$ ls -l ../arm-build/drivers/mfd/as3722* > > -rw-rw-r-- 1 paul paul 13010 Jul 6 12:30 ../arm-build/drivers/mfd/as3722.ko > > -rw-rw-r-- 1 paul paul 1937 Jul 6 12:28 ../arm-build/drivers/mfd/as3722.mod.c > > -rw-rw-r-- 1 paul paul 3820 Jul 6 12:29 ../arm-build/drivers/mfd/as3722.mod.o > > -rw-rw-r-- 1 paul paul 11292 Jul 6 12:13 ../arm-build/drivers/mfd/as3722.o > > paul@yow-builder:~/git/linux-head$ > > > > ...so I'll drop this patch in favour of the one line tristate conversion in v2. > > Did you ever get around to submitting a v2? I made the v2 (as per above build stats) but never sent it out since I wasn't sure if mfd wanted a complete resend (like davem/linux-net) or just resends of the revised content... and then I never came back to it. Here is the one line trivial patch. If you want to punt this to 4.9 given the current timing wrt. the merge window, feel free to do so; and/or I can re-send it myself with other pending 4.9 content I have. Thanks, Paul. -- From 97a056efc6404f612ab83ea900e1aae2cbcac4a6 Mon Sep 17 00:00:00 2001 From: Paul Gortmaker <paul.gortmaker@windriver.com> Date: Tue, 8 Sep 2015 20:53:04 -0400 Subject: [PATCH] mfd: as3722: convert MFD_AS3722 from bool to tristate The Kconfig currently controlling compilation of this code is: drivers/mfd/Kconfig:config MFD_AS3722 drivers/mfd/Kconfig: bool "ams AS3722 Power Management IC" ...meaning that it currently is not being built as a module by anyone. In response to an earlier patch set suggesting removal of the unused modular code, Laxman suggested that this driver be instead moved to tristate. We do that here, and confirm it can build and modpost as a tristate. However there remains to be runtime testing in order to ensure this change is 100% functional for "=m". Cc: Samuel Ortiz <sameo@linux.intel.com> Cc: Lee Jones <lee.jones@linaro.org> Cc: Florian Lobmaier <florian.lobmaier@ams.com> Cc: Laxman Dewangan <ldewangan@nvidia.com> Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> --- drivers/mfd/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig index 2d1fb6420592..0a0ac8c8c14c 100644 --- a/drivers/mfd/Kconfig +++ b/drivers/mfd/Kconfig @@ -50,7 +50,7 @@ config MFD_AS3711 Support for the AS3711 PMIC from AMS config MFD_AS3722 - bool "ams AS3722 Power Management IC" + tristate "ams AS3722 Power Management IC" select MFD_CORE select REGMAP_I2C select REGMAP_IRQ -- 2.8.4
[toc] | [prev] | [next] | [standalone]
| From | Lee Jones <lee.jones@linaro.org> |
|---|---|
| Date | 2016-08-08 11:20 +0200 |
| Message-ID | <s3OSZ-7Ux-3@gated-at.bofh.it> |
| In reply to | #1457115 |
On Fri, 05 Aug 2016, Paul Gortmaker wrote: > [Re: [PATCH 2/6] mfd: as3722: Make it explicitly non-modular] On 05/08/2016 (Fri 13:56) Lee Jones wrote: > > > On Wed, 06 Jul 2016, Paul Gortmaker wrote: > > > > > [Re: [PATCH 2/6] mfd: as3722: Make it explicitly non-modular] On 05/07/2016 (Tue 11:18) Laxman Dewangan wrote: > > > > > > > > > > > On Tuesday 05 July 2016 06:55 AM, Paul Gortmaker wrote: > > > > >The Kconfig currently controlling compilation of this code is: > > > > > > > > > >drivers/mfd/Kconfig:config MFD_AS3722 > > > > >drivers/mfd/Kconfig: bool "ams AS3722 Power Management IC" > > > > > > > > > > > > > > > > > Should we convert this to as "tristate" so that it can be built as module > > > > also. > > > > > > Seems to compile and modpost w/o missing symbols as tristate: > > > > > > paul@yow-builder:~/git/linux-head$ ls -l ../arm-build/drivers/mfd/as3722* > > > -rw-rw-r-- 1 paul paul 13010 Jul 6 12:30 ../arm-build/drivers/mfd/as3722.ko > > > -rw-rw-r-- 1 paul paul 1937 Jul 6 12:28 ../arm-build/drivers/mfd/as3722.mod.c > > > -rw-rw-r-- 1 paul paul 3820 Jul 6 12:29 ../arm-build/drivers/mfd/as3722.mod.o > > > -rw-rw-r-- 1 paul paul 11292 Jul 6 12:13 ../arm-build/drivers/mfd/as3722.o > > > paul@yow-builder:~/git/linux-head$ > > > > > > ...so I'll drop this patch in favour of the one line tristate conversion in v2. > > > > Did you ever get around to submitting a v2? > > I made the v2 (as per above build stats) but never sent it out since I > wasn't sure if mfd wanted a complete resend (like davem/linux-net) or > just resends of the revised content... and then I never came back to it. > > Here is the one line trivial patch. If you want to punt this to 4.9 > given the current timing wrt. the merge window, feel free to do so; > and/or I can re-send it myself with other pending 4.9 content I have. From a personal PoV, I'd prefer if you just submitted patches normally. I'll field them when they land in my Inbox. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog
[toc] | [prev] | [next] | [standalone]
| From | Paul Gortmaker <paul.gortmaker@windriver.com> |
|---|---|
| Date | 2016-08-08 16:10 +0200 |
| Message-ID | <s3TpD-2p3-5@gated-at.bofh.it> |
| In reply to | #1457632 |
[Re: [PATCH 2/6] mfd: as3722: Make it explicitly non-modular] On 08/08/2016 (Mon 10:14) Lee Jones wrote: > On Fri, 05 Aug 2016, Paul Gortmaker wrote: D> [...] > > > > > > > > ...so I'll drop this patch in favour of the one line tristate conversion in v2. > > > > > > Did you ever get around to submitting a v2? > > > > I made the v2 (as per above build stats) but never sent it out since I > > wasn't sure if mfd wanted a complete resend (like davem/linux-net) or > > just resends of the revised content... and then I never came back to it. > > > > Here is the one line trivial patch. If you want to punt this to 4.9 > > given the current timing wrt. the merge window, feel free to do so; > > and/or I can re-send it myself with other pending 4.9 content I have. > > From a personal PoV, I'd prefer if you just submitted patches > normally. I'll field them when they land in my Inbox. Yep, understood. I just didn't want to lose the context of the above discussion wrt. this one-line patch. Now that rc1 was done last night, pending mfd changes will be the normal 0/N followed by the patches; all sent via git send-email as usual. Thanks, Paul. -- > > -- > Lee Jones > Linaro STMicroelectronics Landing Team Lead > Linaro.org │ Open source software for ARM SoCs > Follow Linaro: Facebook | Twitter | Blog
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web