On 22/03/2023 16:04, Martin Kurbanov wrote: > Add YAML devicetree binding for Amlogic Meson A113L (A1 family) > SPIFC Driver. Subject: drop second/last, redundant "binding for". The "dt-bindings" prefix is already stating that these are bindings. > +examples: > + - | > + #include <dt-bindings/clock/amlogic,a1-clkc.h> > + spi@fd000400 { > + compatible = "amlogic,meson-a1-spifc"; Use 4 spaces for example indentation. Best regards, Krzysztof