drm:pl111: Add of_node_put() when breaking out of for_each_available_child_of_node()
[ Upstream commite0686dc6f2] 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:ca454bd42d("drm/pl111: Support the Versatile Express") Signed-off-by: Liang He <windhl@126.com> Reviewed-by: Rob Herring <robh@kernel.org> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch> Link: https://patchwork.freedesktop.org/patch/msgid/20220711131550.361350-1-windhl@126.com Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
ad06d6bed5
commit
5e25bfcd12
@@ -402,6 +402,7 @@ static int pl111_vexpress_clcd_init(struct device *dev, struct device_node *np,
|
|||||||
if (of_device_is_compatible(child, "arm,pl111")) {
|
if (of_device_is_compatible(child, "arm,pl111")) {
|
||||||
has_coretile_clcd = true;
|
has_coretile_clcd = true;
|
||||||
ct_clcd = child;
|
ct_clcd = child;
|
||||||
|
of_node_put(child);
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
if (of_device_is_compatible(child, "arm,hdlcd")) {
|
if (of_device_is_compatible(child, "arm,hdlcd")) {
|
||||||
|
|||||||
Reference in New Issue
Block a user