2014-10-27 09:48:54

by Andrzej Hajda

[permalink] [raw]
Subject: [PATCH 0/5] MAINTAINERS: add entries for few DRM drivers

Hi,

These patches adds MAINTAINERS entries for DRM panel and encoder drivers and DSI bus
I have written.

Regards
Andrzej


Andrzej Hajda (5):
MAINTAINERS: Add entry for MIPI DSI bus
MAINTAINERS: Add entry for LD9040 drm panel driver
MAINTAINERS: Add entry for S6E8AA0 drm panel driver
MAINTAINERS: Add entry for Exynos MIPI DSI DRM driver
MAINTAINERS: Add entry for Exynos MIPI DPI DRM driver

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

--
1.9.1


2014-10-27 09:48:58

by Andrzej Hajda

[permalink] [raw]
Subject: [PATCH 1/5] MAINTAINERS: Add entry for MIPI DSI bus

Signed-off-by: Andrzej Hajda <[email protected]>
---
MAINTAINERS | 7 +++++++
1 file changed, 7 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index dd31933..476679c 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3071,6 +3071,13 @@ F: drivers/gpu/vga/
F: include/drm/
F: include/uapi/drm/

+DRM MIPI DSI BUS
+M: Andrzej Hajda <[email protected]>
+L: [email protected]
+S: Supported
+F: drivers/gpu/drm/drm_mipi_dsi.c
+F: include/drm/drm_mipi_dsi.h
+
RADEON DRM DRIVERS
M: Alex Deucher <[email protected]>
M: Christian König <[email protected]>
--
1.9.1

2014-10-27 09:49:05

by Andrzej Hajda

[permalink] [raw]
Subject: [PATCH 4/5] MAINTAINERS: Add entry for Exynos MIPI DSI DRM driver

Signed-off-by: Andrzej Hajda <[email protected]>
---
MAINTAINERS | 6 ++++++
1 file changed, 6 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 2546fca..256183e 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3650,6 +3650,12 @@ S: Maintained
F: drivers/video/fbdev/exynos/exynos_mipi*
F: include/video/exynos_mipi*

+EXYNOS MIPI DSI DRM DRIVER
+M: Andrzej Hajda <[email protected]>
+L: [email protected]
+S: Supported
+F: drivers/gpu/drm/exynos/exynos_drm_dsi.c
+
F71805F HARDWARE MONITORING DRIVER
M: Jean Delvare <[email protected]>
L: [email protected]
--
1.9.1

2014-10-27 09:49:03

by Andrzej Hajda

[permalink] [raw]
Subject: [PATCH 3/5] MAINTAINERS: Add entry for S6E8AA0 drm panel driver

Signed-off-by: Andrzej Hajda <[email protected]>
---
MAINTAINERS | 7 +++++++
1 file changed, 7 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 6e57cad..2546fca 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3104,6 +3104,13 @@ S: Supported
F: drivers/gpu/drm/panel/panel-ld9040.c
F: Documentation/devicetree/bindings/panel/samsung,ld9040.txt

+DRM PANEL S6E8AA0 DRIVER
+M: Andrzej Hajda <[email protected]>
+L: [email protected]
+S: Supported
+F: drivers/gpu/drm/panel/panel-s6e8aa0.c
+F: Documentation/devicetree/bindings/panel/samsung,s6e8aa0.txt
+
INTEL DRM DRIVERS (excluding Poulsbo, Moorestown and derivative chipsets)
M: Daniel Vetter <[email protected]>
M: Jani Nikula <[email protected]>
--
1.9.1

2014-10-27 09:51:22

by Andrzej Hajda

[permalink] [raw]
Subject: [PATCH 5/5] MAINTAINERS: Add entry for Exynos MIPI DPI DRM driver

Signed-off-by: Andrzej Hajda <[email protected]>
---
MAINTAINERS | 6 ++++++
1 file changed, 6 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 256183e..06cdb5a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3656,6 +3656,12 @@ L: [email protected]
S: Supported
F: drivers/gpu/drm/exynos/exynos_drm_dsi.c

+EXYNOS MIPI DPI DRM DRIVER
+M: Andrzej Hajda <[email protected]>
+L: [email protected]
+S: Supported
+F: drivers/gpu/drm/exynos/exynos_drm_dpi.c
+
F71805F HARDWARE MONITORING DRIVER
M: Jean Delvare <[email protected]>
L: [email protected]
--
1.9.1

