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 <[email protected]>
> > Acked-by: Jani Nikula <[email protected]>
> > ---
> >   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 <[email protected]>
> >   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 <[email protected]>
> 
> 
> 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.

> 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

Attachment: signature.asc
Description: PGP signature

Reply via email to