On Friday 08 March 2013 05:21 PM, Tomi Valkeinen wrote:
We currently setup both DSI and DISPC related things when the DSI bus is enabled. There's no need for DISPC related thing at that point, though, but only later when the video output is enabled. To make it possible to use the DSI bus before DISPC overlay manager is selected, this patch moves DSI's DISPC initialization to dsi_enable_video_output(), from omapdss_dsi_display_enable(). We also move the selection of DISPC's LCD clock to dsi_enable_video_output. This way there are no DISPC dependencies until the video output is enabled.
This is a good patch. I hope CDF also makes sure the Display controller and DSI bus are made more independent in this manner.
One thing which we should eventually add is to ensure that omap_dsi_update() for command mode panels is called only after dsi_enable_video_output() is called. I think we manage this in our panel driver, but maybe some sort of check could help there.
<snip> Archit -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html