Re: [PATCH v3 0/7] drm/bridge_connector: perform HPD enablement automatically

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

 



On 02/11/2022 20:06, Dmitry Baryshkov wrote:
 From all the drivers using drm_bridge_connector only iMX/dcss and OMAP
DRM driver do a proper work of calling
drm_bridge_connector_en/disable_hpd() in right places. Rather than
teaching each and every driver how to properly handle
drm_bridge_connector's HPD, make that automatic.

Add two additional drm_connector helper funcs: enable_hpd() and
disable_hpd(). Make drm_kms_helper_poll_* functions call them (as this
is the time where the drm_bridge_connector's functions are called by the
drivers too).

Since we are at the beginning of the development window, gracious ping for this patchset.

It would be nice to finally handle the bridge_connector's hpd properly. Calling drm_bridge_connector_enable_hpd() from drm_bridge_connector_init() is not a proper way to do this. It results in calling bridge->funcs->hpd_enable() before the rest of the pipeline was set up properly.


Changes since v2:
  - Fixed a typo in the commit message of the second patch.

Changes since v1:
  - Rebased on top of v6.1-rc1
  - Removed the drm_bridge_connector_enable_hpd() from
    drm_bridge_connector_init()
  - Removed extra underscore prefix from
    drm_bridge_connector_en/disable_hpd() helpers

Dmitry Baryshkov (7):
   drm/poll-helper: merge drm_kms_helper_poll_disable() and _fini()
   drm/probe-helper: enable and disable HPD on connectors
   drm/bridge_connector: rely on drm_kms_helper_poll_* for HPD enablement
   drm/imx/dcss: stop using drm_bridge_connector_en/disable_hpd()
   drm/msm/hdmi: stop using drm_bridge_connector_en/disable_hpd()
   drm/omap: stop using drm_bridge_connector_en/disable_hpd()
   drm/bridge_connector: drop drm_bridge_connector_en/disable_hpd()

  drivers/gpu/drm/drm_bridge_connector.c   | 27 +++-------------
  drivers/gpu/drm/drm_probe_helper.c       | 40 ++++++++++++++++++-----
  drivers/gpu/drm/imx/dcss/dcss-dev.c      |  4 ---
  drivers/gpu/drm/imx/dcss/dcss-kms.c      |  2 --
  drivers/gpu/drm/msm/hdmi/hdmi.c          |  2 --
  drivers/gpu/drm/omapdrm/omap_drv.c       | 41 ------------------------
  include/drm/drm_bridge_connector.h       |  2 --
  include/drm/drm_modeset_helper_vtables.h | 22 +++++++++++++
  8 files changed, 59 insertions(+), 81 deletions(-)


--
With best wishes
Dmitry




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux