Re: [PATCH] drivers: net: wireless: realtek: Fix the word association defautly de-faulty

[Bhaskar Chowdhury]

On 21:33 Tue 05 Jan 2021, Julian Calaby wrote:
> Hi Bhaskar,
>
> On Tue, Jan 5, 2021 at 9:19 PM Bhaskar Chowdhury <unixbhaskar@xxxxxxxxx> wrote:
> > s/defautly/de-faulty/p
> >
> >
> > Signed-off-by: Bhaskar Chowdhury <unixbhaskar@xxxxxxxxx>
> > ---
> >  drivers/net/wireless/realtek/rtlwifi/rtl8188ee/trx.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/trx.c b/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/trx.c
> > index c948dafa0c80..7d02d8abb4eb 100644
> > --- a/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/trx.c
> > +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/trx.c
> > @@ -814,7 +814,7 @@ bool rtl88ee_is_tx_desc_closed(struct ieee80211_hw *hw, u8 hw_queue, u16 index)
> >         u8 own = (u8)rtl88ee_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
>
> Really? "de-faultly" isn't any better than "defaultly" and in fact
> it's even worse as it breaks up the word "default".
hey, it was written as "defautly" ...and that was simple spelling mistake ..
so,corrected it.

> This change doesn't make sense and the comment really needs to be
> completely re-written by someone who understands what's going on here
> as it barely makes sense.
>
> Thanks,
>
> --
> Julian Calaby
>
> Email: julian.calaby@xxxxxxxxx
> Profile: http://www.google.com/profiles/julian.calaby/

Attachment: signature.asc
Description: PGP signature