drm/i915: Correct MIPI operation mode as per expected values from VBT
authorShobhit Kumar <shobhit.kumar@intel.com>
Mon, 14 Apr 2014 05:48:24 +0000 (11:18 +0530)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Thu, 15 May 2014 20:43:48 +0000 (22:43 +0200)
In VBT fields operation mode is 0 for Video mode and 1 for command mode.
This field will be directly used as is in generic panel driver. So
adjust accordingly.

Signed-off-by: Shobhit Kumar <shobhit.kumar@intel.com>
Reviewed-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_drv.h
drivers/gpu/drm/i915/intel_dsi.c
drivers/gpu/drm/i915/intel_dsi.h

index 32a74e19d9d93dd86366f5b25022a1af5fc121d6..9ccee19f474177ac66aafec9f3b944885bc5f6e1 100644 (file)
 #define INTEL_DVO_CHIP_TMDS 2
 #define INTEL_DVO_CHIP_TVOUT 4
 
-#define INTEL_DSI_COMMAND_MODE 0
-#define INTEL_DSI_VIDEO_MODE   1
+#define INTEL_DSI_VIDEO_MODE   0
+#define INTEL_DSI_COMMAND_MODE 1
 
 struct intel_framebuffer {
        struct drm_framebuffer base;
index 4e271c768fd0c58c65b60e9e16d5725c9f82653a..2795782b6743ee6c7013ad6891b04caf3f2cbf55 100644 (file)
@@ -59,12 +59,12 @@ static struct intel_dsi *intel_attached_dsi(struct drm_connector *connector)
 
 static inline bool is_vid_mode(struct intel_dsi *intel_dsi)
 {
-       return intel_dsi->dev.type == INTEL_DSI_VIDEO_MODE;
+       return intel_dsi->operation_mode == INTEL_DSI_VIDEO_MODE;
 }
 
 static inline bool is_cmd_mode(struct intel_dsi *intel_dsi)
 {
-       return intel_dsi->dev.type == INTEL_DSI_COMMAND_MODE;
+       return intel_dsi->operation_mode == INTEL_DSI_COMMAND_MODE;
 }
 
 static void intel_dsi_hot_plug(struct intel_encoder *encoder)
index 550714c7860e8ad57a90153e7838830217a92798..16494225ed143e659088d07ae72de480670b6610 100644 (file)
@@ -31,7 +31,6 @@
 struct intel_dsi_device {
        unsigned int panel_id;
        const char *name;
-       int type;
        const struct intel_dsi_dev_ops *dev_ops;
        void *dev_priv;
 };
@@ -85,6 +84,9 @@ struct intel_dsi {
        /* virtual channel */
        int channel;
 
+       /* Video mode or command mode */
+       u16 operation_mode;
+
        /* number of DSI lanes */
        unsigned int lane_count;