Re: [linux-sunxi] [PATCH] video/modedb: fb_find_nearest_mode: take vmode into account

Previously fb_find_nearest_mode() only searched the modelist for a video mode that best matches the
desired resolution and refresh rate.
With this patch it also takes the vmode into account if there is more then one mode with the same
resolution and refresh rate.
Typical use case is HDMI TVs that support both 1080p60 and 1080i60

Signed-off-by: Floris Bos <redacted>
---
  drivers/video/modedb.c |    4 ++++
  1 file changed, 4 insertions(+)

diff --git a/drivers/video/modedb.c b/drivers/video/modedb.c
index a9a907c..e852371 100644
--- a/drivers/video/modedb.c
+++ b/drivers/video/modedb.c

@@ -913,6 +913,8 @@ const struct fb_videomode *fb_find_best_mode(const struct fb_var_screeninfo *var
   * Finds best matching videomode, smaller or greater in dimension.
   * If more than 1 videomode is found, will return the videomode with
   * the closest refresh rate.
+ * If multiple modes with the same resolution and refresh rate are found
+ * pick the one with the matching vmode (e.g. non-interlaced)
   */
  const struct fb_videomode *fb_find_nearest_mode(const struct fb_videomode *mode,
  					        struct list_head *head)

@@ -939,6 +941,8 @@ const struct fb_videomode *fb_find_nearest_mode(const struct fb_videomode *mode,
  			if (diff_refresh > d) {
  				diff_refresh = d;
  				best = cmode;
+			} else if (diff_refresh = d && cmode->vmode = mode->vmode) {
+				best = cmode;
  			}
  		}
  	}