[PATCH 4/4] ARM: dts: imx: add Boundary Devices Nitrogen6_Lite board

[Gary Bisson]

Hi Fabio,

On Tue, Sep 8, 2015 at 3:08 AM, Fabio Estevam <festevam at gmail.com> wrote:
> Hi Gary,
>
> On Mon, Sep 7, 2015 at 1:57 PM, Gary Bisson
> <gary.bisson at boundarydevices.com> 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