* [PATCH v2] ARM: dts: tegra: Add support for 256 MB Colibri-T20 (plus such board on Iris)
@ 2018-05-03 15:08 Krzysztof Kozlowski
2018-05-04 9:19 ` Stefan Agner
2018-05-07 16:58 ` Rob Herring
0 siblings, 2 replies; 9+ messages in thread
From: Krzysztof Kozlowski @ 2018-05-03 15:08 UTC (permalink / raw)
To: Rob Herring, Mark Rutland, Thierry Reding, Jonathan Hunter,
devicetree, linux-tegra, linux-kernel
Cc: Marcel Ziswiler, Stefan Agner, Lucas Stach, Krzysztof Kozlowski
Colibri-T20 can come in 256 MB RAM (with 512 MB NAND) or 512 MB RAM
(with 1024 MB NAND) flavors. Add support for the 256 MB version on Iris
evaluation board.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
Changes since v1:
1. Fix memory size in tegra20-colibri-256.dtsi (was working fine because
my bootloader uses mem= argument).
---
Documentation/devicetree/bindings/arm/tegra.txt | 1 +
arch/arm/boot/dts/Makefile | 1 +
arch/arm/boot/dts/tegra20-colibri-256.dtsi | 11 +
arch/arm/boot/dts/tegra20-colibri-512.dtsi | 531 +--------------------
...gra20-colibri-512.dtsi => tegra20-colibri.dtsi} | 4 -
arch/arm/boot/dts/tegra20-iris-256.dts | 10 +
arch/arm/boot/dts/tegra20-iris-512.dts | 98 +---
...gra20-iris-512.dts => tegra20-iris-common.dtsi} | 11 +-
8 files changed, 31 insertions(+), 636 deletions(-)
create mode 100644 arch/arm/boot/dts/tegra20-colibri-256.dtsi
copy arch/arm/boot/dts/{tegra20-colibri-512.dtsi => tegra20-colibri.dtsi} (99%)
create mode 100644 arch/arm/boot/dts/tegra20-iris-256.dts
copy arch/arm/boot/dts/{tegra20-iris-512.dts => tegra20-iris-common.dtsi} (85%)
diff --git a/Documentation/devicetree/bindings/arm/tegra.txt b/Documentation/devicetree/bindings/arm/tegra.txt
index 32f62bb7006d..d304574ae441 100644
--- a/Documentation/devicetree/bindings/arm/tegra.txt
+++ b/Documentation/devicetree/bindings/arm/tegra.txt
@@ -49,6 +49,7 @@ board-specific compatible values:
toradex,apalis_t30-eval
toradex,apalis-tk1
toradex,apalis-tk1-eval
+ toradex,colibri_t20-256
toradex,colibri_t20-512
toradex,colibri_t30
toradex,colibri_t30-eval-v3
diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
index da7d08c3ae98..d63214c6967c 100644
--- a/arch/arm/boot/dts/Makefile
+++ b/arch/arm/boot/dts/Makefile
@@ -1031,6 +1031,7 @@ dtb-$(CONFIG_ARCH_TANGO) += \
tango4-vantage-1172.dtb
dtb-$(CONFIG_ARCH_TEGRA_2x_SOC) += \
tegra20-harmony.dtb \
+ tegra20-iris-256.dtb \
tegra20-iris-512.dtb \
tegra20-medcom-wide.dtb \
tegra20-paz00.dtb \
diff --git a/arch/arm/boot/dts/tegra20-colibri-256.dtsi b/arch/arm/boot/dts/tegra20-colibri-256.dtsi
new file mode 100644
index 000000000000..9c4b0975628f
--- /dev/null
+++ b/arch/arm/boot/dts/tegra20-colibri-256.dtsi
@@ -0,0 +1,11 @@
+// SPDX-License-Identifier: GPL-2.0
+#include "tegra20-colibri.dtsi"
+
+/ {
+ model = "Toradex Colibri T20 256MB";
+ compatible = "toradex,colibri_t20-256", "nvidia,tegra20";
+
+ memory {
+ reg = <0x00000000 0x10000000>;
+ };
+};
diff --git a/arch/arm/boot/dts/tegra20-colibri-512.dtsi b/arch/arm/boot/dts/tegra20-colibri-512.dtsi
index 5c202b3e3bb1..2011382c9c3d 100644
--- a/arch/arm/boot/dts/tegra20-colibri-512.dtsi
+++ b/arch/arm/boot/dts/tegra20-colibri-512.dtsi
@@ -1,540 +1,11 @@
// SPDX-License-Identifier: GPL-2.0
-#include "tegra20.dtsi"
+#include "tegra20-colibri.dtsi"
/ {
model = "Toradex Colibri T20 512MB";
compatible = "toradex,colibri_t20-512", "nvidia,tegra20";
- aliases {
- rtc0 = "/i2c@7000d000/tps6586x@34";
- rtc1 = "/rtc@7000e000";
- };
-
memory {
reg = <0x00000000 0x20000000>;
};
-
- host1x@50000000 {
- hdmi@54280000 {
- vdd-supply = <&hdmi_vdd_reg>;
- pll-supply = <&hdmi_pll_reg>;
-
- nvidia,ddc-i2c-bus = <&i2c_ddc>;
- nvidia,hpd-gpio = <&gpio TEGRA_GPIO(N, 7)
- GPIO_ACTIVE_HIGH>;
- };
- };
-
- pinmux@70000014 {
- pinctrl-names = "default";
- pinctrl-0 = <&state_default>;
-
- state_default: pinmux {
- audio_refclk {
- nvidia,pins = "cdev1";
- nvidia,function = "plla_out";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_DISABLE>;
- };
- crt {
- nvidia,pins = "crtp";
- nvidia,function = "crt";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_ENABLE>;
- };
- dap3 {
- nvidia,pins = "dap3";
- nvidia,function = "dap3";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_DISABLE>;
- };
- displaya {
- nvidia,pins = "ld0", "ld1", "ld2", "ld3",
- "ld4", "ld5", "ld6", "ld7", "ld8",
- "ld9", "ld10", "ld11", "ld12", "ld13",
- "ld14", "ld15", "ld16", "ld17",
- "lhs", "lpw0", "lpw2", "lsc0",
- "lsc1", "lsck", "lsda", "lspi", "lvs";
- nvidia,function = "displaya";
- nvidia,tristate = <TEGRA_PIN_ENABLE>;
- };
- gpio_dte {
- nvidia,pins = "dte";
- nvidia,function = "rsvd1";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_DISABLE>;
- };
- gpio_gmi {
- nvidia,pins = "ata", "atc", "atd", "ate",
- "dap1", "dap2", "dap4", "gpu", "irrx",
- "irtx", "spia", "spib", "spic";
- nvidia,function = "gmi";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_DISABLE>;
- };
- gpio_pta {
- nvidia,pins = "pta";
- nvidia,function = "rsvd4";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_DISABLE>;
- };
- gpio_uac {
- nvidia,pins = "uac";
- nvidia,function = "rsvd2";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_DISABLE>;
- };
- hdint {
- nvidia,pins = "hdint";
- nvidia,function = "hdmi";
- nvidia,tristate = <TEGRA_PIN_ENABLE>;
- };
- i2c1 {
- nvidia,pins = "rm";
- nvidia,function = "i2c1";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_ENABLE>;
- };
- i2c3 {
- nvidia,pins = "dtf";
- nvidia,function = "i2c3";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_ENABLE>;
- };
- i2cddc {
- nvidia,pins = "ddc";
- nvidia,function = "i2c2";
- nvidia,pull = <TEGRA_PIN_PULL_UP>;
- nvidia,tristate = <TEGRA_PIN_ENABLE>;
- };
- i2cp {
- nvidia,pins = "i2cp";
- nvidia,function = "i2cp";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_DISABLE>;
- };
- irda {
- nvidia,pins = "uad";
- nvidia,function = "irda";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_ENABLE>;
- };
- nand {
- nvidia,pins = "kbca", "kbcc", "kbcd",
- "kbce", "kbcf";
- nvidia,function = "nand";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_DISABLE>;
- };
- owc {
- nvidia,pins = "owc";
- nvidia,function = "owr";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_ENABLE>;
- };
- pmc {
- nvidia,pins = "pmc";
- nvidia,function = "pwr_on";
- nvidia,tristate = <TEGRA_PIN_DISABLE>;
- };
- pwm {
- nvidia,pins = "sdb", "sdc", "sdd";
- nvidia,function = "pwm";
- nvidia,tristate = <TEGRA_PIN_ENABLE>;
- };
- sdio4 {
- nvidia,pins = "atb", "gma", "gme";
- nvidia,function = "sdio4";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_ENABLE>;
- };
- spi1 {
- nvidia,pins = "spid", "spie", "spif";
- nvidia,function = "spi1";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_ENABLE>;
- };
- spi4 {
- nvidia,pins = "slxa", "slxc", "slxd", "slxk";
- nvidia,function = "spi4";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_ENABLE>;
- };
- uarta {
- nvidia,pins = "sdio1";
- nvidia,function = "uarta";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_ENABLE>;
- };
- uartd {
- nvidia,pins = "gmc";
- nvidia,function = "uartd";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_ENABLE>;
- };
- ulpi {
- nvidia,pins = "uaa", "uab", "uda";
- nvidia,function = "ulpi";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_DISABLE>;
- };
- ulpi_refclk {
- nvidia,pins = "cdev2";
- nvidia,function = "pllp_out4";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_DISABLE>;
- };
- usb_gpio {
- nvidia,pins = "spig", "spih";
- nvidia,function = "spi2_alt";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_DISABLE>;
- };
- vi {
- nvidia,pins = "dta", "dtb", "dtc", "dtd";
- nvidia,function = "vi";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_ENABLE>;
- };
- vi_sc {
- nvidia,pins = "csus";
- nvidia,function = "vi_sensor_clk";
- nvidia,pull = <TEGRA_PIN_PULL_NONE>;
- nvidia,tristate = <TEGRA_PIN_ENABLE>;
- };
- };
- };
-
- ac97: ac97@70002000 {
- status = "okay";
- nvidia,codec-reset-gpio = <&gpio TEGRA_GPIO(V, 0)
- GPIO_ACTIVE_HIGH>;
- nvidia,codec-sync-gpio = <&gpio TEGRA_GPIO(P, 0)
- GPIO_ACTIVE_HIGH>;
- };
-
- /*
- * GEN1_I2C: I2C_SDA/SCL on SODIMM pin 194/196 (e.g. RTC on carrier
- * board)
- */
- i2c@7000c000 {
- clock-frequency = <400000>;
- };
-
- /* DDC_SCL/SDA on X3 pin 15/16 (e.g. display EDID) */
- i2c_ddc: i2c@7000c400 {
- clock-frequency = <10000>;
- };
-
- /* GEN2_I2C: unused */
-
- /* CAM/GEN3_I2C: used as EXT_IO1/2 GPIOs on SODIMM pin 133/127 */
-
- /* PWR_I2C: power I2C to PMIC and temperature sensor (On-module) */
- i2c@7000d000 {
- status = "okay";
- clock-frequency = <100000>;
-
- pmic: tps6586x@34 {
- compatible = "ti,tps6586x";
- reg = <0x34>;
- interrupts = <GIC_SPI 86 IRQ_TYPE_LEVEL_HIGH>;
-
- ti,system-power-controller;
-
- #gpio-cells = <2>;
- gpio-controller;
-
- sys-supply = <&vdd_3v3_reg>;
- vin-sm0-supply = <&sys_reg>;
- vin-sm1-supply = <&sys_reg>;
- vin-sm2-supply = <&sys_reg>;
- vinldo01-supply = <&sm2_reg>;
- vinldo23-supply = <&vdd_3v3_reg>;
- vinldo4-supply = <&vdd_3v3_reg>;
- vinldo678-supply = <&vdd_3v3_reg>;
- vinldo9-supply = <&vdd_3v3_reg>;
-
- regulators {
- #address-cells = <1>;
- #size-cells = <0>;
-
- sys_reg: regulator@0 {
- reg = <0>;
- regulator-compatible = "sys";
- regulator-name = "vdd_sys";
- regulator-always-on;
- };
-
- regulator@1 {
- reg = <1>;
- regulator-compatible = "sm0";
- regulator-name = "vdd_sm0,vdd_core";
- regulator-min-microvolt = <1200000>;
- regulator-max-microvolt = <1200000>;
- regulator-always-on;
- };
-
- regulator@2 {
- reg = <2>;
- regulator-compatible = "sm1";
- regulator-name = "vdd_sm1,vdd_cpu";
- regulator-min-microvolt = <1000000>;
- regulator-max-microvolt = <1000000>;
- regulator-always-on;
- };
-
- sm2_reg: regulator@3 {
- reg = <3>;
- regulator-compatible = "sm2";
- regulator-name = "vdd_sm2,vin_ldo*";
- regulator-min-microvolt = <1800000>;
- regulator-max-microvolt = <1800000>;
- regulator-always-on;
- };
-
- /* LDO0 is not connected to anything */
-
- regulator@5 {
- reg = <5>;
- regulator-compatible = "ldo1";
- regulator-name = "vdd_ldo1,avdd_pll*";
- regulator-min-microvolt = <1100000>;
- regulator-max-microvolt = <1100000>;
- regulator-always-on;
- };
-
- regulator@6 {
- reg = <6>;
- regulator-compatible = "ldo2";
- regulator-name = "vdd_ldo2,vdd_rtc";
- regulator-min-microvolt = <1200000>;
- regulator-max-microvolt = <1200000>;
- };
-
- /* LDO3 is not connected to anything */
-
- regulator@8 {
- reg = <8>;
- regulator-compatible = "ldo4";
- regulator-name = "vdd_ldo4,avdd_osc,vddio_sys";
- regulator-min-microvolt = <1800000>;
- regulator-max-microvolt = <1800000>;
- regulator-always-on;
- };
-
- ldo5_reg: regulator@9 {
- reg = <9>;
- regulator-compatible = "ldo5";
- regulator-name = "vdd_ldo5,vdd_fuse";
- regulator-min-microvolt = <3300000>;
- regulator-max-microvolt = <3300000>;
- regulator-always-on;
- };
-
- regulator@10 {
- reg = <10>;
- regulator-compatible = "ldo6";
- regulator-name = "vdd_ldo6,avdd_vdac,vddio_vi,vddio_cam";
- regulator-min-microvolt = <2850000>;
- regulator-max-microvolt = <2850000>;
- };
-
- hdmi_vdd_reg: regulator@11 {
- reg = <11>;
- regulator-compatible = "ldo7";
- regulator-name = "vdd_ldo7,avdd_hdmi";
- regulator-min-microvolt = <3300000>;
- regulator-max-microvolt = <3300000>;
- };
-
- hdmi_pll_reg: regulator@12 {
- reg = <12>;
- regulator-compatible = "ldo8";
- regulator-name = "vdd_ldo8,avdd_hdmi_pll";
- regulator-min-microvolt = <1800000>;
- regulator-max-microvolt = <1800000>;
- };
-
- regulator@13 {
- reg = <13>;
- regulator-compatible = "ldo9";
- regulator-name = "vdd_ldo9,avdd_2v85,vdd_ddr_rx";
- regulator-min-microvolt = <2850000>;
- regulator-max-microvolt = <2850000>;
- regulator-always-on;
- };
-
- regulator@14 {
- reg = <14>;
- regulator-compatible = "ldo_rtc";
- regulator-name = "vdd_rtc_out,vdd_cell";
- regulator-min-microvolt = <3300000>;
- regulator-max-microvolt = <3300000>;
- regulator-always-on;
- };
- };
- };
-
- temperature-sensor@4c {
- compatible = "national,lm95245";
- reg = <0x4c>;
- };
- };
-
- pmc@7000e400 {
- nvidia,suspend-mode = <1>;
- nvidia,cpu-pwr-good-time = <5000>;
- nvidia,cpu-pwr-off-time = <5000>;
- nvidia,core-pwr-good-time = <3845 3845>;
- nvidia,core-pwr-off-time = <3875>;
- nvidia,sys-clock-req-active-high;
- };
-
- memory-controller@7000f400 {
- emc-table@83250 {
- reg = <83250>;
- compatible = "nvidia,tegra20-emc-table";
- clock-frequency = <83250>;
- nvidia,emc-registers = <0x00000005 0x00000011
- 0x00000004 0x00000002 0x00000004 0x00000004
- 0x00000001 0x0000000a 0x00000002 0x00000002
- 0x00000001 0x00000001 0x00000003 0x00000004
- 0x00000003 0x00000009 0x0000000c 0x0000025f
- 0x00000000 0x00000003 0x00000003 0x00000002
- 0x00000002 0x00000001 0x00000008 0x000000c8
- 0x00000003 0x00000005 0x00000003 0x0000000c
- 0x00000002 0x00000000 0x00000000 0x00000002
- 0x00000000 0x00000000 0x00000083 0x00520006
- 0x00000010 0x00000008 0x00000000 0x00000000
- 0x00000000 0x00000000 0x00000000 0x00000000>;
- };
- emc-table@133200 {
- reg = <133200>;
- compatible = "nvidia,tegra20-emc-table";
- clock-frequency = <133200>;
- nvidia,emc-registers = <0x00000008 0x00000019
- 0x00000006 0x00000002 0x00000004 0x00000004
- 0x00000001 0x0000000a 0x00000002 0x00000002
- 0x00000002 0x00000001 0x00000003 0x00000004
- 0x00000003 0x00000009 0x0000000c 0x0000039f
- 0x00000000 0x00000003 0x00000003 0x00000002
- 0x00000002 0x00000001 0x00000008 0x000000c8
- 0x00000003 0x00000007 0x00000003 0x0000000c
- 0x00000002 0x00000000 0x00000000 0x00000002
- 0x00000000 0x00000000 0x00000083 0x00510006
- 0x00000010 0x00000008 0x00000000 0x00000000
- 0x00000000 0x00000000 0x00000000 0x00000000>;
- };
- emc-table@166500 {
- reg = <166500>;
- compatible = "nvidia,tegra20-emc-table";
- clock-frequency = <166500>;
- nvidia,emc-registers = <0x0000000a 0x00000021
- 0x00000008 0x00000003 0x00000004 0x00000004
- 0x00000002 0x0000000a 0x00000003 0x00000003
- 0x00000002 0x00000001 0x00000003 0x00000004
- 0x00000003 0x00000009 0x0000000c 0x000004df
- 0x00000000 0x00000003 0x00000003 0x00000003
- 0x00000003 0x00000001 0x00000009 0x000000c8
- 0x00000003 0x00000009 0x00000004 0x0000000c
- 0x00000002 0x00000000 0x00000000 0x00000002
- 0x00000000 0x00000000 0x00000083 0x004f0006
- 0x00000010 0x00000008 0x00000000 0x00000000
- 0x00000000 0x00000000 0x00000000 0x00000000>;
- };
- emc-table@333000 {
- reg = <333000>;
- compatible = "nvidia,tegra20-emc-table";
- clock-frequency = <333000>;
- nvidia,emc-registers = <0x00000014 0x00000041
- 0x0000000f 0x00000005 0x00000004 0x00000005
- 0x00000003 0x0000000a 0x00000005 0x00000005
- 0x00000004 0x00000001 0x00000003 0x00000004
- 0x00000003 0x00000009 0x0000000c 0x000009ff
- 0x00000000 0x00000003 0x00000003 0x00000005
- 0x00000005 0x00000001 0x0000000e 0x000000c8
- 0x00000003 0x00000011 0x00000006 0x0000000c
- 0x00000002 0x00000000 0x00000000 0x00000002
- 0x00000000 0x00000000 0x00000083 0x00380006
- 0x00000010 0x00000008 0x00000000 0x00000000
- 0x00000000 0x00000000 0x00000000 0x00000000>;
- };
- };
-
- usb@c5004000 {
- status = "okay";
- nvidia,phy-reset-gpio = <&gpio TEGRA_GPIO(V, 1)
- GPIO_ACTIVE_LOW>;
- };
-
- usb-phy@c5004000 {
- status = "okay";
- nvidia,phy-reset-gpio = <&gpio TEGRA_GPIO(V, 1)
- GPIO_ACTIVE_LOW>;
- };
-
- sdhci@c8000600 {
- cd-gpios = <&gpio TEGRA_GPIO(C, 7) GPIO_ACTIVE_LOW>;
- };
-
- clocks {
- compatible = "simple-bus";
- #address-cells = <1>;
- #size-cells = <0>;
-
- clk32k_in: clock@0 {
- compatible = "fixed-clock";
- reg = <0>;
- #clock-cells = <0>;
- clock-frequency = <32768>;
- };
- };
-
- regulators {
- compatible = "simple-bus";
- #address-cells = <1>;
- #size-cells = <0>;
-
- vdd_3v3_reg: regulator@100 {
- compatible = "regulator-fixed";
- reg = <100>;
- regulator-name = "vdd_3v3";
- regulator-min-microvolt = <3300000>;
- regulator-max-microvolt = <3300000>;
- regulator-always-on;
- };
-
- regulator@101 {
- compatible = "regulator-fixed";
- reg = <101>;
- regulator-name = "internal_usb";
- regulator-min-microvolt = <5000000>;
- regulator-max-microvolt = <5000000>;
- enable-active-high;
- regulator-boot-on;
- regulator-always-on;
- gpio = <&gpio TEGRA_GPIO(BB, 1) GPIO_ACTIVE_HIGH>;
- };
- };
-
- sound {
- compatible = "nvidia,tegra-audio-wm9712-colibri_t20",
- "nvidia,tegra-audio-wm9712";
- nvidia,model = "Colibri T20 AC97 Audio";
-
- nvidia,audio-routing =
- "Headphone", "HPOUTL",
- "Headphone", "HPOUTR",
- "LineIn", "LINEINL",
- "LineIn", "LINEINR",
- "Mic", "MIC1";
-
- nvidia,ac97-controller = <&ac97>;
-
- clocks = <&tegra_car TEGRA20_CLK_PLL_A>,
- <&tegra_car TEGRA20_CLK_PLL_A_OUT0>,
- <&tegra_car TEGRA20_CLK_CDEV1>;
- clock-names = "pll_a", "pll_a_out0", "mclk";
- };
};
diff --git a/arch/arm/boot/dts/tegra20-colibri-512.dtsi b/arch/arm/boot/dts/tegra20-colibri.dtsi
similarity index 99%
copy from arch/arm/boot/dts/tegra20-colibri-512.dtsi
copy to arch/arm/boot/dts/tegra20-colibri.dtsi
index 5c202b3e3bb1..a567ea6d442f 100644
--- a/arch/arm/boot/dts/tegra20-colibri-512.dtsi
+++ b/arch/arm/boot/dts/tegra20-colibri.dtsi
@@ -10,10 +10,6 @@
rtc1 = "/rtc@7000e000";
};
- memory {
- reg = <0x00000000 0x20000000>;
- };
-
host1x@50000000 {
hdmi@54280000 {
vdd-supply = <&hdmi_vdd_reg>;
diff --git a/arch/arm/boot/dts/tegra20-iris-256.dts b/arch/arm/boot/dts/tegra20-iris-256.dts
new file mode 100644
index 000000000000..f2df58c81493
--- /dev/null
+++ b/arch/arm/boot/dts/tegra20-iris-256.dts
@@ -0,0 +1,10 @@
+// SPDX-License-Identifier: GPL-2.0
+/dts-v1/;
+
+#include "tegra20-colibri-256.dtsi"
+#include "tegra20-iris-common.dtsi"
+
+/ {
+ model = "Toradex Colibri T20 256MB on Iris";
+ compatible = "toradex,iris", "toradex,colibri_t20-256", "nvidia,tegra20";
+};
diff --git a/arch/arm/boot/dts/tegra20-iris-512.dts b/arch/arm/boot/dts/tegra20-iris-512.dts
index 40126388946d..6841e5af6f26 100644
--- a/arch/arm/boot/dts/tegra20-iris-512.dts
+++ b/arch/arm/boot/dts/tegra20-iris-512.dts
@@ -2,105 +2,9 @@
/dts-v1/;
#include "tegra20-colibri-512.dtsi"
+#include "tegra20-iris-common.dtsi"
/ {
model = "Toradex Colibri T20 512MB on Iris";
compatible = "toradex,iris", "toradex,colibri_t20-512", "nvidia,tegra20";
-
- aliases {
- serial0 = &uarta;
- serial1 = &uartd;
- };
-
- chosen {
- stdout-path = "serial0:115200n8";
- };
-
- host1x@50000000 {
- hdmi@54280000 {
- status = "okay";
- };
- };
-
- pinmux@70000014 {
- state_default: pinmux {
- hdint {
- nvidia,tristate = <TEGRA_PIN_DISABLE>;
- };
-
- i2cddc {
- nvidia,tristate = <TEGRA_PIN_DISABLE>;
- };
-
- sdio4 {
- nvidia,tristate = <TEGRA_PIN_DISABLE>;
- };
-
- uarta {
- nvidia,tristate = <TEGRA_PIN_DISABLE>;
- };
-
- uartd {
- nvidia,tristate = <TEGRA_PIN_DISABLE>;
- };
- };
- };
-
- serial@70006000 {
- status = "okay";
- };
-
- serial@70006300 {
- status = "okay";
- };
-
- i2c_ddc: i2c@7000c400 {
- status = "okay";
- };
-
- usb@c5000000 {
- status = "okay";
- };
-
- usb-phy@c5000000 {
- status = "okay";
- };
-
- usb@c5008000 {
- status = "okay";
- };
-
- usb-phy@c5008000 {
- status = "okay";
- };
-
- sdhci@c8000600 {
- status = "okay";
- bus-width = <4>;
- vmmc-supply = <&vcc_sd_reg>;
- vqmmc-supply = <&vcc_sd_reg>;
- };
-
- regulators {
- regulator@0 {
- compatible = "regulator-fixed";
- reg = <0>;
- regulator-name = "usb_host_vbus";
- regulator-min-microvolt = <5000000>;
- regulator-max-microvolt = <5000000>;
- regulator-boot-on;
- regulator-always-on;
- gpio = <&gpio TEGRA_GPIO(W, 2) GPIO_ACTIVE_HIGH>;
- };
-
- vcc_sd_reg: regulator@1 {
- compatible = "regulator-fixed";
- reg = <1>;
- regulator-name = "vcc_sd";
- regulator-min-microvolt = <3300000>;
- regulator-max-microvolt = <3300000>;
- regulator-boot-on;
- regulator-always-on;
- };
- };
};
diff --git a/arch/arm/boot/dts/tegra20-iris-512.dts b/arch/arm/boot/dts/tegra20-iris-common.dtsi
similarity index 85%
copy from arch/arm/boot/dts/tegra20-iris-512.dts
copy to arch/arm/boot/dts/tegra20-iris-common.dtsi
index 40126388946d..bbc6ab22526e 100644
--- a/arch/arm/boot/dts/tegra20-iris-512.dts
+++ b/arch/arm/boot/dts/tegra20-iris-common.dtsi
@@ -1,11 +1,12 @@
// SPDX-License-Identifier: GPL-2.0
-/dts-v1/;
-
-#include "tegra20-colibri-512.dtsi"
+//
+// This DTSI does not include specific Colibri DTSI but just configures
+// common Iris properties. This DTSI shall be included after including
+// specific Colibri DTSI.
/ {
- model = "Toradex Colibri T20 512MB on Iris";
- compatible = "toradex,iris", "toradex,colibri_t20-512", "nvidia,tegra20";
+ model = "Toradex Colibri T20 256MB/512MB on Iris";
+ compatible = "toradex,iris", "nvidia,tegra20";
aliases {
serial0 = &uarta;
--
2.7.4
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v2] ARM: dts: tegra: Add support for 256 MB Colibri-T20 (plus such board on Iris)
2018-05-03 15:08 [PATCH v2] ARM: dts: tegra: Add support for 256 MB Colibri-T20 (plus such board on Iris) Krzysztof Kozlowski
@ 2018-05-04 9:19 ` Stefan Agner
2018-05-04 9:37 ` Marcel Ziswiler
2018-05-04 10:03 ` Krzysztof Kozlowski
2018-05-07 16:58 ` Rob Herring
1 sibling, 2 replies; 9+ messages in thread
From: Stefan Agner @ 2018-05-04 9:19 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Rob Herring, Mark Rutland, Thierry Reding, Jonathan Hunter,
devicetree, linux-tegra, linux-kernel, Marcel Ziswiler,
Lucas Stach
On 03.05.2018 17:08, Krzysztof Kozlowski wrote:
> Colibri-T20 can come in 256 MB RAM (with 512 MB NAND) or 512 MB RAM
> (with 1024 MB NAND) flavors. Add support for the 256 MB version on Iris
> evaluation board.
To we really need to specify memory size these days? I think all common
boot loaders fill the memory node anyway.
arch/arm/boot/dts/imx6qdl-apalis.dtsi:
/* Will be filled by the bootloader */
memory@10000000 {
reg = <0x10000000 0>;
};
I think we should just rename tegra20-colibri-512.dtsi =>
tegra20-colibri.dtsi and tegra20-iris-512.dts => tegra20-iris.dts to
avoid confusion and add such an empty node.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
>
> ---
>
> Changes since v1:
> 1. Fix memory size in tegra20-colibri-256.dtsi (was working fine because
> my bootloader uses mem= argument).
What boot loader are you using? You even can omit mem= if your boot
loader fills the memory node properly.
--
Stefan
> ---
> Documentation/devicetree/bindings/arm/tegra.txt | 1 +
> arch/arm/boot/dts/Makefile | 1 +
> arch/arm/boot/dts/tegra20-colibri-256.dtsi | 11 +
> arch/arm/boot/dts/tegra20-colibri-512.dtsi | 531 +--------------------
> ...gra20-colibri-512.dtsi => tegra20-colibri.dtsi} | 4 -
> arch/arm/boot/dts/tegra20-iris-256.dts | 10 +
> arch/arm/boot/dts/tegra20-iris-512.dts | 98 +---
> ...gra20-iris-512.dts => tegra20-iris-common.dtsi} | 11 +-
> 8 files changed, 31 insertions(+), 636 deletions(-)
> create mode 100644 arch/arm/boot/dts/tegra20-colibri-256.dtsi
> copy arch/arm/boot/dts/{tegra20-colibri-512.dtsi => tegra20-colibri.dtsi} (99%)
> create mode 100644 arch/arm/boot/dts/tegra20-iris-256.dts
> copy arch/arm/boot/dts/{tegra20-iris-512.dts => tegra20-iris-common.dtsi} (85%)
>
> diff --git a/Documentation/devicetree/bindings/arm/tegra.txt
> b/Documentation/devicetree/bindings/arm/tegra.txt
> index 32f62bb7006d..d304574ae441 100644
> --- a/Documentation/devicetree/bindings/arm/tegra.txt
> +++ b/Documentation/devicetree/bindings/arm/tegra.txt
> @@ -49,6 +49,7 @@ board-specific compatible values:
> toradex,apalis_t30-eval
> toradex,apalis-tk1
> toradex,apalis-tk1-eval
> + toradex,colibri_t20-256
> toradex,colibri_t20-512
> toradex,colibri_t30
> toradex,colibri_t30-eval-v3
> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index da7d08c3ae98..d63214c6967c 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -1031,6 +1031,7 @@ dtb-$(CONFIG_ARCH_TANGO) += \
> tango4-vantage-1172.dtb
> dtb-$(CONFIG_ARCH_TEGRA_2x_SOC) += \
> tegra20-harmony.dtb \
> + tegra20-iris-256.dtb \
> tegra20-iris-512.dtb \
> tegra20-medcom-wide.dtb \
> tegra20-paz00.dtb \
> diff --git a/arch/arm/boot/dts/tegra20-colibri-256.dtsi
> b/arch/arm/boot/dts/tegra20-colibri-256.dtsi
> new file mode 100644
> index 000000000000..9c4b0975628f
> --- /dev/null
> +++ b/arch/arm/boot/dts/tegra20-colibri-256.dtsi
> @@ -0,0 +1,11 @@
> +// SPDX-License-Identifier: GPL-2.0
> +#include "tegra20-colibri.dtsi"
> +
> +/ {
> + model = "Toradex Colibri T20 256MB";
> + compatible = "toradex,colibri_t20-256", "nvidia,tegra20";
> +
> + memory {
> + reg = <0x00000000 0x10000000>;
> + };
> +};
> diff --git a/arch/arm/boot/dts/tegra20-colibri-512.dtsi
> b/arch/arm/boot/dts/tegra20-colibri-512.dtsi
> index 5c202b3e3bb1..2011382c9c3d 100644
> --- a/arch/arm/boot/dts/tegra20-colibri-512.dtsi
> +++ b/arch/arm/boot/dts/tegra20-colibri-512.dtsi
> @@ -1,540 +1,11 @@
> // SPDX-License-Identifier: GPL-2.0
> -#include "tegra20.dtsi"
> +#include "tegra20-colibri.dtsi"
>
> / {
> model = "Toradex Colibri T20 512MB";
> compatible = "toradex,colibri_t20-512", "nvidia,tegra20";
>
> - aliases {
> - rtc0 = "/i2c@7000d000/tps6586x@34";
> - rtc1 = "/rtc@7000e000";
> - };
> -
> memory {
> reg = <0x00000000 0x20000000>;
> };
> -
> - host1x@50000000 {
> - hdmi@54280000 {
> - vdd-supply = <&hdmi_vdd_reg>;
> - pll-supply = <&hdmi_pll_reg>;
> -
> - nvidia,ddc-i2c-bus = <&i2c_ddc>;
> - nvidia,hpd-gpio = <&gpio TEGRA_GPIO(N, 7)
> - GPIO_ACTIVE_HIGH>;
> - };
> - };
> -
> - pinmux@70000014 {
> - pinctrl-names = "default";
> - pinctrl-0 = <&state_default>;
> -
> - state_default: pinmux {
> - audio_refclk {
> - nvidia,pins = "cdev1";
> - nvidia,function = "plla_out";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_DISABLE>;
> - };
> - crt {
> - nvidia,pins = "crtp";
> - nvidia,function = "crt";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_ENABLE>;
> - };
> - dap3 {
> - nvidia,pins = "dap3";
> - nvidia,function = "dap3";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_DISABLE>;
> - };
> - displaya {
> - nvidia,pins = "ld0", "ld1", "ld2", "ld3",
> - "ld4", "ld5", "ld6", "ld7", "ld8",
> - "ld9", "ld10", "ld11", "ld12", "ld13",
> - "ld14", "ld15", "ld16", "ld17",
> - "lhs", "lpw0", "lpw2", "lsc0",
> - "lsc1", "lsck", "lsda", "lspi", "lvs";
> - nvidia,function = "displaya";
> - nvidia,tristate = <TEGRA_PIN_ENABLE>;
> - };
> - gpio_dte {
> - nvidia,pins = "dte";
> - nvidia,function = "rsvd1";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_DISABLE>;
> - };
> - gpio_gmi {
> - nvidia,pins = "ata", "atc", "atd", "ate",
> - "dap1", "dap2", "dap4", "gpu", "irrx",
> - "irtx", "spia", "spib", "spic";
> - nvidia,function = "gmi";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_DISABLE>;
> - };
> - gpio_pta {
> - nvidia,pins = "pta";
> - nvidia,function = "rsvd4";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_DISABLE>;
> - };
> - gpio_uac {
> - nvidia,pins = "uac";
> - nvidia,function = "rsvd2";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_DISABLE>;
> - };
> - hdint {
> - nvidia,pins = "hdint";
> - nvidia,function = "hdmi";
> - nvidia,tristate = <TEGRA_PIN_ENABLE>;
> - };
> - i2c1 {
> - nvidia,pins = "rm";
> - nvidia,function = "i2c1";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_ENABLE>;
> - };
> - i2c3 {
> - nvidia,pins = "dtf";
> - nvidia,function = "i2c3";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_ENABLE>;
> - };
> - i2cddc {
> - nvidia,pins = "ddc";
> - nvidia,function = "i2c2";
> - nvidia,pull = <TEGRA_PIN_PULL_UP>;
> - nvidia,tristate = <TEGRA_PIN_ENABLE>;
> - };
> - i2cp {
> - nvidia,pins = "i2cp";
> - nvidia,function = "i2cp";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_DISABLE>;
> - };
> - irda {
> - nvidia,pins = "uad";
> - nvidia,function = "irda";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_ENABLE>;
> - };
> - nand {
> - nvidia,pins = "kbca", "kbcc", "kbcd",
> - "kbce", "kbcf";
> - nvidia,function = "nand";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_DISABLE>;
> - };
> - owc {
> - nvidia,pins = "owc";
> - nvidia,function = "owr";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_ENABLE>;
> - };
> - pmc {
> - nvidia,pins = "pmc";
> - nvidia,function = "pwr_on";
> - nvidia,tristate = <TEGRA_PIN_DISABLE>;
> - };
> - pwm {
> - nvidia,pins = "sdb", "sdc", "sdd";
> - nvidia,function = "pwm";
> - nvidia,tristate = <TEGRA_PIN_ENABLE>;
> - };
> - sdio4 {
> - nvidia,pins = "atb", "gma", "gme";
> - nvidia,function = "sdio4";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_ENABLE>;
> - };
> - spi1 {
> - nvidia,pins = "spid", "spie", "spif";
> - nvidia,function = "spi1";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_ENABLE>;
> - };
> - spi4 {
> - nvidia,pins = "slxa", "slxc", "slxd", "slxk";
> - nvidia,function = "spi4";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_ENABLE>;
> - };
> - uarta {
> - nvidia,pins = "sdio1";
> - nvidia,function = "uarta";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_ENABLE>;
> - };
> - uartd {
> - nvidia,pins = "gmc";
> - nvidia,function = "uartd";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_ENABLE>;
> - };
> - ulpi {
> - nvidia,pins = "uaa", "uab", "uda";
> - nvidia,function = "ulpi";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_DISABLE>;
> - };
> - ulpi_refclk {
> - nvidia,pins = "cdev2";
> - nvidia,function = "pllp_out4";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_DISABLE>;
> - };
> - usb_gpio {
> - nvidia,pins = "spig", "spih";
> - nvidia,function = "spi2_alt";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_DISABLE>;
> - };
> - vi {
> - nvidia,pins = "dta", "dtb", "dtc", "dtd";
> - nvidia,function = "vi";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_ENABLE>;
> - };
> - vi_sc {
> - nvidia,pins = "csus";
> - nvidia,function = "vi_sensor_clk";
> - nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> - nvidia,tristate = <TEGRA_PIN_ENABLE>;
> - };
> - };
> - };
> -
> - ac97: ac97@70002000 {
> - status = "okay";
> - nvidia,codec-reset-gpio = <&gpio TEGRA_GPIO(V, 0)
> - GPIO_ACTIVE_HIGH>;
> - nvidia,codec-sync-gpio = <&gpio TEGRA_GPIO(P, 0)
> - GPIO_ACTIVE_HIGH>;
> - };
> -
> - /*
> - * GEN1_I2C: I2C_SDA/SCL on SODIMM pin 194/196 (e.g. RTC on carrier
> - * board)
> - */
> - i2c@7000c000 {
> - clock-frequency = <400000>;
> - };
> -
> - /* DDC_SCL/SDA on X3 pin 15/16 (e.g. display EDID) */
> - i2c_ddc: i2c@7000c400 {
> - clock-frequency = <10000>;
> - };
> -
> - /* GEN2_I2C: unused */
> -
> - /* CAM/GEN3_I2C: used as EXT_IO1/2 GPIOs on SODIMM pin 133/127 */
> -
> - /* PWR_I2C: power I2C to PMIC and temperature sensor (On-module) */
> - i2c@7000d000 {
> - status = "okay";
> - clock-frequency = <100000>;
> -
> - pmic: tps6586x@34 {
> - compatible = "ti,tps6586x";
> - reg = <0x34>;
> - interrupts = <GIC_SPI 86 IRQ_TYPE_LEVEL_HIGH>;
> -
> - ti,system-power-controller;
> -
> - #gpio-cells = <2>;
> - gpio-controller;
> -
> - sys-supply = <&vdd_3v3_reg>;
> - vin-sm0-supply = <&sys_reg>;
> - vin-sm1-supply = <&sys_reg>;
> - vin-sm2-supply = <&sys_reg>;
> - vinldo01-supply = <&sm2_reg>;
> - vinldo23-supply = <&vdd_3v3_reg>;
> - vinldo4-supply = <&vdd_3v3_reg>;
> - vinldo678-supply = <&vdd_3v3_reg>;
> - vinldo9-supply = <&vdd_3v3_reg>;
> -
> - regulators {
> - #address-cells = <1>;
> - #size-cells = <0>;
> -
> - sys_reg: regulator@0 {
> - reg = <0>;
> - regulator-compatible = "sys";
> - regulator-name = "vdd_sys";
> - regulator-always-on;
> - };
> -
> - regulator@1 {
> - reg = <1>;
> - regulator-compatible = "sm0";
> - regulator-name = "vdd_sm0,vdd_core";
> - regulator-min-microvolt = <1200000>;
> - regulator-max-microvolt = <1200000>;
> - regulator-always-on;
> - };
> -
> - regulator@2 {
> - reg = <2>;
> - regulator-compatible = "sm1";
> - regulator-name = "vdd_sm1,vdd_cpu";
> - regulator-min-microvolt = <1000000>;
> - regulator-max-microvolt = <1000000>;
> - regulator-always-on;
> - };
> -
> - sm2_reg: regulator@3 {
> - reg = <3>;
> - regulator-compatible = "sm2";
> - regulator-name = "vdd_sm2,vin_ldo*";
> - regulator-min-microvolt = <1800000>;
> - regulator-max-microvolt = <1800000>;
> - regulator-always-on;
> - };
> -
> - /* LDO0 is not connected to anything */
> -
> - regulator@5 {
> - reg = <5>;
> - regulator-compatible = "ldo1";
> - regulator-name = "vdd_ldo1,avdd_pll*";
> - regulator-min-microvolt = <1100000>;
> - regulator-max-microvolt = <1100000>;
> - regulator-always-on;
> - };
> -
> - regulator@6 {
> - reg = <6>;
> - regulator-compatible = "ldo2";
> - regulator-name = "vdd_ldo2,vdd_rtc";
> - regulator-min-microvolt = <1200000>;
> - regulator-max-microvolt = <1200000>;
> - };
> -
> - /* LDO3 is not connected to anything */
> -
> - regulator@8 {
> - reg = <8>;
> - regulator-compatible = "ldo4";
> - regulator-name = "vdd_ldo4,avdd_osc,vddio_sys";
> - regulator-min-microvolt = <1800000>;
> - regulator-max-microvolt = <1800000>;
> - regulator-always-on;
> - };
> -
> - ldo5_reg: regulator@9 {
> - reg = <9>;
> - regulator-compatible = "ldo5";
> - regulator-name = "vdd_ldo5,vdd_fuse";
> - regulator-min-microvolt = <3300000>;
> - regulator-max-microvolt = <3300000>;
> - regulator-always-on;
> - };
> -
> - regulator@10 {
> - reg = <10>;
> - regulator-compatible = "ldo6";
> - regulator-name = "vdd_ldo6,avdd_vdac,vddio_vi,vddio_cam";
> - regulator-min-microvolt = <2850000>;
> - regulator-max-microvolt = <2850000>;
> - };
> -
> - hdmi_vdd_reg: regulator@11 {
> - reg = <11>;
> - regulator-compatible = "ldo7";
> - regulator-name = "vdd_ldo7,avdd_hdmi";
> - regulator-min-microvolt = <3300000>;
> - regulator-max-microvolt = <3300000>;
> - };
> -
> - hdmi_pll_reg: regulator@12 {
> - reg = <12>;
> - regulator-compatible = "ldo8";
> - regulator-name = "vdd_ldo8,avdd_hdmi_pll";
> - regulator-min-microvolt = <1800000>;
> - regulator-max-microvolt = <1800000>;
> - };
> -
> - regulator@13 {
> - reg = <13>;
> - regulator-compatible = "ldo9";
> - regulator-name = "vdd_ldo9,avdd_2v85,vdd_ddr_rx";
> - regulator-min-microvolt = <2850000>;
> - regulator-max-microvolt = <2850000>;
> - regulator-always-on;
> - };
> -
> - regulator@14 {
> - reg = <14>;
> - regulator-compatible = "ldo_rtc";
> - regulator-name = "vdd_rtc_out,vdd_cell";
> - regulator-min-microvolt = <3300000>;
> - regulator-max-microvolt = <3300000>;
> - regulator-always-on;
> - };
> - };
> - };
> -
> - temperature-sensor@4c {
> - compatible = "national,lm95245";
> - reg = <0x4c>;
> - };
> - };
> -
> - pmc@7000e400 {
> - nvidia,suspend-mode = <1>;
> - nvidia,cpu-pwr-good-time = <5000>;
> - nvidia,cpu-pwr-off-time = <5000>;
> - nvidia,core-pwr-good-time = <3845 3845>;
> - nvidia,core-pwr-off-time = <3875>;
> - nvidia,sys-clock-req-active-high;
> - };
> -
> - memory-controller@7000f400 {
> - emc-table@83250 {
> - reg = <83250>;
> - compatible = "nvidia,tegra20-emc-table";
> - clock-frequency = <83250>;
> - nvidia,emc-registers = <0x00000005 0x00000011
> - 0x00000004 0x00000002 0x00000004 0x00000004
> - 0x00000001 0x0000000a 0x00000002 0x00000002
> - 0x00000001 0x00000001 0x00000003 0x00000004
> - 0x00000003 0x00000009 0x0000000c 0x0000025f
> - 0x00000000 0x00000003 0x00000003 0x00000002
> - 0x00000002 0x00000001 0x00000008 0x000000c8
> - 0x00000003 0x00000005 0x00000003 0x0000000c
> - 0x00000002 0x00000000 0x00000000 0x00000002
> - 0x00000000 0x00000000 0x00000083 0x00520006
> - 0x00000010 0x00000008 0x00000000 0x00000000
> - 0x00000000 0x00000000 0x00000000 0x00000000>;
> - };
> - emc-table@133200 {
> - reg = <133200>;
> - compatible = "nvidia,tegra20-emc-table";
> - clock-frequency = <133200>;
> - nvidia,emc-registers = <0x00000008 0x00000019
> - 0x00000006 0x00000002 0x00000004 0x00000004
> - 0x00000001 0x0000000a 0x00000002 0x00000002
> - 0x00000002 0x00000001 0x00000003 0x00000004
> - 0x00000003 0x00000009 0x0000000c 0x0000039f
> - 0x00000000 0x00000003 0x00000003 0x00000002
> - 0x00000002 0x00000001 0x00000008 0x000000c8
> - 0x00000003 0x00000007 0x00000003 0x0000000c
> - 0x00000002 0x00000000 0x00000000 0x00000002
> - 0x00000000 0x00000000 0x00000083 0x00510006
> - 0x00000010 0x00000008 0x00000000 0x00000000
> - 0x00000000 0x00000000 0x00000000 0x00000000>;
> - };
> - emc-table@166500 {
> - reg = <166500>;
> - compatible = "nvidia,tegra20-emc-table";
> - clock-frequency = <166500>;
> - nvidia,emc-registers = <0x0000000a 0x00000021
> - 0x00000008 0x00000003 0x00000004 0x00000004
> - 0x00000002 0x0000000a 0x00000003 0x00000003
> - 0x00000002 0x00000001 0x00000003 0x00000004
> - 0x00000003 0x00000009 0x0000000c 0x000004df
> - 0x00000000 0x00000003 0x00000003 0x00000003
> - 0x00000003 0x00000001 0x00000009 0x000000c8
> - 0x00000003 0x00000009 0x00000004 0x0000000c
> - 0x00000002 0x00000000 0x00000000 0x00000002
> - 0x00000000 0x00000000 0x00000083 0x004f0006
> - 0x00000010 0x00000008 0x00000000 0x00000000
> - 0x00000000 0x00000000 0x00000000 0x00000000>;
> - };
> - emc-table@333000 {
> - reg = <333000>;
> - compatible = "nvidia,tegra20-emc-table";
> - clock-frequency = <333000>;
> - nvidia,emc-registers = <0x00000014 0x00000041
> - 0x0000000f 0x00000005 0x00000004 0x00000005
> - 0x00000003 0x0000000a 0x00000005 0x00000005
> - 0x00000004 0x00000001 0x00000003 0x00000004
> - 0x00000003 0x00000009 0x0000000c 0x000009ff
> - 0x00000000 0x00000003 0x00000003 0x00000005
> - 0x00000005 0x00000001 0x0000000e 0x000000c8
> - 0x00000003 0x00000011 0x00000006 0x0000000c
> - 0x00000002 0x00000000 0x00000000 0x00000002
> - 0x00000000 0x00000000 0x00000083 0x00380006
> - 0x00000010 0x00000008 0x00000000 0x00000000
> - 0x00000000 0x00000000 0x00000000 0x00000000>;
> - };
> - };
> -
> - usb@c5004000 {
> - status = "okay";
> - nvidia,phy-reset-gpio = <&gpio TEGRA_GPIO(V, 1)
> - GPIO_ACTIVE_LOW>;
> - };
> -
> - usb-phy@c5004000 {
> - status = "okay";
> - nvidia,phy-reset-gpio = <&gpio TEGRA_GPIO(V, 1)
> - GPIO_ACTIVE_LOW>;
> - };
> -
> - sdhci@c8000600 {
> - cd-gpios = <&gpio TEGRA_GPIO(C, 7) GPIO_ACTIVE_LOW>;
> - };
> -
> - clocks {
> - compatible = "simple-bus";
> - #address-cells = <1>;
> - #size-cells = <0>;
> -
> - clk32k_in: clock@0 {
> - compatible = "fixed-clock";
> - reg = <0>;
> - #clock-cells = <0>;
> - clock-frequency = <32768>;
> - };
> - };
> -
> - regulators {
> - compatible = "simple-bus";
> - #address-cells = <1>;
> - #size-cells = <0>;
> -
> - vdd_3v3_reg: regulator@100 {
> - compatible = "regulator-fixed";
> - reg = <100>;
> - regulator-name = "vdd_3v3";
> - regulator-min-microvolt = <3300000>;
> - regulator-max-microvolt = <3300000>;
> - regulator-always-on;
> - };
> -
> - regulator@101 {
> - compatible = "regulator-fixed";
> - reg = <101>;
> - regulator-name = "internal_usb";
> - regulator-min-microvolt = <5000000>;
> - regulator-max-microvolt = <5000000>;
> - enable-active-high;
> - regulator-boot-on;
> - regulator-always-on;
> - gpio = <&gpio TEGRA_GPIO(BB, 1) GPIO_ACTIVE_HIGH>;
> - };
> - };
> -
> - sound {
> - compatible = "nvidia,tegra-audio-wm9712-colibri_t20",
> - "nvidia,tegra-audio-wm9712";
> - nvidia,model = "Colibri T20 AC97 Audio";
> -
> - nvidia,audio-routing =
> - "Headphone", "HPOUTL",
> - "Headphone", "HPOUTR",
> - "LineIn", "LINEINL",
> - "LineIn", "LINEINR",
> - "Mic", "MIC1";
> -
> - nvidia,ac97-controller = <&ac97>;
> -
> - clocks = <&tegra_car TEGRA20_CLK_PLL_A>,
> - <&tegra_car TEGRA20_CLK_PLL_A_OUT0>,
> - <&tegra_car TEGRA20_CLK_CDEV1>;
> - clock-names = "pll_a", "pll_a_out0", "mclk";
> - };
> };
> diff --git a/arch/arm/boot/dts/tegra20-colibri-512.dtsi
> b/arch/arm/boot/dts/tegra20-colibri.dtsi
> similarity index 99%
> copy from arch/arm/boot/dts/tegra20-colibri-512.dtsi
> copy to arch/arm/boot/dts/tegra20-colibri.dtsi
> index 5c202b3e3bb1..a567ea6d442f 100644
> --- a/arch/arm/boot/dts/tegra20-colibri-512.dtsi
> +++ b/arch/arm/boot/dts/tegra20-colibri.dtsi
> @@ -10,10 +10,6 @@
> rtc1 = "/rtc@7000e000";
> };
>
> - memory {
> - reg = <0x00000000 0x20000000>;
> - };
> -
> host1x@50000000 {
> hdmi@54280000 {
> vdd-supply = <&hdmi_vdd_reg>;
> diff --git a/arch/arm/boot/dts/tegra20-iris-256.dts
> b/arch/arm/boot/dts/tegra20-iris-256.dts
> new file mode 100644
> index 000000000000..f2df58c81493
> --- /dev/null
> +++ b/arch/arm/boot/dts/tegra20-iris-256.dts
> @@ -0,0 +1,10 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/dts-v1/;
> +
> +#include "tegra20-colibri-256.dtsi"
> +#include "tegra20-iris-common.dtsi"
> +
> +/ {
> + model = "Toradex Colibri T20 256MB on Iris";
> + compatible = "toradex,iris", "toradex,colibri_t20-256", "nvidia,tegra20";
> +};
> diff --git a/arch/arm/boot/dts/tegra20-iris-512.dts
> b/arch/arm/boot/dts/tegra20-iris-512.dts
> index 40126388946d..6841e5af6f26 100644
> --- a/arch/arm/boot/dts/tegra20-iris-512.dts
> +++ b/arch/arm/boot/dts/tegra20-iris-512.dts
> @@ -2,105 +2,9 @@
> /dts-v1/;
>
> #include "tegra20-colibri-512.dtsi"
> +#include "tegra20-iris-common.dtsi"
>
> / {
> model = "Toradex Colibri T20 512MB on Iris";
> compatible = "toradex,iris", "toradex,colibri_t20-512", "nvidia,tegra20";
> -
> - aliases {
> - serial0 = &uarta;
> - serial1 = &uartd;
> - };
> -
> - chosen {
> - stdout-path = "serial0:115200n8";
> - };
> -
> - host1x@50000000 {
> - hdmi@54280000 {
> - status = "okay";
> - };
> - };
> -
> - pinmux@70000014 {
> - state_default: pinmux {
> - hdint {
> - nvidia,tristate = <TEGRA_PIN_DISABLE>;
> - };
> -
> - i2cddc {
> - nvidia,tristate = <TEGRA_PIN_DISABLE>;
> - };
> -
> - sdio4 {
> - nvidia,tristate = <TEGRA_PIN_DISABLE>;
> - };
> -
> - uarta {
> - nvidia,tristate = <TEGRA_PIN_DISABLE>;
> - };
> -
> - uartd {
> - nvidia,tristate = <TEGRA_PIN_DISABLE>;
> - };
> - };
> - };
> -
> - serial@70006000 {
> - status = "okay";
> - };
> -
> - serial@70006300 {
> - status = "okay";
> - };
> -
> - i2c_ddc: i2c@7000c400 {
> - status = "okay";
> - };
> -
> - usb@c5000000 {
> - status = "okay";
> - };
> -
> - usb-phy@c5000000 {
> - status = "okay";
> - };
> -
> - usb@c5008000 {
> - status = "okay";
> - };
> -
> - usb-phy@c5008000 {
> - status = "okay";
> - };
> -
> - sdhci@c8000600 {
> - status = "okay";
> - bus-width = <4>;
> - vmmc-supply = <&vcc_sd_reg>;
> - vqmmc-supply = <&vcc_sd_reg>;
> - };
> -
> - regulators {
> - regulator@0 {
> - compatible = "regulator-fixed";
> - reg = <0>;
> - regulator-name = "usb_host_vbus";
> - regulator-min-microvolt = <5000000>;
> - regulator-max-microvolt = <5000000>;
> - regulator-boot-on;
> - regulator-always-on;
> - gpio = <&gpio TEGRA_GPIO(W, 2) GPIO_ACTIVE_HIGH>;
> - };
> -
> - vcc_sd_reg: regulator@1 {
> - compatible = "regulator-fixed";
> - reg = <1>;
> - regulator-name = "vcc_sd";
> - regulator-min-microvolt = <3300000>;
> - regulator-max-microvolt = <3300000>;
> - regulator-boot-on;
> - regulator-always-on;
> - };
> - };
> };
> diff --git a/arch/arm/boot/dts/tegra20-iris-512.dts
> b/arch/arm/boot/dts/tegra20-iris-common.dtsi
> similarity index 85%
> copy from arch/arm/boot/dts/tegra20-iris-512.dts
> copy to arch/arm/boot/dts/tegra20-iris-common.dtsi
> index 40126388946d..bbc6ab22526e 100644
> --- a/arch/arm/boot/dts/tegra20-iris-512.dts
> +++ b/arch/arm/boot/dts/tegra20-iris-common.dtsi
> @@ -1,11 +1,12 @@
> // SPDX-License-Identifier: GPL-2.0
> -/dts-v1/;
> -
> -#include "tegra20-colibri-512.dtsi"
> +//
> +// This DTSI does not include specific Colibri DTSI but just configures
> +// common Iris properties. This DTSI shall be included after including
> +// specific Colibri DTSI.
>
> / {
> - model = "Toradex Colibri T20 512MB on Iris";
> - compatible = "toradex,iris", "toradex,colibri_t20-512", "nvidia,tegra20";
> + model = "Toradex Colibri T20 256MB/512MB on Iris";
> + compatible = "toradex,iris", "nvidia,tegra20";
>
> aliases {
> serial0 = &uarta;
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] ARM: dts: tegra: Add support for 256 MB Colibri-T20 (plus such board on Iris)
2018-05-04 9:19 ` Stefan Agner
@ 2018-05-04 9:37 ` Marcel Ziswiler
2018-05-04 10:03 ` Krzysztof Kozlowski
1 sibling, 0 replies; 9+ messages in thread
From: Marcel Ziswiler @ 2018-05-04 9:37 UTC (permalink / raw)
To: stefan, krzk
Cc: linux-kernel, robh+dt, jonathanh, devicetree, dev,
thierry.reding, mark.rutland, linux-tegra
Hi Krzysztof
On Fri, 2018-05-04 at 11:19 +0200, Stefan Agner wrote:
> On 03.05.2018 17:08, Krzysztof Kozlowski wrote:
> > Colibri-T20 can come in 256 MB RAM (with 512 MB NAND) or 512 MB RAM
> > (with 1024 MB NAND) flavors. Add support for the 256 MB version on
> > Iris
> > evaluation board.
>
> To we really need to specify memory size these days? I think all
> common
> boot loaders fill the memory node anyway.
>
> arch/arm/boot/dts/imx6qdl-apalis.dtsi:
> /* Will be filled by the bootloader */
> memory@10000000 {
> reg = <0x10000000 0>;
> };
Yes, both mainline as well as our downstream U-Boot do actually set the
memory node.
In the past some people used to set it to the smallest SKU's memory
size just to be on the safe side.
However lately it is agreed that the memory node should no longer even
be present as can be seen from e.g. arch/arm/boot/dts/tegra30-
apalis.dtsi resp. tegra30-apalis-eval.dts.
> I think we should just rename tegra20-colibri-512.dtsi =>
> tegra20-colibri.dtsi and tegra20-iris-512.dts => tegra20-iris.dts
> avoid confusion and add such an empty node.
I would rather suggest going to the regular SoC, module, carrier naming
convention making it tegra20-colibri-iris.dts instead.
> > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> >
> > ---
> >
> > Changes since v1:
> > 1. Fix memory size in tegra20-colibri-256.dtsi (was working fine
> > because
> > my bootloader uses mem= argument).
>
> What boot loader are you using? You even can omit mem= if your boot
> loader fills the memory node properly.
Yes, using mem= should only be used to force memory size for test
purpose [1].
> --
> Stefan
[1] https://www.kernel.org/doc/html/v4.14/admin-guide/kernel-parameters
.html
Cheers
Marcel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] ARM: dts: tegra: Add support for 256 MB Colibri-T20 (plus such board on Iris)
2018-05-04 9:19 ` Stefan Agner
2018-05-04 9:37 ` Marcel Ziswiler
@ 2018-05-04 10:03 ` Krzysztof Kozlowski
2018-05-04 10:14 ` Marcel Ziswiler
1 sibling, 1 reply; 9+ messages in thread
From: Krzysztof Kozlowski @ 2018-05-04 10:03 UTC (permalink / raw)
To: Stefan Agner
Cc: Rob Herring, Mark Rutland, Thierry Reding, Jonathan Hunter,
devicetree, linux-tegra, linux-kernel, Marcel Ziswiler,
Lucas Stach
On Fri, May 4, 2018 at 11:19 AM, Stefan Agner <stefan@agner.ch> wrote:
> On 03.05.2018 17:08, Krzysztof Kozlowski wrote:
>> Colibri-T20 can come in 256 MB RAM (with 512 MB NAND) or 512 MB RAM
>> (with 1024 MB NAND) flavors. Add support for the 256 MB version on Iris
>> evaluation board.
>
> To we really need to specify memory size these days? I think all common
> boot loaders fill the memory node anyway.
>
> arch/arm/boot/dts/imx6qdl-apalis.dtsi:
> /* Will be filled by the bootloader */
> memory@10000000 {
> reg = <0x10000000 0>;
> };
>
>
> I think we should just rename tegra20-colibri-512.dtsi =>
> tegra20-colibri.dtsi and tegra20-iris-512.dts => tegra20-iris.dts to
> avoid confusion and add such an empty node.
Having memory node is requirement of DeviceTree specification (at
least one cpu and memory node). Of course if bootloader fills it, then
we could assume that specification is satisfied... but what if some
bootloader skips it? This creates quite specific dependency between
kernel's DTS and bootloader.
One safe solution would be to specify 256 MB memory node for both
boards and assume that bootloader will bump it to 512 MB for specific
module.
>> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
>>
>> ---
>>
>> Changes since v1:
>> 1. Fix memory size in tegra20-colibri-256.dtsi (was working fine because
>> my bootloader uses mem= argument).
>
> What boot loader are you using? You even can omit mem= if your boot
> loader fills the memory node properly.
I use the one provided by vendor - Toradex. By default it passes both
mem= and few other mem-like arguments (for FB and video processor) to
command line. I believe it does not adjust the DTB itself because
Toradex prepared this bootloader for... 3.1 kernel without DTB. 7
years old kernel. :)
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] ARM: dts: tegra: Add support for 256 MB Colibri-T20 (plus such board on Iris)
2018-05-04 10:03 ` Krzysztof Kozlowski
@ 2018-05-04 10:14 ` Marcel Ziswiler
2018-05-08 12:35 ` Krzysztof Kozlowski
0 siblings, 1 reply; 9+ messages in thread
From: Marcel Ziswiler @ 2018-05-04 10:14 UTC (permalink / raw)
To: stefan, krzk
Cc: linux-kernel, robh+dt, jonathanh, devicetree, dev,
thierry.reding, mark.rutland, linux-tegra
Hi Krzysztof
On Fri, 2018-05-04 at 12:03 +0200, Krzysztof Kozlowski wrote:
> On Fri, May 4, 2018 at 11:19 AM, Stefan Agner <stefan@agner.ch>
> wrote:
> > On 03.05.2018 17:08, Krzysztof Kozlowski wrote:
> > > Colibri-T20 can come in 256 MB RAM (with 512 MB NAND) or 512 MB
> > > RAM
> > > (with 1024 MB NAND) flavors. Add support for the 256 MB version
> > > on Iris
> > > evaluation board.
> >
> > To we really need to specify memory size these days? I think all
> > common
> > boot loaders fill the memory node anyway.
> >
> > arch/arm/boot/dts/imx6qdl-apalis.dtsi:
> > /* Will be filled by the bootloader */
> > memory@10000000 {
> > reg = <0x10000000 0>;
> > };
> >
> >
> > I think we should just rename tegra20-colibri-512.dtsi =>
> > tegra20-colibri.dtsi and tegra20-iris-512.dts => tegra20-iris.dts
> > to
> > avoid confusion and add such an empty node.
>
> Having memory node is requirement of DeviceTree specification (at
> least one cpu and memory node). Of course if bootloader fills it,
> then
> we could assume that specification is satisfied... but what if some
> bootloader skips it? This creates quite specific dependency between
> kernel's DTS and bootloader.
Yes, no system boots without any boot loader e.g. initialising the
memory and what not. So there are obviously quite specific
dependencies.
> One safe solution would be to specify 256 MB memory node for both
> boards and assume that bootloader will bump it to 512 MB for specific
> module.
Like I mentioned this actually used to be how it was done however
lately it has been agreed that any boot loader anyway needs to do the
device tree dance (e.g. even just loading and passing it).
> > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> > >
> > > ---
> > >
> > > Changes since v1:
> > > 1. Fix memory size in tegra20-colibri-256.dtsi (was working fine
> > > because
> > > my bootloader uses mem= argument).
> >
> > What boot loader are you using? You even can omit mem= if your boot
> > loader fills the memory node properly.
>
> I use the one provided by vendor - Toradex. By default it passes both
> mem= and few other mem-like arguments (for FB and video processor) to
> command line. I believe it does not adjust the DTB itself because
> Toradex prepared this bootloader for... 3.1 kernel without DTB. 7
> years old kernel. :)
Remember our regular BSP is still based on NVIDIA's downstream Linux
for Tegra aka L4T R16.5 based on Linux kernel 3.1.10 which does not
even use any device tree yet.
> Best regards,
> Krzysztof
Cheers
Marcel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] ARM: dts: tegra: Add support for 256 MB Colibri-T20 (plus such board on Iris)
2018-05-03 15:08 [PATCH v2] ARM: dts: tegra: Add support for 256 MB Colibri-T20 (plus such board on Iris) Krzysztof Kozlowski
2018-05-04 9:19 ` Stefan Agner
@ 2018-05-07 16:58 ` Rob Herring
2018-05-08 7:26 ` Krzysztof Kozlowski
1 sibling, 1 reply; 9+ messages in thread
From: Rob Herring @ 2018-05-07 16:58 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Mark Rutland, Thierry Reding, Jonathan Hunter, devicetree,
linux-tegra, linux-kernel, Marcel Ziswiler, Stefan Agner,
Lucas Stach
On Thu, May 03, 2018 at 05:08:32PM +0200, Krzysztof Kozlowski wrote:
> Colibri-T20 can come in 256 MB RAM (with 512 MB NAND) or 512 MB RAM
> (with 1024 MB NAND) flavors. Add support for the 256 MB version on Iris
> evaluation board.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
>
> ---
>
> Changes since v1:
> 1. Fix memory size in tegra20-colibri-256.dtsi (was working fine because
> my bootloader uses mem= argument).
> ---
> Documentation/devicetree/bindings/arm/tegra.txt | 1 +
> arch/arm/boot/dts/Makefile | 1 +
> arch/arm/boot/dts/tegra20-colibri-256.dtsi | 11 +
> arch/arm/boot/dts/tegra20-colibri-512.dtsi | 531 +--------------------
> ...gra20-colibri-512.dtsi => tegra20-colibri.dtsi} | 4 -
> arch/arm/boot/dts/tegra20-iris-256.dts | 10 +
> arch/arm/boot/dts/tegra20-iris-512.dts | 98 +---
> ...gra20-iris-512.dts => tegra20-iris-common.dtsi} | 11 +-
> 8 files changed, 31 insertions(+), 636 deletions(-)
> create mode 100644 arch/arm/boot/dts/tegra20-colibri-256.dtsi
> copy arch/arm/boot/dts/{tegra20-colibri-512.dtsi => tegra20-colibri.dtsi} (99%)
> create mode 100644 arch/arm/boot/dts/tegra20-iris-256.dts
> copy arch/arm/boot/dts/{tegra20-iris-512.dts => tegra20-iris-common.dtsi} (85%)
>
> diff --git a/Documentation/devicetree/bindings/arm/tegra.txt b/Documentation/devicetree/bindings/arm/tegra.txt
> index 32f62bb7006d..d304574ae441 100644
> --- a/Documentation/devicetree/bindings/arm/tegra.txt
> +++ b/Documentation/devicetree/bindings/arm/tegra.txt
> @@ -49,6 +49,7 @@ board-specific compatible values:
> toradex,apalis_t30-eval
> toradex,apalis-tk1
> toradex,apalis-tk1-eval
> + toradex,colibri_t20-256
NAK unless there are changes beyond the memory size. Otherwise, just use
the existing string or deprecate the existing one for a new one without
the memory size. I prefer the less churn of the former option.
> toradex,colibri_t20-512
> toradex,colibri_t30
> toradex,colibri_t30-eval-v3
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] ARM: dts: tegra: Add support for 256 MB Colibri-T20 (plus such board on Iris)
2018-05-07 16:58 ` Rob Herring
@ 2018-05-08 7:26 ` Krzysztof Kozlowski
0 siblings, 0 replies; 9+ messages in thread
From: Krzysztof Kozlowski @ 2018-05-08 7:26 UTC (permalink / raw)
To: Rob Herring
Cc: Mark Rutland, Thierry Reding, Jonathan Hunter, devicetree,
linux-tegra, linux-kernel, Marcel Ziswiler, Stefan Agner,
Lucas Stach
On Mon, May 7, 2018 at 6:58 PM, Rob Herring <robh@kernel.org> wrote:
> On Thu, May 03, 2018 at 05:08:32PM +0200, Krzysztof Kozlowski wrote:
>> Colibri-T20 can come in 256 MB RAM (with 512 MB NAND) or 512 MB RAM
>> (with 1024 MB NAND) flavors. Add support for the 256 MB version on Iris
>> evaluation board.
>>
>> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
>>
>> ---
>>
>> Changes since v1:
>> 1. Fix memory size in tegra20-colibri-256.dtsi (was working fine because
>> my bootloader uses mem= argument).
>> ---
>> Documentation/devicetree/bindings/arm/tegra.txt | 1 +
>> arch/arm/boot/dts/Makefile | 1 +
>> arch/arm/boot/dts/tegra20-colibri-256.dtsi | 11 +
>> arch/arm/boot/dts/tegra20-colibri-512.dtsi | 531 +--------------------
>> ...gra20-colibri-512.dtsi => tegra20-colibri.dtsi} | 4 -
>> arch/arm/boot/dts/tegra20-iris-256.dts | 10 +
>> arch/arm/boot/dts/tegra20-iris-512.dts | 98 +---
>> ...gra20-iris-512.dts => tegra20-iris-common.dtsi} | 11 +-
>> 8 files changed, 31 insertions(+), 636 deletions(-)
>> create mode 100644 arch/arm/boot/dts/tegra20-colibri-256.dtsi
>> copy arch/arm/boot/dts/{tegra20-colibri-512.dtsi => tegra20-colibri.dtsi} (99%)
>> create mode 100644 arch/arm/boot/dts/tegra20-iris-256.dts
>> copy arch/arm/boot/dts/{tegra20-iris-512.dts => tegra20-iris-common.dtsi} (85%)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/tegra.txt b/Documentation/devicetree/bindings/arm/tegra.txt
>> index 32f62bb7006d..d304574ae441 100644
>> --- a/Documentation/devicetree/bindings/arm/tegra.txt
>> +++ b/Documentation/devicetree/bindings/arm/tegra.txt
>> @@ -49,6 +49,7 @@ board-specific compatible values:
>> toradex,apalis_t30-eval
>> toradex,apalis-tk1
>> toradex,apalis-tk1-eval
>> + toradex,colibri_t20-256
>
> NAK unless there are changes beyond the memory size. Otherwise, just use
> the existing string or deprecate the existing one for a new one without
> the memory size. I prefer the less churn of the former option.
I'll stick to original compatible. Thanks for feedback.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] ARM: dts: tegra: Add support for 256 MB Colibri-T20 (plus such board on Iris)
2018-05-04 10:14 ` Marcel Ziswiler
@ 2018-05-08 12:35 ` Krzysztof Kozlowski
2018-05-08 13:03 ` Marcel Ziswiler
0 siblings, 1 reply; 9+ messages in thread
From: Krzysztof Kozlowski @ 2018-05-08 12:35 UTC (permalink / raw)
To: Marcel Ziswiler
Cc: stefan, linux-kernel, robh+dt, jonathanh, devicetree, dev,
thierry.reding, mark.rutland, linux-tegra
On Fri, May 4, 2018 at 12:14 PM, Marcel Ziswiler
<marcel.ziswiler@toradex.com> wrote:
> Hi Krzysztof
>
> On Fri, 2018-05-04 at 12:03 +0200, Krzysztof Kozlowski wrote:
>> On Fri, May 4, 2018 at 11:19 AM, Stefan Agner <stefan@agner.ch>
>> wrote:
>> > On 03.05.2018 17:08, Krzysztof Kozlowski wrote:
>> > > Colibri-T20 can come in 256 MB RAM (with 512 MB NAND) or 512 MB
>> > > RAM
>> > > (with 1024 MB NAND) flavors. Add support for the 256 MB version
>> > > on Iris
>> > > evaluation board.
>> >
>> > To we really need to specify memory size these days? I think all
>> > common
>> > boot loaders fill the memory node anyway.
>> >
>> > arch/arm/boot/dts/imx6qdl-apalis.dtsi:
>> > /* Will be filled by the bootloader */
>> > memory@10000000 {
>> > reg = <0x10000000 0>;
>> > };
>> >
>> >
>> > I think we should just rename tegra20-colibri-512.dtsi =>
>> > tegra20-colibri.dtsi and tegra20-iris-512.dts => tegra20-iris.dts
>> > to
>> > avoid confusion and add such an empty node.
>>
>> Having memory node is requirement of DeviceTree specification (at
>> least one cpu and memory node). Of course if bootloader fills it,
>> then
>> we could assume that specification is satisfied... but what if some
>> bootloader skips it? This creates quite specific dependency between
>> kernel's DTS and bootloader.
>
> Yes, no system boots without any boot loader e.g. initialising the
> memory and what not. So there are obviously quite specific
> dependencies.
Sorry for late reply, I missed your emails because unfortunately
Google marks messages coming for Toradex as spam ("This message has a
from address in toradex.com but has failed toradex.com's required
tests for authentication."). Therefore I did not include your comments
and sent v3.
>
>> One safe solution would be to specify 256 MB memory node for both
>> boards and assume that bootloader will bump it to 512 MB for specific
>> module.
>
> Like I mentioned this actually used to be how it was done however
> lately it has been agreed that any boot loader anyway needs to do the
> device tree dance (e.g. even just loading and passing it).
Then the safest way is to provide /memory node with 256 MB. If some
bootloader omits it (e.g. because it is too old or by any mistake) the
FDT will be still usable with reduced memory in case of 512 board.
>> > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
>> > >
>> > > ---
>> > >
>> > > Changes since v1:
>> > > 1. Fix memory size in tegra20-colibri-256.dtsi (was working fine
>> > > because
>> > > my bootloader uses mem= argument).
>> >
>> > What boot loader are you using? You even can omit mem= if your boot
>> > loader fills the memory node properly.
>>
>> I use the one provided by vendor - Toradex. By default it passes both
>> mem= and few other mem-like arguments (for FB and video processor) to
>> command line. I believe it does not adjust the DTB itself because
>> Toradex prepared this bootloader for... 3.1 kernel without DTB. 7
>> years old kernel. :)
>
> Remember our regular BSP is still based on NVIDIA's downstream Linux
> for Tegra aka L4T R16.5 based on Linux kernel 3.1.10 which does not
> even use any device tree yet.
Yes, I noticed. I am trying to bring it fully on new kernels but no
huge successes so far... I'm stuck on NAND trying Lucas Stach's old
patchset.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] ARM: dts: tegra: Add support for 256 MB Colibri-T20 (plus such board on Iris)
2018-05-08 12:35 ` Krzysztof Kozlowski
@ 2018-05-08 13:03 ` Marcel Ziswiler
0 siblings, 0 replies; 9+ messages in thread
From: Marcel Ziswiler @ 2018-05-08 13:03 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: stefan, linux-kernel, robh+dt, jonathanh, devicetree, dev,
thierry.reding, mark.rutland, linux-tegra
On Tue, 2018-05-08 at 14:35 +0200, Krzysztof Kozlowski wrote:
> On Fri, May 4, 2018 at 12:14 PM, Marcel Ziswiler
> <marcel.ziswiler@toradex.com> wrote:
> > Hi Krzysztof
> >
> > On Fri, 2018-05-04 at 12:03 +0200, Krzysztof Kozlowski wrote:
> > > On Fri, May 4, 2018 at 11:19 AM, Stefan Agner <stefan@agner.ch>
> > > wrote:
> > > > On 03.05.2018 17:08, Krzysztof Kozlowski wrote:
> > > > > Colibri-T20 can come in 256 MB RAM (with 512 MB NAND) or 512
> > > > > MB
> > > > > RAM
> > > > > (with 1024 MB NAND) flavors. Add support for the 256 MB
> > > > > version
> > > > > on Iris
> > > > > evaluation board.
> > > >
> > > > To we really need to specify memory size these days? I think
> > > > all
> > > > common
> > > > boot loaders fill the memory node anyway.
> > > >
> > > > arch/arm/boot/dts/imx6qdl-apalis.dtsi:
> > > > /* Will be filled by the bootloader */
> > > > memory@10000000 {
> > > > reg = <0x10000000 0>;
> > > > };
> > > >
> > > >
> > > > I think we should just rename tegra20-colibri-512.dtsi =>
> > > > tegra20-colibri.dtsi and tegra20-iris-512.dts => tegra20-
> > > > iris.dts
> > > > to
> > > > avoid confusion and add such an empty node.
> > >
> > > Having memory node is requirement of DeviceTree specification (at
> > > least one cpu and memory node). Of course if bootloader fills it,
> > > then
> > > we could assume that specification is satisfied... but what if
> > > some
> > > bootloader skips it? This creates quite specific dependency
> > > between
> > > kernel's DTS and bootloader.
> >
> > Yes, no system boots without any boot loader e.g. initialising the
> > memory and what not. So there are obviously quite specific
> > dependencies.
>
> Sorry for late reply, I missed your emails because unfortunately
> Google marks messages coming for Toradex as spam ("This message has a
> from address in toradex.com but has failed toradex.com's required
> tests for authentication.").
Argh, I guess that happens if you have stubborn IT departments which
still believe in M$. Sending via private email now.
> Therefore I did not include your comments
> and sent v3.
Yeah, I noticed that. It's forgiven, don't worry (;-p).
> > > One safe solution would be to specify 256 MB memory node for both
> > > boards and assume that bootloader will bump it to 512 MB for
> > > specific
> > > module.
> >
> > Like I mentioned this actually used to be how it was done however
> > lately it has been agreed that any boot loader anyway needs to do
> > the
> > device tree dance (e.g. even just loading and passing it).
>
> Then the safest way is to provide /memory node with 256 MB. If some
> bootloader omits it (e.g. because it is too old or by any mistake)
> the
> FDT will be still usable with reduced memory in case of 512 board.
Makes sense.
> > > > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> > > > >
> > > > > ---
> > > > >
> > > > > Changes since v1:
> > > > > 1. Fix memory size in tegra20-colibri-256.dtsi (was working
> > > > > fine
> > > > > because
> > > > > my bootloader uses mem= argument).
> > > >
> > > > What boot loader are you using? You even can omit mem= if your
> > > > boot
> > > > loader fills the memory node properly.
> > >
> > > I use the one provided by vendor - Toradex. By default it passes
> > > both
> > > mem= and few other mem-like arguments (for FB and video
> > > processor) to
> > > command line. I believe it does not adjust the DTB itself because
> > > Toradex prepared this bootloader for... 3.1 kernel without DTB. 7
> > > years old kernel. :)
> >
> > Remember our regular BSP is still based on NVIDIA's downstream
> > Linux
> > for Tegra aka L4T R16.5 based on Linux kernel 3.1.10 which does not
> > even use any device tree yet.
>
> Yes, I noticed. I am trying to bring it fully on new kernels but no
> huge successes so far... I'm stuck on NAND trying Lucas Stach's old
> patchset.
Yeah, for a while I kept his patch set alive as well. I believe 4.4 was
still working fine passing all MTD tests. However, lately there were so
many changes it all fell apart. About a month ago I spend a couple
nights trying to get it running again but something is still not quite
right. Yet have to sync with Stefan in my team who is much more
knowledgeable about latest NAND development.
> Best regards,
> Krzysztof
Cheers
Marcel
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2018-05-08 13:03 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-03 15:08 [PATCH v2] ARM: dts: tegra: Add support for 256 MB Colibri-T20 (plus such board on Iris) Krzysztof Kozlowski
2018-05-04 9:19 ` Stefan Agner
2018-05-04 9:37 ` Marcel Ziswiler
2018-05-04 10:03 ` Krzysztof Kozlowski
2018-05-04 10:14 ` Marcel Ziswiler
2018-05-08 12:35 ` Krzysztof Kozlowski
2018-05-08 13:03 ` Marcel Ziswiler
2018-05-07 16:58 ` Rob Herring
2018-05-08 7:26 ` Krzysztof Kozlowski
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.