drivers:staging:ozwpan Replaced wrapper functions with actual spin lock function
authorSurendra Patil <surendra.tux@gmail.com>
Tue, 4 Mar 2014 07:57:47 +0000 (23:57 -0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 5 Mar 2014 00:49:37 +0000 (16:49 -0800)
* Replaced all the spin lock/unlock wrappers from oz_polling_lock_bh()
and oz_polllin_unlock_bh() with spin_lock_bh(&g_polling_lock) and
spin_unlock_bh(&g_polling_lock).Completely erased the wrappers defination
and declaration.
* declared g_polling_lock as global variable in header file and added comments to it.
Module builded successfully with sparse without warnings.

Signed-off-by: Surendra Patil <surendra.tux@gmail.com>
Acked-by: Rupesh Gujare <rupesh.gujare@atmel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/ozwpan/ozpd.c
drivers/staging/ozwpan/ozpd.h
drivers/staging/ozwpan/ozproto.c
drivers/staging/ozwpan/ozproto.h

index 743695077346ff608f617a674459210e94868ced..29a23a325d71eefbbd3652bf3cc01a8c4d33c918 100644 (file)
@@ -284,11 +284,11 @@ int oz_services_start(struct oz_pd *pd, u16 apps, int resume)
                                          ai->app_id);
                                break;
                        }
-                       oz_polling_lock_bh();
+                       spin_lock_bh(&g_polling_lock);
                        pd->total_apps |= (1<<ai->app_id);
                        if (resume)
                                pd->paused_apps &= ~(1<<ai->app_id);
-                       oz_polling_unlock_bh();
+                       spin_unlock_bh(&g_polling_lock);
                }
        }
        return rc;
@@ -304,14 +304,14 @@ void oz_services_stop(struct oz_pd *pd, u16 apps, int pause)
        oz_pd_dbg(pd, ON, "%s: (0x%x) pause(%d)\n", __func__, apps, pause);
        for (ai = g_app_if; ai < &g_app_if[OZ_APPID_MAX]; ai++) {
                if (apps & (1<<ai->app_id)) {
-                       oz_polling_lock_bh();
+                       spin_lock_bh(&g_polling_lock);
                        if (pause) {
                                pd->paused_apps |= (1<<ai->app_id);
                        } else {
                                pd->total_apps &= ~(1<<ai->app_id);
                                pd->paused_apps &= ~(1<<ai->app_id);
                        }
-                       oz_polling_unlock_bh();
+                       spin_unlock_bh(&g_polling_lock);
                        ai->stop(pd, pause);
                }
        }
@@ -349,17 +349,17 @@ void oz_pd_stop(struct oz_pd *pd)
 
        oz_dbg(ON, "oz_pd_stop() State = 0x%x\n", pd->state);
        oz_pd_indicate_farewells(pd);
-       oz_polling_lock_bh();
+       spin_lock_bh(&g_polling_lock);
        stop_apps = pd->total_apps;
        pd->total_apps = 0;
        pd->paused_apps = 0;
-       oz_polling_unlock_bh();
+       spin_unlock_bh(&g_polling_lock);
        oz_services_stop(pd, stop_apps, 0);
-       oz_polling_lock_bh();
+       spin_lock_bh(&g_polling_lock);
        oz_pd_set_state(pd, OZ_PD_S_STOPPED);
        /* Remove from PD list.*/
        list_del(&pd->link);
-       oz_polling_unlock_bh();
+       spin_unlock_bh(&g_polling_lock);
        oz_dbg(ON, "pd ref count = %d\n", atomic_read(&pd->ref_count));
        oz_pd_put(pd);
 }
@@ -372,9 +372,9 @@ int oz_pd_sleep(struct oz_pd *pd)
        int do_stop = 0;
        u16 stop_apps;
 
-       oz_polling_lock_bh();
+       spin_lock_bh(&g_polling_lock);
        if (pd->state & (OZ_PD_S_SLEEP | OZ_PD_S_STOPPED)) {
-               oz_polling_unlock_bh();
+               spin_unlock_bh(&g_polling_lock);
                return 0;
        }
        if (pd->keep_alive && pd->session_id)
@@ -383,7 +383,7 @@ int oz_pd_sleep(struct oz_pd *pd)
                do_stop = 1;
 
        stop_apps = pd->total_apps;
-       oz_polling_unlock_bh();
+       spin_unlock_bh(&g_polling_lock);
        if (do_stop) {
                oz_pd_stop(pd);
        } else {
@@ -999,15 +999,15 @@ void oz_pd_indicate_farewells(struct oz_pd *pd)
        const struct oz_app_if *ai = &g_app_if[OZ_APPID_USB-1];
 
        while (1) {
-               oz_polling_lock_bh();
+               spin_lock_bh(&g_polling_lock);
                if (list_empty(&pd->farewell_list)) {
-                       oz_polling_unlock_bh();
+                       spin_unlock_bh(&g_polling_lock);
                        break;
                }
                f = list_first_entry(&pd->farewell_list,
                                struct oz_farewell, link);
                list_del(&f->link);
-               oz_polling_unlock_bh();
+               spin_unlock_bh(&g_polling_lock);
                if (ai->farewell)
                        ai->farewell(pd, f->ep_num, f->report, f->len);
                kfree(f);
index 12c712956888e6d2f60dc1cbe3fb5eeab27c3f8c..56e6fdf9b0bf5fa2aaea9b79c5ff839768b9e4b6 100644 (file)
 #define OZ_TIMER_HEARTBEAT     2
 #define OZ_TIMER_STOP          3
 
+/* 
+ *External spinlock variable
+ */
+extern spinlock_t g_polling_lock;
+
 /* Data structure that hold information on a frame for transmisson. This is
  * built when the frame is first transmitted and is used to rebuild the frame
  * if a re-transmission is required.
index e7138ed325dda09fa8e1e9b34122af88fd32bf5c..c1325a67d320edc9bfbb5ef46a66ad51ed538802 100644 (file)
@@ -37,10 +37,14 @@ struct oz_binding {
        struct list_head link;
 };
 
+/*
+ * External variable
+ */
+
+DEFINE_SPINLOCK(g_polling_lock);
 /*
  * Static external variables.
  */
-static DEFINE_SPINLOCK(g_polling_lock);
 static LIST_HEAD(g_pd_list);
 static LIST_HEAD(g_binding);
 static DEFINE_SPINLOCK(g_binding_lock);
@@ -794,12 +798,3 @@ int oz_get_pd_list(struct oz_mac_addr *addr, int max_count)
        return count;
 }
 
-void oz_polling_lock_bh(void)
-{
-       spin_lock_bh(&g_polling_lock);
-}
-
-void oz_polling_unlock_bh(void)
-{
-       spin_unlock_bh(&g_polling_lock);
-}
index 0c49c8a0e815f98b395edb516542641294a1bca5..cb38e02c968ea34b866f5016da420a14a04301e9 100644 (file)
@@ -59,8 +59,6 @@ void oz_binding_remove(const char *net_dev);
 void oz_timer_add(struct oz_pd *pd, int type, unsigned long due_time);
 void oz_timer_delete(struct oz_pd *pd, int type);
 void oz_pd_request_heartbeat(struct oz_pd *pd);
-void oz_polling_lock_bh(void);
-void oz_polling_unlock_bh(void);
 void oz_pd_heartbeat_handler(unsigned long data);
 void oz_pd_timeout_handler(unsigned long data);
 enum hrtimer_restart oz_pd_heartbeat_event(struct hrtimer *timer);