[PATCH v2 0/2] "no-hpd" support in CDNS DP bridge driver

Jayesh Choudhary posted 2 patches 2 years, 10 months ago
There is a newer version of this series
.../display/bridge/cdns,mhdp8546.yaml         |  6 +++
.../drm/bridge/cadence/cdns-mhdp8546-core.c   | 37 ++++++++++++++++---
.../drm/bridge/cadence/cdns-mhdp8546-core.h   |  1 +
3 files changed, 39 insertions(+), 5 deletions(-)
[PATCH v2 0/2] "no-hpd" support in CDNS DP bridge driver
Posted by Jayesh Choudhary 2 years, 10 months ago
In J721s2 EVM, DP0 HPD is not connected to correct HPD pin on SOC
which results in HPD detect as always connnected, so when display
is not connected driver continuously retries to read EDID and DPCD
registers.

To handle such cases add support for no hpd configuration in
cdns-mhdp driver.

DT changes for display in j721s2 will be posted once we have no-hpd
support and multilink DP support[1] merged.

[1]:
<https://lore.kernel.org/all/20230227133104.29788-1-sjakhade@cadence.com/>

Changelog v1->v2:
- edit the commit message for the bindings to clarify why we exactly need
  no-hpd indication for mhdp.

v1 patch-link:
<https://lore.kernel.org/all/20230316140823.234263-1-j-choudhary@ti.com/>

Rahul T R (2):
  dt-bindings: drm/bridge: Add no-hpd property
  drm: bridge: cdns-mhdp8546: Add support for no-hpd

 .../display/bridge/cdns,mhdp8546.yaml         |  6 +++
 .../drm/bridge/cadence/cdns-mhdp8546-core.c   | 37 ++++++++++++++++---
 .../drm/bridge/cadence/cdns-mhdp8546-core.h   |  1 +
 3 files changed, 39 insertions(+), 5 deletions(-)

-- 
2.25.1
Re: [PATCH v2 0/2] "no-hpd" support in CDNS DP bridge driver
Posted by Tomi Valkeinen 2 years, 9 months ago
On 05/04/2023 17:24, Jayesh Choudhary wrote:
> In J721s2 EVM, DP0 HPD is not connected to correct HPD pin on SOC
> which results in HPD detect as always connnected, so when display
> is not connected driver continuously retries to read EDID and DPCD
> registers.

Where is the DP0 HPD connected to? Nowhere? If it's connected to a pin, 
can that pin be used as a GPIO? In other words, do we have a HPD, just 
not with mhdp, or is there just no HPD at all?

  Tomi