From: Stefan Agner <stefan-XLVq0VzYD2Y@public.gmane.org>
To: Thierry Reding
<thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
Philipp Zabel <p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
Cc: swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org,
linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
marcel-mitwqZ+T+m9Wk0Htik3J/w@public.gmane.org
Subject: Re: [PATCH v2 2/3] drm/panel: add support for EDT panels
Date: Thu, 15 May 2014 10:24:39 +0200 [thread overview]
Message-ID: <4de1d09b7873e012e43525c1f8ab9c66@agner.ch> (raw)
In-Reply-To: <20140515075102.GE5952@ulmo>
Hi Thierry, hi Philipp,
Am 2014-05-15 09:51, schrieb Thierry Reding:
> On Wed, May 14, 2014 at 11:45:57PM +0200, stefan-XLVq0VzYD2Y@public.gmane.org wrote:
>> From: Stefan Agner <stefan-XLVq0VzYD2Y@public.gmane.org>
>>
>> This panels are sold by Toradex for Colibri T20/T30 and Apalis T30
>> evaluation kits.
>>
>> Signed-off-by: Stefan Agner <stefan-XLVq0VzYD2Y@public.gmane.org>
>
> Panel patches should go to the dri-devel mailing list as well.
>
> Also a patch was posted only yesterday for a panel that seems to be the
> exact same one as this, even though the name differs minimally, see:
>
> https://patchwork.kernel.org/patch/4175251/
>
> Adding Philipp on Cc so you guys can work together whether this is
> indeed the same panel. The only differences seem to be in the vertical
> front and back porches, but I suspect that either settings will work on
> both Tegra and i.MX.
Its etm0700g0dh6 vs. et070080dh6, Philipp's panel is with captive multi
touch, (hence the M I guess). The panel itself really looks the same.
I found this overview of EDT displays:
http://www.dmbtechnics.com/scripts/passthru.php?id=7
There seem to be quite a lot variants with the same panel...
Regarding timings, I just checked the documentation, Philipp's timing
really matches the documented ones, I miscalculated the vertical back
porch.
How do we resolve that?
I would suggest that I split out that patch and remove the et070080dh6
panel and send the other as a single patch to the dri-devel mailing list
as well.
Philipp, could you add my display type (et070080dh6) to the compatible
list of your mode/panel entry?
Does this sound reasonable?
--
Stefan
next prev parent reply other threads:[~2014-05-15 8:24 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-14 21:45 [PATCH v2 0/4] ARM: tegra: initial support for Colibri T30 stefan
2014-05-14 21:45 ` [PATCH v2 2/3] drm/panel: add support for EDT panels stefan
[not found] ` <35c081171a3d9a9401767601c02a12e266b37dd0.1400103181.git.stefan-XLVq0VzYD2Y@public.gmane.org>
2014-05-15 7:51 ` Thierry Reding
2014-05-15 8:24 ` Stefan Agner [this message]
2014-05-15 9:12 ` Thierry Reding
2014-05-15 9:35 ` Philipp Zabel
[not found] ` <1400146519.4102.30.camel-+qGW7pzALmz7o/J7KWpOmN53zsg1cpMQ@public.gmane.org>
2014-05-15 9:39 ` Stefan Agner
2014-05-15 8:27 ` Philipp Zabel
[not found] ` <1400142449.4102.16.camel-+qGW7pzALmz7o/J7KWpOmN53zsg1cpMQ@public.gmane.org>
2014-05-15 9:08 ` Thierry Reding
2014-05-15 9:30 ` Philipp Zabel
2014-05-15 9:36 ` Stefan Agner
[not found] ` <cover.1400103181.git.stefan-XLVq0VzYD2Y@public.gmane.org>
2014-05-14 21:45 ` [PATCH v2 1/3] ARM: tegra: enable MCP251x CAN controller and DS1307 RTC stefan-XLVq0VzYD2Y
[not found] ` <4cc13e0aba03a5b2d3e5f4049d252956342b1dad.1400103181.git.stefan-XLVq0VzYD2Y@public.gmane.org>
2014-05-15 18:15 ` Stephen Warren
2014-05-14 21:45 ` [PATCH v2 3/3] ARM: tegra: initial add of Colibri T30 stefan-XLVq0VzYD2Y
[not found] ` <a8f01e573dcb7d563833ddcda30567e8ac7626ed.1400103181.git.stefan-XLVq0VzYD2Y@public.gmane.org>
2014-05-15 18:37 ` Stephen Warren
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=4de1d09b7873e012e43525c1f8ab9c66@agner.ch \
--to=stefan-xlvq0vzyd2y@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org \
--cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=marcel-mitwqZ+T+m9Wk0Htik3J/w@public.gmane.org \
--cc=p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org \
--cc=swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org \
--cc=thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@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).