[PATCH v2] usb: typec: class: drop PD lookup reference

[PATCH v2] usb: typec: class: drop PD lookup reference

[Shuangpeng Bai]

usb_power_delivery_find() wraps class_find_device_by_name(). That helper
returns a device reference that must be released by the caller.

select_usb_power_delivery_store() only needs this reference while calling
the pd_set callback. Drop it once the callback returns. Otherwise the sysfs
write can pin the selected USB Power Delivery object and prevent it from
being released on unregister.

Fixes: a7cff92f0635 ("usb: typec: USB Power Delivery helpers for ports and partners")
Cc: stable@vger.kernel.org
Signed-off-by: Shuangpeng Bai <shuangpeng.kernel@gmail.com>
---
v2:
- Cc stable@vger.kernel.org as requested.

 drivers/usb/typec/class.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/usb/typec/class.c b/drivers/usb/typec/class.c
index 0977581ad1b6..0595e8cb83aa 100644
--- a/drivers/usb/typec/class.c
+++ b/drivers/usb/typec/class.c
@@ -1619,6 +1619,7 @@ static ssize_t select_usb_power_delivery_store(struct device *dev,
 		return -EINVAL;
 
 	ret = port->ops->pd_set(port, pd);
+	put_device(&pd->dev);
 	if (ret)
 		return ret;
 
-- 
2.43.0

Re: [PATCH v2] usb: typec: class: drop PD lookup reference

[Heikki Krogerus]

On Sat, Jul 04, 2026 at 07:14:36PM -0400, Shuangpeng Bai wrote:
> usb_power_delivery_find() wraps class_find_device_by_name(). That helper
> returns a device reference that must be released by the caller.
> 
> select_usb_power_delivery_store() only needs this reference while calling
> the pd_set callback. Drop it once the callback returns. Otherwise the sysfs
> write can pin the selected USB Power Delivery object and prevent it from
> being released on unregister.
> 
> Fixes: a7cff92f0635 ("usb: typec: USB Power Delivery helpers for ports and partners")
> Cc: stable@vger.kernel.org
> Signed-off-by: Shuangpeng Bai <shuangpeng.kernel@gmail.com>

Revieved-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>

> ---
> v2:
> - Cc stable@vger.kernel.org as requested.
> 
>  drivers/usb/typec/class.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/usb/typec/class.c b/drivers/usb/typec/class.c
> index 0977581ad1b6..0595e8cb83aa 100644
> --- a/drivers/usb/typec/class.c
> +++ b/drivers/usb/typec/class.c
> @@ -1619,6 +1619,7 @@ static ssize_t select_usb_power_delivery_store(struct device *dev,
>  		return -EINVAL;
>  
>  	ret = port->ops->pd_set(port, pd);
> +	put_device(&pd->dev);
>  	if (ret)
>  		return ret;
>  
> -- 
> 2.43.0

-- 
heikki