Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider: - [Low] drm/omap: unupdated hpd_notify() signature in hdmi5_bridge_hpd_notify() --
commit 9cf55ff0f96b440ba05665e3db9e82c5372fd640 Author: Dmitry Baryshkov <[email protected]> drm/bridge: pass down IRQ_HPD to the drivers This commit adds an enum drm_connector_dp_event parameter to the drm_bridge_funcs.hpd_notify callback and updates its callers and bridge implementations so DisplayPort bridges can handle IRQ_HPD events. [ ... ] > diff --git a/drivers/gpu/drm/omapdrm/dss/hdmi4.c > b/drivers/gpu/drm/omapdrm/dss/hdmi4.c > index bd6c4d2dcdc86..72972a6fe989b 100644 > --- a/drivers/gpu/drm/omapdrm/dss/hdmi4.c > +++ b/drivers/gpu/drm/omapdrm/dss/hdmi4.c > @@ -429,7 +429,8 @@ static void hdmi4_bridge_disable(struct drm_bridge > *bridge, > > static void hdmi4_bridge_hpd_notify(struct drm_bridge *bridge, > struct drm_connector *connector, > - enum drm_connector_status status) > + enum drm_connector_status status, > + enum drm_connector_dp_event event) > { > struct omap_hdmi *hdmi = drm_bridge_to_hdmi(bridge); > > diff --git a/include/drm/drm_bridge.h b/include/drm/drm_bridge.h > index 1981d24a700d2..6382ef5f1ea28 100644 > --- a/include/drm/drm_bridge.h > +++ b/include/drm/drm_bridge.h > @@ -509,7 +509,8 @@ struct drm_bridge_funcs { > */ > void (*hpd_notify)(struct drm_bridge *bridge, > struct drm_connector *connector, > - enum drm_connector_status status); > + enum drm_connector_status status, > + enum drm_connector_dp_event event); > > /** > * @hpd_enable: [Severity: Low] While hdmi4_bridge_hpd_notify() in drivers/gpu/drm/omapdrm/dss/hdmi4.c is updated here, it looks like hdmi5_bridge_hpd_notify() in drivers/gpu/drm/omapdrm/dss/hdmi5.c still has the old 3-argument signature: static void hdmi5_bridge_hpd_notify(struct drm_bridge *bridge, struct drm_connector *connector, enum drm_connector_status status) { struct omap_hdmi *hdmi = drm_bridge_to_hdmi(bridge); hdmi_audio_hpd_notify(hdmi, status); } ... static const struct drm_bridge_funcs hdmi5_bridge_funcs = { ... .hpd_notify = hdmi5_bridge_hpd_notify, .edid_read = hdmi5_bridge_edid_read, }; Should hdmi5_bridge_hpd_notify() also be updated to accept enum drm_connector_dp_event event so that builds with CONFIG_OMAP5_DSS_HDMI enabled do not fail with an incompatible function pointer type error? -- Sashiko AI review ยท https://sashiko.dev/#/patchset/[email protected]?part=3
