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.syrj...@linux.intel.com> > > > > 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.syrj...@linux.intel.com> > > 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 <ch...@chris-wilson.co.uk> Thanks. Pushed this one as well. -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
