Message-ID: <55A67FDB.8010602@gmx.at>
Date: Wed, 15 Jul 2015 17:44:27 +0200
From: Manfred Schlaegl <manfred.schlaegl@gmx.at>
User-Agent: Mozilla/5.0 (X11; Linux i686; rv:31.0) Gecko/20100101 Icedove/31.7.0
MIME-Version: 1.0
To: David Airlie <airlied@linux.ie>, Philipp Zabel <p.zabel@pengutronix.de>,
        Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>,
        Tomi Valkeinen <tomi.valkeinen@ti.com>
CC: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org,
        linux-api@vger.kernel.org,
        Manfred Schlaegl <manfred.schlaegl@ginzinger.com>,
        Steve Longerbeam <slongerbeam@gmail.com>,
        Deepak Das <deepak_das@mentor.com>, Jiada Wang <jiada_wang@mentor.com>,
        linux-fbdev@vger.kernel.org
Subject: [RFC PATCH 0/2] drm: add support for for clk and de polarity & gpu:
 ipu-v3: use clock and de polarity
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2076
Lines: 52

Hello!

These patches address a problem we ran into using parallel displays
with Freescale i.MX53 and i.MX6 SoC's.
In short: We wanted to change the clock signal polarity by using
display-timing in the devicetree description, but the output signal stayed
unchanged.

Parallel displays may have different polarities for clock and data enable
signals. (Clock polarity is also a topic on LVDS displays). This is the
reason why there are some properties in devicetree display-timing [1]
called
 * pixelclk-active .. pixel clock polarity
 * de-active .. data enable pulse polarity

This properties are correctly represented in struct display_timings [2]
and struct videomode [3] by using enum display_flags [3].
But when it comes to struct drm_display_mode [4] there are no
representations for this.
The properties are "lost in conversion" and never reach imx drm. (or 
other drm drivers).

Changing this would be especially important for embedded devices where
parallel(RGB) and LVDS displays are still widely used and drm already
plays an important role.

Following two patches will
 1. Introduce representation of clock and data enable polarities
     in struct drm_display_mode analog to hsync/vsync signals.
   * "drm: add support for for clk and de polarity"
 2. Implicitly enable usage of the newly introduced flags in 
     imx drm (with respect to devicetree compatibility.)
   * "gpu: ipu-v3: use clock and de polarity from videomode"

I'm aware that introducing new flags in struct drm_display_mode affects
user space too and therefore has to be considered carefully.
As I'm not really deep in drm user space your suggestions are very
welcome.

best regards,
manfred


[1] Documentation/devicetree/bindings/video/display-timing.txt
[2] include/video/display_timing.h
[3] include/video/videomode.h
[4] include/drm/drm_modes.h
--
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/