Commit e0686dc6 authored by Liang He's avatar Liang He Committed by Daniel Vetter
Browse files

drm:pl111: Add of_node_put() when breaking out of for_each_available_child_of_node() 



The reference 'child' in the iteration of for_each_available_child_of_node()
is only escaped out into a local variable which is only used to check
its value. So we still need to the of_node_put() when breaking of the
for_each_available_child_of_node() which will automatically increase
and decrease the refcount.

Fixes: ca454bd4 ("drm/pl111: Support the Versatile Express")
Signed-off-by: default avatarLiang He <windhl@126.com>
Reviewed-by: default avatarRob Herring <robh@kernel.org>
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/20220711131550.361350-1-windhl@126.com
parent 8759464d

drivers/gpu/drm/pl111/pl111_versatile.c

+1 −0
Original line number Diff line number Diff line
@@ -404,6 +404,7 @@ static int pl111_vexpress_clcd_init(struct device *dev, struct device_node *np, 
		if (of_device_is_compatible(child, "arm,pl111")) {

			has_coretile_clcd = true;

			ct_clcd = child;

			of_node_put(child);

			break;

		}

		if (of_device_is_compatible(child, "arm,hdlcd")) {