You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by xi...@apache.org on 2021/09/12 03:32:10 UTC

[incubator-nuttx] branch master updated: rpmsg_socket: add prefix "rpmsg-socket" for the name of rpmsg endpoint

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

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


The following commit(s) were added to refs/heads/master by this push:
     new ce4b79d  rpmsg_socket: add prefix "rpmsg-socket" for the name of rpmsg endpoint
ce4b79d is described below

commit ce4b79d554d7ff1e653853673042e68c3aac7729
Author: Jiuzhu Dong <do...@xiaomi.com>
AuthorDate: Thu Sep 9 10:52:44 2021 +0800

    rpmsg_socket: add prefix "rpmsg-socket" for the name of rpmsg endpoint
    
    Signed-off-by: Jiuzhu Dong <do...@xiaomi.com>
---
 net/rpmsg/rpmsg_sockif.c | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/net/rpmsg/rpmsg_sockif.c b/net/rpmsg/rpmsg_sockif.c
index 7cfedab..78f248a 100644
--- a/net/rpmsg/rpmsg_sockif.c
+++ b/net/rpmsg/rpmsg_sockif.c
@@ -50,6 +50,7 @@
 
 #define RPMSG_SOCKET_CMD_SYNC       1
 #define RPMSG_SOCKET_CMD_DATA       2
+#define RPMSG_SOCKET_NAME_PREFIX    "rpmsg-socket"
 
 /****************************************************************************
  * Private Types
@@ -406,6 +407,7 @@ static void rpmsg_socket_device_created(FAR struct rpmsg_device *rdev,
                                         FAR void *priv)
 {
   FAR struct rpmsg_socket_conn_s *conn = priv;
+  char buf[RPMSG_SOCKET_NAME_SIZE];
 
   if (conn->ept.rdev)
     {
@@ -415,8 +417,10 @@ static void rpmsg_socket_device_created(FAR struct rpmsg_device *rdev,
   if (strcmp(conn->rpaddr.rp_cpu, rpmsg_get_cpuname(rdev)) == 0)
     {
       conn->ept.priv = conn;
+      snprintf(buf, sizeof(buf), "%s:%s", RPMSG_SOCKET_NAME_PREFIX,
+               conn->rpaddr.rp_name);
 
-      rpmsg_create_ept(&conn->ept, rdev, conn->rpaddr.rp_name,
+      rpmsg_create_ept(&conn->ept, rdev, buf,
                        RPMSG_ADDR_ANY, RPMSG_ADDR_ANY,
                        rpmsg_socket_ept_cb, rpmsg_socket_ns_unbind);
     }
@@ -453,11 +457,13 @@ static void rpmsg_socket_ns_bind(FAR struct rpmsg_device *rdev,
   FAR struct rpmsg_socket_conn_s *server = priv;
   FAR struct rpmsg_socket_conn_s *tmp;
   FAR struct rpmsg_socket_conn_s *new;
+  char buf[RPMSG_SOCKET_NAME_SIZE];
   int cnt = 0;
   int ret;
 
-  if (strncmp(name, server->rpaddr.rp_name,
-              strlen(server->rpaddr.rp_name)))
+  snprintf(buf, sizeof(buf), "%s:%s", RPMSG_SOCKET_NAME_PREFIX,
+           server->rpaddr.rp_name);
+  if (strncmp(name, buf, strlen(buf)))
     {
       return;
     }
@@ -555,8 +561,8 @@ static int rpmsg_socket_setaddr(FAR struct rpmsg_socket_conn_s *conn,
 
       rpaddr = &conn->rpaddr;
       len = strlen(rpaddr->rp_name);
-      snprintf(&rpaddr->rp_name[len], RPMSG_SOCKET_NAME_SIZE - len - 1,
-               "%u", g_rpmsg_id++);
+      snprintf(&rpaddr->rp_name[len], sizeof(rpaddr->rp_name) - len - 1,
+               ":%u", g_rpmsg_id++);
     }
 
   return 0;