Commit 07096bd3 authored by Liu Ying's avatar Liu Ying Committed by Daniel Vetter

drm/crtc_helper/set_config: Remove redundant handling when set->fb is NULL

We've done sanity NULL pointer check on set->fb at the beginning of
drm_crtc_helper_set_config() and bailed out if necessary, thus any later on
check or case handling is redundant.
Signed-off-by: default avatarLiu Ying <gnuiyl@gmail.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1452751210-19216-1-git-send-email-gnuiyl@gmail.comSigned-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent 1df59b84
...@@ -578,8 +578,6 @@ int drm_crtc_helper_set_config(struct drm_mode_set *set) ...@@ -578,8 +578,6 @@ int drm_crtc_helper_set_config(struct drm_mode_set *set)
if (set->crtc->primary->fb == NULL) { if (set->crtc->primary->fb == NULL) {
DRM_DEBUG_KMS("crtc has no fb, full mode set\n"); DRM_DEBUG_KMS("crtc has no fb, full mode set\n");
mode_changed = true; mode_changed = true;
} else if (set->fb == NULL) {
mode_changed = true;
} else if (set->fb->pixel_format != } else if (set->fb->pixel_format !=
set->crtc->primary->fb->pixel_format) { set->crtc->primary->fb->pixel_format) {
mode_changed = true; mode_changed = true;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment