On Mon, Sep 21, 2026 at 09:53:12AM +0800, Chaoyi Chen wrote:
> Hi Dmitry,
>
> On 9/20/2026 11:01 PM, Dmitry Baryshkov wrote:
> > The DisplayPort standard defines a special kind of HPD events called
> > IRQ_HPD. These events are used to notify DP Source about the events on
> > the Sink side.
> >
> > Extend drm_connector_funcs::oob_hotplug_event() to pass the
> > notifications about the IRQ_HPD events down to the individual drivers,
> > letting them handle those as required.
> >
> > Reviewed-by: Pengyu Luo <[email protected]>
> > Acked-by: Heikki Krogerus <[email protected]>
> > Signed-off-by: Dmitry Baryshkov <[email protected]>
> > ---
> > drivers/gpu/drm/display/drm_bridge_connector.c | 17 ++++++++++-------
> > drivers/gpu/drm/drm_connector.c | 2 +-
> > drivers/gpu/drm/i915/display/intel_dp.c | 3 ++-
> > include/drm/drm_connector.h | 3 ++-
> > 4 files changed, 15 insertions(+), 10 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/display/drm_bridge_connector.c
> > b/drivers/gpu/drm/display/drm_bridge_connector.c
> > index 00feb73e63d7..a51567d19550 100644
> > --- a/drivers/gpu/drm/display/drm_bridge_connector.c
> > +++ b/drivers/gpu/drm/display/drm_bridge_connector.c
> > @@ -141,7 +141,8 @@ struct drm_bridge_connector {
> > */
> >
> > static void drm_bridge_connector_hpd_notify(struct drm_connector
> > *connector,
> > - enum drm_connector_status status)
> > + enum drm_connector_status status,
> > + enum drm_connector_dp_event event)
> > {
> > struct drm_bridge_connector *bridge_connector =
> > to_drm_bridge_connector(connector);
> > @@ -154,7 +155,8 @@ static void drm_bridge_connector_hpd_notify(struct
> > drm_connector *connector,
> > }
> >
>
> It looks like you only changed the function signature.
> Don't the "bridge->funcs->hpd_notify" inside this function need to pass the
> event?
This is being handled in one of the next patches. Indeed, maybe I should
squash the patch into this one.
>
> drm_for_each_bridge_in_chain_scoped(bridge_connector->encoder, bridge) {
> if (bridge->funcs->hpd_notify)
> bridge->funcs->hpd_notify(bridge, connector, status);
> }
>
>
--
With best wishes
Dmitry
