* [PATCH 1/3] rtlwifi: rtl8192cu: remove duplicated routine in _rtl92c_phy_rf6052_config_parafile
@ 2015-06-05 14:01 Taehee Yoo
2015-06-05 14:01 ` [PATCH 2/3] rtlwifi: rtl8192cu: debug message change "RTL8192CE" to "RTL8192CU" Taehee Yoo
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Taehee Yoo @ 2015-06-05 14:01 UTC (permalink / raw)
To: Larry.Finger; +Cc: ap420073, linux-wireless
in the _rtl92c_phy_rf6052_config_parafile(), RF90_PATH_A and
RF90_PATH_B are same routine. so i remove one of these routine.
also the return routine is duplicated. so i remove it.
Signed-off-by: Taehee Yoo <ap420073@gmail.com>
---
drivers/net/wireless/rtlwifi/rtl8192cu/rf.c | 4 ----
1 file changed, 4 deletions(-)
diff --git a/drivers/net/wireless/rtlwifi/rtl8192cu/rf.c b/drivers/net/wireless/rtlwifi/rtl8192cu/rf.c
index b878d56..8ce055c 100644
--- a/drivers/net/wireless/rtlwifi/rtl8192cu/rf.c
+++ b/drivers/net/wireless/rtlwifi/rtl8192cu/rf.c
@@ -452,9 +452,6 @@ static bool _rtl92c_phy_rf6052_config_parafile(struct ieee80211_hw *hw)
udelay(1);
switch (rfpath) {
case RF90_PATH_A:
- rtstatus = rtl92cu_phy_config_rf_with_headerfile(hw,
- (enum radio_path) rfpath);
- break;
case RF90_PATH_B:
rtstatus = rtl92cu_phy_config_rf_with_headerfile(hw,
(enum radio_path) rfpath);
@@ -483,7 +480,6 @@ static bool _rtl92c_phy_rf6052_config_parafile(struct ieee80211_hw *hw)
}
}
RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, "<---\n");
- return rtstatus;
phy_rf_cfg_fail:
return rtstatus;
}
--
2.1.4
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/3] rtlwifi: rtl8192cu: debug message change "RTL8192CE" to "RTL8192CU"
2015-06-05 14:01 [PATCH 1/3] rtlwifi: rtl8192cu: remove duplicated routine in _rtl92c_phy_rf6052_config_parafile Taehee Yoo
@ 2015-06-05 14:01 ` Taehee Yoo
2015-06-05 15:33 ` Larry Finger
2015-06-05 14:01 ` [PATCH 3/3] rtlwifi: rtl8192cu: Fix variable isfirst_ampdu Taehee Yoo
2015-06-05 15:37 ` [PATCH 1/3] rtlwifi: rtl8192cu: remove duplicated routine in _rtl92c_phy_rf6052_config_parafile Larry Finger
2 siblings, 1 reply; 7+ messages in thread
From: Taehee Yoo @ 2015-06-05 14:01 UTC (permalink / raw)
To: Larry.Finger; +Cc: ap420073, linux-wireless
In the rtlwifi/rtl8192cu, I change debug message "RTL8192CE" to
"RTL8192CU".
Signed-off-by: Taehee Yoo <ap420073@gmail.com>
---
drivers/net/wireless/rtlwifi/rtl8192cu/phy.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/net/wireless/rtlwifi/rtl8192cu/phy.c b/drivers/net/wireless/rtlwifi/rtl8192cu/phy.c
index 12f6d47..5534dad 100644
--- a/drivers/net/wireless/rtlwifi/rtl8192cu/phy.c
+++ b/drivers/net/wireless/rtlwifi/rtl8192cu/phy.c
@@ -162,7 +162,7 @@ bool _rtl92cu_phy_config_mac_with_headerfile(struct ieee80211_hw *hw)
RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, "Read Rtl819XMACPHY_Array\n");
arraylength = rtlphy->hwparam_tables[MAC_REG].length ;
ptrarray = rtlphy->hwparam_tables[MAC_REG].pdata;
- RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, "Img:RTL8192CEMAC_2T_ARRAY\n");
+ RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, "Img:RTL8192CUMAC_2T_ARRAY\n");
for (i = 0; i < arraylength; i = i + 2)
rtl_write_byte(rtlpriv, ptrarray[i], (u8) ptrarray[i + 1]);
return true;
@@ -259,18 +259,18 @@ bool rtl92cu_phy_config_rf_with_headerfile(struct ieee80211_hw *hw,
radiob_arraylen = rtlphy->hwparam_tables[RADIOB_2T].length;
radiob_array_table = rtlphy->hwparam_tables[RADIOB_2T].pdata;
RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE,
- "Radio_A:RTL8192CERADIOA_2TARRAY\n");
+ "Radio_A:RTL8192CURADIOA_2TARRAY\n");
RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE,
- "Radio_B:RTL8192CE_RADIOB_2TARRAY\n");
+ "Radio_B:RTL8192CU_RADIOB_2TARRAY\n");
} else {
radioa_arraylen = rtlphy->hwparam_tables[RADIOA_1T].length;
radioa_array_table = rtlphy->hwparam_tables[RADIOA_1T].pdata;
radiob_arraylen = rtlphy->hwparam_tables[RADIOB_1T].length;
radiob_array_table = rtlphy->hwparam_tables[RADIOB_1T].pdata;
RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE,
- "Radio_A:RTL8192CE_RADIOA_1TARRAY\n");
+ "Radio_A:RTL8192CU_RADIOA_1TARRAY\n");
RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE,
- "Radio_B:RTL8192CE_RADIOB_1TARRAY\n");
+ "Radio_B:RTL8192CU_RADIOB_1TARRAY\n");
}
RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, "Radio No %x\n", rfpath);
switch (rfpath) {
--
2.1.4
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 3/3] rtlwifi: rtl8192cu: Fix variable isfirst_ampdu
2015-06-05 14:01 [PATCH 1/3] rtlwifi: rtl8192cu: remove duplicated routine in _rtl92c_phy_rf6052_config_parafile Taehee Yoo
2015-06-05 14:01 ` [PATCH 2/3] rtlwifi: rtl8192cu: debug message change "RTL8192CE" to "RTL8192CU" Taehee Yoo
@ 2015-06-05 14:01 ` Taehee Yoo
2015-06-05 15:31 ` Larry Finger
2015-06-05 15:37 ` [PATCH 1/3] rtlwifi: rtl8192cu: remove duplicated routine in _rtl92c_phy_rf6052_config_parafile Larry Finger
2 siblings, 1 reply; 7+ messages in thread
From: Taehee Yoo @ 2015-06-05 14:01 UTC (permalink / raw)
To: Larry.Finger; +Cc: ap420073, linux-wireless
rtl92cu_rx_query_desc set a isampdu twice.
but second code is related to isfirst_ampdu.
so i change it.
Signed-off-by: Taehee Yoo <ap420073@gmail.com>
---
drivers/net/wireless/rtlwifi/rtl8192cu/trx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/wireless/rtlwifi/rtl8192cu/trx.c b/drivers/net/wireless/rtlwifi/rtl8192cu/trx.c
index cbead00..95880fe 100644
--- a/drivers/net/wireless/rtlwifi/rtl8192cu/trx.c
+++ b/drivers/net/wireless/rtlwifi/rtl8192cu/trx.c
@@ -321,7 +321,7 @@ bool rtl92cu_rx_query_desc(struct ieee80211_hw *hw,
stats->rate = (u8) GET_RX_DESC_RX_MCS(pdesc);
stats->shortpreamble = (u16) GET_RX_DESC_SPLCP(pdesc);
stats->isampdu = (bool) (GET_RX_DESC_PAGGR(pdesc) == 1);
- stats->isampdu = (bool) ((GET_RX_DESC_PAGGR(pdesc) == 1)
+ stats->isfirst_ampdu = (bool)((GET_RX_DESC_PAGGR(pdesc) == 1)
&& (GET_RX_DESC_FAGGR(pdesc) == 1));
stats->timestamp_low = GET_RX_DESC_TSFL(pdesc);
stats->rx_is40Mhzpacket = (bool) GET_RX_DESC_BW(pdesc);
--
2.1.4
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 3/3] rtlwifi: rtl8192cu: Fix variable isfirst_ampdu
2015-06-05 14:01 ` [PATCH 3/3] rtlwifi: rtl8192cu: Fix variable isfirst_ampdu Taehee Yoo
@ 2015-06-05 15:31 ` Larry Finger
0 siblings, 0 replies; 7+ messages in thread
From: Larry Finger @ 2015-06-05 15:31 UTC (permalink / raw)
To: Taehee Yoo; +Cc: linux-wireless
On 06/05/2015 09:01 AM, Taehee Yoo wrote:
> rtl92cu_rx_query_desc set a isampdu twice.
> but second code is related to isfirst_ampdu.
> so i change it.
>
> Signed-off-by: Taehee Yoo <ap420073@gmail.com>
> ---
> drivers/net/wireless/rtlwifi/rtl8192cu/trx.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Looks good.
Acked-by: Larry Finger <Larry.Finger@lwfinger.net>
Larry
>
> diff --git a/drivers/net/wireless/rtlwifi/rtl8192cu/trx.c b/drivers/net/wireless/rtlwifi/rtl8192cu/trx.c
> index cbead00..95880fe 100644
> --- a/drivers/net/wireless/rtlwifi/rtl8192cu/trx.c
> +++ b/drivers/net/wireless/rtlwifi/rtl8192cu/trx.c
> @@ -321,7 +321,7 @@ bool rtl92cu_rx_query_desc(struct ieee80211_hw *hw,
> stats->rate = (u8) GET_RX_DESC_RX_MCS(pdesc);
> stats->shortpreamble = (u16) GET_RX_DESC_SPLCP(pdesc);
> stats->isampdu = (bool) (GET_RX_DESC_PAGGR(pdesc) == 1);
> - stats->isampdu = (bool) ((GET_RX_DESC_PAGGR(pdesc) == 1)
> + stats->isfirst_ampdu = (bool)((GET_RX_DESC_PAGGR(pdesc) == 1)
> && (GET_RX_DESC_FAGGR(pdesc) == 1));
> stats->timestamp_low = GET_RX_DESC_TSFL(pdesc);
> stats->rx_is40Mhzpacket = (bool) GET_RX_DESC_BW(pdesc);
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/3] rtlwifi: rtl8192cu: debug message change "RTL8192CE" to "RTL8192CU"
2015-06-05 14:01 ` [PATCH 2/3] rtlwifi: rtl8192cu: debug message change "RTL8192CE" to "RTL8192CU" Taehee Yoo
@ 2015-06-05 15:33 ` Larry Finger
0 siblings, 0 replies; 7+ messages in thread
From: Larry Finger @ 2015-06-05 15:33 UTC (permalink / raw)
To: Taehee Yoo; +Cc: linux-wireless
On 06/05/2015 09:01 AM, Taehee Yoo wrote:
> In the rtlwifi/rtl8192cu, I change debug message "RTL8192CE" to
> "RTL8192CU".
>
> Signed-off-by: Taehee Yoo <ap420073@gmail.com>
> ---
> drivers/net/wireless/rtlwifi/rtl8192cu/phy.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
I am not sure that anyone ever uses this level of debug, but the changes are OK.
Acked-by: Larry Finger <Larry.Finger@lwfinger.net>
Larry
>
> diff --git a/drivers/net/wireless/rtlwifi/rtl8192cu/phy.c b/drivers/net/wireless/rtlwifi/rtl8192cu/phy.c
> index 12f6d47..5534dad 100644
> --- a/drivers/net/wireless/rtlwifi/rtl8192cu/phy.c
> +++ b/drivers/net/wireless/rtlwifi/rtl8192cu/phy.c
> @@ -162,7 +162,7 @@ bool _rtl92cu_phy_config_mac_with_headerfile(struct ieee80211_hw *hw)
> RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, "Read Rtl819XMACPHY_Array\n");
> arraylength = rtlphy->hwparam_tables[MAC_REG].length ;
> ptrarray = rtlphy->hwparam_tables[MAC_REG].pdata;
> - RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, "Img:RTL8192CEMAC_2T_ARRAY\n");
> + RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, "Img:RTL8192CUMAC_2T_ARRAY\n");
> for (i = 0; i < arraylength; i = i + 2)
> rtl_write_byte(rtlpriv, ptrarray[i], (u8) ptrarray[i + 1]);
> return true;
> @@ -259,18 +259,18 @@ bool rtl92cu_phy_config_rf_with_headerfile(struct ieee80211_hw *hw,
> radiob_arraylen = rtlphy->hwparam_tables[RADIOB_2T].length;
> radiob_array_table = rtlphy->hwparam_tables[RADIOB_2T].pdata;
> RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE,
> - "Radio_A:RTL8192CERADIOA_2TARRAY\n");
> + "Radio_A:RTL8192CURADIOA_2TARRAY\n");
> RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE,
> - "Radio_B:RTL8192CE_RADIOB_2TARRAY\n");
> + "Radio_B:RTL8192CU_RADIOB_2TARRAY\n");
> } else {
> radioa_arraylen = rtlphy->hwparam_tables[RADIOA_1T].length;
> radioa_array_table = rtlphy->hwparam_tables[RADIOA_1T].pdata;
> radiob_arraylen = rtlphy->hwparam_tables[RADIOB_1T].length;
> radiob_array_table = rtlphy->hwparam_tables[RADIOB_1T].pdata;
> RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE,
> - "Radio_A:RTL8192CE_RADIOA_1TARRAY\n");
> + "Radio_A:RTL8192CU_RADIOA_1TARRAY\n");
> RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE,
> - "Radio_B:RTL8192CE_RADIOB_1TARRAY\n");
> + "Radio_B:RTL8192CU_RADIOB_1TARRAY\n");
> }
> RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, "Radio No %x\n", rfpath);
> switch (rfpath) {
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/3] rtlwifi: rtl8192cu: remove duplicated routine in _rtl92c_phy_rf6052_config_parafile
2015-06-05 14:01 [PATCH 1/3] rtlwifi: rtl8192cu: remove duplicated routine in _rtl92c_phy_rf6052_config_parafile Taehee Yoo
2015-06-05 14:01 ` [PATCH 2/3] rtlwifi: rtl8192cu: debug message change "RTL8192CE" to "RTL8192CU" Taehee Yoo
2015-06-05 14:01 ` [PATCH 3/3] rtlwifi: rtl8192cu: Fix variable isfirst_ampdu Taehee Yoo
@ 2015-06-05 15:37 ` Larry Finger
2015-06-06 4:57 ` Taehee Yoo
2 siblings, 1 reply; 7+ messages in thread
From: Larry Finger @ 2015-06-05 15:37 UTC (permalink / raw)
To: Taehee Yoo; +Cc: linux-wireless
On 06/05/2015 09:01 AM, Taehee Yoo wrote:
> in the _rtl92c_phy_rf6052_config_parafile(), RF90_PATH_A and
> RF90_PATH_B are same routine. so i remove one of these routine.
> also the return routine is duplicated. so i remove it.
>
> Signed-off-by: Taehee Yoo <ap420073@gmail.com>
> ---
> drivers/net/wireless/rtlwifi/rtl8192cu/rf.c | 4 ----
> 1 file changed, 4 deletions(-)
>
The code changes are OK, but the commit message needs some work. The phrase
"RF90_PATH_A and RF90_PATH_B are same routine" should be changed to "cases
RF90_PATH_A and RF90_PATH_B call the same routine".
Larry
> diff --git a/drivers/net/wireless/rtlwifi/rtl8192cu/rf.c b/drivers/net/wireless/rtlwifi/rtl8192cu/rf.c
> index b878d56..8ce055c 100644
> --- a/drivers/net/wireless/rtlwifi/rtl8192cu/rf.c
> +++ b/drivers/net/wireless/rtlwifi/rtl8192cu/rf.c
> @@ -452,9 +452,6 @@ static bool _rtl92c_phy_rf6052_config_parafile(struct ieee80211_hw *hw)
> udelay(1);
> switch (rfpath) {
> case RF90_PATH_A:
> - rtstatus = rtl92cu_phy_config_rf_with_headerfile(hw,
> - (enum radio_path) rfpath);
> - break;
> case RF90_PATH_B:
> rtstatus = rtl92cu_phy_config_rf_with_headerfile(hw,
> (enum radio_path) rfpath);
> @@ -483,7 +480,6 @@ static bool _rtl92c_phy_rf6052_config_parafile(struct ieee80211_hw *hw)
> }
> }
> RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, "<---\n");
> - return rtstatus;
> phy_rf_cfg_fail:
> return rtstatus;
> }
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/3] rtlwifi: rtl8192cu: remove duplicated routine in _rtl92c_phy_rf6052_config_parafile
2015-06-05 15:37 ` [PATCH 1/3] rtlwifi: rtl8192cu: remove duplicated routine in _rtl92c_phy_rf6052_config_parafile Larry Finger
@ 2015-06-06 4:57 ` Taehee Yoo
0 siblings, 0 replies; 7+ messages in thread
From: Taehee Yoo @ 2015-06-06 4:57 UTC (permalink / raw)
To: Larry Finger; +Cc: linux-wireless@vger.kernel.org
2015-06-06 0:37 GMT+09:00 Larry Finger <Larry.Finger@lwfinger.net>:
> On 06/05/2015 09:01 AM, Taehee Yoo wrote:
>>
>> in the _rtl92c_phy_rf6052_config_parafile(), RF90_PATH_A and
>> RF90_PATH_B are same routine. so i remove one of these routine.
>> also the return routine is duplicated. so i remove it.
>>
>> Signed-off-by: Taehee Yoo <ap420073@gmail.com>
>> ---
>> drivers/net/wireless/rtlwifi/rtl8192cu/rf.c | 4 ----
>> 1 file changed, 4 deletions(-)
>>
>
> The code changes are OK, but the commit message needs some work. The phrase
> "RF90_PATH_A and RF90_PATH_B are same routine" should be changed to "cases
> RF90_PATH_A and RF90_PATH_B call the same routine".
>
> Larry
>
>
>> diff --git a/drivers/net/wireless/rtlwifi/rtl8192cu/rf.c
>> b/drivers/net/wireless/rtlwifi/rtl8192cu/rf.c
>> index b878d56..8ce055c 100644
>> --- a/drivers/net/wireless/rtlwifi/rtl8192cu/rf.c
>> +++ b/drivers/net/wireless/rtlwifi/rtl8192cu/rf.c
>> @@ -452,9 +452,6 @@ static bool _rtl92c_phy_rf6052_config_parafile(struct
>> ieee80211_hw *hw)
>> udelay(1);
>> switch (rfpath) {
>> case RF90_PATH_A:
>> - rtstatus =
>> rtl92cu_phy_config_rf_with_headerfile(hw,
>> - (enum radio_path) rfpath);
>> - break;
>> case RF90_PATH_B:
>> rtstatus =
>> rtl92cu_phy_config_rf_with_headerfile(hw,
>> (enum radio_path) rfpath);
>> @@ -483,7 +480,6 @@ static bool _rtl92c_phy_rf6052_config_parafile(struct
>> ieee80211_hw *hw)
>> }
>> }
>> RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, "<---\n");
>> - return rtstatus;
>> phy_rf_cfg_fail:
>> return rtstatus;
>> }
>>
>
Thank you for your review!
I will resend a patch as V2.
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2015-06-06 4:57 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-06-05 14:01 [PATCH 1/3] rtlwifi: rtl8192cu: remove duplicated routine in _rtl92c_phy_rf6052_config_parafile Taehee Yoo
2015-06-05 14:01 ` [PATCH 2/3] rtlwifi: rtl8192cu: debug message change "RTL8192CE" to "RTL8192CU" Taehee Yoo
2015-06-05 15:33 ` Larry Finger
2015-06-05 14:01 ` [PATCH 3/3] rtlwifi: rtl8192cu: Fix variable isfirst_ampdu Taehee Yoo
2015-06-05 15:31 ` Larry Finger
2015-06-05 15:37 ` [PATCH 1/3] rtlwifi: rtl8192cu: remove duplicated routine in _rtl92c_phy_rf6052_config_parafile Larry Finger
2015-06-06 4:57 ` Taehee Yoo
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).