2023-12-20 14:18:41

by Colin Ian King

[permalink] [raw]
Subject: [PATCH][next] wifi: rtw89: mac: Fix spelling mistakes "notfify" -> "notify"

There are two spelling mistakes in rtw89_err error messages. Fix these
and also add space between [ERR] and message text.

Signed-off-by: Colin Ian King <[email protected]>
---
drivers/net/wireless/realtek/rtw89/mac_be.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/wireless/realtek/rtw89/mac_be.c b/drivers/net/wireless/realtek/rtw89/mac_be.c
index be30c9346293..4befbe06cd15 100644
--- a/drivers/net/wireless/realtek/rtw89/mac_be.c
+++ b/drivers/net/wireless/realtek/rtw89/mac_be.c
@@ -1616,7 +1616,7 @@ static int dbcc_enable_be(struct rtw89_dev *rtwdev, bool enable)
if (test_bit(RTW89_FLAG_FW_RDY, rtwdev->flags)) {
ret = rtw89_fw_h2c_notify_dbcc(rtwdev, true);
if (ret) {
- rtw89_err(rtwdev, "%s:[ERR]notfify dbcc1 fail %d\n",
+ rtw89_err(rtwdev, "%s:[ERR] notify dbcc1 fail %d\n",
__func__, ret);
return ret;
}
@@ -1625,7 +1625,7 @@ static int dbcc_enable_be(struct rtw89_dev *rtwdev, bool enable)
if (test_bit(RTW89_FLAG_FW_RDY, rtwdev->flags)) {
ret = rtw89_fw_h2c_notify_dbcc(rtwdev, false);
if (ret) {
- rtw89_err(rtwdev, "%s:[ERR]notfify dbcc1 fail %d\n",
+ rtw89_err(rtwdev, "%s:[ERR] notify dbcc1 fail %d\n",
__func__, ret);
return ret;
}
--
2.39.2



2023-12-22 00:19:32

by Ping-Ke Shih

[permalink] [raw]
Subject: RE: [PATCH][next] wifi: rtw89: mac: Fix spelling mistakes "notfify" -> "notify"



> -----Original Message-----
> From: Colin Ian King <[email protected]>
> Sent: Wednesday, December 20, 2023 10:19 PM
> To: Ping-Ke Shih <[email protected]>; Kalle Valo <[email protected]>; [email protected]
> Cc: [email protected]; [email protected]
> Subject: [PATCH][next] wifi: rtw89: mac: Fix spelling mistakes "notfify" -> "notify"
>
> There are two spelling mistakes in rtw89_err error messages. Fix these
> and also add space between [ERR] and message text.
>
> Signed-off-by: Colin Ian King <[email protected]>

Acked-by: Ping-Ke Shih <[email protected]>

Thanks for the correctness. Could I know the tool you used to find out these
typo?

Ping-Ke


2023-12-22 12:08:49

by Colin Ian King

[permalink] [raw]
Subject: Re: [PATCH][next] wifi: rtw89: mac: Fix spelling mistakes "notfify" -> "notify"

On 22/12/2023 00:19, Ping-Ke Shih wrote:
>
>
>> -----Original Message-----
>> From: Colin Ian King <[email protected]>
>> Sent: Wednesday, December 20, 2023 10:19 PM
>> To: Ping-Ke Shih <[email protected]>; Kalle Valo <[email protected]>; [email protected]
>> Cc: [email protected]; [email protected]
>> Subject: [PATCH][next] wifi: rtw89: mac: Fix spelling mistakes "notfify" -> "notify"
>>
>> There are two spelling mistakes in rtw89_err error messages. Fix these
>> and also add space between [ERR] and message text.
>>
>> Signed-off-by: Colin Ian King <[email protected]>
>
> Acked-by: Ping-Ke Shih <[email protected]>
>
> Thanks for the correctness. Could I know the tool you used to find out these
> typo?

Sure,

https://github.com/ColinIanKing/kernelscan

It needs the american dictionary, installed in /usr/share/dict e.g.
apt-get install wamerican

and then spellcheck with:

./kernelscan -k path-to-code-you-want-to-scan

I run kernelscan on the entire linux-next source daily and diff the
days's results with the previous day using the meld diff tool.

Colin


>
> Ping-Ke
>
>


2023-12-25 00:19:35

by Ping-Ke Shih

[permalink] [raw]
Subject: RE: [PATCH][next] wifi: rtw89: mac: Fix spelling mistakes "notfify" -> "notify"



> -----Original Message-----
> From: Colin King (gmail) <[email protected]>
> Sent: Friday, December 22, 2023 8:09 PM
> To: Ping-Ke Shih <[email protected]>; Kalle Valo <[email protected]>; [email protected]
> Cc: [email protected]; [email protected]
> Subject: Re: [PATCH][next] wifi: rtw89: mac: Fix spelling mistakes "notfify" -> "notify"
>
> On 22/12/2023 00:19, Ping-Ke Shih wrote:
> >
> >
> >> -----Original Message-----
> >> From: Colin Ian King <[email protected]>
> >> Sent: Wednesday, December 20, 2023 10:19 PM
> >> To: Ping-Ke Shih <[email protected]>; Kalle Valo <[email protected]>; [email protected]
> >> Cc: [email protected]; [email protected]
> >> Subject: [PATCH][next] wifi: rtw89: mac: Fix spelling mistakes "notfify" -> "notify"
> >>
> >> There are two spelling mistakes in rtw89_err error messages. Fix these
> >> and also add space between [ERR] and message text.
> >>
> >> Signed-off-by: Colin Ian King <[email protected]>
> >
> > Acked-by: Ping-Ke Shih <[email protected]>
> >
> > Thanks for the correctness. Could I know the tool you used to find out these
> > typo?
>
> Sure,
>
> https://github.com/ColinIanKing/kernelscan
>
> It needs the american dictionary, installed in /usr/share/dict e.g.
> apt-get install wamerican
>
> and then spellcheck with:
>
> ./kernelscan -k path-to-code-you-want-to-scan

Thanks for the tool. :-)
I will try to scan my driver before sending out patches.

Ping-Ke


2024-01-10 14:52:01

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH][next] wifi: rtw89: mac: Fix spelling mistakes "notfify" -> "notify"

Colin Ian King <[email protected]> wrote:

> There are two spelling mistakes in rtw89_err error messages. Fix these
> and also add space between [ERR] and message text.
>
> Signed-off-by: Colin Ian King <[email protected]>
> Acked-by: Ping-Ke Shih <[email protected]>

Patch applied to wireless-next.git, thanks.

6aeaa379291b wifi: rtw89: mac: Fix spelling mistakes "notfify" -> "notify"

--
https://patchwork.kernel.org/project/linux-wireless/patch/[email protected]/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches