On Wed, Nov 12, 2008 at 03:27:47PM +0800, Wang, Zhenyu Z wrote: > On 2008.11.07 14:23:39 +0800, Wu Fengguang wrote: > > HDMI is compatible with DVI, and we've seen many boards that > > use HDMI port for DVI output. > > > > So Zhenyu proposed this flag: i830_hdmi_priv.has_hdmi_sink > > to indicate the presence of HDMI capable monitors. > > > > Signed-off-by: Wu Fengguang <wfg@xxxxxxxxxxxxxxx> > We should check if the symbol is available for xserver compatiblity. > So below is updated patch for check this. > + > + if (xf86LoaderCheckSymbol("xf86MonitorIsHDMI") && > + xf86MonitorIsHDMI(edid_mon)) > + dev_priv->has_hdmi_sink = TRUE; I'm OK with the backward compatibility check. The side effect is that audio output won't be enabled for old xservers. Thanks, Fengguang _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel