From: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
To: Maxime Ripard
<maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
Cc: Daniel Vetter
<daniel.vetter-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
David Airlie <airlied-cv59FeDIM0c@public.gmane.org>,
Chen-Yu Tsai <wens-jdAy2FN1RRM@public.gmane.org>,
dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
plaes-q/aMd4JkU83YtjvyW6yDsg@public.gmane.org,
icenowy-h8G6r0blFSE@public.gmane.org,
Thomas Petazzoni
<thomas.petazzoni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
jernej.skrabec-gGgVlfcn5nU@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH v3 04/15] dt-bindings: display: sun4i-drm: Add A83T pipeline
Date: Wed, 6 Dec 2017 15:59:22 -0600 [thread overview]
Message-ID: <20171206215922.vtsa4xqa2e5xz7cp@rob-hp-laptop> (raw)
In-Reply-To: <3762a8428aaf207e80ae86d4db2e50473576bb6d.1512486553.git-series.maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
On Tue, Dec 05, 2017 at 04:10:16PM +0100, Maxime Ripard wrote:
> The A83T has two video pipelines in parallel that looks quite similar to
> the other SoCs.
>
> The video planes are handled through a controller called the mixer, and the
> video signal is then passed to the timing controller (TCON).
>
> And while there is two instances of the mixers and TCONs, they have a
> significant number of differences. The TCONs are quite easy to deal with,
> one is supposed to generate TV (in the broader term, so including things
> like HDMI) signals, the other one LCD (so RGB, LVDS, DSI) signals. And
> while they are called TCON0 and TCON1 in the A83t datasheet, newer SoCs
> call them TCON-TV and TCON-LCD, which seems more appropriate.
>
> However, the mixers differ mostly by their capabilities, with some features
> being available only in the first one, or the number of planes they expose,
> but also through their register layout. And while the capabilities could be
> represented as properties, the register layout differences would need to
> express all the registers offsets as properties, which is usually quite
> bad. Especially since documentation on that hardware block is close to
> non-existant and we don't even have the list of all those registers in the
> first place.
>
> So let's call them mixer 0 and 1 in our compatibles, even though the name
> is pretty bad...
>
> At the moment, we only have tested the code on a board that has a single
> display output, so we're leaving the tcon-tv and mixer1 out.
>
> Signed-off-by: Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
> ---
> Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt | 3 +++
> 1 file changed, 3 insertions(+)
Reviewed-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2017-12-06 21:59 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-05 15:10 [PATCH v3 00/15] drm/sun4i: Add A83t LVDS support Maxime Ripard
2017-12-05 15:10 ` [PATCH v3 01/15] dt-bindings: panel: lvds: Document power-supply property Maxime Ripard
[not found] ` <97ebf0615821d79e8d7718594fe3a542a743a05f.1512486553.git-series.maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2017-12-06 21:56 ` Rob Herring
2017-12-05 15:10 ` [PATCH v3 03/15] dt-bindings: display: sun4i-drm: Add LVDS properties Maxime Ripard
[not found] ` <17851ba6277b69aa9cd81de5eead62bfed271661.1512486553.git-series.maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2017-12-06 21:57 ` Rob Herring
2017-12-07 3:57 ` Chen-Yu Tsai
2017-12-05 15:10 ` [PATCH v3 04/15] dt-bindings: display: sun4i-drm: Add A83T pipeline Maxime Ripard
[not found] ` <3762a8428aaf207e80ae86d4db2e50473576bb6d.1512486553.git-series.maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2017-12-06 21:59 ` Rob Herring [this message]
2017-12-07 3:37 ` Chen-Yu Tsai
2017-12-05 15:10 ` [PATCH v3 05/15] drm/sun4i: Fix error path handling Maxime Ripard
[not found] ` <1f2443c41082fc6e086570da38d20744c52fe636.1512486553.git-series.maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2017-12-05 15:42 ` Chen-Yu Tsai
2017-12-05 15:10 ` [PATCH v3 06/15] drm/sun4i: Force the mixer rate at 150MHz Maxime Ripard
2017-12-05 15:10 ` [PATCH v3 07/15] drm/sun4i: Create minimal multipliers and dividers Maxime Ripard
[not found] ` <cover.825a49a4c1c565a548f3de0375e7537e7e8fd3a4.1512486553.git-series.maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2017-12-05 15:10 ` [PATCH v3 02/15] drm/panel: lvds: Add support for the power-supply property Maxime Ripard
2017-12-05 15:10 ` [PATCH v3 08/15] drm/sun4i: Add LVDS support Maxime Ripard
[not found] ` <b900e5963180c94ca02bde178b6674622a127787.1512486553.git-series.maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2017-12-07 6:05 ` Chen-Yu Tsai
2017-12-07 12:25 ` Maxime Ripard
[not found] ` <20171207122521.frv5zgaay3sgol6f-ZC1Zs529Oq4@public.gmane.org>
2017-12-14 3:30 ` Chen-Yu Tsai
2017-12-14 9:37 ` Maxime Ripard
2017-12-07 10:14 ` Philippe Ombredanne
[not found] ` <CAOFm3uEpnBFUDBHK+SqQ-pGUPcWbGXNrWum4HRJ34tMtoX-2Qw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-12-07 10:50 ` Maxime Ripard
2017-12-05 15:10 ` [PATCH v3 14/15] ARM: dts: sun8i: a711: Reinstate the PMIC compatible Maxime Ripard
[not found] ` <25b482f19587ff195582995c11ee9d07f5bf42d2.1512486553.git-series.maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2017-12-07 3:34 ` Chen-Yu Tsai
2017-12-05 15:10 ` [PATCH v3 09/15] drm/sun4i: Add A83T support Maxime Ripard
2017-12-05 15:42 ` Jernej Škrabec
2017-12-06 16:37 ` Jernej Škrabec
2017-12-07 10:55 ` Maxime Ripard
2017-12-05 15:10 ` [PATCH v3 10/15] ARM: dts: sun8i: a83t: Add display pipeline Maxime Ripard
[not found] ` <97d3cee9d0d7a92893f646d72643bac520de5f05.1512486553.git-series.maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2017-12-07 4:02 ` Chen-Yu Tsai
2017-12-05 15:10 ` [PATCH v3 11/15] ARM: dts: sun8i: a83t: Enable the PWM Maxime Ripard
2017-12-05 15:10 ` [PATCH v3 12/15] ARM: dts: sun8i: a83t: Add LVDS pins group Maxime Ripard
2017-12-05 15:10 ` [PATCH v3 13/15] ARM: dts: sun8i: a83t: Add the PWM pin group Maxime Ripard
2017-12-05 15:10 ` [PATCH v3 15/15] ARM: dts: sun8i: a711: Enable the LCD Maxime Ripard
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171206215922.vtsa4xqa2e5xz7cp@rob-hp-laptop \
--to=robh-dgejt+ai2ygdnm+yrofe0a@public.gmane.org \
--cc=airlied-cv59FeDIM0c@public.gmane.org \
--cc=daniel.vetter-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
--cc=icenowy-h8G6r0blFSE@public.gmane.org \
--cc=jernej.skrabec-gGgVlfcn5nU@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
--cc=maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org \
--cc=plaes-q/aMd4JkU83YtjvyW6yDsg@public.gmane.org \
--cc=thomas.petazzoni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org \
--cc=wens-jdAy2FN1RRM@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).