On Thu, Sep 21, 2023 at 10:47:58AM +0200, Maxime Ripard wrote: > Hi, > > Adding Paul in Cc > > On Thu, Sep 21, 2023 at 04:25:50PM +0800, suijingfeng wrote: > > On 2023/9/19 21:12, Maxime Ripard wrote: > > > We've had a number of times when a patch slipped through and we couldn't > > > pick them up either because our MAINTAINERS entry only covers the > > > framework and thus we weren't Cc'd. > > > > > > Let's take another approach where we match everything, and remove all > > > the drivers that are not maintained through drm-misc. > > > > > > Signed-off-by: Maxime Ripard <mripard@xxxxxxxxxx> > > > Acked-by: Jani Nikula <jani.nikula@xxxxxxxxx> > > > --- > > > MAINTAINERS | 23 ++++++++++++++++++++--- > > > 1 file changed, 20 insertions(+), 3 deletions(-) > > > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > > index 90f13281d297..757d4f33e158 100644 > > > --- a/MAINTAINERS > > > +++ b/MAINTAINERS > > > @@ -6860,12 +6860,29 @@ M: Thomas Zimmermann <tzimmermann@xxxxxxx> > > > S: Maintained > > > W: https://01.org/linuxgraphics/gfx-docs/maintainer-tools/drm-misc.html > > > T: git git://anongit.freedesktop.org/drm/drm-misc > > > +F: Documentation/devicetree/bindings/display/ > > > +F: Documentation/devicetree/bindings/gpu/ > > > F: Documentation/gpu/ > > > -F: drivers/gpu/drm/* > > > +F: drivers/gpu/drm/ > > > F: drivers/gpu/vga/ > > > -F: include/drm/drm* > > > +F: include/drm/drm > > > F: include/linux/vga* > > > -F: include/uapi/drm/drm* > > > +F: include/uapi/drm/ > > > +X: drivers/gpu/drm/amd/ > > > +X: drivers/gpu/drm/armada/ > > > +X: drivers/gpu/drm/etnaviv/ > > > +X: drivers/gpu/drm/exynos/ > > > +X: drivers/gpu/drm/gma500/ > > > +X: drivers/gpu/drm/i915/ > > > +X: drivers/gpu/drm/imx/ > > > +X: drivers/gpu/drm/ingenic/ > > > +X: drivers/gpu/drm/kmb/ > > > +X: drivers/gpu/drm/mediatek/ > > > +X: drivers/gpu/drm/msm/ > > > +X: drivers/gpu/drm/nouveau/ > > > +X: drivers/gpu/drm/radeon/ > > > +X: drivers/gpu/drm/renesas/ > > > +X: drivers/gpu/drm/tegra/ > > > DRM DRIVERS FOR ALLWINNER A10 > > > M: Maxime Ripard <mripard@xxxxxxxxxx> > > > > > > Nice patch! > > > > Well, I'm just curious about why the drm/ingenic and drm/gma500 are not maintained through drm-misc? > > > > As far as I know: > > 1) the drm/ingenic driver don't have a "T" annotation (location of the link). > > Yeah, I wasn't sure about that one indeed. I remained conservative since it's a > sensitive topic for some. > > Paul, is drm/ingenic supposed to be maintained through drm-misc? Either way, > could you clarify which git tree is supposed to merge those patches in > MAINTAINERS? > > > 2) the "T" of drm/gma500 is "git git://github.com/patjak/drm-gma500", but the > code for this link is not up to date. > > For gma500, I think it's mostly historical since it was there before drm-misc > was a thing. Yes, that's the reason. I used do PRs from my github before we had drm-misc but now everything gma500 related goes through drm-misc. > > > I think at least the drm/ingenic and drm/gma500 drivers are *actually* > > maintained through drm-misc, So perhaps, these two drivers should not be > > excluded. Am I correct? > > It's likely :) > > Either way, I think it can be solved/clarified later on > > Maxime