linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/2] MAINTAINER entries for few ROHM power devices
@ 2020-05-20  6:10 Matti Vaittinen
  2020-05-20  6:11 ` [PATCH 1/2] MAINTAINERS: Add entry for ROHM power management ICs Matti Vaittinen
                   ` (2 more replies)
  0 siblings, 3 replies; 8+ messages in thread
From: Matti Vaittinen @ 2020-05-20  6:10 UTC (permalink / raw)
  To: matti.vaittinen, mazziesaccount
  Cc: Michael Turquette, SebastianReichel, Liam Girdwood, Mark Brown,
	Stephen Boyd, Wim Van Sebroeck, Guenter Roeck, Lee Jones,
	Linus Walleij, Bartosz Golaszewski, Alessandro Zummo,
	Alexandre Belloni, Rob Herring, linux-kernel

Add maintainer entries to a few ROHM devices and Linear Ranges

Linear Ranges helpers were refactored out of regulator core to lib so
that other drivers could utilize them too. (I guess power/supply drivers 
and possibly clk drivers can benefit from them). As regulators is
currently the main user it makes sense the changes to linear_ranges go
through Mark's tree.

During past two years few ROHM PMIC drivers have been added to
mainstream. They deserve a supporter from ROHM side too :)

Patch 1:
	Maintainer entries for few ROHM IC drivers
Patch 2:
	Maintainer entry for linear ranges helpers

---

Matti Vaittinen (2):
  MAINTAINERS: Add entry for ROHM power management ICs
  MAINTAINERS: Add maintainer entry for linear ranges helper

 MAINTAINERS | 37 +++++++++++++++++++++++++++++++++++++
 1 file changed, 37 insertions(+)


base-commit: b9bbe6ed63b2b9f2c9ee5cbd0f2c946a2723f4ce
-- 
2.21.0


-- 
Matti Vaittinen, Linux device drivers
ROHM Semiconductors, Finland SWDC
Kiviharjunlenkki 1E
90220 OULU
FINLAND

~~~ "I don't think so," said Rene Descartes. Just then he vanished ~~~
Simon says - in Latin please.
~~~ "non cogito me" dixit Rene Descarte, deinde evanescavit ~~~
Thanks to Simon Glass for the translation =] 

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH 1/2] MAINTAINERS: Add entry for ROHM power management ICs
  2020-05-20  6:10 [PATCH 0/2] MAINTAINER entries for few ROHM power devices Matti Vaittinen
