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

[Liviu Dudau]

On Mon, Apr 25, 2016 at 05:00:02PM +0100, Emil Velikov wrote:
> On 25 April 2016 at 15:19, Liviu Dudau <Liviu.Dudau@xxxxxxx> 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@xxxxxxx>
> > ---
> >  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@xxxxxxx>
> >  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@xxxxxxx>
> > +M:     Brian Starkey <brian.starkey@xxxxxxx>
> > +M:     Mali DP Maintainers <malidp@xxxxxxxxxxxx>
> > +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!  /
  ---------------
    Â\_(ã)_/Â