Hi Stefan, On Sun, Dec 17, 2017 at 6:37 PM, Stefan Agner <stefan@xxxxxxxx> wrote: > --- /dev/null > +++ b/arch/arm/boot/dts/imx7d-colibri-emmc-eval-v3.dts > @@ -0,0 +1,20 @@ > +/* > + * Copyright 2017 Toradex AG > + * > + * SPDX-License-Identifier: (GPL-2.0+ OR MIT) > + */ In the previous patch you used GPL-2.0 text, instead of GPL-2.0+. The SPDX line should be the first one and start with a // style comment: // SPDX-License-Identifier: (GPL-2.0 OR MIT) Philippe, please confirm this is the correct way. With this fixed you can add: 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