Re: [PATCH 1/2] drm/bridge: ti-tfp410: support hpd via gpio

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Hi,

On 03/07/2017 03:21 AM, Christopher Spinrath wrote:
Hi Fabio,

On 03/06/2017 10:46 PM, Fabio Estevam wrote:
Hi Christopher,

On Mon, Mar 6, 2017 at 6:40 PM,  <christopher.spinrath@xxxxxxxxxxxxxx> wrote:
From: Christopher Spinrath <christopher.spinrath@xxxxxxxxxxxxxx>

On some boards the hpd pin of a hdmi connector is wired up to a gpio
pin. Since in the DRM world the tfp410 driver is responsible for
handling the connector, add support for hpd gpios in this very driver.

Signed-off-by: Christopher Spinrath <christopher.spinrath@xxxxxxxxxxxxxx>
---
 drivers/gpu/drm/bridge/ti-tfp410.c | 72 ++++++++++++++++++++++++++++++++++++--

It would be nice to add 'hpd-gpios' property in the binding doc as well:
Documentation/devicetree/bindings/display/bridge/ti,tfp410.txt

The hpd-gpios property does *not* belong to the ti,tfp410 binding but to the hdmi-connector one
where it is already documented:
Documentation/devicetree/bindings/display/connector/hdmi-connector.txt

Unfortunatly, there is no dedicated driver for the connector in Linux; the connector properties have
to be handled by the encoder (here  the tfp410) driver. Note that this already happens for the
ddc-i2c-bus property.


The patch looks good to me.

Jyri,

Is it possible for you to verify this on Beaglebone platform?

Thanks,
Archit

--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux