Commit dfefb9f8 authored by Kees Cook's avatar Kees Cook Committed by Kalle Valo

drivers/wireless: rsi: Convert timers to use timer_setup()

In preparation for unconditionally passing the struct timer_list pointer to
all timer callbacks, switch to using the new timer_setup() and from_timer()
to pass the timer pointer explicitly.

Cc: Kalle Valo <kvalo@codeaurora.org>
Cc: Amitkumar Karwar <amit.karwar@redpinesignals.com>
Cc: Prameela Rani Garnepudi <prameela.j04cs@gmail.com>
Cc: Pavani Muthyala <pavani.muthyala@redpinesignals.com>
Cc: Karun Eagalapati <karun256@gmail.com>
Cc: linux-wireless@vger.kernel.org
Cc: netdev@vger.kernel.org
Signed-off-by: default avatarKees Cook <keescook@chromium.org>
Signed-off-by: default avatarKalle Valo <kvalo@codeaurora.org>
parent e3dcf8bb
...@@ -418,9 +418,9 @@ int rsi_prepare_beacon(struct rsi_common *common, struct sk_buff *skb) ...@@ -418,9 +418,9 @@ int rsi_prepare_beacon(struct rsi_common *common, struct sk_buff *skb)
return 0; return 0;
} }
static void bl_cmd_timeout(unsigned long priv) static void bl_cmd_timeout(struct timer_list *t)
{ {
struct rsi_hw *adapter = (struct rsi_hw *)priv; struct rsi_hw *adapter = from_timer(adapter, t, bl_cmd_timer);
adapter->blcmd_timer_expired = true; adapter->blcmd_timer_expired = true;
del_timer(&adapter->bl_cmd_timer); del_timer(&adapter->bl_cmd_timer);
...@@ -428,8 +428,7 @@ static void bl_cmd_timeout(unsigned long priv) ...@@ -428,8 +428,7 @@ static void bl_cmd_timeout(unsigned long priv)
static int bl_start_cmd_timer(struct rsi_hw *adapter, u32 timeout) static int bl_start_cmd_timer(struct rsi_hw *adapter, u32 timeout)
{ {
setup_timer(&adapter->bl_cmd_timer, (void *)&bl_cmd_timeout, timer_setup(&adapter->bl_cmd_timer, bl_cmd_timeout, 0);
(unsigned long)adapter);
adapter->bl_cmd_timer.expires = (msecs_to_jiffies(timeout) + jiffies); adapter->bl_cmd_timer.expires = (msecs_to_jiffies(timeout) + jiffies);
adapter->blcmd_timer_expired = false; adapter->blcmd_timer_expired = false;
......
...@@ -1663,9 +1663,9 @@ static void rsi_resume_conn_channel(struct rsi_common *common) ...@@ -1663,9 +1663,9 @@ static void rsi_resume_conn_channel(struct rsi_common *common)
} }
} }
void rsi_roc_timeout(unsigned long data) void rsi_roc_timeout(struct timer_list *t)
{ {
struct rsi_common *common = (struct rsi_common *)data; struct rsi_common *common = from_timer(common, t, roc_timer);
rsi_dbg(INFO_ZONE, "Remain on channel expired\n"); rsi_dbg(INFO_ZONE, "Remain on channel expired\n");
......
...@@ -262,9 +262,7 @@ struct rsi_hw *rsi_91x_init(void) ...@@ -262,9 +262,7 @@ struct rsi_hw *rsi_91x_init(void)
rsi_default_ps_params(adapter); rsi_default_ps_params(adapter);
spin_lock_init(&adapter->ps_lock); spin_lock_init(&adapter->ps_lock);
common->roc_timer.data = (unsigned long)common; timer_setup(&common->roc_timer, rsi_roc_timeout, 0);
common->roc_timer.function = (void *)&rsi_roc_timeout;
init_timer(&common->roc_timer);
common->init_done = true; common->init_done = true;
return adapter; return adapter;
......
...@@ -85,5 +85,5 @@ void rsi_91x_deinit(struct rsi_hw *adapter); ...@@ -85,5 +85,5 @@ void rsi_91x_deinit(struct rsi_hw *adapter);
int rsi_read_pkt(struct rsi_common *common, s32 rcv_pkt_len); int rsi_read_pkt(struct rsi_common *common, s32 rcv_pkt_len);
struct rsi_sta *rsi_find_sta(struct rsi_common *common, u8 *mac_addr); struct rsi_sta *rsi_find_sta(struct rsi_common *common, u8 *mac_addr);
struct ieee80211_vif *rsi_get_vif(struct rsi_hw *adapter, u8 *mac); struct ieee80211_vif *rsi_get_vif(struct rsi_hw *adapter, u8 *mac);
void rsi_roc_timeout(unsigned long data); void rsi_roc_timeout(struct timer_list *t);
#endif #endif
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment