From: Thierry Reding <thierry.reding@gmail.com>
To: Andrzej Hajda <a.hajda@samsung.com>
Cc: mark.rutland@arm.com, devicetree@vger.kernel.org,
linux-samsung-soc@vger.kernel.org, pawel.moll@arm.com,
ijc+devicetree@hellion.org.uk, sw0312.kim@samsung.com,
dri-devel@lists.freedesktop.org, kyungmin.park@samsung.com,
robh+dt@kernel.org, galak@codeaurora.org, kgene.kim@samsung.com,
s.trumtrar@pengutronix.de, laurent.pinchart@ideasonboard.com
Subject: Re: [PATCH v2 13/18] ARM: dts: s6e3fa0: add DT bindings
Date: Tue, 27 May 2014 09:53:29 +0200 [thread overview]
Message-ID: <20140527075328.GA13081@ulmo> (raw)
In-Reply-To: <538430A4.8020702@samsung.com>
[-- Attachment #1.1: Type: text/plain, Size: 1500 bytes --]
On Tue, May 27, 2014 at 08:28:52AM +0200, Andrzej Hajda wrote:
> Hi Thierry,
>
> On 05/26/2014 03:41 PM, Thierry Reding wrote:
> > On Wed, May 21, 2014 at 01:43:05PM +0900, YoungJun Cho wrote:
> >> This patch adds DT bindings for s6e3fa0 panel.
> >> The bindings describes panel resources, display timings and cpu mode timings.
> >>
> >> Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>
> >> Acked-by: Inki Dae <inki.dae@samsung.com>
> >> Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
> >> ---
> >> .../devicetree/bindings/panel/samsung,s6e3fa0.txt | 45 ++++++++++++++++++++
> >> 1 file changed, 45 insertions(+)
> >> create mode 100644 Documentation/devicetree/bindings/panel/samsung,s6e3fa0.txt
> > You're totally confusing me here. Half of this patch series is about
> > adding i80 support to Exynos FIMD, and then you go and add what is
> > apparently a DSI peripheral driver here that's supposed to be used by
> > this new i80 support. Nothing I've been able to dig up indicates that
> > i80 or DSI are in anyway related.
>
> FIMD can produce parallel RGB output or command mode in i80 style output
> via parallel lines.
> DSIM can accept parallel RGB stream in this case it produces MIPI DSI
> video mode signal or it can accept i80 and in this case it translates it
> to MIPI DSI command mode.
Then the command mode timings aren't a property of the panel at all.
They describe what DSIM expects, so that's where they should be defined.
Thierry
[-- Attachment #1.2: Type: application/pgp-signature, Size: 836 bytes --]
[-- Attachment #2: Type: text/plain, Size: 159 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2014-05-27 7:53 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-21 4:42 [PATCH v2 00/18] drm/exynos: support MIPI DSI command mode display YoungJun Cho
2014-05-21 4:42 ` [PATCH v2 01/18] drm/exynos: dsi: move the EoT packets configuration point YoungJun Cho
2014-05-21 4:42 ` [PATCH v2 02/18] drm/exynos: use wait_event_timeout() for safety usage YoungJun Cho
2014-05-21 6:01 ` Daniel Kurtz
2014-05-21 6:28 ` YoungJun Cho
2014-05-21 6:45 ` Daniel Kurtz
2014-05-21 10:37 ` YoungJun Cho
2014-05-21 4:42 ` [PATCH v2 04/18] video: add command mode and command mode display timing YoungJun Cho
2014-05-21 11:02 ` Thierry Reding
2014-05-21 11:41 ` YoungJun Cho
2014-05-22 13:07 ` Inki Dae
2014-05-22 13:53 ` Thierry Reding
2014-05-26 10:14 ` Thierry Reding
2014-05-26 11:10 ` Heiko Stübner
2014-05-21 4:42 ` [PATCH v2 05/18] drm_modes: add command mode helpers YoungJun Cho
2014-05-21 4:42 ` [PATCH v2 06/18] ARM: dts: sysreg: add exynos5 compatible to DT bindings YoungJun Cho
2014-05-21 4:42 ` [PATCH v2 07/18] ARM: dts: samsung-fimd: add I80 specific properties YoungJun Cho
2014-05-21 4:43 ` [PATCH v2 09/18] drm/exynos: dsi: add TE handler to support command mode interface YoungJun Cho
2014-05-21 4:43 ` [PATCH v2 10/18] drm/exynos: fimd: support I80 interface YoungJun Cho
2014-05-26 9:00 ` Daniel Kurtz
2014-05-29 5:45 ` YoungJun Cho
2014-05-21 4:43 ` [PATCH v2 11/18] ARM: dts: exynos_dsim: add exynos5420 compatible to DT bindings YoungJun Cho
2014-05-21 4:43 ` [PATCH v2 12/18] drm/exynos: dsi: add driver data to support Exynos5420 YoungJun Cho
2014-05-21 4:43 ` [PATCH v2 13/18] ARM: dts: s6e3fa0: add DT bindings YoungJun Cho
2014-05-26 13:41 ` Thierry Reding
2014-05-27 6:28 ` Andrzej Hajda
2014-05-27 7:53 ` Thierry Reding [this message]
2014-05-27 14:24 ` Inki Dae
2014-05-27 20:21 ` Thierry Reding
2014-05-28 4:50 ` Inki Dae
2014-05-28 6:44 ` Andrzej Hajda
2014-05-30 3:08 ` YoungJun Cho
2014-05-21 4:43 ` [PATCH v2 14/18] drm/panel: add S6E3FA0 driver YoungJun Cho
[not found] ` <1400647390-26590-1-git-send-email-yj44.cho-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2014-05-21 4:42 ` [PATCH v2 03/18] ARM: dts: video: add command mode display timing DT bindings YoungJun Cho
2014-05-21 4:43 ` [PATCH v2 08/18] drm/exynos: add TE handler to support MIPI DSI command mode interface YoungJun Cho
2014-05-21 4:43 ` [PATCH v2 15/18] ARM: dts: exynos4: add system register node YoungJun Cho
2014-05-21 4:43 ` [PATCH v2 16/18] ARM: dts: exynos5: add system register support YoungJun Cho
2014-05-21 4:43 ` [PATCH v2 17/18] ARM: dts: exynos5420: add mipi-phy node YoungJun Cho
2014-05-21 4:43 ` [PATCH v2 18/18] ARM: dts: exynos5420: add dsi node YoungJun Cho
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=20140527075328.GA13081@ulmo \
--to=thierry.reding@gmail.com \
--cc=a.hajda@samsung.com \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=galak@codeaurora.org \
--cc=ijc+devicetree@hellion.org.uk \
--cc=kgene.kim@samsung.com \
--cc=kyungmin.park@samsung.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=pawel.moll@arm.com \
--cc=robh+dt@kernel.org \
--cc=s.trumtrar@pengutronix.de \
--cc=sw0312.kim@samsung.com \
/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).