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/01/25 04:04:01 UTC

[incubator-nuttx] branch master updated: drivers: wireless: Print error massages with wlerr() in gs2200m.c

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 8a72ab1  drivers: wireless: Print error massages with wlerr() in gs2200m.c
8a72ab1 is described below

commit 8a72ab1a50b5f449d7717e015b854c7343687c2f
Author: Masayuki Ishikawa <ma...@gmail.com>
AuthorDate: Mon Jan 25 10:27:11 2021 +0900

    drivers: wireless: Print error massages with wlerr() in gs2200m.c
    
    Summary:
    - I noticed that some error messages are printed with wlinfo()
    - This commit prints such error messages with wlerr()
    
    Impact:
    - No impact
    
    Testing:
    - Tested with spresense:wifi
    - NOTE: CONFIG_DEBUG_WIRELESS_ERROR must be enabled
    
    Signed-off-by: Masayuki Ishikawa <Ma...@jp.sony.com>
---
 drivers/wireless/gs2200m.c | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/drivers/wireless/gs2200m.c b/drivers/wireless/gs2200m.c
index ac9c597..0c7b739 100644
--- a/drivers/wireless/gs2200m.c
+++ b/drivers/wireless/gs2200m.c
@@ -1895,8 +1895,8 @@ gs2200m_create_clnt(FAR struct gs2200m_dev_s *dev,
 
   if (r != TYPE_OK || pkt_dat.n == 0)
     {
-      wlinfo("+++ error: r=%d pkt_dat.msg[0]=%s \n",
-             r, pkt_dat.msg[0]);
+      wlerr("+++ error: r=%d pkt_dat.msg[0]=%s \n",
+            r, pkt_dat.msg[0]);
       goto errout;
     }
 
@@ -2165,8 +2165,8 @@ static enum pkt_type_e gs2200m_get_cstatus(FAR struct gs2200m_dev_s *dev,
 
   if (r != TYPE_OK || pkt_dat.n <= 2)
     {
-      wlinfo("+++ error: r=%d pkt_dat.msg[0]=%s \n",
-             r, pkt_dat.msg[0]);
+      wlerr("+++ error: r=%d pkt_dat.msg[0]=%s \n",
+            r, pkt_dat.msg[0]);
 
       goto errout;
     }
@@ -2737,8 +2737,8 @@ static int gs2200m_ioctl_iwreq(FAR struct gs2200m_dev_s *dev,
 
   if (r != TYPE_OK || pkt_dat.n <= 7)
     {
-      wlinfo("+++ error: r=%d pkt_dat.msg[0]=%s \n",
-             r, pkt_dat.msg[0]);
+      wlerr("+++ error: r=%d pkt_dat.msg[0]=%s \n",
+            r, pkt_dat.msg[0]);
 
       goto errout;
     }
@@ -2749,7 +2749,7 @@ static int gs2200m_ioctl_iwreq(FAR struct gs2200m_dev_s *dev,
     {
       if (strstr(pkt_dat.msg[2], "BSSID=") == NULL)
         {
-          wlinfo("+++ error: pkt_dat.msg[2]=%s \n", pkt_dat.msg[2]);
+          wlerr("+++ error: pkt_dat.msg[2]=%s \n", pkt_dat.msg[2]);
           goto errout;
         }
 
@@ -2768,7 +2768,7 @@ static int gs2200m_ioctl_iwreq(FAR struct gs2200m_dev_s *dev,
     {
       if (strstr(pkt_dat.msg[2], "CHANNEL=") == NULL)
         {
-          wlinfo("+++ error: pkt_dat.msg[2]=%s \n", pkt_dat.msg[2]);
+          wlerr("+++ error: pkt_dat.msg[2]=%s \n", pkt_dat.msg[2]);
           goto errout;
         }
 
@@ -2781,7 +2781,7 @@ static int gs2200m_ioctl_iwreq(FAR struct gs2200m_dev_s *dev,
     {
       if (strstr(pkt_dat.msg[3], "RSSI=") == NULL)
         {
-          wlinfo("+++ error: pkt_dat.msg[3]=%s \n", pkt_dat.msg[3]);
+          wlerr("+++ error: pkt_dat.msg[3]=%s \n", pkt_dat.msg[3]);
           goto errout;
         }
 
@@ -3195,8 +3195,8 @@ repeat:
     {
       /* An error event? */
 
-      wlinfo("=== ignore (type=%d msg[0]=%s|) \n",
-             pkt_dat->type, pkt_dat->msg[0]);
+      wlerr("=== ignore (type=%d msg[0]=%s|) \n",
+            pkt_dat->type, pkt_dat->msg[0]);
 
       ignored = true;
       goto errout;