Commit e45d7337 authored by Liang He's avatar Liang He Committed by Greg Kroah-Hartman
Browse files

usb: typec: anx7411: Use of_get_child_by_name() instead of of_find_node_by_name() 



In anx7411_typec_switch_probe(), we should call of_get_child_by_name()
instead of of_find_node_by_name() as of_find_xxx API will decrease the
refcount of the 'from' argument.

Fixes: fe6d8a9c ("usb: typec: anx7411: Add Analogix PD ANX7411 support")
Acked-by: default avatarHeikki Krogerus <heikki.krogerus@linux.intel.com>
Signed-off-by: default avatarLiang He <windhl@126.com>
Link: https://lore.kernel.org/r/20220915092209.4009273-1-windhl@126.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 7489ec86

drivers/usb/typec/anx7411.c

+2 −2
Original line number Diff line number Diff line
@@ -1105,7 +1105,7 @@ static int anx7411_typec_switch_probe(struct anx7411_data *ctx, 
	int ret;

	struct device_node *node;



	node = of_find_node_by_name(dev->of_node, "orientation_switch");

	node = of_get_child_by_name(dev->of_node, "orientation_switch");

	if (!node)

		return 0;
@@ -1115,7 +1115,7 @@ static int anx7411_typec_switch_probe(struct anx7411_data *ctx, 
		return ret;

	}



	node = of_find_node_by_name(dev->of_node, "mode_switch");

	node = of_get_child_by_name(dev->of_node, "mode_switch");

	if (!node) {

		dev_err(dev, "no typec mux exist");

		ret = -ENODEV;