[PATCH 1/3] drm/panel: Add display_timing support

2015-03-24 Thread Philipp Zabel
Hi Thierry, Am Dienstag, den 24.03.2015, 13:40 +0100 schrieb Thierry Reding: > On Tue, Mar 24, 2015 at 12:52:44PM +0100, Philipp Zabel wrote: > > Am Dienstag, den 24.03.2015, 12:34 +0100 schrieb Thierry Reding: > > [...] > > > > > Would you prefer > > > > > struct display_timing

[PATCH 1/3] drm/panel: Add display_timing support

2015-03-24 Thread Thierry Reding
On Tue, Mar 24, 2015 at 12:52:44PM +0100, Philipp Zabel wrote: > Am Dienstag, den 24.03.2015, 12:34 +0100 schrieb Thierry Reding: > [...] > > > > Would you prefer > > > > struct display_timing *(*get_timing)(struct drm_panel *panel); > > > > ? > > > > > > I'd like to resend this. Please

[PATCH 1/3] drm/panel: Add display_timing support

2015-03-24 Thread Philipp Zabel
Am Dienstag, den 24.03.2015, 12:34 +0100 schrieb Thierry Reding: [...] > > > Would you prefer > > > struct display_timing *(*get_timing)(struct drm_panel *panel); > > > ? > > > > I'd like to resend this. Please let me know if you want me to change > > this function prototype. > > I have no

[PATCH 1/3] drm/panel: Add display_timing support

2015-03-24 Thread Thierry Reding
On Tue, Mar 03, 2015 at 12:49:43PM +0100, Philipp Zabel wrote: > Hi Thierry, > > Am Montag, den 23.02.2015, 15:04 +0100 schrieb Philipp Zabel: > > Hi Thierry, > > > > do you have any further thoughts on this? > > [...] > > > Sorry for taking so long to get back on this. I generally like the

[PATCH 1/3] drm/panel: Add display_timing support

2015-03-03 Thread Philipp Zabel
Hi Thierry, Am Montag, den 23.02.2015, 15:04 +0100 schrieb Philipp Zabel: > Hi Thierry, > > do you have any further thoughts on this? [...] > > Sorry for taking so long to get back on this. I generally like the idea, > > though a couple of things are unclear to me. > > > > In struct

[PATCH 1/3] drm/panel: Add display_timing support

2015-02-26 Thread Philipp Zabel
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 wrote: > > > Hi Thierry, > > > > do you have any further thoughts on this? > > > > Am Dienstag, den 03.02.2015, 14:30 +0100 schrieb Thierry

[PATCH 1/3] drm/panel: Add display_timing support

2015-02-26 Thread Boris Brezillon
Hi Philipp, On Mon, 23 Feb 2015 15:04:32 +0100 Philipp Zabel 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

[PATCH 1/3] drm/panel: Add display_timing support

2015-02-23 Thread Philipp Zabel
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

[PATCH 1/3] drm/panel: Add display_timing support

2015-02-03 Thread Philipp Zabel
Hi Thierry, 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

[PATCH 1/3] drm/panel: Add display_timing support

2015-02-03 Thread 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

[PATCH 1/3] drm/panel: Add display_timing support

2014-12-11 Thread Philipp Zabel
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