[PATCH] MAINTAINERS: add include/dt-bindings/leds/ to linux-leds list

[PATCH] MAINTAINERS: add include/dt-bindings/leds/ to linux-leds list

[Adrian Schmutzler]

The content of include/dt-bindings/leds/, particularly common.h,
is apparantly maintained by the linux-leds list. This is also
explicitly stated in the Documentation.

Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 190c7fa2ea01..415f0e089bbe 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -9809,6 +9809,7 @@ S:	Maintained
 T:	git git://git.kernel.org/pub/scm/linux/kernel/git/pavel/linux-leds.git
 F:	Documentation/devicetree/bindings/leds/
 F:	drivers/leds/
+F:	include/dt-bindings/leds/
 F:	include/linux/leds.h
 
 LEGACY EEPROM DRIVER
-- 
2.20.1

Re: [PATCH] MAINTAINERS: add include/dt-bindings/leds/ to linux-leds list

[Marek Behun]

On Sat, 26 Sep 2020 18:47:45 +0200
Adrian Schmutzler <freifunk@adrianschmutzler.de> wrote:

> The content of include/dt-bindings/leds/, particularly common.h,
> is apparantly maintained by the linux-leds list. This is also
> explicitly stated in the Documentation.
> 
> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 190c7fa2ea01..415f0e089bbe 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -9809,6 +9809,7 @@ S:	Maintained
>  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/pavel/linux-leds.git
>  F:	Documentation/devicetree/bindings/leds/
>  F:	drivers/leds/
> +F:	include/dt-bindings/leds/
>  F:	include/linux/leds.h
>  
>  LEGACY EEPROM DRIVER

Some subsystem maintainers do not want to touch the MAINTAINERS file
because of potential conflict. If Pavel refuses to do this, I was told
that the best way is to send such to Andrew Morton
<akpm@linux-foundation.org>.

Marek

Re: [PATCH] MAINTAINERS: add include/dt-bindings/leds/ to linux-leds list

[Pavel Machek]

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

Hi!

> The content of include/dt-bindings/leds/, particularly common.h,
> is apparantly maintained by the linux-leds list. This is also
> explicitly stated in the Documentation.
> 
> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 190c7fa2ea01..415f0e089bbe 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -9809,6 +9809,7 @@ S:	Maintained
>  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/pavel/linux-leds.git
>  F:	Documentation/devicetree/bindings/leds/
>  F:	drivers/leds/
> +F:	include/dt-bindings/leds/
>  F:	include/linux/leds.h

I'd not say this is right. I'm not sure what the docs says, but I
don't commit dt changes without ack from Rob. With yaml stuff, I'm
even less confident doing so.

So I'd say Rob maintains this.

Best regards,
									Pavel

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

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

RE: [PATCH] MAINTAINERS: add include/dt-bindings/leds/ to linux-leds list

[Adrian Schmutzler]

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

> > +F:   include/dt-bindings/leds/ 
> >  F:   include/linux/leds.h 
> I'd not say this is right. I'm not sure what the docs says, but I 
> don't commit dt changes without ack from Rob. With yaml stuff, I'm 
> even less confident doing so. 
> So I'd say Rob maintains this. 

https://www.kernel.org/doc/Documentation/leds/leds-class.rst:

"
[...]
If required color or function is missing, please submit a patch
to linux-leds@vger.kernel.org.
[...]
"

This was the only thing pointing me to the linux-leds list when I submitted my patches for LED_FUNCTION_RSSI etc. last week.
I only found it when I had already submitted to Rob and devicetree@ beforehand. However, all of the discussion actually took place at linux-leds@ after I resent there as well.

I do not care who merges this or whom I should send this to, but I think get_maintainers.sh should point me to the correct list where proposed changes are actually discussed (which happened on the linux-leds list).

Best

Adrian

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