Search Linux Wireless

Re: [PATCH] rtlwifi: rtl8192cu: Modify REG_BCN_MAX_ERR related code.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2015-05-09 21:02 GMT+09:00 Kalle Valo <kvalo@xxxxxxxxxxxxxx>:
> Taehee Yoo <ap420073@xxxxxxxxx> writes:
>
>> 2015-05-08 22:01 GMT+09:00 Kalle Valo <kvalo@xxxxxxxxxxxxxx>:
>>> Taehee Yoo <ap420073@xxxxxxxxx> writes:
>>>
>>>> 1. Remove setting REG_BCN_MAX_ERR code in _rtl92cu_set_media_status().
>>>> Initialization routine set REG_BCN_MAX_ERR to 0xFF.
>>>> also _rtl92cu_set_media_status set REG_BCN_MAX_ERR to same value.
>>>> so i remove this code.
>>>>
>>>> 2. Remove rtl92c_init_beacon_max_error's parameter.
>>>> because parameter "infra_mode" is not used.
>>>>
>>>> Signed-off-by: Taehee Yoo <ap420073@xxxxxxxxx>
>>>
>>> Please split the patches into two. Having a numbered list of changes in
>>> the commit log is a strong indication that the patch should be split
>>> into smaller patches.
>>>
>>> --
>>> Kalle Valo
>>
>> Thank you for your review!
>> I'll resend two patchs.
>
> Thanks. But when you send new versions please mark them as v2, v3 and so
> on as that makes it easy for me to see what is the latest version.
>
> --
> Kalle Valo


I already sent two patchs as "rtlwifi: rtl8192cu: Remove setting
REG_BCN_MAX_ERR code in _rtl92cu_set_media_status()." and "rtlwifi:
rtl8192cu: Remove rtl92c_init_beacon_max_error's parameter"

is it right? or would I resend as V2?
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux