Commit 65544caf authored by Sean Anderson's avatar Sean Anderson Committed by Zheng Zengkai
Browse files

usb: ulpi: Move of_node_put to ulpi_dev_release 

stable inclusion
from stable-v5.10.101
commit 8b89a6916681a110403961a2bb923f1789340933
bugzilla: https://gitee.com/openeuler/kernel/issues/I5669Z

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=8b89a6916681a110403961a2bb923f1789340933



--------------------------------

commit 092f45b1 upstream.

Drivers are not unbound from the device when ulpi_unregister_interface
is called. Move of_node-freeing code to ulpi_dev_release which is called
only after all users are gone.

Fixes: ef6a7bcf ("usb: ulpi: Support device discovery via DT")
Cc: stable <stable@vger.kernel.org>
Reviewed-by: default avatarHeikki Krogerus <heikki.krogerus@linux.intel.com>
Signed-off-by: default avatarSean Anderson <sean.anderson@seco.com>
Link: https://lore.kernel.org/r/20220127190004.1446909-2-sean.anderson@seco.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarYu Liao <liaoyu15@huawei.com>
Reviewed-by: default avatarWei Li <liwei391@huawei.com>
Signed-off-by: default avatarZheng Zengkai <zhengzengkai@huawei.com>
parent 5e70e53e

drivers/usb/common/ulpi.c

+1 −1
Original line number Diff line number Diff line
@@ -132,6 +132,7 @@ static const struct attribute_group *ulpi_dev_attr_groups[] = { 


static void ulpi_dev_release(struct device *dev)

{

	of_node_put(dev->of_node);

	kfree(to_ulpi_dev(dev));

}
@@ -301,7 +302,6 @@ EXPORT_SYMBOL_GPL(ulpi_register_interface); 
 */

void ulpi_unregister_interface(struct ulpi *ulpi)

{

	of_node_put(ulpi->dev.of_node);

	device_unregister(&ulpi->dev);

}

EXPORT_SYMBOL_GPL(ulpi_unregister_interface);