@ 2020-05-20  6:11 ` Matti Vaittinen
  2020-05-20  6:35   ` Lee Jones
  2020-05-20 10:52   ` Sebastian Reichel
  2020-05-20  6:12 ` [PATCH 2/2] MAINTAINERS: Add maintainer entry for linear ranges helper Matti Vaittinen
  2020-05-20 17:17 ` [PATCH 0/2] MAINTAINER entries for few ROHM power devices Mark Brown
  2 siblings, 2 replies; 8+ messages in thread
From: Matti Vaittinen @ 2020-05-20  6:11 UTC (permalink / raw)
  To: matti.vaittinen, mazziesaccount
  Cc: Michael Turquette, SebastianReichel, Liam Girdwood, Mark Brown,
	Stephen Boyd, Wim Van Sebroeck, Guenter Roeck, Lee Jones,
	Linus Walleij, Bartosz Golaszewski, Alessandro Zummo,
	Alexandre Belloni, Rob Herring, linux-kernel

Add entry for maintaining power management IC drivers for ROHM
BD71837, BD71847, BD71850, BD71828, BD71878, BD70528 and BD99954.

Signed-off-by: Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
---
 MAINTAINERS | 30 ++++++++++++++++++++++++++++++
 1 file changed, 30 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index ecc0749810b0..63a2ca70540e 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -14490,6 +14490,12 @@ L:	linux-serial@vger.kernel.org
 S:	Odd Fixes
 F:	drivers/tty/serial/rp2.*
 
+ROHM BD99954 CHARGER IC
+R:	Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
+S:	Supported
+F:	drivers/power/supply/bd99954-charger.c
+F:	drivers/power/supply/bd99954-charger.h
+
 ROHM BH1750 AMBIENT LIGHT SENSOR DRIVER
 M:	Tomasz Duszynski <tduszyns@gmail.com>
 S:	Maintained
@@ -14507,6 +14513,30 @@ F:	drivers/mfd/bd9571mwv.c
 F:	drivers/regulator/bd9571mwv-regulator.c
 F:	include/linux/mfd/bd9571mwv.h
 
+ROHM POWER MANAGEMENT IC DEVICE DRIVERS
+R:	Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
+S:	Supported
+F:	Documentation/devicetree/bindings/mfd/rohm,bd70528-pmic.txt
+F:	Documentation/devicetree/bindings/regulator/rohm,bd70528-regulator.txt
+F:	drivers/clk/clk-bd718x7.c
+F:	drivers/gpio/gpio-bd70528.c
+F:	drivers/gpio/gpio-bd71828.c
+F:	drivers/mfd/rohm-bd70528.c
+F:	drivers/mfd/rohm-bd71828.c
+F:	drivers/mfd/rohm-bd718x7.c
+F:	drivers/power/supply/bd70528-charger.c
+F:	drivers/regulator/bd70528-regulator.c
+F:	drivers/regulator/bd71828-regulator.c
+F:	drivers/regulator/bd718x7-regulator.c
+F:	drivers/regulator/rohm-regulator.c
+F:	drivers/rtc/rtc-bd70528.c
+F:	drivers/watchdog/bd70528_wdt.c
+F:	include/linux/mfd/rohm-shared.h
+F:	include/linux/mfd/rohm-bd71828.h
+F:	include/linux/mfd/rohm-bd70528.h
+F:	include/linux/mfd/rohm-generic.h
+F:	include/linux/mfd/rohm-bd718x7.h
+
 ROSE NETWORK LAYER
 M:	Ralf Baechle <ralf@linux-mips.org>
 L:	linux-hams@vger.kernel.org
-- 
2.21.0


-- 
Matti Vaittinen, Linux device drivers
ROHM Semiconductors, Finland SWDC
Kiviharjunlenkki 1E
90220 OULU
FINLAND

~~~ "I don't think so," said Rene Descartes. Just then he vanished ~~~
Simon says - in Latin please.
~~~ "non cogito me" dixit Rene Descarte, deinde evanescavit ~~~
Thanks to Simon Glass for the translation =] 

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [PATCH 2/2] MAINTAINERS: Add maintainer entry for linear ranges helper
  2020-05-20  6:10 [PATCH 0/2] MAINTAINER entries for few ROHM power devices Matti Vaittinen
  2020-05-20  6:11 ` [PATCH 1/2] MAINTAINERS: Add entry for ROHM power management ICs Matti Vaittinen
@ 2020-05-20  6:12 ` Matti Vaittinen
  2020-05-20 17:17 ` [PATCH 0/2] MAINTAINER entries for few ROHM power devices Mark Brown
  2 siblings, 0 replies; 8+ messages in thread
From: Matti Vaittinen @ 2020-05-20  6:12 UTC (permalink / raw)
  To: matti.vaittinen, mazziesaccount
  Cc: Michael Turquette, SebastianReichel, Liam Girdwood, Mark Brown,
	Stephen Boyd, Wim Van Sebroeck, Guenter Roeck, Lee Jones,
	Linus Walleij, Bartosz Golaszewski, Alessandro Zummo,
	Alexandre Belloni, Rob Herring, linux-kernel

The linear ranges helpers were refactored out of regulator core
for other drivers to enjoy. Add regulator maintainer Mark Brown as
maintainer and myself as a reviewer.

Signed-off-by: Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
---
 MAINTAINERS | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 63a2ca70540e..e103e7db1522 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -9720,6 +9720,13 @@ F:	drivers/lightnvm/
 F:	include/linux/lightnvm.h
 F:	include/uapi/linux/lightnvm.h
 
+LINEAR RANGES HELPERS
+M:	Mark Brown <broonie@kernel.org>
+R:	Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
+F:	lib/linear_ranges.c
+F:	lib/test_linear_ranges.c
+F:	include/linux/linear_range.h
+
 LINUX FOR POWER MACINTOSH
 M:	Benjamin Herrenschmidt <benh@kernel.crashing.org>
 L:	linuxppc-dev@lists.ozlabs.org
-- 
2.21.0


-- 
Matti Vaittinen, Linux device drivers
ROHM Semiconductors, Finland SWDC
Kiviharjunlenkki 1E
90220 OULU
FINLAND

~~~ "I don't think so," said Rene Descartes. Just then he vanished ~~~
Simon says - in Latin please.
~~~ "non cogito me" dixit Rene Descarte, deinde evanescavit ~~~
Thanks to Simon Glass for the translation =] 

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] MAINTAINERS: Add entry for ROHM power management ICs
  2020-05-20  6:11 ` [PATCH 1/2] MAINTAINERS: Add entry for ROHM power management ICs Matti Vaittinen
@ 2020-05-20  6:35   ` Lee Jones
  2020-05-20  7:07     ` Vaittinen, Matti
  2020-05-20 10:52   ` Sebastian Reichel
  1 sibling, 1 reply; 8+ messages in thread
From: Lee Jones @ 2020-05-20  6:35 UTC (permalink / raw)
  To: Matti Vaittinen
  Cc: mazziesaccount, Michael Turquette, SebastianReichel,
	Liam Girdwood, Mark Brown, Stephen Boyd, Wim Van Sebroeck,
	Guenter Roeck, Linus Walleij, Bartosz Golaszewski,
	Alessandro Zummo, Alexandre Belloni, Rob Herring, linux-kernel

On Wed, 20 May 2020, Matti Vaittinen wrote:

> Add entry for maintaining power management IC drivers for ROHM
> BD71837, BD71847, BD71850, BD71828, BD71878, BD70528 and BD99954.
> 
> Signed-off-by: Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
> ---
>  MAINTAINERS | 30 ++++++++++++++++++++++++++++++
>  1 file changed, 30 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index ecc0749810b0..63a2ca70540e 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -14490,6 +14490,12 @@ L:	linux-serial@vger.kernel.org
>  S:	Odd Fixes
>  F:	drivers/tty/serial/rp2.*
>  
> +ROHM BD99954 CHARGER IC
> +R:	Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
> +S:	Supported
> +F:	drivers/power/supply/bd99954-charger.c
> +F:	drivers/power/supply/bd99954-charger.h
> +
>  ROHM BH1750 AMBIENT LIGHT SENSOR DRIVER
>  M:	Tomasz Duszynski <tduszyns@gmail.com>
>  S:	Maintained
> @@ -14507,6 +14513,30 @@ F:	drivers/mfd/bd9571mwv.c
>  F:	drivers/regulator/bd9571mwv-regulator.c
>  F:	include/linux/mfd/bd9571mwv.h
>  
> +ROHM POWER MANAGEMENT IC DEVICE DRIVERS
> +R:	Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
> +S:	Supported
> +F:	Documentation/devicetree/bindings/mfd/rohm,bd70528-pmic.txt
> +F:	Documentation/devicetree/bindings/regulator/rohm,bd70528-regulator.txt
> +F:	drivers/clk/clk-bd718x7.c
> +F:	drivers/gpio/gpio-bd70528.c
> +F:	drivers/gpio/gpio-bd71828.c
> +F:	drivers/mfd/rohm-bd70528.c
> +F:	drivers/mfd/rohm-bd71828.c
> +F:	drivers/mfd/rohm-bd718x7.c
> +F:	drivers/power/supply/bd70528-charger.c
> +F:	drivers/regulator/bd70528-regulator.c
> +F:	drivers/regulator/bd71828-regulator.c
> +F:	drivers/regulator/bd718x7-regulator.c
> +F:	drivers/regulator/rohm-regulator.c
> +F:	drivers/rtc/rtc-bd70528.c
> +F:	drivers/watchdog/bd70528_wdt.c
> +F:	include/linux/mfd/rohm-shared.h
> +F:	include/linux/mfd/rohm-bd71828.h
> +F:	include/linux/mfd/rohm-bd70528.h
> +F:	include/linux/mfd/rohm-generic.h
> +F:	include/linux/mfd/rohm-bd718x7.h

How small can you get this list using wildcards?

+F:	drivers/clk/clk-bd718x7.c
+F:	drivers/gpio/gpio-bd7*
+F:	drivers/mfd/rohm-bd7*
+F:	drivers/power/supply/bd7*
+F:	drivers/regulator/bd7*
+F:	drivers/regulator/rohm-regulator.c
+F:	drivers/rtc/rtc-bd7*
+F:	drivers/watchdog/bd7*
+F:	include/linux/mfd/rohm-*

Or

+F:	drivers/*/bd7*
+F:	drivers/*/*-bd7*
+F:	drivers/*/rohm-*
+F:	drivers/*/*rohm-*
+F:	include/linux/*/rohm-*
+F:	include/linux/*/*rohm-*

Not checked either of these.  They are just an example.

>  ROSE NETWORK LAYER
>  M:	Ralf Baechle <ralf@linux-mips.org>
>  L:	linux-hams@vger.kernel.org

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] MAINTAINERS: Add entry for ROHM power management ICs
  2020-05-20  6:35   ` Lee Jones
@ 2020-05-20  7:07     ` Vaittinen, Matti
  2020-05-20  8:25       ` Lee Jones
  0 siblings, 1 reply; 8+ messages in thread
From: Vaittinen, Matti @ 2020-05-20  7:07 UTC (permalink / raw)
  To: lee.jones
  Cc: alexandre.belloni, linux, wim, mazziesaccount, broonie, sboyd,
	linus.walleij, linux-kernel, sre, bgolaszewski, lgirdwood,
	mturquette, a.zummo, robh+dt

Morning Lee,

On Wed, 2020-05-20 at 07:35 +0100, Lee Jones wrote:
> On Wed, 20 May 2020, Matti Vaittinen wrote:
> 
> > Add entry for maintaining power management IC drivers for ROHM
> > BD71837, BD71847, BD71850, BD71828, BD71878, BD70528 and BD99954.
> > 
> > Signed-off-by: Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
> > ---
> >  MAINTAINERS | 30 ++++++++++++++++++++++++++++++
> >  1 file changed, 30 insertions(+)
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index ecc0749810b0..63a2ca70540e 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -14490,6 +14490,12 @@ L:	linux-serial@vger.kernel.org
> >  S:	Odd Fixes
> >  F:	drivers/tty/serial/rp2.*
> >  
> > +ROHM BD99954 CHARGER IC
> > +R:	Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
> > +S:	Supported
> > +F:	drivers/power/supply/bd99954-charger.c
> > +F:	drivers/power/supply/bd99954-charger.h
> > +
> >  ROHM BH1750 AMBIENT LIGHT SENSOR DRIVER
> >  M:	Tomasz Duszynski <tduszyns@gmail.com>
> >  S:	Maintained
> > @@ -14507,6 +14513,30 @@ F:	drivers/mfd/bd9571mwv.c
> >  F:	drivers/regulator/bd9571mwv-regulator.c
> >  F:	include/linux/mfd/bd9571mwv.h
> >  
> > +ROHM POWER MANAGEMENT IC DEVICE DRIVERS
> > +R:	Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
> > +S:	Supported
> > +F:	Documentation/devicetree/bindings/mfd/rohm,bd70528-pmic.txt
> > +F:	Documentation/devicetree/bindings/regulator/rohm,bd70528-
> > regulator.txt
> > +F:	drivers/clk/clk-bd718x7.c
> > +F:	drivers/gpio/gpio-bd70528.c
> > +F:	drivers/gpio/gpio-bd71828.c
> > +F:	drivers/mfd/rohm-bd70528.c
> > +F:	drivers/mfd/rohm-bd71828.c
> > +F:	drivers/mfd/rohm-bd718x7.c
> > +F:	drivers/power/supply/bd70528-charger.c
> > +F:	drivers/regulator/bd70528-regulator.c
> > +F:	drivers/regulator/bd71828-regulator.c
> > +F:	drivers/regulator/bd718x7-regulator.c
> > +F:	drivers/regulator/rohm-regulator.c
> > +F:	drivers/rtc/rtc-bd70528.c
> > +F:	drivers/watchdog/bd70528_wdt.c
> > +F:	include/linux/mfd/rohm-shared.h
> > +F:	include/linux/mfd/rohm-bd71828.h
> > +F:	include/linux/mfd/rohm-bd70528.h
> > +F:	include/linux/mfd/rohm-generic.h
> > +F:	include/linux/mfd/rohm-bd718x7.h
> 
> How small can you get this list using wildcards?
I was considering this but wildcards are somewhat error prone. I can't
guarantee they don't match to drivers which I am not working with :(
The BDxxxxx is common for pretty much all ROHM ICs (well, theres few
other like BUxxxxx and BHxxxxx) - and there is huge amount of ICs.
Hence I think it is likely the wildcards (will) match to drivers
maintained by others.

One already existing example is:
ROHM MULTIFUNCTION BD9571MWV-M PMIC DEVICE DRIVERS
M:      Marek Vasut <marek.vasut+renesas@gmail.com>
L:      linux-kernel@vger.kernel.org
L:      linux-renesas-soc@vger.kernel.org
S:      Supported          
F:      Documentation/devicetree/bindings/mfd/bd9571mwv.txt          
F:      drivers/gpio/gpio-bd9571mwv.c
F:      drivers/mfd/bd9571mwv.c          
F:      drivers/regulator/bd9571mwv-regulator.c
F:      include/linux/mfd/bd9571mwv.h

also LEDs have some ROHM IC drivers which I am not familiar with.

> 
> +F:	drivers/clk/clk-bd718x7.c
> +F:	drivers/gpio/gpio-bd7*
> +F:	drivers/mfd/rohm-bd7*
> +F:	drivers/power/supply/bd7*
> +F:	drivers/regulator/bd7*
> +F:	drivers/regulator/rohm-regulator.c
> +F:	drivers/rtc/rtc-bd7*
> +F:	drivers/watchdog/bd7*
> +F:	include/linux/mfd/rohm-*
> 
> Or
> 
> +F:	drivers/*/bd7*
> +F:	drivers/*/*-bd7*
> +F:	drivers/*/rohm-*
> +F:	drivers/*/*rohm-*
> +F:	include/linux/*/rohm-*
> +F:	include/linux/*/*rohm-*
> 
> Not checked either of these.  They are just an example.

So, I am sure I can find a working match pattern for a few of the files
- but if using wildcards is not compulsory I'd rather not do that as I
have a feeling this will cause wrong matches...

> 
> >  ROSE NETWORK LAYER
> >  M:	Ralf Baechle <ralf@linux-mips.org>
> >  L:	linux-hams@vger.kernel.org

Br,
	Matti Vaittinen


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] MAINTAINERS: Add entry for ROHM power management ICs
  2020-05-20  7:07     ` Vaittinen, Matti
