Re: MAINTAINERS: Update drm-misc entry to match all drivers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

Le jeudi 21 septembre 2023 à 10:47 +0200, Maxime Ripard a écrit :
> 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?

drm/ingenic is maintained through drm-misc, yes.

Looking at the MAINTAINERS file, it seems to be one of the only DRM
drivers without its own entry. I'll add one then.

Cheers,
-Paul

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





[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux