Commit 0885e76c authored by Luca Ceresoli's avatar Luca Ceresoli
Browse files

drm/imx: parallel-display: put the bridge returned by drm_bridge_get_next_bridge() 



The bridge returned by drm_bridge_get_next_bridge() is refcounted. Put it
when done. We need to ensure it is not put before either next_bridge or
next_bridge_state is in use, thus for simplicity use a cleanup action.

Reviewed-by: default avatarMaxime Ripard <mripard@kernel.org>
Reviewed-by: default avatarPhilipp Zabel <p.zabel@pengutronix.de>
Link: https://lore.kernel.org/r/20250801-drm-bridge-alloc-getput-drm_bridge_get_next_bridge-v2-9-888912b0be13@bootlin.com


Signed-off-by: default avatarLuca Ceresoli <luca.ceresoli@bootlin.com>
parent da550b5a

drivers/gpu/drm/imx/ipuv3/parallel-display.c

+2 −2
Original line number Diff line number Diff line
@@ -134,10 +134,10 @@ static int imx_pd_bridge_atomic_check(struct drm_bridge *bridge, 
	struct imx_crtc_state *imx_crtc_state = to_imx_crtc_state(crtc_state);

	struct drm_display_info *di = &conn_state->connector->display_info;

	struct drm_bridge_state *next_bridge_state = NULL;

	struct drm_bridge *next_bridge;

	u32 bus_flags, bus_fmt;



	next_bridge = drm_bridge_get_next_bridge(bridge);

	struct drm_bridge *next_bridge __free(drm_bridge_put) = drm_bridge_get_next_bridge(bridge);



	if (next_bridge)

		next_bridge_state = drm_atomic_get_new_bridge_state(crtc_state->state,

								    next_bridge);