(nuttx) branch master updated: drivers/rpmsg: take the ping fill byte as unsigned char

[archer]

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

anchao 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 55090afb01e drivers/rpmsg: take the ping fill byte as unsigned char
55090afb01e is described below

commit 55090afb01e8ace90315bbb1de8046b01b62eb7c
Author: Bowen Wang <wangbow...@xiaomi.com>
AuthorDate: Wed Oct 28 21:30:37 2026 +0800

    drivers/rpmsg: take the ping fill byte as unsigned char
    
    rpmsg_ping_once() received its fill byte as plain char, whose
    signedness is implementation-defined, and passed it to memset().
    memset() converts the value to unsigned char, so the bytes written are
    the same either way; use unsigned char to make the intent explicit.
    No functional change.
    
    Signed-off-by: Bowen Wang <wangbow...@xiaomi.com>
---
 drivers/rpmsg/rpmsg_ping.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/rpmsg/rpmsg_ping.c b/drivers/rpmsg/rpmsg_ping.c
index b20ff0a1afc..addfb3a239f 100644
--- a/drivers/rpmsg/rpmsg_ping.c
+++ b/drivers/rpmsg/rpmsg_ping.c
@@ -121,7 +121,7 @@ static int rpmsg_ping_ept_cb(FAR struct rpmsg_endpoint *ept,
 }
 
 static int rpmsg_ping_once(FAR struct rpmsg_endpoint *ept, int len,
-                           int cmd, uint32_t *buf_len, char i)
+                           int cmd, uint32_t *buf_len, unsigned char i)
 {
   FAR struct rpmsg_ping_msg_s *msg;
   int ret;