This is an automated email from the ASF dual-hosted git repository.

acassis pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx.git

commit f1f2c719d65587e51580695025d485abcc4c8e1e
Author: Yongrong Wang <[email protected]>
AuthorDate: Thu Aug 21 20:40:00 2025 +0800

    rpmsg/rpmsg_virtio: uninit the PM wakelock on remove
    
    rpmsg_virtio_probe() initializes priv->wakelock under
    CONFIG_RPMSG_VIRTIO_PM, but remove freed priv without uninitializing it,
    leaving the wakelock registered with the PM framework.
    
    Signed-off-by: Yongrong Wang <[email protected]>
---
 drivers/rpmsg/rpmsg_virtio.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/rpmsg/rpmsg_virtio.c b/drivers/rpmsg/rpmsg_virtio.c
index 62ebede1817..6f839fa1bd4 100644
--- a/drivers/rpmsg/rpmsg_virtio.c
+++ b/drivers/rpmsg/rpmsg_virtio.c
@@ -785,6 +785,10 @@ void rpmsg_virtio_remove(FAR struct virtio_device *vdev)
     metal_container_of(vdev->priv, struct rpmsg_virtio_priv_s, rvdev);
   char name[64];
 
+#ifdef CONFIG_RPMSG_VIRTIO_PM
+  pm_wakelock_uninit(&priv->wakelock);
+#endif
+
   /* Unregister the rpmsg */
 
   snprintf(name, sizeof(name), "/dev/rpmsg/%s", priv->rpmsg.cpuname);

Reply via email to