On Tue, Feb 25, 2025 at 05:48:22PM +0530, Ayushi Makhija wrote:
> When device enters the suspend state, it prevents
> HPD interrupts from occurring. To address this,
> add an additional PM runtime vote during bridge
> attach for DisplayPort. This vote is removed on
> bridge detach.
Well.. You can guess. The bridges have .hpd_enable() and .hpd_disable()
callbacks. Please use those instead.
>
> Signed-off-by: Ayushi Makhija <quic_amakhija@quicinc.com>
> ---
> drivers/gpu/drm/bridge/analogix/anx7625.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/drivers/gpu/drm/bridge/analogix/anx7625.c b/drivers/gpu/drm/bridge/analogix/anx7625.c
> index 4be34d5c7a3b..d2655bf46842 100644
> --- a/drivers/gpu/drm/bridge/analogix/anx7625.c
> +++ b/drivers/gpu/drm/bridge/analogix/anx7625.c
> @@ -2168,6 +2168,9 @@ static int anx7625_bridge_attach(struct drm_bridge *bridge,
>
> ctx->bridge_attached = 1;
>
> + if (ctx->pdata.panel_bridge->type == DRM_MODE_CONNECTOR_DisplayPort)
> + pm_runtime_get_sync(dev);
> +
> return 0;
> }
>
> @@ -2175,6 +2178,9 @@ static void anx7625_bridge_detach(struct drm_bridge *bridge)
> {
> struct anx7625_data *ctx = bridge_to_anx7625(bridge);
>
> + if (ctx->pdata.panel_bridge->type == DRM_MODE_CONNECTOR_DisplayPort)
> + pm_runtime_put_sync(ctx->dev);
> +
> drm_dp_aux_unregister(&ctx->aux);
> }
>
> --
> 2.34.1
>
--
With best wishes
Dmitry