[GitHub] [incubator-nuttx] zhhyu7 commented on a diff in pull request #6330: tcp: move wd_timer from wifi driver to tcp stack

[GitBox <gi...@apache.org>]

zhhyu7 commented on code in PR #6330:
URL: https://github.com/apache/incubator-nuttx/pull/6330#discussion_r882471174


##########
net/devif/devif_poll.c:
##########
@@ -849,29 +798,7 @@ int devif_poll(FAR struct net_driver_s *dev, devif_poll_callback_t callback)
 int devif_timer(FAR struct net_driver_s *dev, int delay,
                 devif_poll_callback_t callback)
 {
-#if defined(NET_TCP_HAVE_STACK)
-  int hsec = TICK2HSEC(delay);
-#endif
-  int bstop = false;
-
-#ifdef NET_TCP_HAVE_STACK
-  /* Traverse all of the active TCP connections and perform the
-   * timer action.
-   */
-
-  bstop = devif_poll_tcp_timer(dev, callback, hsec);
-#endif
-
-  /* If possible, continue with a normal poll checking for pending
-   * network driver actions.
-   */
-
-  if (!bstop)
-    {
-      bstop = devif_poll(dev, callback);
-    }
-
-  return bstop;
+  return devif_poll(dev, callback);

Review Comment:
   > so `delay` parameter now is ignored?
   
   yes, we will remove all devif_timer in the next change.



-- 
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-unsubscribe@nuttx.apache.org

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