Changeset 8a64320e in mainline for uspace/lib/drv/generic/remote_ieee80211.c

Timestamp:

2015-04-23T23:40:14Z (10 years ago)

Author:

Martin Decky <martin@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

dcba819

Parents:

09044cb

Message:

pre-merge coding style cleanup and code review

File:

• uspace/lib/drv/generic/remote_ieee80211.c (modified) (15 diffs)

uspace/lib/drv/generic/remote_ieee80211.c

@@ -40 +40 @@
 #include <inet/dhcp.h>
 #include <inet/inetcfg.h>
-
 #include "ops/ieee80211.h"
 #include "ieee80211_iface.h"
 #include "nic_iface.h"
 
-#define MAX_STRING_SIZE 32
+#define MAX_STRING_SIZE  32
 
 /** IEEE 802.11 RPC functions IDs. */
@@ -56 +55 @@
 /** Get scan results from IEEE 802.11 device
  *
- * @param[in] dev_sess Device session.
- * @param[out] results Structure where to put scan results.
+ * @param[in]  dev_sess Device session.
+ * @param[out] results  Structure where to put scan results.
  *
  * @return EOK If the operation was successfully completed,
- * negative error code otherwise.
- */
-int ieee80211_get_scan_results(async_sess_t *dev_sess, 
-        ieee80211_scan_results_t *results, bool now)
+ *         negative error code otherwise.
+ *
+ */
+int ieee80211_get_scan_results(async_sess_t *dev_sess,
+    ieee80211_scan_results_t *results, bool now)
 {
         assert(results);
@@ -71 +71 @@
         aid_t aid = async_send_2(exch, DEV_IFACE_ID(IEEE80211_DEV_IFACE),
             IEEE80211_GET_SCAN_RESULTS, now, NULL);
-        int rc = async_data_read_start(exch, results, sizeof(ieee80211_scan_results_t));
+        int rc = async_data_read_start(exch, results,
+            sizeof(ieee80211_scan_results_t));
         async_exchange_end(exch);
-
+        
         sysarg_t res;
         async_wait_for(aid, &res);
@@ -79 +80 @@
         if(res != EOK)
                 return (int) res;
-        else
-                return rc;
+        
+        return rc;
 }
 
 static bool mac_matches(uint8_t *mac1, uint8_t *mac2)
 {
-        for(size_t i = 0; i < ETH_ADDR; i++) {
-                if(mac1[i] != mac2[i])
+        for (size_t i = 0; i < ETH_ADDR; i++) {
+                if (mac1[i] != mac2[i])
                         return false;
         }
@@ -108 +109 @@
                         return -1;
                 
-                if(mac_matches(mac, link_info.mac_addr)) {
+                if (mac_matches(mac, link_info.mac_addr))
                         return link_list[i];
-                }
         }
         
@@ -118 +118 @@
 /** Connect to specified network.
  *
- * @param[in] dev_sess Device session.
+ * @param[in] dev_sess   Device session.
  * @param[in] ssid_start Network SSID prefix.
- * @param[in] password Network password (pass empty string if not needed).
+ * @param[in] password   Network password (pass empty string if not needed).
  *
  * @return EOK If the operation was successfully completed,
- * negative error code otherwise.
+ *         negative error code otherwise.
+ *
  */
 int ieee80211_connect(async_sess_t *dev_sess, char *ssid_start, char *password)
@@ -136 +137 @@
             IEEE80211_CONNECT, NULL);
         
-        sysarg_t rc = async_data_write_start(exch, ssid_start, 
-                str_size(ssid_start) + 1);
+        sysarg_t rc = async_data_write_start(exch, ssid_start,
+            str_size(ssid_start) + 1);
         if (rc != EOK) {
                 async_exchange_end(exch);
@@ -144 +145 @@
                 if (rc_orig == EOK)
                         return (int) rc;
-                else
-                        return (int) rc_orig;
-        }
-        
-        if(password == NULL) {
+                
+                return (int) rc_orig;
+        }
+        
+        // FIXME: Typecasting string literal
+        if (password == NULL)
                 password = (char *) "";
-        }
         
         rc = async_data_write_start(exch, password, str_size(password) + 1);
@@ -159 +160 @@
                 if (rc_orig == EOK)
                         return (int) rc;
-                else
-                        return (int) rc_orig;
+                
+                return (int) rc_orig;
         }
         
         async_exchange_end(exch);
-
+        
         async_wait_for(aid, &rc);
         if (rc != EOK)
@@ -176 +177 @@
         
         sysarg_t link_id = get_link_id(wifi_mac.address);
-        if(link_id == ((sysarg_t) -1))
+        if (link_id == ((sysarg_t) -1))
                 return EINVAL;
         
@@ -189 +190 @@
  *
  * @return EOK If the operation was successfully completed,
- * negative error code otherwise.
+ *         negative error code otherwise.
+ *
  */
 int ieee80211_disconnect(async_sess_t *dev_sess)
@@ -198 +200 @@
         async_exchange_end(exch);
         
-        if(rc != EOK)
+        if (rc != EOK)
                 return rc;
         
@@ -227 +229 @@
                         return rc;
                 
-                if(mac_matches(wifi_mac.address, link_info.mac_addr)) {
-                        if(str_test_prefix(addr_info.name, "dhcp")) {
+                if (mac_matches(wifi_mac.address, link_info.mac_addr)) {
+                        if (str_test_prefix(addr_info.name, "dhcp")) {
                                 rc = inetcfg_addr_delete(addr_list[i]);
-                                if(rc != EOK)
+                                if (rc != EOK)
                                         return rc;
+                                
                                 break;
                         }
@@ -237 +240 @@
         }
         
-        /* 
+        /*
          * TODO: At this moment there can be only one DHCP route,
          * so it must be reimplemented after this limitation will be
@@ -252 +255 @@
                         return rc;
                 
-                if(str_test_prefix(route_info.name, "dhcp")) {
+                if (str_test_prefix(route_info.name, "dhcp")) {
                         rc = inetcfg_sroute_delete(route_list[i]);
-                        if(rc != EOK)
+                        if (rc != EOK)
                                 return rc;
+                        
                         break;
                 }