Commit c571cb70 authored by Luca Ceresoli's avatar Luca Ceresoli
Browse files

drm/bridge: display-connector: put the bridge obtained by drm_bridge_get_prev_bridge() 



The bridge returned by drm_bridge_get_prev_bridge() is refcounted. Put it
when done.

To keep the code clean and future-proof use a scope-based free.

Reviewed-by: default avatarMaxime Ripard <mripard@kernel.org>
Link: https://lore.kernel.org/r/20250709-drm-bridge-alloc-getput-drm_bridge_get_prev_bridge-v1-3-34ba6f395aaa@bootlin.com


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

drivers/gpu/drm/bridge/display-connector.c

+2 −2
Original line number Diff line number Diff line
@@ -108,7 +108,7 @@ static u32 *display_connector_get_output_bus_fmts(struct drm_bridge *bridge, 
					struct drm_connector_state *conn_state,

					unsigned int *num_output_fmts)

{

	struct drm_bridge *prev_bridge = drm_bridge_get_prev_bridge(bridge);

	struct drm_bridge *prev_bridge __free(drm_bridge_put) = drm_bridge_get_prev_bridge(bridge);

	struct drm_bridge_state *prev_bridge_state;



	if (!prev_bridge || !prev_bridge->funcs->atomic_get_output_bus_fmts) {
@@ -151,7 +151,7 @@ static u32 *display_connector_get_input_bus_fmts(struct drm_bridge *bridge, 
					u32 output_fmt,

					unsigned int *num_input_fmts)

{

	struct drm_bridge *prev_bridge = drm_bridge_get_prev_bridge(bridge);

	struct drm_bridge *prev_bridge __free(drm_bridge_put) = drm_bridge_get_prev_bridge(bridge);

	struct drm_bridge_state *prev_bridge_state;



	if (!prev_bridge || !prev_bridge->funcs->atomic_get_input_bus_fmts) {