drm: sti: remove gpio for HDMI hot plug detection
authorBenjamin Gaignard <benjamin.gaignard@linaro.org>
Thu, 9 Oct 2014 06:53:35 +0000 (08:53 +0200)
committerBenjamin Gaignard <benjamin.gaignard@linaro.org>
Thu, 11 Dec 2014 12:58:12 +0000 (13:58 +0100)
gpio used for HDMI hot plug detection is useless,
HDMI_STI register contains an hot plug detection status bit.
Fix binding documentation.

Signed-off-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>
Documentation/devicetree/bindings/gpu/st,stih4xx.txt
drivers/gpu/drm/sti/sti_hdmi.c
drivers/gpu/drm/sti/sti_hdmi.h

index 8885d9e203fcb298dce24a0d534e347628495e9e..32cfc7b7631b9b9d85295ee613b2082c0c55da74 100644 (file)
@@ -68,7 +68,6 @@ STMicroelectronics stih4xx platforms
     number of clocks may depend of the SoC type.
   - clock-names: names of the clocks listed in clocks property in the same
     order.
-  - hdmi,hpd-gpio: gpio id to detect if an hdmi cable is plugged or not.
   - ddc: phandle of an I2C controller used for DDC EDID probing
 
 sti-hda:
@@ -174,7 +173,6 @@ Example:
                                interrupt-names = "irq";
                                clock-names     = "pix", "tmds", "phy", "audio";
                                clocks          = <&clockgen_c_vcc CLK_S_PIX_HDMI>, <&clockgen_c_vcc CLK_S_TMDS_HDMI>, <&clockgen_c_vcc CLK_S_HDMI_REJECT_PLL>, <&clockgen_b1 CLK_S_PCM_0>;
-                               hdmi,hpd-gpio   = <&PIO2 5>;
                        };
 
                        sti-hda@fe85a000 {
index fed1b5fe4842d72da235626f8162006d7061fda9..192119761c147a39f87e482b927e1d3e5f9276e9 100644 (file)
@@ -130,8 +130,7 @@ static irqreturn_t hdmi_irq_thread(int irq, void *arg)
 
        /* Hot plug/unplug IRQ */
        if (hdmi->irq_status & HDMI_INT_HOT_PLUG) {
-               /* read gpio to get the status */
-               hdmi->hpd = gpio_get_value(hdmi->hpd_gpio);
+               hdmi->hpd = readl(hdmi->regs + HDMI_STA) & HDMI_STA_HOT_PLUG;
                if (hdmi->drm_dev)
                        drm_helper_hpd_irq_event(hdmi->drm_dev);
        }
@@ -766,13 +765,7 @@ static int sti_hdmi_probe(struct platform_device *pdev)
                return PTR_ERR(hdmi->clk_audio);
        }
 
-       hdmi->hpd_gpio = of_get_named_gpio(np, "hdmi,hpd-gpio", 0);
-       if (hdmi->hpd_gpio < 0) {
-               DRM_ERROR("Failed to get hdmi hpd-gpio\n");
-               return -EIO;
-       }
-
-       hdmi->hpd = gpio_get_value(hdmi->hpd_gpio);
+       hdmi->hpd = readl(hdmi->regs + HDMI_STA) & HDMI_STA_HOT_PLUG;
 
        init_waitqueue_head(&hdmi->wait_event);
 
index d00a3e0d807f16e5aaa70537f984c50febc42975..3d22390e1f3b3a3b5ad122398853a8a999b66fa1 100644 (file)
@@ -14,6 +14,9 @@
 #define HDMI_STA           0x0010
 #define HDMI_STA_DLL_LCK   BIT(5)
 
+#define HDMI_STA_HOT_PLUG_SHIFT 4
+#define HDMI_STA_HOT_PLUG      (1 << HDMI_STA_HOT_PLUG_SHIFT)
+
 struct sti_hdmi;
 
 struct hdmi_phy_ops {
@@ -37,7 +40,6 @@ struct hdmi_phy_ops {
  * @irq_status: interrupt status register
  * @phy_ops: phy start/stop operations
  * @enabled: true if hdmi is enabled else false
- * @hpd_gpio: hdmi hot plug detect gpio number
  * @hpd: hot plug detect status
  * @wait_event: wait event
  * @event_received: wait event status
@@ -57,7 +59,6 @@ struct sti_hdmi {
        u32 irq_status;
        struct hdmi_phy_ops *phy_ops;
        bool enabled;
-       int hpd_gpio;
        bool hpd;
        wait_queue_head_t wait_event;
        bool event_received;