diff mbox

MAINTAINERS: Add the stlinux kernel mailing list for the STi drm driver.

Message ID 20160317132622.GB13692@x1
State New
Headers show

Commit Message

Lee Jones March 17, 2016, 1:26 p.m. UTC
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 <peter.griffin@linaro.org>

> ---

>  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 <benjamin.gaignard@linaro.org>

>  M:	Vincent Abriou <vincent.abriou@st.com>

>  L:	dri-devel@lists.freedesktop.org

> +L:	kernel@stlinux.com

>  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?



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

Comments

Peter Griffin March 17, 2016, 3:07 p.m. UTC | #1
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 <peter.griffin@linaro.org>

> > ---

> >  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 <benjamin.gaignard@linaro.org>

> >  M:	Vincent Abriou <vincent.abriou@st.com>

> >  L:	dri-devel@lists.freedesktop.org

> > +L:	kernel@stlinux.com

> >  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 kernel@stlinux.com 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.
Maxime COQUELIN March 17, 2016, 3:12 p.m. UTC | #2
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 <peter.griffin@linaro.org>

>>> ---

>>>   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 <benjamin.gaignard@linaro.org>

>>>   M:	Vincent Abriou <vincent.abriou@st.com>

>>>   L:	dri-devel@lists.freedesktop.org

>>> +L:	kernel@stlinux.com

>>>   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 <maxime.coquelin@st.com>


Regards,
Maxime
diff mbox

Patch

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