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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




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.

> + *
> + * 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.

Regards,

Fabio Estevam
--
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



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux