drm/edid: Rewrite drm_mode_find_dmt search loop
authorAdam Jackson <ajax@redhat.com>
Fri, 13 Apr 2012 20:33:30 +0000 (16:33 -0400)
committerDave Airlie <airlied@redhat.com>
Fri, 20 Apr 2012 11:56:56 +0000 (12:56 +0100)
No functional change, but will make an upcoming change clearer.

Signed-off-by: Adam Jackson <ajax@redhat.com>
Tested-by: Takashi Iwai <tiwai@suse.de>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
drivers/gpu/drm/drm_edid.c

index a75aefef528e6601523344b80438fc70e34dbd9b..9c8fa8860f6b9cd947c36a5ac22fea42cac685c8 100644 (file)
@@ -499,20 +499,21 @@ static void edid_fixup_preferred(struct drm_connector *connector,
 struct drm_display_mode *drm_mode_find_dmt(struct drm_device *dev,
                                           int hsize, int vsize, int fresh)
 {
-       struct drm_display_mode *mode = NULL;
        int i;
 
        for (i = 0; i < drm_num_dmt_modes; i++) {
                const struct drm_display_mode *ptr = &drm_dmt_modes[i];
-               if (hsize == ptr->hdisplay &&
-                       vsize == ptr->vdisplay &&
-                       fresh == drm_mode_vrefresh(ptr)) {
-                       /* get the expected default mode */
-                       mode = drm_mode_duplicate(dev, ptr);
-                       break;
-               }
+               if (hsize != ptr->hdisplay)
+                       continue;
+               if (vsize != ptr->vdisplay)
+                       continue;
+               if (fresh != drm_mode_vrefresh(ptr))
+                       continue;
+
+               return drm_mode_duplicate(dev, ptr);
        }
-       return mode;
+
+       return NULL;
 }
 EXPORT_SYMBOL(drm_mode_find_dmt);