On Thu, Sep 12, 2013 at 09:09:05PM +0100, Chris Wilson wrote: > On Thu, Sep 12, 2013 at 10:45:41PM +0300, ville.syrjala@xxxxxxxxxxxxxxx wrote: > > From: Jani Nikula <jani.nikula@xxxxxxxxx> > > > > The cursor is disabled before crtc mode set in crtc disable (and we > > assert this is the case), and enabled afterwards in crtc enable. Do not > > update it in crtc mode set. > > > > On HSW enabling a plane on a disabled pipe may hang the entire system. > > And there's no good reason for doing it ever, so just don't. > > > > v2: Add note about HSW hangs - vsyrjala > > > > Cc: stable@xxxxxxxxxxxxxxx > > Suggested-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx> > > Where did the asserts go? :) I asked Jani to move them around a bit, which AFAIK he didn't yet. So I figured we'd just go with a minimal fix for stable first and then slap on more asserts later in dinq. -- Ville Syrjälä Intel OTC -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html