Hi, On Sun, Feb 02, 2020 at 01:18:23PM -0800, Florian Fainelli wrote: > Update the Broadcom BCM4708 SoC family binding document for boards/SoCs > to use YAML. Verified with dt_binding_check and dtbs_check. > > Signed-off-by: Florian Fainelli <f.fainelli@xxxxxxxxx> > --- > .../bindings/arm/bcm/brcm,bcm4708.txt | 15 ---- > .../bindings/arm/bcm/brcm,bcm4708.yaml | 88 +++++++++++++++++++ > 2 files changed, 88 insertions(+), 15 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/arm/bcm/brcm,bcm4708.txt > create mode 100644 Documentation/devicetree/bindings/arm/bcm/brcm,bcm4708.yaml > > diff --git a/Documentation/devicetree/bindings/arm/bcm/brcm,bcm4708.txt b/Documentation/devicetree/bindings/arm/bcm/brcm,bcm4708.txt > deleted file mode 100644 > index 8608a776caa7..000000000000 > --- a/Documentation/devicetree/bindings/arm/bcm/brcm,bcm4708.txt > +++ /dev/null > @@ -1,15 +0,0 @@ > -Broadcom BCM4708 device tree bindings > -------------------------------------------- > - > -Boards with the BCM4708 SoC shall have the following properties: > - > -Required root node property: > - > -bcm4708 > -compatible = "brcm,bcm4708"; > - > -bcm4709 > -compatible = "brcm,bcm4709"; > - > -bcm53012 > -compatible = "brcm,bcm53012"; > diff --git a/Documentation/devicetree/bindings/arm/bcm/brcm,bcm4708.yaml b/Documentation/devicetree/bindings/arm/bcm/brcm,bcm4708.yaml > new file mode 100644 > index 000000000000..d48313c7ae45 > --- /dev/null > +++ b/Documentation/devicetree/bindings/arm/bcm/brcm,bcm4708.yaml > @@ -0,0 +1,88 @@ > +# SPDX-License-Identifier: GPL-2.0 > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/arm/bcm/brcm,bcm4708.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Broadcom BCM4708 device tree bindings > + > +description: > + Broadcom BCM4708/47081/4709/47094/53012 Wi-Fi/network SoCs based > + on the iProc architecture (Northstar). > + > +maintainers: > + - Florian Fainelli <f.fainelli@xxxxxxxxx> > + - Hauke Mehrtens <hauke@xxxxxxxxxx> > + - Rafal Milecki <zajec5@xxxxxxxxx> > + > +properties: > + $nodename: > + const: '/' > + compatible: > + oneOf: > + - description: BCM4708 based boards > + items: > + - enum: > + - asus,rt-ac56u > + - asus,rt-ac68u > + - buffalo,wzr-1750dhp > + - linksys,ea6300-v1 > + - linksys,ea6500-v2 > + - luxul,xap-1510v1 > + - luxul,xwc-1000 > + - netgear,r6250v1 > + - netgear,r6300v2 > + - smartrg,sr400ac > + - brcm,bcm94708 > + - const: brcm,bcm4708 I'm under the impression that while the SoC compatible description in your other bindings is obvious enough, if anything, you should have a description of the boards. But it's up to you :) Maxime
