2009-07-02 15:43:22

by Mark Brown

[permalink] [raw]
Subject: [PATCH] MAINTAINERS: Add entry for Wolfson PMIC drivers

Signed-off-by: Mark Brown <[email protected]>
---
MAINTAINERS | 18 ++++++++++++++++++
1 files changed, 18 insertions(+), 0 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index cf4abdd..b9a1e2d 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -6278,6 +6278,24 @@ S: Supported
F: drivers/input/touchscreen/*wm97*
F: include/linux/wm97xx.h

+WOLFSON MICROELECTRONICS PMIC DRIVERS
+P: Mark Brown
+M: [email protected]
+L: [email protected]
+T: git git://opensource.wolfsonmicro.com/linux-2.6-audioplus
+W: http://opensource.wolfsonmicro.com/node/8
+S: Supported
+F: drivers/leds/leds-wm83*.c
+F: drivers/mfd/wm8*.c
+F: drivers/power/wm83*.c
+F: drivers/rtc/rtc-wm83*.c
+F: drivers/regulator/wm8*.c
+F: drivers/watchdog/wm83*_wdt.c
+F: include/linux/mfd/wm8350/
+F: include/linux/mfd/wm8400/
+F: sound/soc/codecs/wm8350.c
+F: sound/soc/codecs/wm8400.c
+
X.25 NETWORK LAYER
P: Henner Eisen
M: [email protected]
--
1.6.3.3


2009-07-02 16:08:34

by Joe Perches

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Add entry for Wolfson PMIC drivers

On Thu, 2009-07-02 at 16:43 +0100, Mark Brown wrote:
> Signed-off-by: Mark Brown <[email protected]>

Hi Mark.

> diff --git a/MAINTAINERS b/MAINTAINERS

[]

> +F: drivers/leds/leds-wm83*.c
> +F: drivers/mfd/wm8*.c
> +F: drivers/power/wm83*.c
> +F: drivers/rtc/rtc-wm83*.c
> +F: drivers/regulator/wm8*.c
> +F: drivers/watchdog/wm83*_wdt.c

Maybe this pattern is complete enough?

F: drivers/*/*wm8*.c

> +F: sound/soc/codecs/wm8350.c
> +F: sound/soc/codecs/wm8400.c

What about the other wm8 codecs? Maybe:

F: sound/soc/codecs/wm8* ?

2009-07-02 17:36:40

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Add entry for Wolfson PMIC drivers

On Thu, Jul 02, 2009 at 09:08:23AM -0700, Joe Perches wrote:
> On Thu, 2009-07-02 at 16:43 +0100, Mark Brown wrote:

> > +F: drivers/leds/leds-wm83*.c
> > +F: drivers/mfd/wm8*.c
> > +F: drivers/power/wm83*.c
> > +F: drivers/rtc/rtc-wm83*.c
> > +F: drivers/regulator/wm8*.c
> > +F: drivers/watchdog/wm83*_wdt.c

> Maybe this pattern is complete enough?

> F: drivers/*/*wm8*.c

That misses the LED, RTC and watchdog drivers. Making the match more
general with a leading * in the filename is just asking for false
positives.

> > +F: sound/soc/codecs/wm8350.c
> > +F: sound/soc/codecs/wm8400.c

> What about the other wm8 codecs? Maybe:

Those are not PMICs and therefore not relevant here. Since I am the
most active maintainer for the ASoC subsystem at the minute a separate
entry for the drivers within it would be a bit redundant. The CODEC
drivers for the PMICs are included here for completeness more than
anything else.

2009-07-03 10:47:22

by Samuel Ortiz

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Add entry for Wolfson PMIC drivers

On Thu, Jul 02, 2009 at 04:43:08PM +0100, Mark Brown wrote:
> Signed-off-by: Mark Brown <[email protected]>
Applied, thanks Mark.

Cheers,
Samuel.

> ---
> MAINTAINERS | 18 ++++++++++++++++++
> 1 files changed, 18 insertions(+), 0 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index cf4abdd..b9a1e2d 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -6278,6 +6278,24 @@ S: Supported
> F: drivers/input/touchscreen/*wm97*
> F: include/linux/wm97xx.h
>
> +WOLFSON MICROELECTRONICS PMIC DRIVERS
> +P: Mark Brown
> +M: [email protected]
> +L: [email protected]
> +T: git git://opensource.wolfsonmicro.com/linux-2.6-audioplus
> +W: http://opensource.wolfsonmicro.com/node/8
> +S: Supported
> +F: drivers/leds/leds-wm83*.c
> +F: drivers/mfd/wm8*.c
> +F: drivers/power/wm83*.c
> +F: drivers/rtc/rtc-wm83*.c
> +F: drivers/regulator/wm8*.c
> +F: drivers/watchdog/wm83*_wdt.c
> +F: include/linux/mfd/wm8350/
> +F: include/linux/mfd/wm8400/
> +F: sound/soc/codecs/wm8350.c
> +F: sound/soc/codecs/wm8400.c
> +
> X.25 NETWORK LAYER
> P: Henner Eisen
> M: [email protected]
> --
> 1.6.3.3
>

--
Intel Open Source Technology Centre
http://oss.intel.com/