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. 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> -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx