2019-01-21 08:24:24

by Matteo Croce

[permalink] [raw]
Subject: [PATCH net-next] wireless: fix typos

Fix spelling mistakes in brcm80211 and cfg80211: "lenght" -> "length".
The typos are also in the special comment blocks which
translates to documentation.

Signed-off-by: Matteo Croce <[email protected]>
---
drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.h | 2 +-
drivers/net/wireless/broadcom/brcm80211/brcmfmac/firmware.c | 2 +-
drivers/net/wireless/broadcom/brcm80211/brcmfmac/fweh.h | 2 +-
drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c | 2 +-
include/net/cfg80211.h | 2 +-
5 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.h b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.h
index dcf6e27cc16f..d8085ce579f4 100644
--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.h
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.h
@@ -36,7 +36,7 @@
#define BRCMF_DCMD_MEDLEN 1536
#define BRCMF_DCMD_MAXLEN 8192

-/* IOCTL from host to device are limited in lenght. A device can only handle
+/* IOCTL from host to device are limited in length. A device can only handle
* ethernet frame size. This limitation is to be applied by protocol layer.
*/
#define BRCMF_TX_IOCTL_MAX_MSG_SIZE (ETH_FRAME_LEN+ETH_FCS_LEN)
diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/firmware.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/firmware.c
index 14b948917a1a..cf1ddf988476 100644
--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/firmware.c
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/firmware.c
@@ -47,7 +47,7 @@ enum nvram_parser_state {
* @state: current parser state.
* @data: input buffer being parsed.
* @nvram: output buffer with parse result.
- * @nvram_len: lenght of parse result.
+ * @nvram_len: length of parse result.
* @line: current line.
* @column: current column in line.
* @pos: byte offset in input buffer.
diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/fweh.h b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/fweh.h
index 816f80ea925b..31f3e8e83a21 100644
--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/fweh.h
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/fweh.h
@@ -266,7 +266,7 @@ struct brcmf_event {
* @status: status information.
* @reason: reason code.
* @auth_type: authentication type.
- * @datalen: lenght of event data buffer.
+ * @datalen: length of event data buffer.
* @addr: ether address.
* @ifname: interface name.
* @ifidx: interface index.
diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c
index a4308c6e72d7..0b7d977510c8 100644
--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c
@@ -508,7 +508,7 @@ static void brcmf_usb_rx_complete(struct urb *urb)
skb = req->skb;
req->skb = NULL;

- /* zero lenght packets indicate usb "failure". Do not refill */
+ /* zero length packets indicate usb "failure". Do not refill */
if (urb->status != 0 || !urb->actual_length) {
brcmu_pkt_buf_free_skb(skb);
brcmf_usb_enq(devinfo, &devinfo->rx_freeq, req, NULL);
diff --git a/include/net/cfg80211.h b/include/net/cfg80211.h
index e0c41eb1c860..237839210a36 100644
--- a/include/net/cfg80211.h
+++ b/include/net/cfg80211.h
@@ -5568,7 +5568,7 @@ void cfg80211_ibss_joined(struct net_device *dev, const u8 *bssid,
* @dev: network device
* @macaddr: the MAC address of the new candidate
* @ie: information elements advertised by the peer candidate
- * @ie_len: lenght of the information elements buffer
+ * @ie_len: length of the information elements buffer
* @gfp: allocation flags
*
* This function notifies cfg80211 that the mesh peer candidate has been
--
2.20.1



2019-01-21 07:48:06

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH net-next] wireless: fix typos

Matteo Croce <[email protected]> writes:

> Fix spelling mistakes in brcm80211 and cfg80211: "lenght" -> "length".
> The typos are also in the special comment blocks which
> translates to documentation.
>
> Signed-off-by: Matteo Croce <[email protected]>
> ---
> drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.h | 2 +-
> drivers/net/wireless/broadcom/brcm80211/brcmfmac/firmware.c | 2 +-
> drivers/net/wireless/broadcom/brcm80211/brcmfmac/fweh.h | 2 +-
> drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c | 2 +-
> include/net/cfg80211.h | 2 +-

cfg80211 changes go through mac80211-next. Can you please split these in
two patches, cfg80211 and brcmfmac?

--
Kalle Valo

2019-01-29 12:50:07

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH net-next] wireless: fix typos

Kalle Valo <[email protected]> writes:

> Matteo Croce <[email protected]> writes:
>
>> Fix spelling mistakes in brcm80211 and cfg80211: "lenght" -> "length".
>> The typos are also in the special comment blocks which
>> translates to documentation.
>>
>> Signed-off-by: Matteo Croce <[email protected]>
>> ---
>> drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.h | 2 +-
>> drivers/net/wireless/broadcom/brcm80211/brcmfmac/firmware.c | 2 +-
>> drivers/net/wireless/broadcom/brcm80211/brcmfmac/fweh.h | 2 +-
>> drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c | 2 +-
>> include/net/cfg80211.h | 2 +-
>
> cfg80211 changes go through mac80211-next. Can you please split these in
> two patches, cfg80211 and brcmfmac?

Strange, my previous comment didn't make it to patchwork. Let's try
again :)

https://patchwork.kernel.org/patch/10773337/

--
Kalle Valo