2021-01-05 10:30:08

by Bhaskar Chowdhury

[permalink] [raw]
Subject: [PATCH] drivers: wireless: rtlwifi: rtl8192ce: Fix construction of word rtl8192ce/trx.c

s/defautly/de-faulty/p

Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/net/wireless/realtek/rtlwifi/rtl8192ce/trx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/trx.c b/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/trx.c
index 4165175cf5c0..d53397e7eb2e 100644
--- a/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/trx.c
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/trx.c
@@ -671,7 +671,7 @@ bool rtl92ce_is_tx_desc_closed(struct ieee80211_hw *hw,
u8 own = (u8)rtl92ce_get_desc(hw, entry, true, HW_DESC_OWN);

/*beacon packet will only use the first
- *descriptor defautly,and the own may not
+ *descriptor de-faulty,and the own may not
*be cleared by the hardware
*/
if (own)
--
2.26.2


2021-01-05 10:38:13

by Julian Calaby

[permalink] [raw]
Subject: Re: [PATCH] drivers: wireless: rtlwifi: rtl8192ce: Fix construction of word rtl8192ce/trx.c

Hi Bhaskar,

On Tue, Jan 5, 2021 at 9:32 PM Bhaskar Chowdhury <[email protected]> wrote:
>
> s/defautly/de-faulty/p
>
> Signed-off-by: Bhaskar Chowdhury <[email protected]>
> ---
> drivers/net/wireless/realtek/rtlwifi/rtl8192ce/trx.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/trx.c b/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/trx.c
> index 4165175cf5c0..d53397e7eb2e 100644
> --- a/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/trx.c
> +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/trx.c
> @@ -671,7 +671,7 @@ bool rtl92ce_is_tx_desc_closed(struct ieee80211_hw *hw,
> u8 own = (u8)rtl92ce_get_desc(hw, entry, true, HW_DESC_OWN);
>
> /*beacon packet will only use the first
> - *descriptor defautly,and the own may not
> + *descriptor de-faulty,and the own may not

Same comments here as the previous patch:

"de-faultly" makes less sense than "defaultly". This comment needs to
be re-written by someone who knows what's going on here.

Thanks,

--
Julian Calaby

Email: [email protected]
Profile: http://www.google.com/profiles/julian.calaby/

2021-01-05 10:53:52

by Bhaskar Chowdhury

[permalink] [raw]
Subject: Re: [PATCH] drivers: wireless: rtlwifi: rtl8192ce: Fix construction of word rtl8192ce/trx.c

On 21:34 Tue 05 Jan 2021, Julian Calaby wrote:
>Hi Bhaskar,
>
>On Tue, Jan 5, 2021 at 9:32 PM Bhaskar Chowdhury <[email protected]> wrote:
>>
>> s/defautly/de-faulty/p
>>
>> Signed-off-by: Bhaskar Chowdhury <[email protected]>
>> ---
>> drivers/net/wireless/realtek/rtlwifi/rtl8192ce/trx.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/trx.c b/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/trx.c
>> index 4165175cf5c0..d53397e7eb2e 100644
>> --- a/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/trx.c
>> +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/trx.c
>> @@ -671,7 +671,7 @@ bool rtl92ce_is_tx_desc_closed(struct ieee80211_hw *hw,
>> u8 own = (u8)rtl92ce_get_desc(hw, entry, true, HW_DESC_OWN);
>>
>> /*beacon packet will only use the first
>> - *descriptor defautly,and the own may not
>> + *descriptor de-faulty,and the own may not
>
>Same comments here as the previous patch:
>
>"de-faultly" makes less sense than "defaultly". This comment needs to
>be re-written by someone who knows what's going on here.
>
Again, it was written "defautly" ..which is a wrong spelling , it has got
nothing do with other thing.
>Thanks,
>
>--
>Julian Calaby
>
>Email: [email protected]
>Profile: http://www.google.com/profiles/julian.calaby/


Attachments:
(No filename) (1.37 kB)
signature.asc (499.00 B)
Download all attachments