On Mon, Dec 05, 2016 at 02:04:08PM +0200, Marta Lofstedt wrote: > The drm_atomic_helper_connector_dpms > will set the connector back the old DPMS state > before returning. This makes it impossible to change > DPMS state of the connector. > > Fixes: 0853695c3ba46f97dfc0b5885f7b7e640ca212dd > v2: edit of commit message > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > Cc: Daniel Vetter <daniel.vetter@xxxxxxxx> > Cc: Eric Engestrom <eric.engestrom@xxxxxxxxxx> > Cc: Sean Paul <seanpaul@xxxxxxxxxxxx> > Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx > Cc: <drm-intel-fixes@xxxxxxxxxxxxxxxxxxxxx> > Signed-off-by: Marta Lofstedt <marta.lofstedt@xxxxxxxxx> Applied to drm-misc, thanks. -Daniel > --- > drivers/gpu/drm/drm_atomic_helper.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c > index 494680c..6a5acb9 100644 > --- a/drivers/gpu/drm/drm_atomic_helper.c > +++ b/drivers/gpu/drm/drm_atomic_helper.c > @@ -2885,8 +2885,8 @@ int drm_atomic_helper_connector_dpms(struct drm_connector *connector, > fail: > if (ret == -EDEADLK) > goto backoff; > - > - connector->dpms = old_mode; > + if (ret != 0) > + connector->dpms = old_mode; > drm_atomic_state_put(state); > return ret; > > -- > 2.9.3 > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel