Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Thu, 15 Apr 2021 16:34:06 -0500
From:   Rob Herring <robh@kernel.org>
To:     Shawn Guo <shawn.guo@linaro.org>
Cc:     Kalle Valo <kvalo@codeaurora.org>,
        =?utf-8?B?UmFmYcWCIE1pxYJlY2tp?= <rafal@milecki.pl>,
        Arend van Spriel <aspriel@gmail.com>,
        Franky Lin <franky.lin@broadcom.com>,
        Hante Meuleman <hante.meuleman@broadcom.com>,
        Chi-hsien Lin <chi-hsien.lin@infineon.com>,
        Wright Feng <wright.feng@infineon.com>,
        Chung-hsien Hsu <chung-hsien.hsu@infineon.com>,
        linux-wireless@vger.kernel.org, netdev@vger.kernel.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        brcm80211-dev-list.pdl@broadcom.com,
        SHA-cyfmac-dev-list@infineon.com
Subject: Re: [PATCH v2 0/2] brcmfmac: support parse country code map from DT
Message-ID: <20210415213406.GA1912098@robh.at.kernel.org>
References: <20210415104728.8471-1-shawn.guo@linaro.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20210415104728.8471-1-shawn.guo@linaro.org>
Precedence: bulk

On Thu, Apr 15, 2021 at 06:47:26PM +0800, Shawn Guo wrote:
> This is a couple of patches adding optional brcm,ccode-map bindings for
> brcmfmac driver to parse country code map from DT.
> 
> Changes for v2:
>  - Rebase bindings patch on top of yaml conversion patch [1].
>  - Improve commit log with Arend's explanation on why this data could
>    be put in device tree.
>  - Use pattern to define mapping string as suggested by Rob.
>  - Use brcmf_err() instead of dev_warn() and print error code.
>  - Use sscanf() to validate mapping string.
>  - Use brcmf_dbg(INFO, ...) to print country code entry.
>  - Separate BRCMF_BUSTYPE_SDIO bus_type check from general DT validation.
> 
> [1] https://patchwork.kernel.org/project/linux-wireless/patch/20210315105911.138553-1-linus.walleij@linaro.org/

As that's in my tree, I'm applying patch 1. Patch 2 should go via the 
wireless tree.

Rob