Hi,

At 2026-07-01 11:08:35, "Chen Ni" <[email protected]> wrote:
>The drm_atomic_get_bridge_state() function returns an error pointer on
>failure, not NULL. Fix the check to use IS_ERR() to correctly handle
>allocation failures.
>
>Fixes: dc70272cc184 ("drm/rockchip: dw_hdmi: Propagate bus format to display 
>driver")
>Signed-off-by: Chen Ni <[email protected]>


   Reviewed-by:Andy Yan <[email protected]>

>---
> drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c 
>b/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c
>index cffc0af5190f..766cf21a4be7 100644
>--- a/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c
>+++ b/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c
>@@ -287,7 +287,7 @@ static u32 dw_hdmi_rockchip_get_bus_format(struct 
>drm_encoder *encoder,
>               return 0;
> 
>       bridge_state = drm_atomic_get_bridge_state(conn_state->state, bridge);
>-      if (!bridge_state)
>+      if (IS_ERR(bridge_state))
>               return 0;
> 
>       if (bridge_state->input_bus_cfg.format != MEDIA_BUS_FMT_FIXED)
>-- 
>2.25.1
>
>
>_______________________________________________
>Linux-rockchip mailing list
>[email protected]
>http://lists.infradead.org/mailman/listinfo/linux-rockchip

Reply via email to