Re: [PATCH v4] of: Add videomode helper

2012-09-25 Thread Laurent Pinchart
Hi Rob, On Monday 24 September 2012 08:42:12 Rob Herring wrote: > On 09/19/2012 03:20 AM, Steffen Trumtrar wrote: > > This patch adds a helper function for parsing videomodes from the > > devicetree. The videomode can be either converted to a struct > > drm_display_mode or a struct fb_videomode. >

Re: [PATCH v4] of: Add videomode helper

2012-09-25 Thread Steffen Trumtrar
On Mon, Sep 24, 2012 at 05:09:30PM -0600, Stephen Warren wrote: > On 09/24/2012 12:26 PM, Rob Herring wrote: > > On 09/24/2012 10:45 AM, Stephen Warren wrote: > >> On 09/24/2012 07:42 AM, Rob Herring wrote: > >>> On 09/19/2012 03:20 AM, Steffen Trumtrar wrote: > This patch adds a helper functi

Re: [PATCH v4] of: Add videomode helper

2012-09-25 Thread Steffen Trumtrar
On Mon, Sep 24, 2012 at 09:16:28PM +0200, Sascha Hauer wrote: > On Mon, Sep 24, 2012 at 10:18:39AM -0500, Rob Herring wrote: > > On 09/24/2012 09:12 AM, Sascha Hauer wrote: > > >> > > >> A major piece missing is the LCD controller to display interface width > > >> and component ordering. > > > > >

Re: [PATCH v4] of: Add videomode helper

2012-09-25 Thread Stephen Warren
On 09/24/2012 12:26 PM, Rob Herring wrote: > On 09/24/2012 10:45 AM, Stephen Warren wrote: >> On 09/24/2012 07:42 AM, Rob Herring wrote: >>> On 09/19/2012 03:20 AM, Steffen Trumtrar wrote: This patch adds a helper function for parsing videomodes from the devicetree. The videomode ca

Re: [PATCH v4] of: Add videomode helper

2012-09-25 Thread Rob Herring
On 09/24/2012 10:45 AM, Stephen Warren wrote: > On 09/24/2012 07:42 AM, Rob Herring wrote: >> On 09/19/2012 03:20 AM, Steffen Trumtrar wrote: >>> This patch adds a helper function for parsing videomodes from the >>> devicetree. >>> The videomode can be either converted to a struct drm_display_mode

Re: [PATCH v4] of: Add videomode helper

2012-09-24 Thread Sascha Hauer
On Mon, Sep 24, 2012 at 10:18:39AM -0500, Rob Herring wrote: > On 09/24/2012 09:12 AM, Sascha Hauer wrote: > >> > >> A major piece missing is the LCD controller to display interface width > >> and component ordering. > > > > Psst. We silently skipped this for now... > > > > I think having such a

Re: [PATCH v4] of: Add videomode helper

2012-09-24 Thread Stephen Warren
On 09/24/2012 07:42 AM, Rob Herring wrote: > On 09/19/2012 03:20 AM, Steffen Trumtrar wrote: >> This patch adds a helper function for parsing videomodes from the devicetree. >> The videomode can be either converted to a struct drm_display_mode or a >> struct fb_videomode. >> +++ b/Documentation/de

Re: [PATCH v4] of: Add videomode helper

2012-09-24 Thread Rob Herring
On 09/24/2012 09:12 AM, Sascha Hauer wrote: > Hi Rob, > > On Mon, Sep 24, 2012 at 08:42:12AM -0500, Rob Herring wrote: >> On 09/19/2012 03:20 AM, Steffen Trumtrar wrote: >>> This patch adds a helper function for parsing videomodes from the >>> devicetree. >>> The videomode can be either converted

Re: [PATCH v4] of: Add videomode helper

2012-09-24 Thread Rob Herring
On 09/19/2012 03:20 AM, Steffen Trumtrar wrote: > This patch adds a helper function for parsing videomodes from the devicetree. > The videomode can be either converted to a struct drm_display_mode or a > struct fb_videomode. > > Signed-off-by: Sascha Hauer > Signed-off-by: Steffen Trumtrar > ---

Re: [PATCH v4] of: Add videomode helper

2012-09-24 Thread Sascha Hauer
Hi Rob, On Mon, Sep 24, 2012 at 08:42:12AM -0500, Rob Herring wrote: > On 09/19/2012 03:20 AM, Steffen Trumtrar wrote: > > This patch adds a helper function for parsing videomodes from the > > devicetree. > > The videomode can be either converted to a struct drm_display_mode or a > > struct fb_vi

Re: [PATCH v4] of: Add videomode helper

2012-09-21 Thread Steffen Trumtrar
On Thu, Sep 20, 2012 at 11:29:42PM +0200, Laurent Pinchart wrote: > Hi, > > (CC'ing the linux-media mailing list, as video modes are of interest there as > well) > > On Wednesday 19 September 2012 12:19:22 Tomi Valkeinen wrote: > > On Wed, 2012-09-19 at 10:20 +0200, Steffen Trumtrar wrote: > >

Re: [PATCH v4] of: Add videomode helper

2012-09-21 Thread Hans Verkuil
On Wed September 19 2012 10:20:43 Steffen Trumtrar wrote: > This patch adds a helper function for parsing videomodes from the devicetree. > The videomode can be either converted to a struct drm_display_mode or a > struct fb_videomode. > > Signed-off-by: Sascha Hauer > Signed-off-by: Steffen Trumt

Re: [PATCH v4] of: Add videomode helper

2012-09-20 Thread Laurent Pinchart
Hi, (CC'ing the linux-media mailing list, as video modes are of interest there as well) On Wednesday 19 September 2012 12:19:22 Tomi Valkeinen wrote: > On Wed, 2012-09-19 at 10:20 +0200, Steffen Trumtrar wrote: > > This patch adds a helper function for parsing videomodes from the > > devicetree.

Re: [PATCH v4] of: Add videomode helper

2012-09-19 Thread Tomi Valkeinen
On Wed, 2012-09-19 at 10:20 +0200, Steffen Trumtrar wrote: > This patch adds a helper function for parsing videomodes from the devicetree. > The videomode can be either converted to a struct drm_display_mode or a > struct fb_videomode. > > Signed-off-by: Sascha Hauer > Signed-off-by: Steffen Trum