diff mbox

drm/msm/hdmi: enable lpm-mux if it is present

Message ID 1406818005-21953-1-git-send-email-sviau@codeaurora.org (mailing list archive)
State Not Applicable, archived
Headers show

Commit Message

Stephane Viau July 31, 2014, 2:46 p.m. UTC
From: Beeresh Gopal <gbeeresh@codeaurora.org>

lpm-mux is programmed to enable HDMI connector
on the docking station for S805 chipset based
devices.

Signed-off-by: Beeresh Gopal <gbeeresh@codeaurora.org>
---
 drivers/gpu/drm/msm/hdmi/hdmi.c           |  1 +
 drivers/gpu/drm/msm/hdmi/hdmi.h           |  1 +
 drivers/gpu/drm/msm/hdmi/hdmi_connector.c | 24 +++++++++++++++++++++++-
 3 files changed, 25 insertions(+), 1 deletion(-)

Comments

Andreas Färber July 31, 2014, 3:06 p.m. UTC | #1
Hi,

Am 31.07.2014 16:46, schrieb Stephane Viau:
> From: Beeresh Gopal <gbeeresh@codeaurora.org>
> 
> lpm-mux is programmed to enable HDMI connector
> on the docking station for S805 chipset based
> devices.
> 
> Signed-off-by: Beeresh Gopal <gbeeresh@codeaurora.org>

You forgot to sign off yourself.

[...]
> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_connector.c b/drivers/gpu/drm/msm/hdmi/hdmi_connector.c
> index 93d1551..1301d03 100644
> --- a/drivers/gpu/drm/msm/hdmi/hdmi_connector.c
> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_connector.c
> @@ -63,7 +63,8 @@ static int gpio_config(struct hdmi *hdmi, bool on)
>  			ret = gpio_request(config->mux_en_gpio, "HDMI_MUX_EN");
>  			if (ret) {
>  				dev_err(dev->dev, "'%s'(%d) gpio_request failed: %d\n",
> -					"HDMI_MUX_SEL", config->mux_en_gpio, ret);
> +					"HDMI_MUX_EN",
> +					config->mux_en_gpio, ret);
>  				goto error4;
>  			}
>  			gpio_set_value_cansleep(config->mux_en_gpio, 1);

This hunk looks like an unrelated typo fix, which should then probably
go into its own patch.

Cheers,
Andreas
Rob Clark July 31, 2014, 3:19 p.m. UTC | #2
On Thu, Jul 31, 2014 at 10:46 AM, Stephane Viau <sviau@codeaurora.org> wrote:
> From: Beeresh Gopal <gbeeresh@codeaurora.org>
>
> lpm-mux is programmed to enable HDMI connector
> on the docking station for S805 chipset based
> devices.
>
> Signed-off-by: Beeresh Gopal <gbeeresh@codeaurora.org>

other than the issues Andreas mentioned, it looks good, so with those addressed:

Reviewed-by: Rob Clark <robdclark@gmail.com>


