Hi Rob, > > + clock-names: > > + items: > > + - const: nand_hclk > > + - const: nand_eclk > > 'nand_' is redundant. Actually the nand_ prefix was voluntary to be consistent with the specification which called these two clocks NAND_HCLK and NAND_EXLK. But whatever, I'll change the names. > > + "#address-cells": true > > + "#size-cells": true > > + > > +required: > > + - compatible > > + - reg > > + - clocks > > + - clock-names > > + - interrupts > > + > > +additionalProperties: true > > unevaluatedProperties: false Not sure why I proposed this property in the first place. I'll correct. Thanks, Miquèl