re: drm/crtc-helper: explicit DPMS on after modeset

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Daniel Vetter,

The patch 25f397a429df: "drm/crtc-helper: explicit DPMS on after 
modeset" from Jul 19, 2013, leads to the following warning:
"drivers/gpu/drm/drm_crtc_helper.c:681 drm_crtc_helper_set_config()
	 info: ignoring unreachable code."

drivers/gpu/drm/drm_crtc_helper.c
   671                  for (ro = 0; ro < set->num_connectors; ro++) {
   672                          if (set->connectors[ro] == connector) {
   673                                  new_encoder = connector_funcs->best_encoder(connector);
   674                                  /* if we can't get an encoder for a connector
   675                                     we are setting now - then fail */
   676                                  if (new_encoder == NULL)
   677                                          /* don't break so fail path works correct */
   678                                          fail = 1;
   679                                  break;
   680  

The new code below comes after a break statement and won't ever be run.

   681                                  if (connector->dpms != DRM_MODE_DPMS_ON) {
   682                                          DRM_DEBUG_KMS("connector dpms not on, full mode switch\n");
   683                                          mode_changed = true;
   684                                  }
   685                          }
   686                  }

regards,
dan carpenter

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux