On 1/6/21 4:07 AM, Rafał Miłecki wrote: > From: Rafał Miłecki <rafal@xxxxxxxxxx> > > 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@xxxxxxxxxx> > --- 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