Hi Liu, Thanks for the review.
On Wed, 6 May 2026 at 15:03, Liu Ying <[email protected]> wrote: > > Can you provide a minimal fix for stable tree by not using the cleanup > action? You can add the cleanup action with follow-up patch(es). > Yes, I agree. I will make the stable fix minimal by not changing the helper function pattern. In v5, I will only avoid using the cleanup action for the endpoint node in imx8qxp_pxl2dpi_find_next_bridge() and release it explicitly with of_node_put() after of_graph_get_remote_port_parent(). > Why do you need to initialize ep to NULL? > The NULL initialization was only needed because v4 changed the helper to use an output argument together with a cleanup variable. With the minimal fix, ep will be a normal pointer, so the NULL initialization is not needed. I will send a v5. Best regards, Guangshuo
