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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CACvgo50nqzcSbJVSBx_dk+yUTxKpNLXs0hbdOjX_X3OYwt2+Tg@mail.gmail.com>
Date:	Mon, 25 Apr 2016 18:53:26 +0100
From:	Emil Velikov <emil.l.velikov@...il.com>
To:	Liviu Dudau <Liviu.Dudau@....com>
Cc:	Rob Herring <robh+dt@...nel.org>, Pawel Moll <pawel.moll@....com>,
	Mark Rutland <mark.rutland@....com>,
	devicetree <devicetree@...r.kernel.org>,
	David Airlie <airlied@...ux.ie>,
	DRI devel <dri-devel@...ts.freedesktop.org>,
	LKML <linux-kernel@...r.kernel.org>,
	Daniel Vetter <daniel@...ll.ch>,
	Brian Starkey <Brian.Starkey@....com>,
	David Brown <David.Brown@....com>
Subject: Re: [PATCH v2 3/3] MAINTAINERS: Add entry for Mali-DP driver

On 25 April 2016 at 18:08, Liviu Dudau <Liviu.Dudau@....com> wrote:
> On Mon, Apr 25, 2016 at 05:00:02PM +0100, Emil Velikov wrote:
>> On 25 April 2016 at 15:19, Liviu Dudau <Liviu.Dudau@....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@....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@....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@....com>
>> > +M:     Brian Starkey <brian.starkey@....com>
>> > +M:     Mali DP Maintainers <malidp@...s.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).
>
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