dri-devel.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
From: Philipp Zabel <p.zabel@pengutronix.de>
To: Boris Brezillon <boris.brezillon@free-electrons.com>
Cc: Steffen Trumtrar <s.trumtrar@pengutronix.de>,
	dri-devel@lists.freedesktop.org, kernel@pengutronix.de,
	Anthony Harivel <anthony.harivel@emtrion.de>
Subject: Re: [PATCH 1/3] drm/panel: Add display_timing support
Date: Thu, 26 Feb 2015 19:33:04 +0100	[thread overview]
Message-ID: <1424975584.3073.56.camel@pengutronix.de> (raw)
In-Reply-To: <20150226145113.56e633a3@bbrezillon>

Hi Boris,

Am Donnerstag, den 26.02.2015, 14:51 +0100 schrieb Boris Brezillon:
> Hi Philipp,
> 
> On Mon, 23 Feb 2015 15:04:32 +0100
> Philipp Zabel <p.zabel@pengutronix.de> wrote:
> 
> > Hi Thierry,
> > 
> > do you have any further thoughts on this?
> > 
> > Am Dienstag, den 03.02.2015, 14:30 +0100 schrieb Thierry Reding:
> > > On Thu, Dec 11, 2014 at 06:32:44PM +0100, Philipp Zabel wrote:
> > > > Many panel data sheets additionally to typical values list allowed ranges for
> > > > timings such as hsync/vsync lengths, porches, and the pixel clock rate. These
> > > > can be stored in a struct display_timing, to be used by an encoder mode_fixup
> > > > callback to clamp user provided timing values or to validate workarounds for
> > > > clock source limitations.
> > > > 
> > > > This patch adds a new drm_panel_funcs callback that returns the panels'
> > > > available display_timing entries.
> 
> Anthony recently noticed that the edt_etm0700g0dh6 panel he's
> connecting on his sama5 board defines typical timings that are not
> supported by Atmel's display controller.
> This patch could help us deal with such cases.
> Could you keep me (and Anthony) in Cc of your future versions ?

Will do.

regards
Philipp

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2015-02-26 18:33 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-11 17:32 [PATCH 1/3] drm/panel: Add display_timing support Philipp Zabel
2014-12-11 17:32 ` [PATCH 2/3] drm/panel: Add display_timing support to simple panel driver Philipp Zabel
2014-12-11 17:32 ` [PATCH 3/3] drm/panel: Add display_timing entry for the HannStar HSD070PWW1 panel Philipp Zabel
2015-02-03 13:30 ` [PATCH 1/3] drm/panel: Add display_timing support Thierry Reding
2015-02-03 16:56   ` Philipp Zabel
2015-02-23 14:04   ` Philipp Zabel
2015-02-26 13:51     ` Boris Brezillon
2015-02-26 18:33       ` Philipp Zabel [this message]
2015-03-03 11:49     ` Philipp Zabel
2015-03-24 11:34       ` Thierry Reding
2015-03-24 11:52         ` Philipp Zabel
2015-03-24 12:40           ` Thierry Reding
2015-03-24 16:36             ` Philipp Zabel

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=1424975584.3073.56.camel@pengutronix.de \
    --to=p.zabel@pengutronix.de \
    --cc=anthony.harivel@emtrion.de \
    --cc=boris.brezillon@free-electrons.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=kernel@pengutronix.de \
    --cc=s.trumtrar@pengutronix.de \
    /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).