From mboxrd@z Thu Jan  1 00:00:00 1970
Return-path: <linux-wireless-owner@vger.kernel.org>
Received: from mail-qk0-f174.google.com ([209.85.220.174]:35085 "EHLO
        mail-qk0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1751687AbeCOJdY (ORCPT
        <rfc822;linux-wireless@vger.kernel.org>);
        Thu, 15 Mar 2018 05:33:24 -0400
Received: by mail-qk0-f174.google.com with SMTP id s188so6563707qkb.2
        for <linux-wireless@vger.kernel.org>; Thu, 15 Mar 2018 02:33:24 -0700 (PDT)
Subject: Re: [PATCH] wireless: brcmfmac: cfg80211: Fix check for ISO3166 code
To: Kalle Valo <kvalo@codeaurora.org>
References: <1521054179-4030-1-git-send-email-stefan.wahren@i2se.com>
 <87o9jp7l95.fsf@kamboji.qca.qualcomm.com> <5AAA3C9A.3040907@broadcom.com>
 <87fu517kk1.fsf@kamboji.qca.qualcomm.com>
Cc: Stefan Wahren <stefan.wahren@i2se.com>,
        Franky Lin <franky.lin@broadcom.com>,
        Hante Meuleman <hante.meuleman@broadcom.com>,
        Chi-Hsien Lin <chi-hsien.lin@cypress.com>,
        Wright Feng <wright.feng@cypress.com>,
        Johannes Berg <johannes.berg@intel.com>,
        Hans de Goede <hdegoede@redhat.com>,
        linux-wireless@vger.kernel.org,
        brcm80211-dev-list.pdl@broadcom.com, brcm80211-dev-list@cypress.com
From: Arend van Spriel <arend.vanspriel@broadcom.com>
Message-ID: <5AAA3DE0.5040108@broadcom.com> (sfid-20180315_103328_648353_67B157FB)
Date: Thu, 15 Mar 2018 10:33:20 +0100
MIME-Version: 1.0
In-Reply-To: <87fu517kk1.fsf@kamboji.qca.qualcomm.com>
Content-Type: text/plain; charset=windows-1252; format=flowed
Sender: linux-wireless-owner@vger.kernel.org
List-ID: <linux-wireless.vger.kernel.org>

On 3/15/2018 10:30 AM, Kalle Valo wrote:
> Arend van Spriel <arend.vanspriel@broadcom.com> writes:
>
>> On 3/15/2018 10:15 AM, Kalle Valo wrote:
>>> Stefan Wahren <stefan.wahren@i2se.com> writes:
>>>
>>>> The commit "regulatory: add NUL to request alpha2" increases the length of
>>>> alpha2 to 3. This causes a regression on brcmfmac, because
>>>> brcmf_cfg80211_reg_notifier() expect valid ISO3166 codes in the complete
>>>> array. So fix this accordingly.
>>>>
>>>> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
>>>
>>> Arend, the first release for 657308f73e67 is v4.16-rc3 so should this go
>>> to v4.16 as well?
>>>
>>> And "wireless:" in the title is useless, I can remove that.
>>
>> And ": cfg80211" as well?
>
> Ok, I'll remove that.

Thanks