linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] drivers: wireless: rtlwifi: rtl8192ce: Fix construction of word rtl8192ce/trx.c
@ 2021-01-05 10:27 Bhaskar Chowdhury
  2021-01-05 10:34 ` Julian Calaby
  0 siblings, 1 reply; 3+ messages in thread
From: Bhaskar Chowdhury @ 2021-01-05 10:27 UTC (permalink / raw)
  To: pkshih, kvalo, davem, kuba, christophe.jaillet, Larry.Finger,
	baijiaju1990, linux-wireless, netdev, linux-kernel
  Cc: Bhaskar Chowdhury

s/defautly/de-faulty/p

Signed-off-by: Bhaskar Chowdhury <unixbhaskar@gmail.com>
---
 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


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] drivers: wireless: rtlwifi: rtl8192ce: Fix construction of word rtl8192ce/trx.c
  2021-01-05 10:27 [PATCH] drivers: wireless: rtlwifi: rtl8192ce: Fix construction of word rtl8192ce/trx.c Bhaskar Chowdhury
@ 2021-01-05 10:34 ` Julian Calaby
  2021-01-05 10:50   ` Bhaskar Chowdhury
  0 siblings, 1 reply; 3+ messages in thread
From: Julian Calaby @ 2021-01-05 10:34 UTC (permalink / raw)
  To: Bhaskar Chowdhury
  Cc: Ping-Ke Shih, Kalle Valo, David S. Miller, Jakub Kicinski,
	Christophe JAILLET, Larry Finger, baijiaju1990, linux-wireless,
	netdev, LKML

Hi Bhaskar,

On Tue, Jan 5, 2021 at 9:32 PM Bhaskar Chowdhury <unixbhaskar@gmail.com> wrote:
>
> s/defautly/de-faulty/p
>
> Signed-off-by: Bhaskar Chowdhury <unixbhaskar@gmail.com>
> ---
>  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: julian.calaby@gmail.com
Profile: http://www.google.com/profiles/julian.calaby/

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] drivers: wireless: rtlwifi: rtl8192ce: Fix construction of word rtl8192ce/trx.c
  2021-01-05 10:34 ` Julian Calaby
@ 2021-01-05 10:50   ` Bhaskar Chowdhury
  0 siblings, 0 replies; 3+ messages in thread
From: Bhaskar Chowdhury @ 2021-01-05 10:50 UTC (permalink / raw)
  To: Julian Calaby
  Cc: Ping-Ke Shih, Kalle Valo, David S. Miller, Jakub Kicinski,
	Christophe JAILLET, Larry Finger, baijiaju1990, linux-wireless,
	netdev, LKML

[-- Attachment #1: Type: text/plain, Size: 1363 bytes --]

On 21:34 Tue 05 Jan 2021, Julian Calaby wrote:
>Hi Bhaskar,
>
>On Tue, Jan 5, 2021 at 9:32 PM Bhaskar Chowdhury <unixbhaskar@gmail.com> wrote:
>>
>> s/defautly/de-faulty/p
>>
>> Signed-off-by: Bhaskar Chowdhury <unixbhaskar@gmail.com>
>> ---
>>  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: julian.calaby@gmail.com
>Profile: http://www.google.com/profiles/julian.calaby/

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-01-05 10:51 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-05 10:27 [PATCH] drivers: wireless: rtlwifi: rtl8192ce: Fix construction of word rtl8192ce/trx.c Bhaskar Chowdhury
2021-01-05 10:34 ` Julian Calaby
2021-01-05 10:50   ` Bhaskar Chowdhury

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).