linux-kernel - Re: [PATCH v4 0/4] drm: Add support for the ARM HDLCD display controller

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: <20151203165606.GS8644@n2100.arm.linux.org.uk>
Date:	Thu, 3 Dec 2015 16:56:07 +0000
From:	Russell King - ARM Linux <linux@....linux.org.uk>
To:	Liviu Dudau <Liviu.Dudau@....com>
Cc:	David Airlie <airlied@...ux.ie>,
	Catalin Marinas <catalin.marinas@....com>,
	Will Deacon <will.deacon@....com>,
	Rob Herring <robh+dt@...nel.org>,
	Sudeep Holla <sudeep.holla@....com>,
	Jon Medhurst <tixy@...aro.org>,
	Mark Rutland <mark.rutland@....com>,
	Ian Campbell <ijc+devicetree@...lion.org.uk>,
	Kumar Gala <galak@...eaurora.org>,
	Rob Herring <robh@...nel.org>, Pawel Moll <pawel.moll@....com>,
	Arnd Bergmann <arnd@...db.de>, Olof Johansson <olof@...om.net>,
	Punit Agrawal <punit.agrawal@....com>,
	DRI devel <dri-devel@...ts.freedesktop.org>,
	devicetree <devicetree@...r.kernel.org>,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	Andrew Morton <akpm@...ux-foundation.org>,
	LAKML <linux-arm-kernel@...ts.infradead.org>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v4 0/4] drm: Add support for the ARM HDLCD display
 controller

On Thu, Dec 03, 2015 at 10:40:45AM +0000, Liviu Dudau wrote:
> This series depends on Sudeep Holla's SCPI driver (now in mainline) and on
> the tda998x patches that have been queued on Russell's patch system here [1].

Now merged into my tree.

Can I ask a fairly obvious question...

>  drivers/gpu/drm/Kconfig                            |   2 +
>  drivers/gpu/drm/Makefile                           |   1 +
>  drivers/gpu/drm/arm/Kconfig                        |  29 ++
>  drivers/gpu/drm/arm/Makefile                       |   2 +
>  drivers/gpu/drm/arm/hdlcd_crtc.c                   | 327 ++++++++++++
>  drivers/gpu/drm/arm/hdlcd_drv.c                    | 555 +++++++++++++++++++++
>  drivers/gpu/drm/arm/hdlcd_drv.h                    |  42 ++
>  drivers/gpu/drm/arm/hdlcd_regs.h                   |  87 ++++

Why is the subdirectory called "arm" and not "hdlcd" ?

ARM Ltd have more than one display controller (they have the AMBA PL110
and PL111 in addition to this), and (afaics) this doesn't drive these.
HDLCD probably won't be the last display hardware which ARM Ltd comes
out with either.

So, I think naming the subdirectory after the vendor is probably a
mistake.

-- 
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/