[PATCH] drm/rockchip: dw_hdmi: Fix error handling for drm_atomic_get_bridge_state()

[Andy Yan]

Hi，


At 2026-07-01 11:08:35, "Chen Ni" <nichen at iscas.ac.cn> 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 <nichen at iscas.ac.cn>


   Reviewed-by：Andy Yan <andyshrk at 163.com>

>---
> 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
>Linux-rockchip at lists.infradead.org
>http://lists.infradead.org/mailman/listinfo/linux-rockchip