drm/sti: add hdmi_mode property for HDMI connector
authorVincent Abriou <vincent.abriou@st.com>
Wed, 10 Feb 2016 10:21:37 +0000 (11:21 +0100)
committerVincent Abriou <vincent.abriou@st.com>
Fri, 26 Feb 2016 09:06:20 +0000 (10:06 +0100)
Configures the framer of the HDMI connection.
By default starts in HDMI mode and can be swtich to DVI.

Signed-off-by: Vincent Abriou <vincent.abriou@st.com>
Reviewed-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>
drivers/gpu/drm/sti/sti_hdmi.c
drivers/gpu/drm/sti/sti_hdmi.h

index 1f37dc4f3b859c632e35131ef091452198e3bedb..69a2286ce2f63614892b575e42719a26517af61d 100644 (file)
@@ -129,6 +129,7 @@ struct sti_hdmi_connector {
        struct drm_encoder *encoder;
        struct sti_hdmi *hdmi;
        struct drm_property *colorspace_property;
+       struct drm_property *hdmi_mode_property;
 };
 
 #define to_sti_hdmi_connector(x) \
@@ -229,8 +230,10 @@ static void hdmi_config(struct sti_hdmi *hdmi)
        /* Clear overrun and underrun fifo */
        conf = HDMI_CFG_FIFO_OVERRUN_CLR | HDMI_CFG_FIFO_UNDERRUN_CLR;
 
-       /* Enable HDMI mode not DVI */
-       conf |= HDMI_CFG_HDMI_NOT_DVI | HDMI_CFG_ESS_NOT_OESS;
+       /* Select encryption type and the framing mode */
+       conf |= HDMI_CFG_ESS_NOT_OESS;
+       if (hdmi->hdmi_mode == HDMI_MODE_HDMI)
+               conf |= HDMI_CFG_HDMI_NOT_DVI;
 
        /* Enable sink term detection */
        conf |= HDMI_CFG_SINK_TERM_DET_EN;
@@ -791,6 +794,19 @@ static void sti_hdmi_connector_init_property(struct drm_device *drm_dev,
        }
        hdmi_connector->colorspace_property = prop;
        drm_object_attach_property(&connector->base, prop, hdmi->colorspace);
+
+       /* hdmi_mode property */
+       hdmi->hdmi_mode = DEFAULT_HDMI_MODE;
+       prop = drm_property_create_enum(drm_dev, 0, "hdmi_mode",
+                                       hdmi_mode_names,
+                                       ARRAY_SIZE(hdmi_mode_names));
+       if (!prop) {
+               DRM_ERROR("fails to create colorspace property\n");
+               return;
+       }
+       hdmi_connector->hdmi_mode_property = prop;
+       drm_object_attach_property(&connector->base, prop, hdmi->hdmi_mode);
+
 }
 
 static int
@@ -808,6 +824,11 @@ sti_hdmi_connector_set_property(struct drm_connector *connector,
                return 0;
        }
 
+       if (property == hdmi_connector->hdmi_mode_property) {
+               hdmi->hdmi_mode = val;
+               return 0;
+       }
+
        DRM_ERROR("failed to set hdmi connector property\n");
        return -EINVAL;
 }
@@ -827,6 +848,11 @@ sti_hdmi_connector_get_property(struct drm_connector *connector,
                return 0;
        }
 
+       if (property == hdmi_connector->hdmi_mode_property) {
+               *val = hdmi->hdmi_mode;
+               return 0;
+       }
+
        DRM_ERROR("failed to get hdmi connector property\n");
        return -EINVAL;
 }
index f621cd73b0b8e5458e999dbdddf65c54dfae100a..77edb73a56e4a6aa2e06b464ca8d802c62c3ea87 100644 (file)
@@ -25,6 +25,19 @@ struct hdmi_phy_ops {
        void (*stop)(struct sti_hdmi *hdmi);
 };
 
+/* values for the framing mode property */
+enum sti_hdmi_modes {
+       HDMI_MODE_HDMI,
+       HDMI_MODE_DVI,
+};
+
+static const struct drm_prop_enum_list hdmi_mode_names[] = {
+       { HDMI_MODE_HDMI, "hdmi" },
+       { HDMI_MODE_DVI, "dvi" },
+};
+
+#define DEFAULT_HDMI_MODE HDMI_MODE_HDMI
+
 static const struct drm_prop_enum_list colorspace_mode_names[] = {
        { HDMI_COLORSPACE_RGB, "rgb" },
        { HDMI_COLORSPACE_YUV422, "yuv422" },
@@ -55,6 +68,7 @@ static const struct drm_prop_enum_list colorspace_mode_names[] = {
  * @reset: reset control of the hdmi phy
  * @ddc_adapt: i2c ddc adapter
  * @colorspace: current colorspace selected
+ * @hdmi_mode: select framing for HDMI or DVI
  */
 struct sti_hdmi {
        struct device dev;
@@ -76,6 +90,7 @@ struct sti_hdmi {
        struct reset_control *reset;
        struct i2c_adapter *ddc_adapt;
        enum hdmi_colorspace colorspace;
+       enum sti_hdmi_modes hdmi_mode;
 };
 
 u32 hdmi_read(struct sti_hdmi *hdmi, int offset);