@ 2020-05-20  8:25       ` Lee Jones
  0 siblings, 0 replies; 8+ messages in thread
From: Lee Jones @ 2020-05-20  8:25 UTC (permalink / raw)
  To: Vaittinen, Matti
  Cc: alexandre.belloni, linux, wim, mazziesaccount, broonie, sboyd,
	linus.walleij, linux-kernel, sre, bgolaszewski, lgirdwood,
	mturquette, a.zummo, robh+dt

On Wed, 20 May 2020, Vaittinen, Matti wrote:

> Morning Lee,
> 
> On Wed, 2020-05-20 at 07:35 +0100, Lee Jones wrote:
> > On Wed, 20 May 2020, Matti Vaittinen wrote:
> > 
> > > Add entry for maintaining power management IC drivers for ROHM
> > > BD71837, BD71847, BD71850, BD71828, BD71878, BD70528 and BD99954.
> > > 
> > > Signed-off-by: Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
> > > ---
> > >  MAINTAINERS | 30 ++++++++++++++++++++++++++++++
> > >  1 file changed, 30 insertions(+)
> > > 
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index ecc0749810b0..63a2ca70540e 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -14490,6 +14490,12 @@ L:	linux-serial@vger.kernel.org
> > >  S:	Odd Fixes
> > >  F:	drivers/tty/serial/rp2.*
> > >  
> > > +ROHM BD99954 CHARGER IC
> > > +R:	Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
> > > +S:	Supported
> > > +F:	drivers/power/supply/bd99954-charger.c
> > > +F:	drivers/power/supply/bd99954-charger.h
> > > +
> > >  ROHM BH1750 AMBIENT LIGHT SENSOR DRIVER
> > >  M:	Tomasz Duszynski <tduszyns@gmail.com>
> > >  S:	Maintained
> > > @@ -14507,6 +14513,30 @@ F:	drivers/mfd/bd9571mwv.c
> > >  F:	drivers/regulator/bd9571mwv-regulator.c
> > >  F:	include/linux/mfd/bd9571mwv.h
> > >  
> > > +ROHM POWER MANAGEMENT IC DEVICE DRIVERS
> > > +R:	Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
> > > +S:	Supported
> > > +F:	Documentation/devicetree/bindings/mfd/rohm,bd70528-pmic.txt
> > > +F:	Documentation/devicetree/bindings/regulator/rohm,bd70528-
> > > regulator.txt
> > > +F:	drivers/clk/clk-bd718x7.c
> > > +F:	drivers/gpio/gpio-bd70528.c
> > > +F:	drivers/gpio/gpio-bd71828.c
> > > +F:	drivers/mfd/rohm-bd70528.c
> > > +F:	drivers/mfd/rohm-bd71828.c
> > > +F:	drivers/mfd/rohm-bd718x7.c
> > > +F:	drivers/power/supply/bd70528-charger.c
> > > +F:	drivers/regulator/bd70528-regulator.c
> > > +F:	drivers/regulator/bd71828-regulator.c
> > > +F:	drivers/regulator/bd718x7-regulator.c
> > > +F:	drivers/regulator/rohm-regulator.c
> > > +F:	drivers/rtc/rtc-bd70528.c
> > > +F:	drivers/watchdog/bd70528_wdt.c
> > > +F:	include/linux/mfd/rohm-shared.h
> > > +F:	include/linux/mfd/rohm-bd71828.h
> > > +F:	include/linux/mfd/rohm-bd70528.h
> > > +F:	include/linux/mfd/rohm-generic.h
> > > +F:	include/linux/mfd/rohm-bd718x7.h
> > 
> > How small can you get this list using wildcards?
> I was considering this but wildcards are somewhat error prone. I can't
> guarantee they don't match to drivers which I am not working with :(
> The BDxxxxx is common for pretty much all ROHM ICs (well, theres few
> other like BUxxxxx and BHxxxxx) - and there is huge amount of ICs.
> Hence I think it is likely the wildcards (will) match to drivers
> maintained by others.
> 
> One already existing example is:
> ROHM MULTIFUNCTION BD9571MWV-M PMIC DEVICE DRIVERS
> M:      Marek Vasut <marek.vasut+renesas@gmail.com>
> L:      linux-kernel@vger.kernel.org
> L:      linux-renesas-soc@vger.kernel.org
> S:      Supported          
> F:      Documentation/devicetree/bindings/mfd/bd9571mwv.txt          
> F:      drivers/gpio/gpio-bd9571mwv.c
> F:      drivers/mfd/bd9571mwv.c          
> F:      drivers/regulator/bd9571mwv-regulator.c
> F:      include/linux/mfd/bd9571mwv.h
> 
> also LEDs have some ROHM IC drivers which I am not familiar with.
> 
> > +F:	drivers/clk/clk-bd718x7.c
> > +F:	drivers/gpio/gpio-bd7*
> > +F:	drivers/mfd/rohm-bd7*
> > +F:	drivers/power/supply/bd7*
> > +F:	drivers/regulator/bd7*
> > +F:	drivers/regulator/rohm-regulator.c
> > +F:	drivers/rtc/rtc-bd7*
> > +F:	drivers/watchdog/bd7*
> > +F:	include/linux/mfd/rohm-*
> > 
> > Or
> > 
> > +F:	drivers/*/bd7*
> > +F:	drivers/*/*-bd7*
> > +F:	drivers/*/rohm-*
> > +F:	drivers/*/*rohm-*
> > +F:	include/linux/*/rohm-*
> > +F:	include/linux/*/*rohm-*
> > 
> > Not checked either of these.  They are just an example.
> 
> So, I am sure I can find a working match pattern for a few of the files
> - but if using wildcards is not compulsory I'd rather not do that as I
> have a feeling this will cause wrong matches...

That's fine.  So long as you've thought about it.

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] MAINTAINERS: Add entry for ROHM power management ICs
  2020-05-20  6:11 ` [PATCH 1/2] MAINTAINERS: Add entry for ROHM power management ICs Matti Vaittinen
  2020-05-20  6:35   ` Lee Jones
@ 2020-05-20 10:52   ` Sebastian Reichel
  1 sibling, 0 replies; 8+ messages in thread
From: Sebastian Reichel @ 2020-05-20 10:52 UTC (permalink / raw)
  To: Matti Vaittinen
  Cc: mazziesaccount, Michael Turquette, Liam Girdwood, Mark Brown,
	Stephen Boyd, Wim Van Sebroeck, Guenter Roeck, Lee Jones,
	Linus Walleij, Bartosz Golaszewski, Alessandro Zummo,
	Alexandre Belloni, Rob Herring, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 2593 bytes --]

Hi,

On Wed, May 20, 2020 at 09:11:28AM +0300, Matti Vaittinen wrote:
> Add entry for maintaining power management IC drivers for ROHM
> BD71837, BD71847, BD71850, BD71828, BD71878, BD70528 and BD99954.
> 
> Signed-off-by: Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
> ---

Acked-by: Sebastian Reichel <sre@kernel.org>

-- Sebastian

>  MAINTAINERS | 30 ++++++++++++++++++++++++++++++
>  1 file changed, 30 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index ecc0749810b0..63a2ca70540e 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -14490,6 +14490,12 @@ L:	linux-serial@vger.kernel.org
>  S:	Odd Fixes
>  F:	drivers/tty/serial/rp2.*
>  
> +ROHM BD99954 CHARGER IC
> +R:	Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
> +S:	Supported
> +F:	drivers/power/supply/bd99954-charger.c
> +F:	drivers/power/supply/bd99954-charger.h
> +
>  ROHM BH1750 AMBIENT LIGHT SENSOR DRIVER
>  M:	Tomasz Duszynski <tduszyns@gmail.com>
>  S:	Maintained
> @@ -14507,6 +14513,30 @@ F:	drivers/mfd/bd9571mwv.c
>  F:	drivers/regulator/bd9571mwv-regulator.c
>  F:	include/linux/mfd/bd9571mwv.h
>  
> +ROHM POWER MANAGEMENT IC DEVICE DRIVERS
> +R:	Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
> +S:	Supported
> +F:	Documentation/devicetree/bindings/mfd/rohm,bd70528-pmic.txt
> +F:	Documentation/devicetree/bindings/regulator/rohm,bd70528-regulator.txt
> +F:	drivers/clk/clk-bd718x7.c
> +F:	drivers/gpio/gpio-bd70528.c
> +F:	drivers/gpio/gpio-bd71828.c
> +F:	drivers/mfd/rohm-bd70528.c
> +F:	drivers/mfd/rohm-bd71828.c
> +F:	drivers/mfd/rohm-bd718x7.c
> +F:	drivers/power/supply/bd70528-charger.c
> +F:	drivers/regulator/bd70528-regulator.c
> +F:	drivers/regulator/bd71828-regulator.c
> +F:	drivers/regulator/bd718x7-regulator.c
> +F:	drivers/regulator/rohm-regulator.c
> +F:	drivers/rtc/rtc-bd70528.c
> +F:	drivers/watchdog/bd70528_wdt.c
> +F:	include/linux/mfd/rohm-shared.h
> +F:	include/linux/mfd/rohm-bd71828.h
> +F:	include/linux/mfd/rohm-bd70528.h
> +F:	include/linux/mfd/rohm-generic.h
> +F:	include/linux/mfd/rohm-bd718x7.h
> +
>  ROSE NETWORK LAYER
>  M:	Ralf Baechle <ralf@linux-mips.org>
>  L:	linux-hams@vger.kernel.org
> -- 
> 2.21.0
> 
> 
> -- 
> Matti Vaittinen, Linux device drivers
> ROHM Semiconductors, Finland SWDC
> Kiviharjunlenkki 1E
> 90220 OULU
> FINLAND
> 
> ~~~ "I don't think so," said Rene Descartes. Just then he vanished ~~~
> Simon says - in Latin please.
> ~~~ "non cogito me" dixit Rene Descarte, deinde evanescavit ~~~
> Thanks to Simon Glass for the translation =] 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 0/2] MAINTAINER entries for few ROHM power devices
  2020-05-20  6:10 [PATCH 0/2] MAINTAINER entries for few ROHM power devices Matti Vaittinen
  2020-05-20  6:11 ` [PATCH 1/2] MAINTAINERS: Add entry for ROHM power management ICs Matti Vaittinen
  2020-05-20  6:12 ` [PATCH 2/2] MAINTAINERS: Add maintainer entry for linear ranges helper Matti Vaittinen
@ 2020-05-20 17:17 ` Mark Brown
  2 siblings, 0 replies; 8+ messages in thread
From: Mark Brown @ 2020-05-20 17:17 UTC (permalink / raw)
  To: Matti Vaittinen, mazziesaccount
  Cc: Michael Turquette, Alessandro Zummo, SebastianReichel,
	Wim Van Sebroeck, linux-kernel, Rob Herring, Alexandre Belloni,
	Stephen Boyd, Lee Jones, Guenter Roeck, Liam Girdwood,
	Linus Walleij, Bartosz Golaszewski

On Wed, 20 May 2020 09:10:22 +0300, Matti Vaittinen wrote:
> Add maintainer entries to a few ROHM devices and Linear Ranges
> 
> Linear Ranges helpers were refactored out of regulator core to lib so
> that other drivers could utilize them too. (I guess power/supply drivers
> and possibly clk drivers can benefit from them). As regulators is
> currently the main user it makes sense the changes to linear_ranges go
> through Mark's tree.
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git for-next

Thanks!

[1/1] MAINTAINERS: Add maintainer entry for linear ranges helper
      commit: 7e73861eb40d591a98628592c6f0182fbf2f6c4d

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2020-05-20 17:17 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-20  6:10 [PATCH 0/2] MAINTAINER entries for few ROHM power devices Matti Vaittinen
2020-05-20  6:11 ` [PATCH 1/2] MAINTAINERS: Add entry for ROHM power management ICs Matti Vaittinen
2020-05-20  6:35   ` Lee Jones
2020-05-20  7:07     ` Vaittinen, Matti
2020-05-20  8:25       ` Lee Jones
2020-05-20 10:52   ` Sebastian Reichel
2020-05-20  6:12 ` [PATCH 2/2] MAINTAINERS: Add maintainer entry for linear ranges helper Matti Vaittinen
2020-05-20 17:17 ` [PATCH 0/2] MAINTAINER entries for few ROHM power devices Mark Brown

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).