Hi Andrey, On Tue, Jul 17, 2018 at 1:06 AM, Andrey Smirnov <andrew.smirnov@xxxxxxxxx> wrote: > +/dts-v1/; > +#include "vf610.dtsi" > + > +/ { > + model = "ZII VF610 SSMB SPU3 Board"; > + compatible = "zii,vf610spu3", "zii,vf610dev", "fsl,vf610"; Looks good to me. Just a minor comment: Is " "zii,vf610dev" really needed here? Anyway: Reviewed-by: Fabio Estevam <fabio.estevam@xxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html