Hi Fabio, On Tue, Sep 8, 2015 at 3:08 AM, Fabio Estevam <festevam@xxxxxxxxx> wrote: > Hi Gary, > > On Mon, Sep 7, 2015 at 1:57 PM, Gary Bisson > <gary.bisson@xxxxxxxxxxxxxxxxxxx> wrote: > >> --- /dev/null >> +++ b/arch/arm/boot/dts/imx6dl-nit6xlite.dts >> @@ -0,0 +1,21 @@ >> +/* >> + * Copyright 2015 Boundary Devices, Inc. >> + * Copyright 2012 Freescale Semiconductor, Inc. >> + * Copyright 2011 Linaro Ltd. >> + * >> + * The code contained herein is licensed under the GNU General Public >> + * License. You may obtain a copy of the GNU General Public License >> + * Version 2 or later at the following locations: > > Could you do this dual license GPLv2/X11? > > Take a look at arch/arm/boot/dts/imx6qdl-cubox-i.dtsi for a reference. Ok will do in v2. Should I relicense Nitrogen6x and SabreLite as well? >> + * >> + * http://www.opensource.org/licenses/gpl-license.html >> + * http://www.gnu.org/copyleft/gpl.html >> + */ >> + >> +/dts-v1/; >> +#include "imx6dl.dtsi" >> +#include "imx6qdl-nit6xlite.dtsi" >> + >> +/ { >> + model = "Freescale i.MX6 Solo Nitrogen6_Lite Board"; >> + compatible = "fsl,imx6dl-nit6xlite", "fsl,imx6dl"; > > As this board is manufactured by Boundary Devices and not by Freescale > shouldn't this be: > > model = "Boundary Devices i.MX6 Solo Nitrogen6_Lite Board"; > compatible = "boundarydevices,imx6dl-nit6xlite", "fsl,imx6dl"; > > Then you can add the symbol for Boundary Devices at > Documentation/devicetree/bindings/vendor-prefixes.txt in a separate > patch. Yes, I actually just copied what was done for the Nitrogen6x, should I update the latter as well? Thanks, Gary -- 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