Re: [PATCH] brcmfmac: iso cfg80211: Fix grammar issue in error message

[Arend van Spriel <arend.vanspriel@broadcom.com>]

Hah. That is (unintended?) fun. I actually meant to change the subject
from "cfg80211: Fix grammar issue in error message" into "brcmfmac: Fix
grammar issue in error message". Using the abbreviation "iso" in this
context was maybe a bit misleading of me.

One more purely educational note: when sending a reworked patch it is
good practice to use "[PATCH V2]" in the subject and add a changelog
between the signature tags and the diff separated by "---" so people now
what changed, ie.:

---
 changes:
  V2:
   - updated subject prefix.
---

Thanks,
Arend

On 10-06-17 11:22, Martin Michlmayr wrote:
> Fix grammar issue in error message about ISO3166.
> 
> Signed-off-by: Martin Michlmayr <tbm@cyrius.com>
> Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com>
> 
> diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c
> index cd1d6730eab7..c1ad81f34658 100644
> --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c
> +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c
> @@ -6767,7 +6767,7 @@ static void brcmf_cfg80211_reg_notifier(struct wiphy *wiphy,
>  	/* ignore non-ISO3166 country codes */
>  	for (i = 0; i < sizeof(req->alpha2); i++)
>  		if (req->alpha2[i] < 'A' || req->alpha2[i] > 'Z') {
> -			brcmf_err("not a ISO3166 code (0x%02x 0x%02x)\n",
> +			brcmf_err("not an ISO3166 code (0x%02x 0x%02x)\n",
>  				  req->alpha2[0], req->alpha2[1]);
>  			return;
>  		}
>