Re: [PATCH net-next 1/3] dt-bindings: net: convert Broadcom Starfighter 2 binding to the json-schema

On 1/6/21 4:07 AM, Rafał Miłecki wrote:

From: Rafał Miłecki <rafal@milecki.pl>

This helps validating DTS files. Only the current (not deprecated one)
binding was converted.

Minor changes:
1. Dropped dsa/dsa.txt references
2. Updated node name to match dsa.yaml requirement
3. Fixed 2 typos in examples

The new binding was validated using the dt_binding_check.

Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
---

This looks good to me, but we would want Rob to review this binding
obviously. Just one comment below.

[snip]

+
+  reset-names:
+    const: switch
+
+  clocks:
+    items:
+      - description: switch's main clock (valid for both BCM7445 and BCM7278)
+      - description: only applicable to BCM7445 and is to support dividing the switch core clock.

Since this is a YAML binding we could adjust the number of min/max clock
items here based on the compatible string.
-- 
Florian