On Fri, Jun 06, 2014 at 02:04:21PM +0300, Jani Nikula wrote: > Do we have any need to differentiate between this and what VBT has? We > could just treat dev_priv->vbt as "VBT or what BIOS did", and do > dev_priv->vbt.lvds_use_ssc = true; here, possibly with a > DRM_DEBUG_KMS. It would simplify code elsewhere and be less error prone. > We already do a similar thing for the eDP bpp (see intel_dp.c, look for > "This is a big fat ugly hack").
Indeed, that is a nice simplification. Conceptually, at least. -Chris -- Chris Wilson, Intel Open Source Technology Centre