> ---
>  drivers/gpu/drm/msm/hdmi/hdmi.c           |  1 +
>  drivers/gpu/drm/msm/hdmi/hdmi.h           |  1 +
>  drivers/gpu/drm/msm/hdmi/hdmi_connector.c | 24 +++++++++++++++++++++++-
>  3 files changed, 25 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi.c b/drivers/gpu/drm/msm/hdmi/hdmi.c
> index bb1f696..f2c92e6 100644
> --- a/drivers/gpu/drm/msm/hdmi/hdmi.c
> +++ b/drivers/gpu/drm/msm/hdmi/hdmi.c
> @@ -453,6 +453,7 @@ static int hdmi_bind(struct device *dev, struct device *master, void *data)
>         config.hpd_gpio      = get_gpio("qcom,hdmi-tx-hpd");
>         config.mux_en_gpio   = get_gpio("qcom,hdmi-tx-mux-en");
>         config.mux_sel_gpio  = get_gpio("qcom,hdmi-tx-mux-sel");
> +       config.mux_lpm_gpio  = get_gpio("qcom,hdmi-tx-mux-lpm");
>         config.shared_irq    = true;
>
>  #else
> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi.h b/drivers/gpu/drm/msm/hdmi/hdmi.h
> index 0a077b0..323ceb7 100644
> --- a/drivers/gpu/drm/msm/hdmi/hdmi.h
> +++ b/drivers/gpu/drm/msm/hdmi/hdmi.h
> @@ -103,6 +103,7 @@ struct hdmi_platform_config {
>
>         /* gpio's: */
>         int ddc_clk_gpio, ddc_data_gpio, hpd_gpio, mux_en_gpio, mux_sel_gpio;
> +       int mux_lpm_gpio;
>
>         /* older devices had their own irq, mdp5+ it is shared w/ mdp: */
>         bool shared_irq;
> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_connector.c b/drivers/gpu/drm/msm/hdmi/hdmi_connector.c
> index 93d1551..1301d03 100644
> --- a/drivers/gpu/drm/msm/hdmi/hdmi_connector.c
> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_connector.c
> @@ -63,7 +63,8 @@ static int gpio_config(struct hdmi *hdmi, bool on)
>                         ret = gpio_request(config->mux_en_gpio, "HDMI_MUX_EN");
>                         if (ret) {
>                                 dev_err(dev->dev, "'%s'(%d) gpio_request failed: %d\n",
> -                                       "HDMI_MUX_SEL", config->mux_en_gpio, ret);
> +                                       "HDMI_MUX_EN",
> +                                       config->mux_en_gpio, ret);
>                                 goto error4;
>                         }
>                         gpio_set_value_cansleep(config->mux_en_gpio, 1);
> @@ -78,6 +79,19 @@ static int gpio_config(struct hdmi *hdmi, bool on)
>                         }
>                         gpio_set_value_cansleep(config->mux_sel_gpio, 0);
>                 }
> +
> +               if (config->mux_lpm_gpio != -1) {
> +                       ret = gpio_request(config->mux_lpm_gpio,
> +                                       "HDMI_MUX_LPM");
> +                       if (ret) {
> +                               dev_err(dev->dev,
> +                                       "'%s'(%d) gpio_request failed: %d\n",
> +                                       "HDMI_MUX_LPM",
> +                                       config->mux_lpm_gpio, ret);
> +                               goto error6;
> +                       }
> +                       gpio_set_value_cansleep(config->mux_lpm_gpio, 1);
> +               }
>                 DBG("gpio on");
>         } else {
>                 gpio_free(config->ddc_clk_gpio);
> @@ -93,11 +107,19 @@ static int gpio_config(struct hdmi *hdmi, bool on)
>                         gpio_set_value_cansleep(config->mux_sel_gpio, 1);
>                         gpio_free(config->mux_sel_gpio);
>                 }
> +
> +               if (config->mux_lpm_gpio != -1) {
> +                       gpio_set_value_cansleep(config->mux_lpm_gpio, 0);
> +                       gpio_free(config->mux_lpm_gpio);
> +               }
>                 DBG("gpio off");
>         }
>
>         return 0;
>
> +error6:
> +       if (config->mux_sel_gpio != -1)
> +               gpio_free(config->mux_sel_gpio);
>  error5:
>         if (config->mux_en_gpio != -1)
>                 gpio_free(config->mux_en_gpio);
> --
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> hosted by The Linux Foundation
>
--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Stephane Viau July 31, 2014, 3:48 p.m. UTC | #3
version 2 (response to Andreas' review): 
- patch split into two (fix typo + new lpm-mux config)
- added my Signed-off-by in patches

Beeresh Gopal (2):
  drm/msm/hdmi: enable lpm-mux if it is present
  drm/msm/hdmi: fix HDMI_MUX_EN gpio request typo

 drivers/gpu/drm/msm/hdmi/hdmi.c           |  1 +
 drivers/gpu/drm/msm/hdmi/hdmi.h           |  1 +
 drivers/gpu/drm/msm/hdmi/hdmi_connector.c | 23 ++++++++++++++++++++++-
 3 files changed, 24 insertions(+), 1 deletion(-)
diff mbox

Patch

diff --git a/drivers/gpu/drm/msm/hdmi/hdmi.c b/drivers/gpu/drm/msm/hdmi/hdmi.c
index bb1f696..f2c92e6 100644
--- a/drivers/gpu/drm/msm/hdmi/hdmi.c
+++ b/drivers/gpu/drm/msm/hdmi/hdmi.c
@@ -453,6 +453,7 @@  static int hdmi_bind(struct device *dev, struct device *master, void *data)
 	config.hpd_gpio      = get_gpio("qcom,hdmi-tx-hpd");
 	config.mux_en_gpio   = get_gpio("qcom,hdmi-tx-mux-en");
 	config.mux_sel_gpio  = get_gpio("qcom,hdmi-tx-mux-sel");
+	config.mux_lpm_gpio  = get_gpio("qcom,hdmi-tx-mux-lpm");
 	config.shared_irq    = true;
 
 #else
diff --git a/drivers/gpu/drm/msm/hdmi/hdmi.h b/drivers/gpu/drm/msm/hdmi/hdmi.h
index 0a077b0..323ceb7 100644
--- a/drivers/gpu/drm/msm/hdmi/hdmi.h
+++ b/drivers/gpu/drm/msm/hdmi/hdmi.h
@@ -103,6 +103,7 @@  struct hdmi_platform_config {
 
 	/* gpio's: */
 	int ddc_clk_gpio, ddc_data_gpio, hpd_gpio, mux_en_gpio, mux_sel_gpio;
+	int mux_lpm_gpio;
 
 	/* older devices had their own irq, mdp5+ it is shared w/ mdp: */
 	bool shared_irq;
diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_connector.c b/drivers/gpu/drm/msm/hdmi/hdmi_connector.c
index 93d1551..1301d03 100644
--- a/drivers/gpu/drm/msm/hdmi/hdmi_connector.c
+++ b/drivers/gpu/drm/msm/hdmi/hdmi_connector.c
@@ -63,7 +63,8 @@  static int gpio_config(struct hdmi *hdmi, bool on)
 			ret = gpio_request(config->mux_en_gpio, "HDMI_MUX_EN");
 			if (ret) {
 				dev_err(dev->dev, "'%s'(%d) gpio_request failed: %d\n",
-					"HDMI_MUX_SEL", config->mux_en_gpio, ret);
+					"HDMI_MUX_EN",
+					config->mux_en_gpio, ret);
 				goto error4;
 			}
 			gpio_set_value_cansleep(config->mux_en_gpio, 1);
@@ -78,6 +79,19 @@  static int gpio_config(struct hdmi *hdmi, bool on)
 			}
 			gpio_set_value_cansleep(config->mux_sel_gpio, 0);
 		}
+
+		if (config->mux_lpm_gpio != -1) {
+			ret = gpio_request(config->mux_lpm_gpio,
+					"HDMI_MUX_LPM");
+			if (ret) {
+				dev_err(dev->dev,
+					"'%s'(%d) gpio_request failed: %d\n",
+					"HDMI_MUX_LPM",
+					config->mux_lpm_gpio, ret);
+				goto error6;
+			}
+			gpio_set_value_cansleep(config->mux_lpm_gpio, 1);
+		}
 		DBG("gpio on");
 	} else {
 		gpio_free(config->ddc_clk_gpio);
@@ -93,11 +107,19 @@  static int gpio_config(struct hdmi *hdmi, bool on)
 			gpio_set_value_cansleep(config->mux_sel_gpio, 1);
 			gpio_free(config->mux_sel_gpio);
 		}
+
+		if (config->mux_lpm_gpio != -1) {
+			gpio_set_value_cansleep(config->mux_lpm_gpio, 0);
+			gpio_free(config->mux_lpm_gpio);
+		}
 		DBG("gpio off");
 	}
 
 	return 0;
 
+error6:
+	if (config->mux_sel_gpio != -1)
+		gpio_free(config->mux_sel_gpio);
 error5:
 	if (config->mux_en_gpio != -1)
 		gpio_free(config->mux_en_gpio);