Re: [PATCH v2 4/5] ARM: dts: Add support for emtrion emCON-MX6 series

2017-12-29 Thread Alexandre Belloni
On 26/12/2017 at 11:16:34 -0600, Rob Herring wrote: > On Fri, Dec 22, 2017 at 4:56 AM, Alexandre Belloni > wrote: > > + Philippe > > > > On 22/12/2017 at 11:43:33 +0100, Andreas Färber wrote: > >> >> I'll change it for v3 of this patch however it will end up like this: > >> >> //SPDX-License... >

Re: [PATCH v2 4/5] ARM: dts: Add support for emtrion emCON-MX6 series

2017-12-26 Thread Rob Herring
On Fri, Dec 22, 2017 at 4:56 AM, Alexandre Belloni wrote: > + Philippe > > On 22/12/2017 at 11:43:33 +0100, Andreas Färber wrote: >> >> I'll change it for v3 of this patch however it will end up like this: >> >> //SPDX-License... >> > >> > That should be /* SPDX-License */, // is for c files. >> >

Re: [PATCH v2 4/5] ARM: dts: Add support for emtrion emCON-MX6 series

2017-12-23 Thread Andreas Färber
Am 22.12.2017 um 11:34 schrieb Türk, Jan: >> On Wed, Dec 20, 2017 at 02:47:04PM +0100, jan.tu...@emtrion.com wrote: >>> + * SPDX-License-Identifier: GPL-2.0 >> >> You have this. >> >> Also, the rules around this are getting a bit stricter saying the SPDX >> tag should be the first line of the file

Re: [PATCH v2 4/5] ARM: dts: Add support for emtrion emCON-MX6 series

2017-12-23 Thread Alexandre Belloni
Hi, On 22/12/2017 at 11:34:31 +0100, Türk, Jan wrote: > > > diff --git a/arch/arm/boot/dts/imx6q-emcon.dtsi b/arch/arm/boot/dts/imx6q- > > emcon.dtsi > > > new file mode 100644 > > > index ..64fc0cd74c05 > > > --- /dev/null > > > +++ b/arch/arm/boot/dts/imx6q-emcon.dtsi > > > @@ -0,0 +

AW: [PATCH v2 4/5] ARM: dts: Add support for emtrion emCON-MX6 series

2017-12-23 Thread Türk , Jan
Hi, > On 22/12/2017 at 11:43:33 +0100, Andreas Färber wrote: > > >> I'll change it for v3 of this patch however it will end up like this: > > >> //SPDX-License... > > > > > > That should be /* SPDX-License */, // is for c files. > > > > Got any reference for that? Since we're using the C preproce

Re: [PATCH v2 4/5] ARM: dts: Add support for emtrion emCON-MX6 series

2017-12-23 Thread Türk , Jan
> -Ursprüngliche Nachricht- > Von: Rob Herring [mailto:r...@kernel.org] > Gesendet: Freitag, 22. Dezember 2017 00:19 > An: Türk, Jan > Betreff: Re: [PATCH v2 4/5] ARM: dts: Add support for emtrion emCON-MX6 > series > > On Wed, Dec 20, 2017 at 02:47:04PM +0100, jan

Re: [PATCH v2 4/5] ARM: dts: Add support for emtrion emCON-MX6 series

2017-12-23 Thread Alexandre Belloni
+ Philippe On 22/12/2017 at 11:43:33 +0100, Andreas Färber wrote: > >> I'll change it for v3 of this patch however it will end up like this: > >> //SPDX-License... > > > > That should be /* SPDX-License */, // is for c files. > > Got any reference for that? Since we're using the C preprocessor b

Re: [PATCH v2 4/5] ARM: dts: Add support for emtrion emCON-MX6 series

2017-12-23 Thread Andreas Färber
Hi, Am 22.12.2017 um 11:40 schrieb Alexandre Belloni: > On 22/12/2017 at 11:34:31 +0100, Türk, Jan wrote: diff --git a/arch/arm/boot/dts/imx6q-emcon.dtsi b/arch/arm/boot/dts/imx6q- >>> emcon.dtsi new file mode 100644 index ..64fc0cd74c05 --- /dev/null +++ b/arc

Re: [PATCH v2 4/5] ARM: dts: Add support for emtrion emCON-MX6 series

2017-12-21 Thread Rob Herring
On Wed, Dec 20, 2017 at 02:47:04PM +0100, jan.tu...@emtrion.com wrote: > From: Jan Tuerk > > This patch adds support for the emtrion GmbH emCON-MX6 modules. > They are available with imx.6 Solo, Dual-Lite, Dual and Quad > equipped with Memory from 512MB to 2GB (configured by U-Boot). > > Our def

[PATCH v2 4/5] ARM: dts: Add support for emtrion emCON-MX6 series

2017-12-21 Thread jan.tuerk
From: Jan Tuerk This patch adds support for the emtrion GmbH emCON-MX6 modules. They are available with imx.6 Solo, Dual-Lite, Dual and Quad equipped with Memory from 512MB to 2GB (configured by U-Boot). Our default developer-Kit ships with the Avari baseboard and the EDT ETM0700G0BDH6 Display (