On Fri, Jan 23, 2015 at 09:04:23PM +0200, ville.syrjala@xxxxxxxxxxxxxxx wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > Currently we don't call valleyview_set_rps() when changing the min/max > limits through sysfs if the current frequency is still within the new > limits. However that means we sometimes forget to update PMINTRMSK. > Eg. if the current frequency is at the old minimum, and then we reduce > the minum further we should then enable the 'down' interrupts in PMINTRMSK > but currently we don't. > > Fix it up by always calling valleyview_set_rps() (just like we do for > !vlv/chv platforms). This also allows the code to be simplified a bit. > > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
