Re: [PATCH v2 3/3] MAINTAINERS: Add entry for Mali-DP driver

From: Liviu Dudau <Liviu.Dudau@arm.com>
To: Emil Velikov <emil.l.velikov@gmail.com>
Cc: Rob Herring <robh+dt@kernel.org>, Pawel Moll <pawel.moll@arm.com>,
	Mark Rutland <mark.rutland@arm.com>,
	devicetree <devicetree@vger.kernel.org>,
	David Airlie <airlied@linux.ie>,
	DRI devel <dri-devel@lists.freedesktop.org>,
	LKML <linux-kernel@vger.kernel.org>,
	Daniel Vetter <daniel@ffwll.ch>,
	Brian Starkey <Brian.Starkey@arm.com>,
	David Brown <David.Brown@arm.com>
Subject: Re: [PATCH v2 3/3] MAINTAINERS: Add entry for Mali-DP driver
Date: Mon, 25 Apr 2016 18:08:11 +0100	[thread overview]
Message-ID: <20160425170811.GC28464@e106497-lin.cambridge.arm.com> (raw)
In-Reply-To: <CACvgo51obFzx05SWUPZYbEj-LcNGq_NaCvF_iYsg9gpyLPj48Q@mail.gmail.com>

On Mon, Apr 25, 2016 at 05:00:02PM +0100, Emil Velikov wrote:
> On 25 April 2016 at 15:19, Liviu Dudau <Liviu.Dudau@arm.com> wrote:
> > Add MAINTAINERS entry for ARM Mali-DP driver and update the
> > HDLCD file matching pattern to cover only HDLCD rather than
> > the whole drivers/gpu/drm/arm directory.
> >
> > Signed-off-by: Liviu Dudau <Liviu.Dudau@arm.com>
> > ---
> >  MAINTAINERS | 10 +++++++++-
> >  1 file changed, 9 insertions(+), 1 deletion(-)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 1d5b4be..a5a4346 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -850,9 +850,17 @@ F: include/uapi/linux/if_arcnet.h
> >  ARM HDLCD DRM DRIVER
> >  M:     Liviu Dudau <liviu.dudau@arm.com>
> >  S:     Supported
> > -F:     drivers/gpu/drm/arm/
> > +F:     drivers/gpu/drm/arm/hdlcd_*
> >  F:     Documentation/devicetree/bindings/display/arm,hdlcd.txt
> >
> > +ARM MALI-DP DRM DRIVER
> > +M:     Liviu Dudau <liviu.dudau@arm.com>
> > +M:     Brian Starkey <brian.starkey@arm.com>
> > +M:     Mali DP Maintainers <malidp@foss.arm.com>
> > +S:     Supported
> > +F:     drivers/gpu/drm/arm/
> Woohoo. Thanks Liviu !
> 
> Note that having both drivers' sources in the same folder will cause
> some grief wrt getting the correct maintainer. If you want to avoid
> that the easiest way is to move things in subdirectories roughly like:
> 
> drivers/gpu/arm/hdlcd/
> drivers/gpu/arm/mapidp/

I'm surprised to hear that there might be problems. All HDLCD files start
with hdlcd_ and the rest will go to the Mali-DP DRM maintainers (which includes
me as well).

I understand your suggestion and I thank for it, but I really hate to have
to navigate too many directories (for now). Lets see how things evolve and then
we can move stuff around.

Best regards,
Liviu

> 
> Regards,
> Emil
> 

-- 
====================
| I would like to |
| fix the world,  |
| but they're not |
| giving me the   |
 \ source code!  /
  ---------------
    ¯\_(ツ)_/¯