mbox series

[v2,0/2] brcmfmac: support parse country code map from DT

Message ID 20210415104728.8471-1-shawn.guo@linaro.org
Headers show
Series brcmfmac: support parse country code map from DT | expand

Message

Shawn Guo April 15, 2021, 10:47 a.m. UTC 
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/


Shawn Guo (2):
  dt-bindings: bcm4329-fmac: add optional brcm,ccode-map
  brcmfmac: support parse country code map from DT

 .../net/wireless/brcm,bcm4329-fmac.yaml       |  8 +++
 .../wireless/broadcom/brcm80211/brcmfmac/of.c | 57 ++++++++++++++++++-
 2 files changed, 63 insertions(+), 2 deletions(-)

-- 
2.17.1

Comments

Rob Herring (Arm) April 15, 2021, 9:34 p.m. UTC | #1 
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