From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933006AbcDYRx3 (ORCPT ); Mon, 25 Apr 2016 13:53:29 -0400 Received: from mail-wm0-f45.google.com ([74.125.82.45]:34502 "EHLO mail-wm0-f45.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754651AbcDYRx1 (ORCPT ); Mon, 25 Apr 2016 13:53:27 -0400 MIME-Version: 1.0 In-Reply-To: <20160425170811.GC28464@e106497-lin.cambridge.arm.com> References: <1461593964-27722-1-git-send-email-Liviu.Dudau@arm.com> <1461593964-27722-4-git-send-email-Liviu.Dudau@arm.com> <20160425170811.GC28464@e106497-lin.cambridge.arm.com> Date: Mon, 25 Apr 2016 18:53:26 +0100 Message-ID: Subject: Re: [PATCH v2 3/3] MAINTAINERS: Add entry for Mali-DP driver From: Emil Velikov To: Liviu Dudau Cc: Rob Herring , Pawel Moll , Mark Rutland , devicetree , David Airlie , DRI devel , LKML , Daniel Vetter , Brian Starkey , David Brown Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 25 April 2016 at 18:08, Liviu Dudau wrote: > On Mon, Apr 25, 2016 at 05:00:02PM +0100, Emil Velikov wrote: >> On 25 April 2016 at 15:19, Liviu Dudau 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 >> > --- >> > 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 >> > 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 >> > +M: Brian Starkey >> > +M: Mali DP Maintainers >> > +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). > You're absolutely right. I read things the wrong way - the MaliDP maintainers will be added for the HDLCD patches. Which obviously isn't the case. > 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. > IIRC DRM had the same structure (all drivers living in a single a folder) before 2008, before people got fed up. Feel free to do as you wish, just pointing out some historical observations ;-) Regardless of my suggestions/nitpicks, thanks for all the work Liviu ! It's really nice to see more vendors contributing to DRM. Regards, Emil From mboxrd@z Thu Jan 1 00:00:00 1970 From: Emil Velikov Subject: Re: [PATCH v2 3/3] MAINTAINERS: Add entry for Mali-DP driver Date: Mon, 25 Apr 2016 18:53:26 +0100 Message-ID: References: <1461593964-27722-1-git-send-email-Liviu.Dudau@arm.com> <1461593964-27722-4-git-send-email-Liviu.Dudau@arm.com> <20160425170811.GC28464@e106497-lin.cambridge.arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <20160425170811.GC28464-2JSQmVVBSi7ZROr8t4l/smS4ubULX0JqMm0uRHvK7Nw@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Liviu Dudau Cc: Rob Herring , Pawel Moll , Mark Rutland , devicetree , David Airlie , DRI devel , LKML , Daniel Vetter , Brian Starkey , David Brown List-Id: devicetree@vger.kernel.org On 25 April 2016 at 18:08, Liviu Dudau wrote: > On Mon, Apr 25, 2016 at 05:00:02PM +0100, Emil Velikov wrote: >> On 25 April 2016 at 15:19, Liviu Dudau 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 >> > --- >> > 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 >> > 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 >> > +M: Brian Starkey >> > +M: Mali DP Maintainers >> > +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). > You're absolutely right. I read things the wrong way - the MaliDP maintainers will be added for the HDLCD patches. Which obviously isn't the case. > 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. > IIRC DRM had the same structure (all drivers living in a single a folder) before 2008, before people got fed up. Feel free to do as you wish, just pointing out some historical observations ;-) Regardless of my suggestions/nitpicks, thanks for all the work Liviu ! It's really nice to see more vendors contributing to DRM. Regards, Emil -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html