Re: [PATCH 3/3] drm/bridge: imx8qxp-pixel-combiner: Fix bailout for imx8qxp_pc_bridge_probe()

[Liu Ying <victor.liu@nxp.com>]

On Mon, Jan 26, 2026 at 09:33:38AM +0100, Luca Ceresoli wrote:
> On Fri Jan 23, 2026 at 10:22 AM CET, Liu Ying wrote:
>> In case the channel0 is unavailable and bailing out from free_child is
>> needed when we fail to add a DRM bridge for the available channel1,
>> pointer pc->ch[0] in the bailout path would be NULL and it would be
>> dereferenced as pc->ch[0]->bridge.next_bridge.  Fix this by checking
>> pc->ch[0] before dereferencing it.
> 
> Thanks for checking this code and providing fixes! These drivers have a
> complex code path so I had a hard time in following the logic without neing
> able to test on hardware. Apologies for the issues emerged.

Thanks for your quick review and hard work!

> 
>> Fixes: ae754f049ce1 ("drm/bridge: imx8qxp-pixel-combiner: get/put the next bridge")
>> Fixes: 99764593528f ("drm/bridge: imx8qxp-pixel-combiner: convert to devm_drm_bridge_alloc() API")
> 
> The bug was introduced by 99764593528f, while ae754f049ce1 is not changing
> the bug, so for the sake of precision we can remove the first 'Fixes:'
> line.

I added Fixes tag for ae754f049ce1 because it is the first commit which
this commit could be applied upon properly.  Maybe this helps some tools
apply this commit or maybe tools are smart enough to find ae754f049ce1
automatically?

> 
> With that:
>  Reviewed-by: Luca Ceresoli <luca.ceresoli@bootlin.com>

I hope I can have this R-b.  Please let me know if I need to drop the
Fixes tag for ae754f049ce1.

> 
> --
> Luca Ceresoli, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com/

-- 
Regards,
Liu Ying