diff mbox series

[V4,2/3] dt-bindings: arm: bcm: document BCM4908 bindings

Message ID 20201112150832.26474-2-zajec5@gmail.com (mailing list archive)
State New, archived
Headers show
Series [V4,1/3] arm64: add config for Broadcom BCM4908 SoCs | expand

Commit Message

Rafał Miłecki Nov. 12, 2020, 3:08 p.m. UTC 
From: Rafał Miłecki <rafal@milecki.pl>

BCM4908 is a new family that includes BCM4906, BCM4908 and BCM49408.
It's mostly used in home routers and often replaces Northstar in vendors
portfolio.

Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
---
V3: New patch in the series
---
 .../bindings/arm/bcm/brcm,bcm4908.yaml        | 38 +++++++++++++++++++
 1 file changed, 38 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/arm/bcm/brcm,bcm4908.yaml

Comments

Florian Fainelli Nov. 24, 2020, 10:41 p.m. UTC | #1 
On Thu, 12 Nov 2020 16:08:31 +0100, Rafał Miłecki <zajec5@gmail.com> wrote:
> From: Rafał Miłecki <rafal@milecki.pl>
> 
> BCM4908 is a new family that includes BCM4906, BCM4908 and BCM49408.
> It's mostly used in home routers and often replaces Northstar in vendors
> portfolio.
> 
> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> ---

Applied to devicetree-arm64/next, thanks!
--
Florian
Florian Fainelli Nov. 25, 2020, 4:20 a.m. UTC | #2 
On 11/24/2020 2:41 PM, Florian Fainelli wrote:
> On Thu, 12 Nov 2020 16:08:31 +0100, Rafał Miłecki <zajec5@gmail.com> wrote:
>> From: Rafał Miłecki <rafal@milecki.pl>
>>
>> BCM4908 is a new family that includes BCM4906, BCM4908 and BCM49408.
>> It's mostly used in home routers and often replaces Northstar in vendors
>> portfolio.
>>
>> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
>> ---
> 
> Applied to devicetree-arm64/next, thanks!

Can you send me an incremental fix for this warning:

  CHECK
arch/arm64/boot/dts/broadcom/bcm4908/bcm4908-asus-gt-ac5300.dt.yaml
/home/fainelli/dev/linux/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908-asus-gt-ac5300.dt.yaml:
/: '#address-cells', '#size-cells', 'aliases', 'axi@81000000',
'bus@ff800000', 'chosen', 'clocks', 'cpus', 'gpio-keys-polled',
'interrupt-parent', 'memory@0', 'model', 'pmu', 'soc', 'timer' do not
match any of the regexes: 'pinctrl-[0-9]+'
        From schema:
/home/fainelli/dev/linux/Documentation/devicetree/bindings/arm/bcm/brcm,bcm4908.yaml


reproduced with:

ARCH=arm64 make dtbs_check
DT_SCHEMA_FILES=Documentation/devicetree/bindings/arm/bcm/brcm,bcm4908.yaml

Thanks
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/bcm/brcm,bcm4908.yaml b/Documentation/devicetree/bindings/arm/bcm/brcm,bcm4908.yaml
new file mode 100644
index 000000000000..dfe1c1c6bb8c
--- /dev/null
+++ b/Documentation/devicetree/bindings/arm/bcm/brcm,bcm4908.yaml
@@ -0,0 +1,38 @@ 
+# SPDX-License-Identifier: GPL-2.0 OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/arm/bcm/brcm,bcm4908.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Broadcom BCM4908 device tree bindings
+
+description:
+  Broadcom BCM4906 / BCM4908 / BCM49408 Wi-Fi/network SoCs with Brahma CPUs.
+
+maintainers:
+  - Rafał Miłecki <rafal@milecki.pl>
+
+properties:
+  $nodename:
+    const: '/'
+  compatible:
+    oneOf:
+      - description: BCM4906 based boards
+        items:
+          - const: brcm,bcm4906
+          - const: brcm,bcm4908
+
+      - description: BCM4908 based boards
+        items:
+          - enum:
+              - asus,gt-ac5300
+          - const: brcm,bcm4908
+
+      - description: BCM49408 based boards
+        items:
+          - const: brcm,bcm49408
+          - const: brcm,bcm4908
+
+additionalProperties: false
+
+...