Browse Source

i915: remove unneeded null checks

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>
Dan Carpenter 15 years ago
parent
commit
d8201ab651
1 changed files with 2 additions and 2 deletions
  1. 2 2
      drivers/gpu/drm/i915/intel_dp.c

+ 2 - 2
drivers/gpu/drm/i915/intel_dp.c

@@ -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_encoder *intel_encoder;
 		struct intel_dp_priv *dp_priv;
 		struct intel_dp_priv *dp_priv;
 
 
-		if (!encoder || encoder->crtc != crtc)
+		if (encoder->crtc != crtc)
 			continue;
 			continue;
 
 
 		intel_encoder = enc_to_intel_encoder(encoder);
 		intel_encoder = enc_to_intel_encoder(encoder);
@@ -1353,7 +1353,7 @@ intel_trans_dp_port_sel (struct drm_crtc *crtc)
 	struct intel_encoder *intel_encoder = NULL;
 	struct intel_encoder *intel_encoder = NULL;
 
 
 	list_for_each_entry(encoder, &mode_config->encoder_list, head) {
 	list_for_each_entry(encoder, &mode_config->encoder_list, head) {
-		if (!encoder || encoder->crtc != crtc)
+		if (encoder->crtc != crtc)
 			continue;
 			continue;
 
 
 		intel_encoder = enc_to_intel_encoder(encoder);
 		intel_encoder = enc_to_intel_encoder(encoder);