2014-10-27 09:51:45

by Andrzej Hajda

[permalink] [raw]
Subject: [PATCH 2/5] MAINTAINERS: Add entry for LD9040 drm panel driver

Signed-off-by: Andrzej Hajda <[email protected]>
---
MAINTAINERS | 7 +++++++
1 file changed, 7 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 476679c..6e57cad 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3097,6 +3097,13 @@ F: drivers/gpu/drm/panel/
F: include/drm/drm_panel.h
F: Documentation/devicetree/bindings/panel/

+DRM PANEL LD9040 DRIVER
+M: Andrzej Hajda <[email protected]>
+L: [email protected]
+S: Supported
+F: drivers/gpu/drm/panel/panel-ld9040.c
+F: Documentation/devicetree/bindings/panel/samsung,ld9040.txt
+
INTEL DRM DRIVERS (excluding Poulsbo, Moorestown and derivative chipsets)
M: Daniel Vetter <[email protected]>
M: Jani Nikula <[email protected]>
--
1.9.1

2014-10-27 14:17:09

by Daniel Vetter

[permalink] [raw]
Subject: Re: [PATCH 1/5] MAINTAINERS: Add entry for MIPI DSI bus

On Mon, Oct 27, 2014 at 10:48:26AM +0100, Andrzej Hajda wrote:
> Signed-off-by: Andrzej Hajda <[email protected]>
> ---
> MAINTAINERS | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index dd31933..476679c 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -3071,6 +3071,13 @@ F: drivers/gpu/vga/
> F: include/drm/
> F: include/uapi/drm/
>
> +DRM MIPI DSI BUS
> +M: Andrzej Hajda <[email protected]>
> +L: [email protected]
> +S: Supported
> +F: drivers/gpu/drm/drm_mipi_dsi.c
> +F: include/drm/drm_mipi_dsi.h

I really prefer when common infrastructure like this doesn't get
maintainer by a vendor. And I care with my intel hat on since we also ahve
a dsi driver that sooner or later needs to be converted to to this stuff.

Why doesn't the usual model of merging stuff through Dave's tree, or if
that doesn't work well due to patch depencies, through vendor trees with
Dave's ack?

Thanks, Daniel

> +
> RADEON DRM DRIVERS
> M: Alex Deucher <[email protected]>
> M: Christian K?nig <[email protected]>
> --
> 1.9.1
>
> _______________________________________________
> dri-devel mailing list
> [email protected]
> http://lists.freedesktop.org/mailman/listinfo/dri-devel

--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch

2014-10-27 15:20:31

by Andrzej Hajda

[permalink] [raw]
Subject: Re: [PATCH 1/5] MAINTAINERS: Add entry for MIPI DSI bus

