[PATCH] imx-drm: parallel-display: honor 'native-mode' property when selecting video mode from DT

[PATCH] imx-drm: parallel-display: honor 'native-mode' property when selecting video mode from DT

[Lothar Waßmann]

This patch allows to select a specific video mode from a list of modes
defined in DT by setting the 'native-mode' property appropriately.

Since all current users of this driver have only one mode defined in
their .dts files, the patch does not change the behaviour of this
driver on the affected platforms.

Signed-off-by: Lothar Wa?mann <LW@KARO-electronics.de>
---
 drivers/staging/imx-drm/parallel-display.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/drivers/staging/imx-drm/parallel-display.c b/drivers/staging/imx-drm/parallel-display.c
index 24aa9be..351d61d 100644
--- a/drivers/staging/imx-drm/parallel-display.c
+++ b/drivers/staging/imx-drm/parallel-display.c
@@ -23,6 +23,7 @@
 #include <drm/drm_fb_helper.h>
 #include <drm/drm_crtc_helper.h>
 #include <linux/videodev2.h>
+#include <video/of_display_timing.h>
 
 #include "imx-drm.h"
 
@@ -74,7 +75,7 @@ static int imx_pd_connector_get_modes(struct drm_connector *connector)
 
 	if (np) {
 		struct drm_display_mode *mode = drm_mode_create(connector->dev);
-		of_get_drm_display_mode(np, &imxpd->mode, 0);
+		of_get_drm_display_mode(np, &imxpd->mode, OF_USE_NATIVE_MODE);
 		drm_mode_copy(mode, &imxpd->mode);
 		mode->type |= DRM_MODE_TYPE_DRIVER | DRM_MODE_TYPE_PREFERRED,
 		drm_mode_probed_add(connector, mode);
-- 
1.7.2.5

[PATCH] imx-drm: parallel-display: honor 'native-mode' property when selecting video mode from DT

[Marek Vasut]

On Monday, January 13, 2014 at 11:24:33 AM, Lothar Wa?mann wrote:
> This patch allows to select a specific video mode from a list of modes
> defined in DT by setting the 'native-mode' property appropriately.
> 
> Since all current users of this driver have only one mode defined in
> their .dts files, the patch does not change the behaviour of this
> driver on the affected platforms.
> 
> Signed-off-by: Lothar Wa?mann <LW@KARO-electronics.de>

Looks OK to me, thanks.

Reviewed-by: Marek Vasut <marex@denx.de>

Best regards,
Marek Vasut