* [PATCH V2] rtlwifi: rtl8192c: Add init codes for "fw_version" and "fw_subversion".
@ 2015-06-07 13:33 Taehee Yoo
2015-06-07 15:05 ` Larry Finger
2015-06-15 9:38 ` [V2] rtlwifi: rtl8192c: Add init codes for "fw_version" and"fw_subversion" Kalle Valo
0 siblings, 2 replies; 7+ messages in thread
From: Taehee Yoo @ 2015-06-07 13:33 UTC (permalink / raw)
To: Larry.Finger; +Cc: ap420073, linux-wireless
The variable "fw_version" is used in the _ResetDigitalProcedure1().
but It is not initialized. so I add init codes for "fw_version" and
"fw_subversion".
Signed-off-by: Taehee Yoo <ap420073@gmail.com>
---
drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c b/drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c
index 29983bc..14b819e 100644
--- a/drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c
+++ b/drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c
@@ -233,13 +233,14 @@ int rtl92c_download_fw(struct ieee80211_hw *hw)
pfwheader = (struct rtl92c_firmware_header *)rtlhal->pfirmware;
pfwdata = (u8 *)rtlhal->pfirmware;
fwsize = rtlhal->fwsize;
-
if (IS_FW_HEADER_EXIST(pfwheader)) {
RT_TRACE(rtlpriv, COMP_FW, DBG_DMESG,
"Firmware Version(%d), Signature(%#x),Size(%d)\n",
pfwheader->version, pfwheader->signature,
(int)sizeof(struct rtl92c_firmware_header));
+ rtlhal->fw_version = pfwheader->version;
+ rtlhal->fw_subversion = pfwheader->subversion;
pfwdata = pfwdata + sizeof(struct rtl92c_firmware_header);
fwsize = fwsize - sizeof(struct rtl92c_firmware_header);
}
--
2.1.4
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH V2] rtlwifi: rtl8192c: Add init codes for "fw_version" and "fw_subversion".
2015-06-07 13:33 [PATCH V2] rtlwifi: rtl8192c: Add init codes for "fw_version" and "fw_subversion" Taehee Yoo
@ 2015-06-07 15:05 ` Larry Finger
2015-06-07 15:18 ` Taehee Yoo
2015-06-15 9:38 ` [V2] rtlwifi: rtl8192c: Add init codes for "fw_version" and"fw_subversion" Kalle Valo
1 sibling, 1 reply; 7+ messages in thread
From: Larry Finger @ 2015-06-07 15:05 UTC (permalink / raw)
To: Taehee Yoo; +Cc: linux-wireless
On 06/07/2015 08:33 AM, Taehee Yoo wrote:
> The variable "fw_version" is used in the _ResetDigitalProcedure1().
> but It is not initialized. so I add init codes for "fw_version" and
> "fw_subversion".
>
> Signed-off-by: Taehee Yoo <ap420073@gmail.com>
> ---
V2 - Moved new statements into location where header is known to exist.
You should have something like this after the separator (---) in the commit
message for any resubmission.
Acked-by: Larry Finger <Larry.Finger@lwfinger.net>
Larry
> drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c b/drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c
> index 29983bc..14b819e 100644
> --- a/drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c
> +++ b/drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c
> @@ -233,13 +233,14 @@ int rtl92c_download_fw(struct ieee80211_hw *hw)
> pfwheader = (struct rtl92c_firmware_header *)rtlhal->pfirmware;
> pfwdata = (u8 *)rtlhal->pfirmware;
> fwsize = rtlhal->fwsize;
> -
> if (IS_FW_HEADER_EXIST(pfwheader)) {
> RT_TRACE(rtlpriv, COMP_FW, DBG_DMESG,
> "Firmware Version(%d), Signature(%#x),Size(%d)\n",
> pfwheader->version, pfwheader->signature,
> (int)sizeof(struct rtl92c_firmware_header));
>
> + rtlhal->fw_version = pfwheader->version;
> + rtlhal->fw_subversion = pfwheader->subversion;
> pfwdata = pfwdata + sizeof(struct rtl92c_firmware_header);
> fwsize = fwsize - sizeof(struct rtl92c_firmware_header);
> }
>
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH V2] rtlwifi: rtl8192c: Add init codes for "fw_version" and "fw_subversion".
2015-06-07 15:05 ` Larry Finger
@ 2015-06-07 15:18 ` Taehee Yoo
2015-06-07 16:23 ` Larry Finger
0 siblings, 1 reply; 7+ messages in thread
From: Taehee Yoo @ 2015-06-07 15:18 UTC (permalink / raw)
To: Larry Finger; +Cc: linux-wireless@vger.kernel.org
2015-06-08 0:05 GMT+09:00 Larry Finger <Larry.Finger@lwfinger.net>:
> On 06/07/2015 08:33 AM, Taehee Yoo wrote:
>>
>> The variable "fw_version" is used in the _ResetDigitalProcedure1().
>> but It is not initialized. so I add init codes for "fw_version" and
>> "fw_subversion".
>>
>> Signed-off-by: Taehee Yoo <ap420073@gmail.com>
>> ---
>
>
> V2 - Moved new statements into location where header is known to exist.
>
> You should have something like this after the separator (---) in the commit
> message for any resubmission.
>
> Acked-by: Larry Finger <Larry.Finger@lwfinger.net>
>
> Larry
>
>
>> drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c
>> b/drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c
>> index 29983bc..14b819e 100644
>> --- a/drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c
>> +++ b/drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c
>> @@ -233,13 +233,14 @@ int rtl92c_download_fw(struct ieee80211_hw *hw)
>> pfwheader = (struct rtl92c_firmware_header *)rtlhal->pfirmware;
>> pfwdata = (u8 *)rtlhal->pfirmware;
>> fwsize = rtlhal->fwsize;
>> -
>> if (IS_FW_HEADER_EXIST(pfwheader)) {
>> RT_TRACE(rtlpriv, COMP_FW, DBG_DMESG,
>> "Firmware Version(%d),
>> Signature(%#x),Size(%d)\n",
>> pfwheader->version, pfwheader->signature,
>> (int)sizeof(struct rtl92c_firmware_header));
>>
>> + rtlhal->fw_version = pfwheader->version;
>> + rtlhal->fw_subversion = pfwheader->subversion;
>> pfwdata = pfwdata + sizeof(struct rtl92c_firmware_header);
>> fwsize = fwsize - sizeof(struct rtl92c_firmware_header);
>> }
>>
>
Thank you for your feedback!
Would i resend it?
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH V2] rtlwifi: rtl8192c: Add init codes for "fw_version" and "fw_subversion".
2015-06-07 15:18 ` Taehee Yoo
@ 2015-06-07 16:23 ` Larry Finger
0 siblings, 0 replies; 7+ messages in thread
From: Larry Finger @ 2015-06-07 16:23 UTC (permalink / raw)
To: Taehee Yoo; +Cc: linux-wireless@vger.kernel.org
On 06/07/2015 10:18 AM, Taehee Yoo wrote:
> 2015-06-08 0:05 GMT+09:00 Larry Finger <Larry.Finger@lwfinger.net>:
>> On 06/07/2015 08:33 AM, Taehee Yoo wrote:
>>>
>>> The variable "fw_version" is used in the _ResetDigitalProcedure1().
>>> but It is not initialized. so I add init codes for "fw_version" and
>>> "fw_subversion".
>>>
>>> Signed-off-by: Taehee Yoo <ap420073@gmail.com>
>>> ---
>>
>>
>> V2 - Moved new statements into location where header is known to exist.
>>
>> You should have something like this after the separator (---) in the commit
>> message for any resubmission.
>>
>> Acked-by: Larry Finger <Larry.Finger@lwfinger.net>
>>
>> Larry
>>
>>
>>> drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c | 3 ++-
>>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c
>>> b/drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c
>>> index 29983bc..14b819e 100644
>>> --- a/drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c
>>> +++ b/drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c
>>> @@ -233,13 +233,14 @@ int rtl92c_download_fw(struct ieee80211_hw *hw)
>>> pfwheader = (struct rtl92c_firmware_header *)rtlhal->pfirmware;
>>> pfwdata = (u8 *)rtlhal->pfirmware;
>>> fwsize = rtlhal->fwsize;
>>> -
>>> if (IS_FW_HEADER_EXIST(pfwheader)) {
>>> RT_TRACE(rtlpriv, COMP_FW, DBG_DMESG,
>>> "Firmware Version(%d),
>>> Signature(%#x),Size(%d)\n",
>>> pfwheader->version, pfwheader->signature,
>>> (int)sizeof(struct rtl92c_firmware_header));
>>>
>>> + rtlhal->fw_version = pfwheader->version;
>>> + rtlhal->fw_subversion = pfwheader->subversion;
>>> pfwdata = pfwdata + sizeof(struct rtl92c_firmware_header);
>>> fwsize = fwsize - sizeof(struct rtl92c_firmware_header);
>>> }
>>>
>>
>
> Thank you for your feedback!
> Would i resend it?
No, but remember that for next time.
Larry
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [V2] rtlwifi: rtl8192c: Add init codes for "fw_version" and"fw_subversion".
2015-06-07 13:33 [PATCH V2] rtlwifi: rtl8192c: Add init codes for "fw_version" and "fw_subversion" Taehee Yoo
2015-06-07 15:05 ` Larry Finger
@ 2015-06-15 9:38 ` Kalle Valo
2015-06-15 12:35 ` Kalle Valo
1 sibling, 1 reply; 7+ messages in thread
From: Kalle Valo @ 2015-06-15 9:38 UTC (permalink / raw)
To: Taehee Yoo; +Cc: Larry.Finger, ap420073, linux-wireless
> The variable "fw_version" is used in the _ResetDigitalProcedure1().
> but It is not initialized. so I add init codes for "fw_version" and
> "fw_subversion".
>
> Signed-off-by: Taehee Yoo <ap420073@gmail.com>
> Acked-by: Larry Finger <Larry.Finger@lwfinger.net>
Thanks, applied to wireless-drivers-next.git.
Kalle Valo
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [V2] rtlwifi: rtl8192c: Add init codes for "fw_version" and"fw_subversion".
2015-06-15 9:38 ` [V2] rtlwifi: rtl8192c: Add init codes for "fw_version" and"fw_subversion" Kalle Valo
@ 2015-06-15 12:35 ` Kalle Valo
2015-06-15 15:50 ` Larry Finger
0 siblings, 1 reply; 7+ messages in thread
From: Kalle Valo @ 2015-06-15 12:35 UTC (permalink / raw)
To: Taehee Yoo; +Cc: Larry.Finger, linux-wireless
Kalle Valo <kvalo@codeaurora.org> writes:
>> The variable "fw_version" is used in the _ResetDigitalProcedure1().
>> but It is not initialized. so I add init codes for "fw_version" and
>> "fw_subversion".
>>
>> Signed-off-by: Taehee Yoo <ap420073@gmail.com>
>> Acked-by: Larry Finger <Larry.Finger@lwfinger.net>
>
> Thanks, applied to wireless-drivers-next.git.
This patch adds a new sparse warning. Please send a new patch to fix
this.
>> drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c:242:36: sparse: incorrect type in assignment (different base types)
drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c:242:36: expected unsigned short [unsigned] [usertype] fw_version
drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c:242:36: got restricted __le16 [usertype] version
--
Kalle Valo
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [V2] rtlwifi: rtl8192c: Add init codes for "fw_version" and"fw_subversion".
2015-06-15 12:35 ` Kalle Valo
@ 2015-06-15 15:50 ` Larry Finger
0 siblings, 0 replies; 7+ messages in thread
From: Larry Finger @ 2015-06-15 15:50 UTC (permalink / raw)
To: Kalle Valo, Taehee Yoo; +Cc: linux-wireless
On 06/15/2015 07:35 AM, Kalle Valo wrote:
> Kalle Valo <kvalo@codeaurora.org> writes:
>
>>> The variable "fw_version" is used in the _ResetDigitalProcedure1().
>>> but It is not initialized. so I add init codes for "fw_version" and
>>> "fw_subversion".
>>>
>>> Signed-off-by: Taehee Yoo <ap420073@gmail.com>
>>> Acked-by: Larry Finger <Larry.Finger@lwfinger.net>
>>
>> Thanks, applied to wireless-drivers-next.git.
>
> This patch adds a new sparse warning. Please send a new patch to fix
> this.
>
>>> drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c:242:36: sparse: incorrect type in assignment (different base types)
> drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c:242:36: expected unsigned short [unsigned] [usertype] fw_version
> drivers/net/wireless/rtlwifi/rtl8192c/fw_common.c:242:36: got restricted __le16 [usertype] version
Sorry, I missed that when I reviewed the patch.
Larry
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2015-06-15 15:50 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-06-07 13:33 [PATCH V2] rtlwifi: rtl8192c: Add init codes for "fw_version" and "fw_subversion" Taehee Yoo
2015-06-07 15:05 ` Larry Finger
2015-06-07 15:18 ` Taehee Yoo
2015-06-07 16:23 ` Larry Finger
2015-06-15 9:38 ` [V2] rtlwifi: rtl8192c: Add init codes for "fw_version" and"fw_subversion" Kalle Valo
2015-06-15 12:35 ` Kalle Valo
2015-06-15 15:50 ` Larry Finger
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).