* [PATCH] mfd: da9063: Allow modular build
@ 2015-12-07 16:16 Geert Uytterhoeven
2015-12-08 13:56 ` Lee Jones
2016-01-11 6:41 ` Lee Jones
0 siblings, 2 replies; 5+ messages in thread
From: Geert Uytterhoeven @ 2015-12-07 16:16 UTC (permalink / raw)
To: Lee Jones, Support Opensource
Cc: Liam Girdwood, Mark Brown, linux-kernel, Geert Uytterhoeven
Allow support for the DA9063 PMIC to be modular, cfr. DA9062, which
allows to decrease size of multi-platform kernels (e.g.
multi_v7_defconfig).
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
drivers/mfd/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
index 4d92df6ef9fe92ac..527ee7168617c933 100644
--- a/drivers/mfd/Kconfig
+++ b/drivers/mfd/Kconfig
@@ -211,7 +211,7 @@ config MFD_DA9062
of the device.
config MFD_DA9063
- bool "Dialog Semiconductor DA9063 PMIC Support"
+ tristate "Dialog Semiconductor DA9063 PMIC Support"
select MFD_CORE
select REGMAP_I2C
select REGMAP_IRQ
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] mfd: da9063: Allow modular build
2015-12-07 16:16 [PATCH] mfd: da9063: Allow modular build Geert Uytterhoeven
@ 2015-12-08 13:56 ` Lee Jones
2015-12-08 14:53 ` Opensource [Steve Twiss]
2015-12-17 13:26 ` Opensource [Steve Twiss]
2016-01-11 6:41 ` Lee Jones
1 sibling, 2 replies; 5+ messages in thread
From: Lee Jones @ 2015-12-08 13:56 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Support Opensource, Liam Girdwood, Mark Brown, linux-kernel
On Mon, 07 Dec 2015, Geert Uytterhoeven wrote:
> Allow support for the DA9063 PMIC to be modular, cfr. DA9062, which
> allows to decrease size of multi-platform kernels (e.g.
> multi_v7_defconfig).
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> drivers/mfd/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
The DA9063 driver appears to contain all module code. In fact, I'm
surprised that it wasn't enabled for tristate in the first place.
In any case, it would be good to get a Reviewed/Acked-by from one of
the Dialog guys.
> diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
> index 4d92df6ef9fe92ac..527ee7168617c933 100644
> --- a/drivers/mfd/Kconfig
> +++ b/drivers/mfd/Kconfig
> @@ -211,7 +211,7 @@ config MFD_DA9062
> of the device.
>
> config MFD_DA9063
> - bool "Dialog Semiconductor DA9063 PMIC Support"
> + tristate "Dialog Semiconductor DA9063 PMIC Support"
> select MFD_CORE
> select REGMAP_I2C
> select REGMAP_IRQ
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH] mfd: da9063: Allow modular build
2015-12-08 13:56 ` Lee Jones
@ 2015-12-08 14:53 ` Opensource [Steve Twiss]
2015-12-17 13:26 ` Opensource [Steve Twiss]
1 sibling, 0 replies; 5+ messages in thread
From: Opensource [Steve Twiss] @ 2015-12-08 14:53 UTC (permalink / raw)
To: Lee Jones, Geert Uytterhoeven
Cc: Support Opensource, Liam Girdwood, Mark Brown, linux-kernel,
David Dajun Chen
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 1483 bytes --]
On 08 December 2015 @ 13:56 Lee Jones wrote:
> To: Geert Uytterhoeven
> Cc: Support Opensource; Liam Girdwood; Mark Brown; linux-
> kernel@vger.kernel.org
> Subject: Re: [PATCH] mfd: da9063: Allow modular build
>
> On Mon, 07 Dec 2015, Geert Uytterhoeven wrote:
>
> > Allow support for the DA9063 PMIC to be modular, cfr. DA9062, which
> > allows to decrease size of multi-platform kernels (e.g.
> > multi_v7_defconfig).
> >
> > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > ---
> > drivers/mfd/Kconfig | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
>
> The DA9063 driver appears to contain all module code. In fact, I'm
> surprised that it wasn't enabled for tristate in the first place.
>
> In any case, it would be good to get a Reviewed/Acked-by from one of
> the Dialog guys.
>
Hi Lee; Geert,
I'll take a look at this.
Regards,
Steve
> > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
> > index 4d92df6ef9fe92ac..527ee7168617c933 100644
> > --- a/drivers/mfd/Kconfig
> > +++ b/drivers/mfd/Kconfig
> > @@ -211,7 +211,7 @@ config MFD_DA9062
> > of the device.
> >
> > config MFD_DA9063
> > - bool "Dialog Semiconductor DA9063 PMIC Support"
> > + tristate "Dialog Semiconductor DA9063 PMIC Support"
> > select MFD_CORE
> > select REGMAP_I2C
> > select REGMAP_IRQ
ÿôèº{.nÇ+‰·Ÿ®‰†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a¢f£¢·hšïêÿ‘êçz_è®\x03(階ŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨èÚ&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥
^ permalink raw reply [flat|nested] 5+ messages in thread* RE: [PATCH] mfd: da9063: Allow modular build
2015-12-08 13:56 ` Lee Jones
2015-12-08 14:53 ` Opensource [Steve Twiss]
@ 2015-12-17 13:26 ` Opensource [Steve Twiss]
1 sibling, 0 replies; 5+ messages in thread
From: Opensource [Steve Twiss] @ 2015-12-17 13:26 UTC (permalink / raw)
To: Lee Jones, Geert Uytterhoeven
Cc: Support Opensource, Liam Girdwood, Mark Brown, linux-kernel
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 1704 bytes --]
On 08 December 2015 13:56, Lee Jones wrote:
> To: Geert Uytterhoeven
> Subject: Re: [PATCH] mfd: da9063: Allow modular build
>
> On Mon, 07 Dec 2015, Geert Uytterhoeven wrote:
>
> > Allow support for the DA9063 PMIC to be modular, cfr. DA9062, which
> > allows to decrease size of multi-platform kernels (e.g.
> > multi_v7_defconfig).
> >
> > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > ---
> > drivers/mfd/Kconfig | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
>
> The DA9063 driver appears to contain all module code. In fact, I'm
> surprised that it wasn't enabled for tristate in the first place.
>
> In any case, it would be good to get a Reviewed/Acked-by from one of
> the Dialog guys.
Yes.
Without this we cannot load any of the sub-devices as modules for the DA9063.
Sorry this has taken so long for me to look at.
Acked-by: Steve Twiss <stwiss.opensource@diasemi.com>
Regards,
Steve
>
> > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
> > index 4d92df6ef9fe92ac..527ee7168617c933 100644
> > --- a/drivers/mfd/Kconfig
> > +++ b/drivers/mfd/Kconfig
> > @@ -211,7 +211,7 @@ config MFD_DA9062
> > of the device.
> >
> > config MFD_DA9063
> > - bool "Dialog Semiconductor DA9063 PMIC Support"
> > + tristate "Dialog Semiconductor DA9063 PMIC Support"
> > select MFD_CORE
> > select REGMAP_I2C
> > select REGMAP_IRQ
>
> --
> Lee Jones
> Linaro STMicroelectronics Landing Team Lead
> Linaro.org │ Open source software for ARM SoCs
> Follow Linaro: Facebook | Twitter | Blog
ÿôèº{.nÇ+‰·Ÿ®‰†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a¢f£¢·hšïêÿ‘êçz_è®\x03(階ŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨èÚ&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] mfd: da9063: Allow modular build
2015-12-07 16:16 [PATCH] mfd: da9063: Allow modular build Geert Uytterhoeven
2015-12-08 13:56 ` Lee Jones
@ 2016-01-11 6:41 ` Lee Jones
1 sibling, 0 replies; 5+ messages in thread
From: Lee Jones @ 2016-01-11 6:41 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Support Opensource, Liam Girdwood, Mark Brown, linux-kernel
On Mon, 07 Dec 2015, Geert Uytterhoeven wrote:
> Allow support for the DA9063 PMIC to be modular, cfr. DA9062, which
> allows to decrease size of multi-platform kernels (e.g.
> multi_v7_defconfig).
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> drivers/mfd/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Applied, thanks.
> diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
> index 4d92df6ef9fe92ac..527ee7168617c933 100644
> --- a/drivers/mfd/Kconfig
> +++ b/drivers/mfd/Kconfig
> @@ -211,7 +211,7 @@ config MFD_DA9062
> of the device.
>
> config MFD_DA9063
> - bool "Dialog Semiconductor DA9063 PMIC Support"
> + tristate "Dialog Semiconductor DA9063 PMIC Support"
> select MFD_CORE
> select REGMAP_I2C
> select REGMAP_IRQ
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-01-11 6:41 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-12-07 16:16 [PATCH] mfd: da9063: Allow modular build Geert Uytterhoeven
2015-12-08 13:56 ` Lee Jones
2015-12-08 14:53 ` Opensource [Steve Twiss]
2015-12-17 13:26 ` Opensource [Steve Twiss]
2016-01-11 6:41 ` Lee Jones
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).