linux-kernel - Re: [PATCH 4/4] drm/panel: Add helper for simple panel connector

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]

Date:	Fri, 6 May 2016 16:15:32 +0200
From:	Thierry Reding <thierry.reding@...il.com>
To:	Noralf Trønnes <noralf@...nnes.org>,
	treding@...dia.com, linux-kernel@...r.kernel.org,
	dri-devel@...ts.freedesktop.org
Subject: Re: [PATCH 4/4] drm/panel: Add helper for simple panel connector

On Fri, May 06, 2016 at 04:08:16PM +0200, Daniel Vetter wrote:
> On Fri, May 06, 2016 at 04:03:47PM +0200, Thierry Reding wrote:
> > On Thu, May 05, 2016 at 03:24:34PM +0200, Noralf Trønnes wrote:
> > > Add function to create a simple connector for a panel.
> > 
> > I'm not sure I see the usefulness of this. Typically you'd attach a
> > panel to an encoder/connector, in which case you already have the
> > connector.
> > 
> > Perhaps it would become more obvious why we need this if you posted
> > patches that show where this is used?
> 
> The other helpers give you a simple drm pipeline with plane, crtc &
> encoder all baked into on drm_simple_pipeline structure. The only thing
> variable you have to hook up to that is the drm_connector. And I think for
> dead-simple panels avoiding the basic boilerplate in that does indeed make
> some sense.

Avoiding boilerplate is good, but I have a difficult time envisioning
how you might want to use this. At the same time I'm asking myself how
we know that this helper is any good if we haven't seen it used anywhere
and actually see the boilerplate go away.

Thierry

Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)