From mboxrd@z Thu Jan 1 00:00:00 1970 From: Manfred Schlaegl Date: Thu, 26 Nov 2015 14:20:51 +0000 Subject: Re: [RFC PATCH 1/2] drm: add support for for clk and de polarity Message-Id: <56571543.20701@gmx.at> List-Id: References: <55A67FDB.8010602@gmx.at> <55A6813A.8090706@gmx.at> <1448472141.3390.17.camel@pengutronix.de> In-Reply-To: <1448472141.3390.17.camel@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Philipp Zabel Cc: linux-fbdev@vger.kernel.org, linux-api@vger.kernel.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Manfred Schlaegl , Tomi Valkeinen , Steve Longerbeam , Deepak Das , Jean-Christophe Plagniol-Villard On 2015-11-25 18:22, Philipp Zabel wrote: > Am Mittwoch, den 15.07.2015, 17:50 +0200 schrieb Manfred Schlaegl: >> To get full support for parallel and LVDS displays with drm: >> Add representation for clock and data enable polarity in drm_display_mode >> flags (similar to HSYNC/VSYNC polarity) and update conversion functions >> from/to videomode accordingly. >> >> This is especially important for embedded devices where parallel(RGB) and >> LVDS displays are still widely used and drm already plays an important >> role. >> >> Tested on Freescale i.MX53(parallel) and i.MX6(LVDS). >> >> Background: >> There was the ability to set polarity of clock and data enable signals >> in devicetree(display-timing), struct display_timing and struct videomode, >> but there was no representation for this in struct drm_display_mode. >> Example on Freescale i.MX53/i.MX6 SoC's: >> * A parallel display using different clock polarity is set up using >> display-timing in devicetree >> * ipuv3 parallel outputs clock with wrong polarity >> >> Signed-off-by: Manfred Schlaegl > > Any comments on whether data enable and pixel clock polarity flags can > be added to the visible DRM_MODE_FLAGs, and if not, where else this > information should be kept? struct drm_display_info? > > This patch and the following IPUv3 patch are useful and necessary for > quite some panels connected to i.MX SoCs, but adding DRM_MODE_FLAGs is > somewhat out of my jurisdiction. > > best regards > Philipp > Good to see that this discussion is triggered. State of the sent patches respective to git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git (78c4a49) * [RFC PATCH 1/2] drm: add support for for clk and de polarity * applies unmodified * [RFC PATCH 2/2] gpu: ipu-v3: use clock and de polarity from videomode * does not apply -> but this can be fixed with corrections of line offsets (using patch) >From a code review I think the modifications must work, but I did no testing on kernel versions newer than 4.1.13. On demand, I can redo tests on some of our i.MX53 (parallel) and i.MX6 (lvds) boards and send actual patches. best regards, Manfred