linux-kernel - Re: [PATCH v5 00/11] Add DRM for stih4xx platforms

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:	Thu, 3 Jul 2014 22:52:00 +0100
From:	Russell King - ARM Linux <linux@....linux.org.uk>
To:	Rob Clark <robdclark@...il.com>
Cc:	Benjamin Gaignard <benjamin.gaignard@...aro.org>,
	Laurent Pinchart <laurent.pinchart@...asonboard.com>,
	Thierry Reding <thierry.reding@...il.com>,
	"dri-devel@...ts.freedesktop.org" <dri-devel@...ts.freedesktop.org>,
	"linaro-mm-sig@...ts.linaro.org" <linaro-mm-sig@...ts.linaro.org>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	David Airlie <airlied@...ux.ie>
Subject: Re: [PATCH v5 00/11] Add DRM for stih4xx platforms

On Thu, Jul 03, 2014 at 05:31:21PM -0400, Rob Clark wrote:
> >From a brief look, it looks like my comments have been addressed, so I
> think this is starting to shape up..
> 
> Laurent/Thierry/Russell, I don't suppose any of you are likely to have
> time before 3.17 merge window to give sti one last once-over to see
> what I missed ;-)

As this appears to be using the component helpers, it would be good for
the author to be pre-warned about the change to the API.  I'm going to
be doing this as a two step thing.

The first stage of it has just been taken by Greg this evening.  This
retires old behaviour that was needed for the initial versions of imx-drm,
and fixes a bug.  More importantly, it introduces a new interface where
you build the list of expected components before calling
component_master_add(), passing it into that function.

It is slightly less flexible from the driver writer's view point, but
allows us to do more in the component helper, and support some features
that others would like to see (such as being able to add additional
optional components after the initial bind (that's not much interest
to DRM though.)

This should make the v3.17 merge window.  I'm hoping that by forewarning
about the change, that people can be aware of it and have patches
prepared for v3.18 (or maybe even v3.17) to convert over to it, so we
can get rid of the older interface before we get too many users of it.

I've just peeked at the binding document in the first patch.  I notice
that it isn't making use of the of_graph stuff, which is a bit of a
shame because I've sent a RFC patch adding drivers/gpu/drm/drm_of.c
to provide a helper which encoders can use to get the CRTC mask from
an of_graph representation.

I'm using this not only for imx-drm, but also for tda998x - and that's
where the problems are likely to start if we end up with different
DRM implementations doing different solutions to the connectivity
problem.

-- 
FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly
improving, and getting towards what was expected from it.
--
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/