* [PATCH v2] Staging: rtl8192e: Timer setup using macro rather assignment
@ 2015-06-15 15:00 DHANAPAL, GNANACHANDRAN (G.)
2015-06-16 6:30 ` Sudip Mukherjee
0 siblings, 1 reply; 3+ messages in thread
From: DHANAPAL, GNANACHANDRAN (G.) @ 2015-06-15 15:00 UTC (permalink / raw)
To: gregkh
Cc: mateusz.kulikowski, vthakkar1994, mdcasey, kuleshovmail,
mahati.chamarthy, lambert.quentin, kolbeinnkarls, joe,
himangi774, behanw, devel, linux-kernel
This patch shall replaces user defined timer setup function with
standard timer setup macro. Also removes init_timer, because timer can
be initialized in setup_timer macro as well.
Signed-off-by: Gnanachandran Dhanapal <gdhanapa@visteon.com>
---
Addressed review comments from v1
1. setup_timer also subsumes init_timer.
2. Probably delete this function completely (_setup_timer)
---
drivers/staging/rtl8192e/rtl8192e/rtl_core.c | 2 --
drivers/staging/rtl8192e/rtl8192e/rtl_dm.c | 1 -
drivers/staging/rtl8192e/rtl819x_TS.h | 1 -
drivers/staging/rtl8192e/rtl819x_TSProc.c | 18 +++++++++---------
drivers/staging/rtl8192e/rtllib_module.c | 6 ------
drivers/staging/rtl8192e/rtllib_softmac.c | 4 ++--
6 files changed, 11 insertions(+), 21 deletions(-)
diff --git a/drivers/staging/rtl8192e/rtl8192e/rtl_core.c b/drivers/staging/rtl8192e/rtl8192e/rtl_core.c
index 4c53c87..5584b86 100644
--- a/drivers/staging/rtl8192e/rtl8192e/rtl_core.c
+++ b/drivers/staging/rtl8192e/rtl8192e/rtl_core.c
@@ -1275,12 +1275,10 @@ static short rtl8192_init(struct net_device *dev)
init_hal_dm(dev);
- init_timer(&priv->watch_dog_timer);
setup_timer(&priv->watch_dog_timer,
watch_dog_timer_callback,
(unsigned long) dev);
- init_timer(&priv->gpio_polling_timer);
setup_timer(&priv->gpio_polling_timer,
check_rfctrl_gpio_timer,
(unsigned long)dev);
diff --git a/drivers/staging/rtl8192e/rtl8192e/rtl_dm.c b/drivers/staging/rtl8192e/rtl8192e/rtl_dm.c
index 8532e0c..502a699 100644
--- a/drivers/staging/rtl8192e/rtl8192e/rtl_dm.c
+++ b/drivers/staging/rtl8192e/rtl8192e/rtl_dm.c
@@ -2180,7 +2180,6 @@ static void dm_init_fsync(struct net_device *dev)
priv->rtllib->fsync_state = Default_Fsync;
priv->framesyncMonitor = 1;
- init_timer(&priv->fsync_timer);
setup_timer(&priv->fsync_timer, dm_fsync_timer_callback,
(unsigned long) dev);
}
diff --git a/drivers/staging/rtl8192e/rtl819x_TS.h b/drivers/staging/rtl8192e/rtl819x_TS.h
index b3e721b..b8fed55 100644
--- a/drivers/staging/rtl8192e/rtl819x_TS.h
+++ b/drivers/staging/rtl8192e/rtl819x_TS.h
@@ -67,7 +67,6 @@ struct rx_ts_record {
u8 num;
};
-void _setup_timer(struct timer_list *, void *, unsigned long);
#endif
diff --git a/drivers/staging/rtl8192e/rtl819x_TSProc.c b/drivers/staging/rtl8192e/rtl819x_TSProc.c
index 8a5dd6e..05aea43 100644
--- a/drivers/staging/rtl8192e/rtl819x_TSProc.c
+++ b/drivers/staging/rtl8192e/rtl819x_TSProc.c
@@ -154,22 +154,22 @@ void TSInitialize(struct rtllib_device *ieee)
for (count = 0; count < TOTAL_TS_NUM; count++) {
pTxTS->num = count;
- _setup_timer(&pTxTS->TsCommonInfo.SetupTimer,
+ setup_timer(&pTxTS->TsCommonInfo.SetupTimer,
TsSetupTimeOut,
(unsigned long) pTxTS);
- _setup_timer(&pTxTS->TsCommonInfo.InactTimer,
+ setup_timer(&pTxTS->TsCommonInfo.InactTimer,
TsInactTimeout,
(unsigned long) pTxTS);
- _setup_timer(&pTxTS->TsAddBaTimer,
+ setup_timer(&pTxTS->TsAddBaTimer,
TsAddBaProcess,
(unsigned long) pTxTS);
- _setup_timer(&pTxTS->TxPendingBARecord.Timer,
+ setup_timer(&pTxTS->TxPendingBARecord.Timer,
BaSetupTimeOut,
(unsigned long) pTxTS);
- _setup_timer(&pTxTS->TxAdmittedBARecord.Timer,
+ setup_timer(&pTxTS->TxAdmittedBARecord.Timer,
TxBaInactTimeout,
(unsigned long) pTxTS);
@@ -186,19 +186,19 @@ void TSInitialize(struct rtllib_device *ieee)
pRxTS->num = count;
INIT_LIST_HEAD(&pRxTS->RxPendingPktList);
- _setup_timer(&pRxTS->TsCommonInfo.SetupTimer,
+ setup_timer(&pRxTS->TsCommonInfo.SetupTimer,
TsSetupTimeOut,
(unsigned long) pRxTS);
- _setup_timer(&pRxTS->TsCommonInfo.InactTimer,
+ setup_timer(&pRxTS->TsCommonInfo.InactTimer,
TsInactTimeout,
(unsigned long) pRxTS);
- _setup_timer(&pRxTS->RxAdmittedBARecord.Timer,
+ setup_timer(&pRxTS->RxAdmittedBARecord.Timer,
RxBaInactTimeout,
(unsigned long) pRxTS);
- _setup_timer(&pRxTS->RxPktPendingTimer,
+ setup_timer(&pRxTS->RxPktPendingTimer,
RxPktPendingTimeout,
(unsigned long) pRxTS);
diff --git a/drivers/staging/rtl8192e/rtllib_module.c b/drivers/staging/rtl8192e/rtllib_module.c
index 845d9b8..7704177 100644
--- a/drivers/staging/rtl8192e/rtllib_module.c
+++ b/drivers/staging/rtl8192e/rtllib_module.c
@@ -57,12 +57,6 @@ u32 rt_global_debug_component = COMP_ERR;
EXPORT_SYMBOL(rt_global_debug_component);
-void _setup_timer(struct timer_list *ptimer, void *fun, unsigned long data)
-{
- ptimer->function = fun;
- ptimer->data = data;
- init_timer(ptimer);
-}
static inline int rtllib_networks_allocate(struct rtllib_device *ieee)
{
diff --git a/drivers/staging/rtl8192e/rtllib_softmac.c b/drivers/staging/rtl8192e/rtllib_softmac.c
index 7ce58e2..6341a83 100644
--- a/drivers/staging/rtl8192e/rtllib_softmac.c
+++ b/drivers/staging/rtl8192e/rtllib_softmac.c
@@ -3127,11 +3127,11 @@ void rtllib_softmac_init(struct rtllib_device *ieee)
ieee->enable_rx_imm_BA = true;
ieee->tx_pending.txb = NULL;
- _setup_timer(&ieee->associate_timer,
+ setup_timer(&ieee->associate_timer,
rtllib_associate_abort_cb,
(unsigned long) ieee);
- _setup_timer(&ieee->beacon_timer,
+ setup_timer(&ieee->beacon_timer,
rtllib_send_beacon_cb,
(unsigned long) ieee);
--
1.7.9.5
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2] Staging: rtl8192e: Timer setup using macro rather assignment
2015-06-15 15:00 [PATCH v2] Staging: rtl8192e: Timer setup using macro rather assignment DHANAPAL, GNANACHANDRAN (G.)
@ 2015-06-16 6:30 ` Sudip Mukherjee
2015-06-16 6:40 ` DHANAPAL, GNANACHANDRAN (G.)
0 siblings, 1 reply; 3+ messages in thread
From: Sudip Mukherjee @ 2015-06-16 6:30 UTC (permalink / raw)
To: DHANAPAL, GNANACHANDRAN (G.)
Cc: gregkh, devel, mdcasey, mateusz.kulikowski, kuleshovmail,
vthakkar1994, lambert.quentin, linux-kernel, himangi774, joe,
behanw, mahati.chamarthy, kolbeinnkarls
On Mon, Jun 15, 2015 at 03:00:34PM +0000, DHANAPAL, GNANACHANDRAN (G.) wrote:
> This patch shall replaces user defined timer setup function with
> standard timer setup macro. Also removes init_timer, because timer can
> be initialized in setup_timer macro as well.
>
> Signed-off-by: Gnanachandran Dhanapal <gdhanapa@visteon.com>
You forgot to add your From: and as a result your email From: name
is not matching with your Signed-off-by: name.
regards
sudip
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] Staging: rtl8192e: Timer setup using macro rather assignment
2015-06-16 6:30 ` Sudip Mukherjee
@ 2015-06-16 6:40 ` DHANAPAL, GNANACHANDRAN (G.)
0 siblings, 0 replies; 3+ messages in thread
From: DHANAPAL, GNANACHANDRAN (G.) @ 2015-06-16 6:40 UTC (permalink / raw)
To: Sudip Mukherjee
Cc: gregkh, devel, mdcasey, mateusz.kulikowski, kuleshovmail,
vthakkar1994, lambert.quentin, linux-kernel, himangi774, joe,
behanw, mahati.chamarthy, kolbeinnkarls
On Tue, Jun 16, 2015 at 12:00:28PM +0530, Sudip Mukherjee wrote:
> On Mon, Jun 15, 2015 at 03:00:34PM +0000, DHANAPAL, GNANACHANDRAN (G.) wrote:
> > This patch shall replaces user defined timer setup function with
> > standard timer setup macro. Also removes init_timer, because timer can
> > be initialized in setup_timer macro as well.
> >
> > Signed-off-by: Gnanachandran Dhanapal <gdhanapa@visteon.com>
> You forgot to add your From: and as a result your email From: name
> is not matching with your Signed-off-by: name.
>
> regards
> sudip
Thanks for the comments. I'm sending v3 with valid From: field added.
Cheers
Gnanachandran Dhanapal
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-06-16 6:40 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-06-15 15:00 [PATCH v2] Staging: rtl8192e: Timer setup using macro rather assignment DHANAPAL, GNANACHANDRAN (G.)
2015-06-16 6:30 ` Sudip Mukherjee
2015-06-16 6:40 ` DHANAPAL, GNANACHANDRAN (G.)
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).