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

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


The following commit(s) were added to refs/heads/master by this push:
     new ae2f4f9d87f drivers/rpmsg/rpmsg_port: log the register and unregister 
steps
ae2f4f9d87f is described below

commit ae2f4f9d87f6404fc45f25cd61986b62599bec7d
Author: liaoao <[email protected]>
AuthorDate: Thu Oct 23 22:27:42 2025 +0800

    drivers/rpmsg/rpmsg_port: log the register and unregister steps
    
    Print a line before and after rpmsg_port_register() and
    rpmsg_port_unregister(), so a log shows whether a port came up or went
    down and where it stopped. Move rpmsgvbs from rpmsg_port_uart.c to
    rpmsg_port.h for that.
    
    Signed-off-by: liaoao <[email protected]>
---
 drivers/rpmsg/rpmsg_port.c      | 4 ++++
 drivers/rpmsg/rpmsg_port.h      | 2 ++
 drivers/rpmsg/rpmsg_port_uart.c | 1 -
 3 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/drivers/rpmsg/rpmsg_port.c b/drivers/rpmsg/rpmsg_port.c
index fa139ee077d..62cacf033d0 100644
--- a/drivers/rpmsg/rpmsg_port.c
+++ b/drivers/rpmsg/rpmsg_port.c
@@ -812,6 +812,7 @@ int rpmsg_port_register(FAR struct rpmsg_port_s *port,
       strlcpy(port->rpmsg.local_cpuname, local_cpuname, RPMSG_NAME_SIZE);
     }
 
+  rpmsgvbs("start register %s\n", port->rpmsg.cpuname);
   snprintf(name, sizeof(name), "/dev/rpmsg/%s", port->rpmsg.cpuname);
   ret = rpmsg_register(name, &port->rpmsg, &g_rpmsg_port_ops);
   if (ret < 0)
@@ -823,6 +824,7 @@ int rpmsg_port_register(FAR struct rpmsg_port_s *port,
                           RPMSG_NS_EPT_ADDR, RPMSG_NS_EPT_ADDR,
                           rpmsg_port_ns_callback, NULL, port);
   rpmsg_device_created(&port->rpmsg);
+  rpmsgvbs("register %s success\n", port->rpmsg.cpuname);
   return ret;
 }
 
@@ -834,10 +836,12 @@ void rpmsg_port_unregister(FAR struct rpmsg_port_s *port)
 {
   char name[64];
 
+  rpmsgvbs("start unregister %s\n", port->rpmsg.cpuname);
   snprintf(name, sizeof(name), "/dev/rpmsg/%s", port->rpmsg.cpuname);
 
   rpmsg_device_destroy(&port->rpmsg);
   rpmsg_unregister(name, &port->rpmsg);
+  rpmsgvbs("unregister %s success\n", port->rpmsg.cpuname);
 }
 
 /****************************************************************************
diff --git a/drivers/rpmsg/rpmsg_port.h b/drivers/rpmsg/rpmsg_port.h
index ed14b78f1ab..acb3e9c7311 100644
--- a/drivers/rpmsg/rpmsg_port.h
+++ b/drivers/rpmsg/rpmsg_port.h
@@ -40,6 +40,8 @@
  * Pre-processor Definitions
  ****************************************************************************/
 
+#define rpmsgvbs                rpmsgerr
+
 #define RPMSG_PORT_DROP_TXQ     0x01
 #define RPMSG_PORT_DROP_RXQ     0x02
 #define RPMSG_PORT_DROP_ALL     0x03
diff --git a/drivers/rpmsg/rpmsg_port_uart.c b/drivers/rpmsg/rpmsg_port_uart.c
index cc3e8200402..f6e03fbc509 100644
--- a/drivers/rpmsg/rpmsg_port_uart.c
+++ b/drivers/rpmsg/rpmsg_port_uart.c
@@ -99,7 +99,6 @@
 #endif
 
 #define rpmsgerrdump(m,b,s)                lib_dumpbuffer(m, (FAR const 
uint8_t *)b, s)
-#define rpmsgvbs                           rpmsgerr
 
 #define rpmsg_port_uart_staywake(r)        rpmsg_port_uart_staywake_times(r, 
~0)
 

Reply via email to