Re: [PATCH v3 4/4] OMAP_VOUT: Don't trigger updates in omap_vout_probe

2011-09-27 Thread Archit Taneja
On Tuesday 27 September 2011 12:37 PM, Valkeinen, Tomi wrote: On Tue, 2011-09-27 at 12:32 +0530, Archit Taneja wrote: On Tuesday 27 September 2011 11:40 AM, Valkeinen, Tomi wrote: On Mon, 2011-09-26 at 17:29 +0530, Archit Taneja wrote: Remove the code in omap_vout_probe() which calls display->

Re: [PATCH v3 4/4] OMAP_VOUT: Don't trigger updates in omap_vout_probe

2011-09-27 Thread Tomi Valkeinen
On Tue, 2011-09-27 at 12:32 +0530, Archit Taneja wrote: > On Tuesday 27 September 2011 11:40 AM, Valkeinen, Tomi wrote: > > On Mon, 2011-09-26 at 17:29 +0530, Archit Taneja wrote: > >> Remove the code in omap_vout_probe() which calls display->driver->update() > >> for > >> all the displays. This i

Re: [PATCH v3 4/4] OMAP_VOUT: Don't trigger updates in omap_vout_probe

2011-09-27 Thread Archit Taneja
On Tuesday 27 September 2011 11:40 AM, Valkeinen, Tomi wrote: On Mon, 2011-09-26 at 17:29 +0530, Archit Taneja wrote: Remove the code in omap_vout_probe() which calls display->driver->update() for all the displays. This isn't correct because: - An update in probe doesn't make sense, because we

RE: [PATCH v3 4/4] OMAP_VOUT: Don't trigger updates in omap_vout_probe

2011-09-26 Thread Hiremath, Vaibhav
> -Original Message- > From: Taneja, Archit > Sent: Monday, September 26, 2011 5:29 PM > To: Hiremath, Vaibhav > Cc: Valkeinen, Tomi; linux-o...@vger.kernel.org; Semwal, Sumit; linux- > me...@vger.kernel.org; Taneja, Archit > Subject: [PATCH v3 4/4] OMAP_VOUT: Don

Re: [PATCH v3 4/4] OMAP_VOUT: Don't trigger updates in omap_vout_probe

2011-09-26 Thread Tomi Valkeinen
On Mon, 2011-09-26 at 17:29 +0530, Archit Taneja wrote: > Remove the code in omap_vout_probe() which calls display->driver->update() for > all the displays. This isn't correct because: > > - An update in probe doesn't make sense, because we don't have any valid > content > to show at this time.

[PATCH v3 4/4] OMAP_VOUT: Don't trigger updates in omap_vout_probe

2011-09-26 Thread Archit Taneja
Remove the code in omap_vout_probe() which calls display->driver->update() for all the displays. This isn't correct because: - An update in probe doesn't make sense, because we don't have any valid content to show at this time. - Calling update for a panel which isn't enabled is not supported by