| Message ID | 20160317132622.GB13692@x1 |
|---|---|
| State | New |
| Headers | show |
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.
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 --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