On Wed, Oct 11, 2017 at 07:06:45PM +0100, Chris Wilson wrote: > Quoting Ville Syrjala (2017-10-09 17:19:51) > > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > > > With intel_encoder_current_mode() using the normal state readout code it > > actually works on PCH platforms as well. So let's nuke the PCH check from > > intel_lvds_init(). I suppose there aren't any machines that actually > > need this, but at least we get to eliminate a few lines of code, and one > > FIXME. > > > > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > I have to admit the FIXME comment confused me utterly. It does seem to be something not quite English. > Having read the > function, this should now work given the conversion of > intel_encoder_current_mode() to use the common encoder->get_hw_state() > routine. > > Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> Thanks. Pushed this one as well. -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
