Re: [PATCH 5/6] ARM: dts: tegra: apalis-tk1: copyright period, spurious newlines
On Sat, Feb 10, 2018 at 9:40 AM, Marcel Ziswiler wrote: > Yes, I'm already in the process of preparing a separate patch set fixing this > across all Toradex files. However I'm running this through our legal as well > first to make absolutely sure it's all proper. That's the way to go and thank you for this. Do not hesitate to ping here or privately if there is any legal concern that comes up. -- Cordially Philippe Ombredanne
Re: [PATCH 5/6] ARM: dts: tegra: apalis-tk1: copyright period, spurious newlines
On February 10, 2018 8:07:58 AM GMT+01:00, Philippe Ombredanne wrote: >Marcel, > >On Sat, Feb 10, 2018 at 2:38 AM, Marcel Ziswiler >wrote: >> From: Marcel Ziswiler >> >> Update the copyright period and get rid of some spurious newlines. >> >> Signed-off-by: Marcel Ziswiler >> >> --- >> >> arch/arm/boot/dts/tegra124-apalis-eval.dts | 6 ++ >> arch/arm/boot/dts/tegra124-apalis.dtsi | 11 +-- >> 2 files changed, 3 insertions(+), 14 deletions(-) >> >> diff --git a/arch/arm/boot/dts/tegra124-apalis-eval.dts >b/arch/arm/boot/dts/tegra124-apalis-eval.dts >> index f1010cefb993..a6ad759dddb4 100644 >> --- a/arch/arm/boot/dts/tegra124-apalis-eval.dts >> +++ b/arch/arm/boot/dts/tegra124-apalis-eval.dts >> @@ -1,5 +1,5 @@ >> /* >> - * Copyright 2016 Toradex AG >> + * Copyright 2016-2018 Toradex AG >> * >> * This file is dual-licensed: you can use it either under the terms >> * of the GPL or the X11 license, at your option. Note that this >dual > >Since you are fixing copyrights, would you consider also fixing the >license to use a proper SPDX Id instead? > >It would be super gentle of you! > >And you will get extra good karma point if you feel like doing the >same for every Toradex-copyrighted files ;) > >The (still new and fresh) license documentation contributed by tglx >--the only real-time docu-mentalist-- is in: >Documentation/process/license-rules.rst > >Thanks! Yes, I'm already in the process of preparing a separate patch set fixing this across all Toradex files. However I'm running this through our legal as well first to make absolutely sure it's all proper. Thanks!
Re: [PATCH 5/6] ARM: dts: tegra: apalis-tk1: copyright period, spurious newlines
Marcel, On Sat, Feb 10, 2018 at 2:38 AM, Marcel Ziswiler wrote: > From: Marcel Ziswiler > > Update the copyright period and get rid of some spurious newlines. > > Signed-off-by: Marcel Ziswiler > > --- > > arch/arm/boot/dts/tegra124-apalis-eval.dts | 6 ++ > arch/arm/boot/dts/tegra124-apalis.dtsi | 11 +-- > 2 files changed, 3 insertions(+), 14 deletions(-) > > diff --git a/arch/arm/boot/dts/tegra124-apalis-eval.dts > b/arch/arm/boot/dts/tegra124-apalis-eval.dts > index f1010cefb993..a6ad759dddb4 100644 > --- a/arch/arm/boot/dts/tegra124-apalis-eval.dts > +++ b/arch/arm/boot/dts/tegra124-apalis-eval.dts > @@ -1,5 +1,5 @@ > /* > - * Copyright 2016 Toradex AG > + * Copyright 2016-2018 Toradex AG > * > * This file is dual-licensed: you can use it either under the terms > * of the GPL or the X11 license, at your option. Note that this dual Since you are fixing copyrights, would you consider also fixing the license to use a proper SPDX Id instead? It would be super gentle of you! And you will get extra good karma point if you feel like doing the same for every Toradex-copyrighted files ;) The (still new and fresh) license documentation contributed by tglx --the only real-time docu-mentalist-- is in: Documentation/process/license-rules.rst Thanks! -- Cordially Philippe Ombredanne
[PATCH 5/6] ARM: dts: tegra: apalis-tk1: copyright period, spurious newlines
From: Marcel Ziswiler Update the copyright period and get rid of some spurious newlines. Signed-off-by: Marcel Ziswiler --- arch/arm/boot/dts/tegra124-apalis-eval.dts | 6 ++ arch/arm/boot/dts/tegra124-apalis.dtsi | 11 +-- 2 files changed, 3 insertions(+), 14 deletions(-) diff --git a/arch/arm/boot/dts/tegra124-apalis-eval.dts b/arch/arm/boot/dts/tegra124-apalis-eval.dts index f1010cefb993..a6ad759dddb4 100644 --- a/arch/arm/boot/dts/tegra124-apalis-eval.dts +++ b/arch/arm/boot/dts/tegra124-apalis-eval.dts @@ -1,5 +1,5 @@ /* - * Copyright 2016 Toradex AG + * Copyright 2016-2018 Toradex AG * * This file is dual-licensed: you can use it either under the terms * of the GPL or the X11 license, at your option. Note that this dual @@ -225,9 +225,7 @@ backlight: backlight { compatible = "pwm-backlight"; - - /* BKL1_PWM */ - pwms = <&pwm 3 500>; + pwms = <&pwm 3 500>; /* BKL1_PWM */ brightness-levels = <255 231 223 207 191 159 127 0>; default-brightness-level = <6>; /* BKL1_ON */ diff --git a/arch/arm/boot/dts/tegra124-apalis.dtsi b/arch/arm/boot/dts/tegra124-apalis.dtsi index b7648ce4565d..65a2161b9b8e 100644 --- a/arch/arm/boot/dts/tegra124-apalis.dtsi +++ b/arch/arm/boot/dts/tegra124-apalis.dtsi @@ -1,5 +1,5 @@ /* - * Copyright 2016 Toradex AG + * Copyright 2016-2018 Toradex AG * * This file is dual-licensed: you can use it either under the terms * of the GPL or the X11 license, at your option. Note that this dual @@ -56,7 +56,6 @@ pcie@1003000 { status = "okay"; - avddio-pex-supply = <&vdd_1v05>; avdd-pex-pll-supply = <&vdd_1v05>; avdd-pll-erefe-supply = <&avdd_1v05>; @@ -85,7 +84,6 @@ hdmi@5428 { pll-supply = <®_1v05_avdd_hdmi_pll>; vdd-supply = <®_3v3_avdd_hdmi>; - nvidia,ddc-i2c-bus = <&hdmi_ddc>; nvidia,hpd-gpio = <&gpio TEGRA_GPIO(N, 7) GPIO_ACTIVE_HIGH>; @@ -1600,15 +1598,11 @@ compatible = "ams,as3722"; reg = <0x40>; interrupts = <0 86 IRQ_TYPE_LEVEL_HIGH>; - ams,system-power-controller; - #interrupt-cells = <2>; interrupt-controller; - gpio-controller; #gpio-cells = <2>; - pinctrl-names = "default"; pinctrl-0 = <&as3722_default>; @@ -1783,7 +1777,6 @@ reg = <0x4c>; interrupt-parent = <&gpio>; interrupts = ; - #thermal-sensor-cells = <1>; }; }; @@ -1816,7 +1809,6 @@ sata@7002 { phys = <&{/padctl@7009f000/pads/sata/lanes/sata-0}>; phy-names = "sata-0"; - avdd-supply = <&vdd_1v05>; hvdd-supply = <®_3v3>; vddio-supply = <&vdd_1v05>; @@ -1830,7 +1822,6 @@ <&{/padctl@7009f000/pads/usb2/lanes/usb2-2}>, <&{/padctl@7009f000/pads/pcie/lanes/pcie-0}>; phy-names = "usb2-0", "usb3-1", "usb2-1", "usb2-2", "usb3-0"; - avddio-pex-supply = <&vdd_1v05>; avdd-pll-erefe-supply = <&avdd_1v05>; avdd-pll-utmip-supply = <&vddio_1v8>; -- 2.14.3