xiaoxiang781216 commented on code in PR #17807:
URL: https://github.com/apache/nuttx/pull/17807#discussion_r2672462929


##########
drivers/rpmsg/rpmsg.c:
##########
@@ -124,6 +125,11 @@ static int rpmsg_dev_ioctl_(FAR struct rpmsg_s *rpmsg, int 
cmd,
       case RPMSGIOC_PING:
         ret = rpmsg_ping(&rpmsg->ping, (FAR const struct rpmsg_ping_s *)arg);
         break;
+#endif
+#ifdef CONFIG_RPMSG_TEST

Review Comment:
   rpmsg_dev_ioctl_ is an internal api, don't need add the additional check. 
@cederom 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to