[PATCH] omapdrm: Make fixed resolution panels work

2013-02-18 Thread Archit Taneja
On Monday 18 February 2013 02:01 PM, Tomi Valkeinen wrote: > On 2013-02-18 09:23, Archit Taneja wrote: > diff --git a/drivers/staging/omapdrm/omap_connector.c b/drivers/staging/omapdrm/omap_connector.c index 4cc9ee7..839c6e4 100644 --- a/drivers/staging/omapdrm/omap_connector.c

Re: [PATCH] omapdrm: Make fixed resolution panels work

2013-02-18 Thread Tomi Valkeinen
On 2013-02-18 09:23, Archit Taneja wrote: >>> diff --git a/drivers/staging/omapdrm/omap_connector.c >>> b/drivers/staging/omapdrm/omap_connector.c >>> index 4cc9ee7..839c6e4 100644 >>> --- a/drivers/staging/omapdrm/omap_connector.c >>> +++ b/drivers/staging/omapdrm/omap_connector.c >>> @@ -110,6 +

[PATCH] omapdrm: Make fixed resolution panels work

2013-02-18 Thread Archit Taneja
On Thursday 14 February 2013 09:24 PM, Rob Clark wrote: > On Thu, Feb 14, 2013 at 6:52 AM, Archit Taneja wrote: >> The omapdrm driver requires omapdss panel drivers to expose ops like detect, >> set_timings and check_timings. These can be NULL for fixed panel DPI, DBI, >> DSI >> and SDI drivers.

[PATCH] omapdrm: Make fixed resolution panels work

2013-02-18 Thread Tomi Valkeinen
On 2013-02-18 09:23, Archit Taneja wrote: >>> diff --git a/drivers/staging/omapdrm/omap_connector.c >>> b/drivers/staging/omapdrm/omap_connector.c >>> index 4cc9ee7..839c6e4 100644 >>> --- a/drivers/staging/omapdrm/omap_connector.c >>> +++ b/drivers/staging/omapdrm/omap_connector.c >>> @@ -110,6 +

Re: [PATCH] omapdrm: Make fixed resolution panels work

2013-02-18 Thread Archit Taneja
On Monday 18 February 2013 02:01 PM, Tomi Valkeinen wrote: On 2013-02-18 09:23, Archit Taneja wrote: diff --git a/drivers/staging/omapdrm/omap_connector.c b/drivers/staging/omapdrm/omap_connector.c index 4cc9ee7..839c6e4 100644 --- a/drivers/staging/omapdrm/omap_connector.c +++ b/drivers/stagin

Re: [PATCH] omapdrm: Make fixed resolution panels work

2013-02-17 Thread Archit Taneja
On Thursday 14 February 2013 09:24 PM, Rob Clark wrote: On Thu, Feb 14, 2013 at 6:52 AM, Archit Taneja wrote: The omapdrm driver requires omapdss panel drivers to expose ops like detect, set_timings and check_timings. These can be NULL for fixed panel DPI, DBI, DSI and SDI drivers. At some plac

[PATCH] omapdrm: Make fixed resolution panels work

2013-02-14 Thread Archit Taneja
The omapdrm driver requires omapdss panel drivers to expose ops like detect, set_timings and check_timings. These can be NULL for fixed panel DPI, DBI, DSI and SDI drivers. At some places, there are no checks to see if the panel driver has these ops or not, and that leads to a crash. The following

[PATCH] omapdrm: Make fixed resolution panels work

2013-02-14 Thread Rob Clark
On Thu, Feb 14, 2013 at 6:52 AM, Archit Taneja wrote: > The omapdrm driver requires omapdss panel drivers to expose ops like detect, > set_timings and check_timings. These can be NULL for fixed panel DPI, DBI, DSI > and SDI drivers. At some places, there are no checks to see if the panel > driver

Re: [PATCH] omapdrm: Make fixed resolution panels work

2013-02-14 Thread Rob Clark
On Thu, Feb 14, 2013 at 6:52 AM, Archit Taneja wrote: > The omapdrm driver requires omapdss panel drivers to expose ops like detect, > set_timings and check_timings. These can be NULL for fixed panel DPI, DBI, DSI > and SDI drivers. At some places, there are no checks to see if the panel > driver

[PATCH] omapdrm: Make fixed resolution panels work

2013-02-14 Thread Archit Taneja
The omapdrm driver requires omapdss panel drivers to expose ops like detect, set_timings and check_timings. These can be NULL for fixed panel DPI, DBI, DSI and SDI drivers. At some places, there are no checks to see if the panel driver has these ops or not, and that leads to a crash. The following