[PATCH] mfd: da9063: Allow modular build

[PATCH] mfd: da9063: Allow modular build

[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

Re: [PATCH] mfd: da9063: Allow modular build

[Lee Jones]

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

RE: [PATCH] mfd: da9063: Allow modular build

[Opensource [Steve Twiss]]

[-- 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¥

RE: [PATCH] mfd: da9063: Allow modular build

[Opensource [Steve Twiss]]

[-- 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¥

Re: [PATCH] mfd: da9063: Allow modular build

[Lee Jones]

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