[PATCH v4] drm/bridge: imx8qxp-pxl2dpi: avoid ERR_PTR with device_node cleanup
Guangshuo Li
lgs201920130244 at gmail.com
Wed May 6 02:09:47 PDT 2026
Hi Liu,
Thanks for the review.
On Wed, 6 May 2026 at 15:03, Liu Ying <victor.liu at nxp.com> 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
More information about the linux-arm-kernel
mailing list