2016-03-17 11:13:37

by Peter Griffin

[permalink] [raw]
Subject: [PATCH] MAINTAINERS: Add the stlinux kernel mailing list for the STi drm driver.

Most drivers for STi are under the ARM/STI Architecture section which
includes this mailing list.

Signed-off-by: Peter Griffin <[email protected]>
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 6ee06ea..c541a89 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3814,6 +3814,7 @@ DRM DRIVERS FOR STI
M: Benjamin Gaignard <[email protected]>
M: Vincent Abriou <[email protected]>
L: [email protected]
+L: [email protected]
T: git http://git.linaro.org/people/benjamin.gaignard/kernel.git
S: Maintained
F: drivers/gpu/drm/sti
--
1.9.1


2016-03-17 13:26:28

by Lee Jones

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Add the stlinux kernel mailing list for the STi drm driver.

On Thu, 17 Mar 2016, Peter Griffin wrote:

> Most drivers for STi are under the ARM/STI Architecture section which
> includes this mailing list.
>
> Signed-off-by: Peter Griffin <[email protected]>
> ---
> MAINTAINERS | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 6ee06ea..c541a89 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -3814,6 +3814,7 @@ DRM DRIVERS FOR STI
> M: Benjamin Gaignard <[email protected]>
> M: Vincent Abriou <[email protected]>
> L: [email protected]
> +L: [email protected]
> T: git http://git.linaro.org/people/benjamin.gaignard/kernel.git
> S: Maintained
> F: drivers/gpu/drm/sti

Doesn't it make more sense to add drivers/gpu/drm/sti/ to the ARM/STI
ARCHITECTURE list of supported files?

diff --git a/MAINTAINERS b/MAINTAINERS
index 3a1a569..78201c9 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1505,6 +1505,7 @@ F: arch/arm/mach-sti/
F: arch/arm/boot/dts/sti*
F: drivers/clocksource/arm_global_timer.c
F: drivers/i2c/busses/i2c-st.c
+F: drivers/gpu/drm/sti/
F: drivers/media/rc/st_rc.c
F: drivers/mmc/host/sdhci-st.c
F: drivers/phy/phy-miphy28lp.c


--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

2016-03-17 15:07:19

by Peter Griffin

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Add the stlinux kernel mailing list for the STi drm driver.

Hi Lee,

On Thu, 17 Mar 2016, Lee Jones wrote:

> On Thu, 17 Mar 2016, Peter Griffin wrote:
>
> > Most drivers for STi are under the ARM/STI Architecture section which
> > includes this mailing list.
> >
> > Signed-off-by: Peter Griffin <[email protected]>
> > ---
> > MAINTAINERS | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 6ee06ea..c541a89 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -3814,6 +3814,7 @@ DRM DRIVERS FOR STI
> > M: Benjamin Gaignard <[email protected]>
> > M: Vincent Abriou <[email protected]>
> > L: [email protected]
> > +L: [email protected]
> > T: git http://git.linaro.org/people/benjamin.gaignard/kernel.git
> > S: Maintained
> > F: drivers/gpu/drm/sti
>
> Doesn't it make more sense to add drivers/gpu/drm/sti/ to the ARM/STI
> ARCHITECTURE list of supported files?

I assumed Vincent and Benjamin deliberately added a new entry because they
want to be maintainers of this driver (which I think probably makes sense given
that the display IP is quite complicated).

Not seeing these patches on [email protected] list though has caught me out,
especially recent changes applied to the firmware loading mechanism in drm/sti which
also need similar changes in c8sectpfe and fdma drivers.

regards,

Peter.

2016-03-17 15:13:19

by Maxime Coquelin

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Add the stlinux kernel mailing list for the STi drm driver.



On 03/17/2016 04:07 PM, Peter Griffin wrote:
> Hi Lee,
>
> On Thu, 17 Mar 2016, Lee Jones wrote:
>
>> On Thu, 17 Mar 2016, Peter Griffin wrote:
>>
>>> Most drivers for STi are under the ARM/STI Architecture section which
>>> includes this mailing list.
>>>
>>> Signed-off-by: Peter Griffin <[email protected]>
>>> ---
>>> MAINTAINERS | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index 6ee06ea..c541a89 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -3814,6 +3814,7 @@ DRM DRIVERS FOR STI
>>> M: Benjamin Gaignard <[email protected]>
>>> M: Vincent Abriou <[email protected]>
>>> L: [email protected]
>>> +L: [email protected]
>>> T: git http://git.linaro.org/people/benjamin.gaignard/kernel.git
>>> S: Maintained
>>> F: drivers/gpu/drm/sti
>> Doesn't it make more sense to add drivers/gpu/drm/sti/ to the ARM/STI
>> ARCHITECTURE list of supported files?
> I assumed Vincent and Benjamin deliberately added a new entry because they
> want to be maintainers of this driver (which I think probably makes sense given
> that the display IP is quite complicated).
I agree with Peter, let's just add kernel@stlinux entry in the DRM STi
entry.
Acked-by: Maxime Coquelin <[email protected]>

Regards,
Maxime