Skip to content

Commit

Permalink
i915: remove unneeded null checks
Browse files Browse the repository at this point in the history
The "encoder" variable can never be null because it is used as loop
cursor in a list_for_each_entry() loop.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Eric Anholt <eric@anholt.net>
  • Loading branch information
Dan Carpenter authored and Eric Anholt committed May 26, 2010
1 parent 90a78e8 commit d8201ab
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions drivers/gpu/drm/i915/intel_dp.c
Original file line number Diff line number Diff line change
Expand Up @@ -576,7 +576,7 @@ intel_dp_set_m_n(struct drm_crtc *crtc, struct drm_display_mode *mode,
struct intel_encoder *intel_encoder;
struct intel_dp_priv *dp_priv;

if (!encoder || encoder->crtc != crtc)
if (encoder->crtc != crtc)
continue;

intel_encoder = enc_to_intel_encoder(encoder);
Expand Down Expand Up @@ -1353,7 +1353,7 @@ intel_trans_dp_port_sel (struct drm_crtc *crtc)
struct intel_encoder *intel_encoder = NULL;

list_for_each_entry(encoder, &mode_config->encoder_list, head) {
if (!encoder || encoder->crtc != crtc)
if (encoder->crtc != crtc)
continue;

intel_encoder = enc_to_intel_encoder(encoder);
Expand Down

0 comments on commit d8201ab

Please sign in to comment.