[PATCH] drm/bridge_connector: use drm_kms_helper_connector_hotplug_event()
Sam Ravnborg
sam at ravnborg.org
Tue Jun 20 18:58:20 UTC 2023
Hi Simon,
On Tue, Jun 20, 2023 at 05:55:09PM +0000, Simon Ser wrote:
> This adds more information to the hotplug uevent and lets user-space
> know that it's about a particular connector only.
Will this change have impact on user space?
They will start to see CONNECTOR=n where they did not occur before.
Asking as I have not really worked with the hotplug stuff.
The code change looks fine - no comment there.
And if there are no user space worries then:
Reviewed-by: Sam Ravnborg <sam at ravnborg.org>
Sam
>
> Signed-off-by: Simon Ser <contact at emersion.fr>
> Cc: Dmitry Baryshkov <dmitry.baryshkov at linaro.org>
> Cc: Paul Cercueil <paul at crapouillou.net>
> Cc: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> Cc: Sam Ravnborg <sam at ravnborg.org>
> ---
> drivers/gpu/drm/drm_bridge_connector.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/drm_bridge_connector.c b/drivers/gpu/drm/drm_bridge_connector.c
> index 19ae4a177ac3..fc6f16e14f36 100644
> --- a/drivers/gpu/drm/drm_bridge_connector.c
> +++ b/drivers/gpu/drm/drm_bridge_connector.c
> @@ -125,7 +125,7 @@ static void drm_bridge_connector_hpd_cb(void *cb_data,
>
> drm_bridge_connector_hpd_notify(connector, status);
>
> - drm_kms_helper_hotplug_event(dev);
> + drm_kms_helper_connector_hotplug_event(connector);
> }
>
> static void drm_bridge_connector_enable_hpd(struct drm_connector *connector)
> --
> 2.41.0
>
More information about the dri-devel
mailing list