Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759513AbaGCVwN (ORCPT ); Thu, 3 Jul 2014 17:52:13 -0400 Received: from gw-1.arm.linux.org.uk ([78.32.30.217]:53521 "EHLO pandora.arm.linux.org.uk" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753201AbaGCVwL (ORCPT ); Thu, 3 Jul 2014 17:52:11 -0400 Date: Thu, 3 Jul 2014 22:52:00 +0100 From: Russell King - ARM Linux To: Rob Clark Cc: Benjamin Gaignard , Laurent Pinchart , Thierry Reding , "dri-devel@lists.freedesktop.org" , "linaro-mm-sig@lists.linaro.org" , Linux Kernel Mailing List , David Airlie Subject: Re: [PATCH v5 00/11] Add DRM for stih4xx platforms Message-ID: <20140703215200.GG32514@n2100.arm.linux.org.uk> References: <1403081488-13114-1-git-send-email-benjamin.gaignard@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.19 (2009-01-05) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/