Hi, On Thu, 2012-08-09 at 17:19 +0530, Archit Taneja wrote: > Add function omapdss_hdmi_display_get_timing() which returns the timings > maintained by the HDMI interface driver in it's hdmi_config field. This > prevents the need for the panel driver to configure default timings in it's > probe. > > This function is just intended to be used once during the panel driver's probe. > It makes sense for those interfaces which can be configured to a default timing. I'm not sure about this patch. So I think the basic idea here is that HDMI will use VGA video mode if, for whatever reason, no better mode is found out. After this change, the panel driver doesn't seem to do that. Instead it uses whatever mode was used previously by the hdmi output driver. Is the only reason for this patch to clean up the hdmi_panel driver? We could have a dss helper function which returns the timings for VGA (well, just a public const variable would be enough), and the panel driver could use that to initialize the timings struct. Tomi
Attachment:
signature.asc
Description: This is a digitally signed message part
