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 <[email protected]> > Acked-by: Arend van Spriel <[email protected]> > > 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; > } >