On 10/27/2014 03:17 PM, Daniel Vetter wrote:
> On Mon, Oct 27, 2014 at 10:48:26AM +0100, Andrzej Hajda wrote:
>> Signed-off-by: Andrzej Hajda <[email protected]>
>> ---
>> MAINTAINERS | 7 +++++++
>> 1 file changed, 7 insertions(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index dd31933..476679c 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -3071,6 +3071,13 @@ F: drivers/gpu/vga/
>> F: include/drm/
>> F: include/uapi/drm/
>>
>> +DRM MIPI DSI BUS
>> +M: Andrzej Hajda <[email protected]>
>> +L: [email protected]
>> +S: Supported
>> +F: drivers/gpu/drm/drm_mipi_dsi.c
>> +F: include/drm/drm_mipi_dsi.h
>
> I really prefer when common infrastructure like this doesn't get
> maintainer by a vendor. And I care with my intel hat on since we also ahve
> a dsi driver that sooner or later needs to be converted to to this stuff.
>
> Why doesn't the usual model of merging stuff through Dave's tree, or if
> that doesn't work well due to patch depencies, through vendor trees with
> Dave's ack?

The main intention of this patch is to be on CC in case of patches
touching MIPI DSI bus.
I do not want to create separate tree for such small subsystem.

So if such things are not allowed please drop the patch and sorry for
the noise.

Regards
Andrzej

>
> Thanks, Daniel
>
>> +
>> RADEON DRM DRIVERS
>> M: Alex Deucher <[email protected]>
>> M: Christian König <[email protected]>
>> --
>> 1.9.1
>>
>> _______________________________________________
>> dri-devel mailing list
>> [email protected]
>> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>

2014-10-27 18:53:00

by Daniel Vetter

[permalink] [raw]
Subject: Re: [PATCH 1/5] MAINTAINERS: Add entry for MIPI DSI bus

On Mon, Oct 27, 2014 at 04:20:21PM +0100, Andrzej Hajda wrote:
> On 10/27/2014 03:17 PM, Daniel Vetter wrote:
> > On Mon, Oct 27, 2014 at 10:48:26AM +0100, Andrzej Hajda wrote:
> >> Signed-off-by: Andrzej Hajda <[email protected]>
> >> ---
> >> MAINTAINERS | 7 +++++++
> >> 1 file changed, 7 insertions(+)
> >>
> >> diff --git a/MAINTAINERS b/MAINTAINERS
> >> index dd31933..476679c 100644
> >> --- a/MAINTAINERS
> >> +++ b/MAINTAINERS
> >> @@ -3071,6 +3071,13 @@ F: drivers/gpu/vga/
> >> F: include/drm/
> >> F: include/uapi/drm/
> >>
> >> +DRM MIPI DSI BUS
> >> +M: Andrzej Hajda <[email protected]>
> >> +L: [email protected]
> >> +S: Supported
> >> +F: drivers/gpu/drm/drm_mipi_dsi.c
> >> +F: include/drm/drm_mipi_dsi.h
> >
> > I really prefer when common infrastructure like this doesn't get
> > maintainer by a vendor. And I care with my intel hat on since we also ahve
> > a dsi driver that sooner or later needs to be converted to to this stuff.
> >
> > Why doesn't the usual model of merging stuff through Dave's tree, or if
> > that doesn't work well due to patch depencies, through vendor trees with
> > Dave's ack?
>
> The main intention of this patch is to be on CC in case of patches
> touching MIPI DSI bus.
> I do not want to create separate tree for such small subsystem.
>
> So if such things are not allowed please drop the patch and sorry for
> the noise.

For that stuff you can just create a mail filter that shovels everything
touching anything dsi related on dri-devel (people arleady should send all
stuff there anyway) into a special folder/mark it as important. If you
rely on people to always use get_maintainers.pl that won't work properly
anyway.

So if all you want is to notice all dsi related patches then adding a
maintainer entry isn't a good solution anyway. So best to drop this one.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch

2014-10-29 12:39:49

by Thierry Reding

[permalink] [raw]
Subject: Re: [PATCH 1/5] MAINTAINERS: Add entry for MIPI DSI bus

On Mon, Oct 27, 2014 at 03:17:17PM +0100, Daniel Vetter wrote:
> On Mon, Oct 27, 2014 at 10:48:26AM +0100, Andrzej Hajda wrote:
> > Signed-off-by: Andrzej Hajda <[email protected]>
> > ---
> > MAINTAINERS | 7 +++++++
> > 1 file changed, 7 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index dd31933..476679c 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -3071,6 +3071,13 @@ F: drivers/gpu/vga/
> > F: include/drm/
> > F: include/uapi/drm/
> >
> > +DRM MIPI DSI BUS
> > +M: Andrzej Hajda <[email protected]>
> > +L: [email protected]
> > +S: Supported
> > +F: drivers/gpu/drm/drm_mipi_dsi.c
> > +F: include/drm/drm_mipi_dsi.h
>
> I really prefer when common infrastructure like this doesn't get
> maintainer by a vendor. And I care with my intel hat on since we also ahve
> a dsi driver that sooner or later needs to be converted to to this stuff.
>
> Why doesn't the usual model of merging stuff through Dave's tree, or if
> that doesn't work well due to patch depencies, through vendor trees with
> Dave's ack?

FWIW, I've merged DSI changes through the panel tree in the past after
discussing it with Dave on IRC. It seemed like a good fit and it was
most relevant for panels anyway. Given that I had expected that Dave
would give it a closer look before pulling than any of the vendor trees
anyway, so it would be as good as an explicit Acked-by anyway.

Thierry


Attachments:
(No filename) (1.39 kB)
(No filename) (819.00 B)
Download all attachments