| Message ID | 20210105102751.21237-1-unixbhaskar@gmail.com (mailing list archive) |
|---|---|
| State | Changes Requested |
| Delegated to: | Kalle Valo |
| Headers | show |
| Series | drivers: wireless: rtlwifi: rtl8192ce: Fix construction of word rtl8192ce/trx.c | expand |
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,
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/
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)
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(-) -- 2.26.2