xiaoxiang781216 commented on a change in pull request #1064:
URL: 
https://github.com/apache/incubator-nuttx-apps/pull/1064#discussion_r825814903



##########
File path: netutils/usrsock_rpmsg/usrsock_rpmsg_client.c
##########
@@ -155,6 +161,41 @@ static int usrsock_rpmsg_ept_cb(struct rpmsg_endpoint 
*ept, void *data,
   return ret;
 }
 
+#ifdef CONFIG_NETDEV_WIRELESS_IOCTL
+
+static void usersock_coherent_cache(FAR void *buf,
+                                    enum usrsock_cache_action_e action)
+{
+  FAR struct usrsock_request_ioctl_s *req = buf;
+  FAR struct iwreq *wlreq;
+
+  if (req->head.reqid == USRSOCK_REQUEST_IOCTL)
+    {
+      if (WL_IS80211POINTERCMD(req->cmd))
+        {
+          wlreq = (FAR struct iwreq *)(req + 1);
+          if (action == USRSOCK_COHERENT_BEFORE)
+            {
+              metal_cache_flush(wlreq->u.data.pointer,

Review comment:
       usrsock rpmag is built on the top of openamp/libmetal, it's reasonable 
to call meta_ API instead of NuttX specific API to improve the portability.




-- 
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: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to