* [PATCH v1 0/3] Tegra ARM32 device-tree improvements for 5.16
@ 2021-09-26 23:37 Dmitry Osipenko
2021-09-26 23:37 ` [PATCH v1 1/3] ARM: tegra: acer-a500: Correct compatible of ak8975 magnetometer Dmitry Osipenko
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Dmitry Osipenko @ 2021-09-26 23:37 UTC (permalink / raw)
To: Thierry Reding, Jonathan Hunter, David Heidelberg
Cc: devicetree, linux-kernel, linux-tegra
This series fixes couple warnings reported by the DT schema checker for
ARM32 Tegra device-trees.
David Heidelberg (2):
ARM: tegra: acer-a500: Correct compatible of ak8975 magnetometer
ARM: tegra: Name clock and regulator nodes according to DT-schema
Dmitry Osipenko (1):
ARM: tegra: Update Broadcom Bluetooth device-tree nodes
arch/arm/boot/dts/tegra114-dalmore.dts | 16 +++++------
arch/arm/boot/dts/tegra114-roth.dts | 14 +++++-----
arch/arm/boot/dts/tegra114-tn7.dts | 8 +++---
arch/arm/boot/dts/tegra124-jetson-tk1.dts | 26 ++++++++---------
arch/arm/boot/dts/tegra124-nyan.dtsi | 28 +++++++++----------
arch/arm/boot/dts/tegra124-venice2.dts | 28 +++++++++----------
.../boot/dts/tegra20-acer-a500-picasso.dts | 19 +++++++------
arch/arm/boot/dts/tegra20-harmony.dts | 16 +++++------
arch/arm/boot/dts/tegra20-medcom-wide.dts | 8 +++---
arch/arm/boot/dts/tegra20-paz00.dts | 6 ++--
arch/arm/boot/dts/tegra20-plutux.dts | 8 +++---
arch/arm/boot/dts/tegra20-seaboard.dts | 16 +++++------
arch/arm/boot/dts/tegra20-tamonten.dtsi | 4 +--
arch/arm/boot/dts/tegra20-tec.dts | 8 +++---
arch/arm/boot/dts/tegra20-trimslice.dts | 12 ++++----
arch/arm/boot/dts/tegra20-ventana.dts | 12 ++++----
.../tegra30-asus-nexus7-grouper-common.dtsi | 15 ++++++----
...egra30-asus-nexus7-grouper-maxim-pmic.dtsi | 4 +--
.../tegra30-asus-nexus7-grouper-ti-pmic.dtsi | 2 +-
arch/arm/boot/dts/tegra30-beaver.dts | 20 ++++++-------
arch/arm/boot/dts/tegra30-cardhu-a02.dts | 12 ++++----
arch/arm/boot/dts/tegra30-cardhu-a04.dts | 14 +++++-----
arch/arm/boot/dts/tegra30-cardhu.dtsi | 28 +++++++++----------
arch/arm/boot/dts/tegra30-ouya.dts | 5 +++-
24 files changed, 169 insertions(+), 160 deletions(-)
--
2.32.0
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v1 1/3] ARM: tegra: acer-a500: Correct compatible of ak8975 magnetometer
2021-09-26 23:37 [PATCH v1 0/3] Tegra ARM32 device-tree improvements for 5.16 Dmitry Osipenko
@ 2021-09-26 23:37 ` Dmitry Osipenko
2021-10-04 20:40 ` Thierry Reding
2021-09-26 23:37 ` [PATCH v1 2/3] ARM: tegra: Name clock and regulator nodes according to DT-schema Dmitry Osipenko
2021-09-26 23:37 ` [PATCH v1 3/3] ARM: tegra: Update Broadcom Bluetooth device-tree nodes Dmitry Osipenko
2 siblings, 1 reply; 8+ messages in thread
From: Dmitry Osipenko @ 2021-09-26 23:37 UTC (permalink / raw)
To: Thierry Reding, Jonathan Hunter, David Heidelberg
Cc: devicetree, linux-kernel, linux-tegra
From: David Heidelberg <david@ixit.cz>
The "ak,ak8975" compatible is not recognized by dt-bindings, it's
deprecated. Use supported "asahi-kasei,ak8975" compatible.
Signed-off-by: David Heidelberg <david@ixit.cz>
---
arch/arm/boot/dts/tegra20-acer-a500-picasso.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/tegra20-acer-a500-picasso.dts b/arch/arm/boot/dts/tegra20-acer-a500-picasso.dts
index 2280d75b66ab..5ac288fa9c5e 100644
--- a/arch/arm/boot/dts/tegra20-acer-a500-picasso.dts
+++ b/arch/arm/boot/dts/tegra20-acer-a500-picasso.dts
@@ -543,7 +543,7 @@ i2c@7000d000 {
status = "okay";
magnetometer@c {
- compatible = "ak,ak8975";
+ compatible = "asahi-kasei,ak8975";
reg = <0x0c>;
interrupt-parent = <&gpio>;
--
2.32.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH v1 2/3] ARM: tegra: Name clock and regulator nodes according to DT-schema
2021-09-26 23:37 [PATCH v1 0/3] Tegra ARM32 device-tree improvements for 5.16 Dmitry Osipenko
2021-09-26 23:37 ` [PATCH v1 1/3] ARM: tegra: acer-a500: Correct compatible of ak8975 magnetometer Dmitry Osipenko
@ 2021-09-26 23:37 ` Dmitry Osipenko
2021-10-04 20:56 ` Thierry Reding
2021-09-26 23:37 ` [PATCH v1 3/3] ARM: tegra: Update Broadcom Bluetooth device-tree nodes Dmitry Osipenko
2 siblings, 1 reply; 8+ messages in thread
From: Dmitry Osipenko @ 2021-09-26 23:37 UTC (permalink / raw)
To: Thierry Reding, Jonathan Hunter, David Heidelberg
Cc: devicetree, linux-kernel, linux-tegra
From: David Heidelberg <david@ixit.cz>
Name clocks and regulators according to DT-schema to fix warnings such as:
arch/arm/boot/dts/tegra20-acer-a500-picasso.dt.yaml: /: clock@0: 'anyOf' conditional failed, one must be fixed:
'reg' is a required property
'ranges' is a required property
From schema: /home/runner/.local/lib/python3.8/site-packages/dtschema/schemas/root-node.yaml
Signed-off-by: David Heidelberg <david@ixit.cz>
---
arch/arm/boot/dts/tegra114-dalmore.dts | 16 +++++------
arch/arm/boot/dts/tegra114-roth.dts | 14 +++++-----
arch/arm/boot/dts/tegra114-tn7.dts | 8 +++---
arch/arm/boot/dts/tegra124-jetson-tk1.dts | 26 ++++++++---------
arch/arm/boot/dts/tegra124-nyan.dtsi | 28 +++++++++----------
arch/arm/boot/dts/tegra124-venice2.dts | 28 +++++++++----------
.../boot/dts/tegra20-acer-a500-picasso.dts | 12 ++++----
arch/arm/boot/dts/tegra20-harmony.dts | 16 +++++------
arch/arm/boot/dts/tegra20-medcom-wide.dts | 8 +++---
arch/arm/boot/dts/tegra20-paz00.dts | 6 ++--
arch/arm/boot/dts/tegra20-plutux.dts | 8 +++---
arch/arm/boot/dts/tegra20-seaboard.dts | 16 +++++------
arch/arm/boot/dts/tegra20-tamonten.dtsi | 4 +--
arch/arm/boot/dts/tegra20-tec.dts | 8 +++---
arch/arm/boot/dts/tegra20-trimslice.dts | 12 ++++----
arch/arm/boot/dts/tegra20-ventana.dts | 12 ++++----
.../tegra30-asus-nexus7-grouper-common.dtsi | 10 +++----
...egra30-asus-nexus7-grouper-maxim-pmic.dtsi | 4 +--
.../tegra30-asus-nexus7-grouper-ti-pmic.dtsi | 2 +-
arch/arm/boot/dts/tegra30-beaver.dts | 20 ++++++-------
arch/arm/boot/dts/tegra30-cardhu-a02.dts | 12 ++++----
arch/arm/boot/dts/tegra30-cardhu-a04.dts | 14 +++++-----
arch/arm/boot/dts/tegra30-cardhu.dtsi | 28 +++++++++----------
23 files changed, 156 insertions(+), 156 deletions(-)
diff --git a/arch/arm/boot/dts/tegra114-dalmore.dts b/arch/arm/boot/dts/tegra114-dalmore.dts
index 7fd901f8d39a..d23050bfaba6 100644
--- a/arch/arm/boot/dts/tegra114-dalmore.dts
+++ b/arch/arm/boot/dts/tegra114-dalmore.dts
@@ -1151,7 +1151,7 @@ backlight: backlight {
default-brightness-level = <6>;
};
- clk32k_in: clock@0 {
+ clk32k_in: clock-32k {
compatible = "fixed-clock";
clock-frequency = <32768>;
#clock-cells = <0>;
@@ -1186,7 +1186,7 @@ volume_up {
};
};
- vdd_ac_bat_reg: regulator@0 {
+ vdd_ac_bat_reg: regulator-acbat {
compatible = "regulator-fixed";
regulator-name = "vdd_ac_bat";
regulator-min-microvolt = <5000000>;
@@ -1194,7 +1194,7 @@ vdd_ac_bat_reg: regulator@0 {
regulator-always-on;
};
- dvdd_ts_reg: regulator@1 {
+ dvdd_ts_reg: regulator-ts {
compatible = "regulator-fixed";
regulator-name = "dvdd_ts";
regulator-min-microvolt = <1800000>;
@@ -1203,7 +1203,7 @@ dvdd_ts_reg: regulator@1 {
gpio = <&gpio TEGRA_GPIO(H, 5) GPIO_ACTIVE_HIGH>;
};
- usb1_vbus_reg: regulator@3 {
+ usb1_vbus_reg: regulator-usb1 {
compatible = "regulator-fixed";
regulator-name = "usb1_vbus";
regulator-min-microvolt = <5000000>;
@@ -1214,7 +1214,7 @@ usb1_vbus_reg: regulator@3 {
vin-supply = <&tps65090_dcdc1_reg>;
};
- usb3_vbus_reg: regulator@4 {
+ usb3_vbus_reg: regulator-usb3 {
compatible = "regulator-fixed";
regulator-name = "usb2_vbus";
regulator-min-microvolt = <5000000>;
@@ -1225,7 +1225,7 @@ usb3_vbus_reg: regulator@4 {
vin-supply = <&tps65090_dcdc1_reg>;
};
- vdd_hdmi_reg: regulator@5 {
+ vdd_hdmi_reg: regulator-hdmi {
compatible = "regulator-fixed";
regulator-name = "vdd_hdmi_5v0";
regulator-min-microvolt = <5000000>;
@@ -1233,7 +1233,7 @@ vdd_hdmi_reg: regulator@5 {
vin-supply = <&tps65090_dcdc1_reg>;
};
- vdd_cam_1v8_reg: regulator@6 {
+ vdd_cam_1v8_reg: regulator-cam {
compatible = "regulator-fixed";
regulator-name = "vdd_cam_1v8_reg";
regulator-min-microvolt = <1800000>;
@@ -1242,7 +1242,7 @@ vdd_cam_1v8_reg: regulator@6 {
gpio = <&palmas_gpio 6 0>;
};
- vdd_5v0_hdmi: regulator@7 {
+ vdd_5v0_hdmi: regulator-hdmicon {
compatible = "regulator-fixed";
regulator-name = "VDD_5V0_HDMI_CON";
regulator-min-microvolt = <5000000>;
diff --git a/arch/arm/boot/dts/tegra114-roth.dts b/arch/arm/boot/dts/tegra114-roth.dts
index 07960171fabe..18521707ff5b 100644
--- a/arch/arm/boot/dts/tegra114-roth.dts
+++ b/arch/arm/boot/dts/tegra114-roth.dts
@@ -1016,7 +1016,7 @@ backlight: backlight {
enable-gpios = <&gpio TEGRA_GPIO(H, 2) GPIO_ACTIVE_HIGH>;
};
- clk32k_in: clock@0 {
+ clk32k_in: clock-32k {
compatible = "fixed-clock";
clock-frequency = <32768>;
#clock-cells = <0>;
@@ -1045,7 +1045,7 @@ power {
};
};
- lcd_bl_en: regulator@0 {
+ lcd_bl_en: regulator-lcden {
compatible = "regulator-fixed";
regulator-name = "lcd_bl_en";
regulator-min-microvolt = <5000000>;
@@ -1053,7 +1053,7 @@ lcd_bl_en: regulator@0 {
regulator-boot-on;
};
- vdd_lcd: regulator@1 {
+ vdd_lcd: regulator-lcd {
compatible = "regulator-fixed";
regulator-name = "vdd_lcd_1v8";
regulator-min-microvolt = <1800000>;
@@ -1064,7 +1064,7 @@ vdd_lcd: regulator@1 {
regulator-boot-on;
};
- regulator@2 {
+ regulator-1v8ts {
compatible = "regulator-fixed";
regulator-name = "vdd_1v8_ts";
regulator-min-microvolt = <1800000>;
@@ -1073,7 +1073,7 @@ regulator@2 {
regulator-boot-on;
};
- regulator@3 {
+ regulator-3v3ts {
compatible = "regulator-fixed";
regulator-name = "vdd_3v3_ts";
regulator-min-microvolt = <3300000>;
@@ -1083,7 +1083,7 @@ regulator@3 {
regulator-boot-on;
};
- regulator@4 {
+ regulator-1v8com {
compatible = "regulator-fixed";
regulator-name = "vdd_1v8_com";
regulator-min-microvolt = <1800000>;
@@ -1094,7 +1094,7 @@ regulator@4 {
regulator-boot-on;
};
- regulator@5 {
+ regulator-3v3com {
compatible = "regulator-fixed";
regulator-name = "vdd_3v3_com";
regulator-min-microvolt = <3300000>;
diff --git a/arch/arm/boot/dts/tegra114-tn7.dts b/arch/arm/boot/dts/tegra114-tn7.dts
index 745d234b105b..0534247458f0 100644
--- a/arch/arm/boot/dts/tegra114-tn7.dts
+++ b/arch/arm/boot/dts/tegra114-tn7.dts
@@ -273,7 +273,7 @@ backlight: backlight {
power-supply = <&lcd_bl_en>;
};
- clk32k_in: clock@0 {
+ clk32k_in: clock-32k {
compatible = "fixed-clock";
clock-frequency = <32768>;
#clock-cells = <0>;
@@ -303,7 +303,7 @@ volume_up {
};
/* FIXME: output of BQ24192 */
- vs_sys: regulator@0 {
+ vs_sys: regulator-vs {
compatible = "regulator-fixed";
regulator-name = "VS_SYS";
regulator-min-microvolt = <4200000>;
@@ -312,7 +312,7 @@ vs_sys: regulator@0 {
regulator-boot-on;
};
- lcd_bl_en: regulator@1 {
+ lcd_bl_en: regulator-lcden {
compatible = "regulator-fixed";
regulator-name = "VDD_LCD_BL";
regulator-min-microvolt = <16500000>;
@@ -323,7 +323,7 @@ lcd_bl_en: regulator@1 {
regulator-boot-on;
};
- vdd_lcd: regulator@2 {
+ vdd_lcd: regulator-lcd {
compatible = "regulator-fixed";
regulator-name = "VD_LCD_1V8";
regulator-min-microvolt = <1800000>;
diff --git a/arch/arm/boot/dts/tegra124-jetson-tk1.dts b/arch/arm/boot/dts/tegra124-jetson-tk1.dts
index 35ab296408e1..094f94a627b2 100644
--- a/arch/arm/boot/dts/tegra124-jetson-tk1.dts
+++ b/arch/arm/boot/dts/tegra124-jetson-tk1.dts
@@ -1868,7 +1868,7 @@ usb-phy@7d008000 {
vbus-supply = <&vdd_usb3_vbus>;
};
- clk32k_in: clock@0 {
+ clk32k_in: clock-32k {
compatible = "fixed-clock";
clock-frequency = <32768>;
#clock-cells = <0>;
@@ -1892,7 +1892,7 @@ power {
};
};
- vdd_mux: regulator@0 {
+ vdd_mux: regulator-mux {
compatible = "regulator-fixed";
regulator-name = "+VDD_MUX";
regulator-min-microvolt = <12000000>;
@@ -1901,7 +1901,7 @@ vdd_mux: regulator@0 {
regulator-boot-on;
};
- vdd_5v0_sys: regulator@1 {
+ vdd_5v0_sys: regulator-5v0sys {
compatible = "regulator-fixed";
regulator-name = "+5V_SYS";
regulator-min-microvolt = <5000000>;
@@ -1911,7 +1911,7 @@ vdd_5v0_sys: regulator@1 {
vin-supply = <&vdd_mux>;
};
- vdd_3v3_sys: regulator@2 {
+ vdd_3v3_sys: regulator-3v3sys {
compatible = "regulator-fixed";
regulator-name = "+3.3V_SYS";
regulator-min-microvolt = <3300000>;
@@ -1921,7 +1921,7 @@ vdd_3v3_sys: regulator@2 {
vin-supply = <&vdd_mux>;
};
- vdd_3v3_run: regulator@3 {
+ vdd_3v3_run: regulator-3v3run {
compatible = "regulator-fixed";
regulator-name = "+3.3V_RUN";
regulator-min-microvolt = <3300000>;
@@ -1933,7 +1933,7 @@ vdd_3v3_run: regulator@3 {
vin-supply = <&vdd_3v3_sys>;
};
- vdd_3v3_hdmi: regulator@4 {
+ vdd_3v3_hdmi: regulator-3v3hdmi {
compatible = "regulator-fixed";
regulator-name = "+3.3V_AVDD_HDMI_AP_GATED";
regulator-min-microvolt = <3300000>;
@@ -1941,7 +1941,7 @@ vdd_3v3_hdmi: regulator@4 {
vin-supply = <&vdd_3v3_run>;
};
- vdd_usb1_vbus: regulator@5 {
+ vdd_usb1_vbus: regulator-usb1 {
compatible = "regulator-fixed";
regulator-name = "+USB0_VBUS_SW";
regulator-min-microvolt = <5000000>;
@@ -1952,7 +1952,7 @@ vdd_usb1_vbus: regulator@5 {
vin-supply = <&vdd_5v0_sys>;
};
- vdd_usb3_vbus: regulator@6 {
+ vdd_usb3_vbus: regulator-usb3 {
compatible = "regulator-fixed";
regulator-name = "+5V_USB_HS";
regulator-min-microvolt = <5000000>;
@@ -1963,7 +1963,7 @@ vdd_usb3_vbus: regulator@6 {
vin-supply = <&vdd_5v0_sys>;
};
- vdd_3v3_lp0: regulator@7 {
+ vdd_3v3_lp0: regulator-lp0 {
compatible = "regulator-fixed";
regulator-name = "+3.3V_LP0";
regulator-min-microvolt = <3300000>;
@@ -1975,7 +1975,7 @@ vdd_3v3_lp0: regulator@7 {
vin-supply = <&vdd_3v3_sys>;
};
- vdd_hdmi_pll: regulator@8 {
+ vdd_hdmi_pll: regulator-hdmipll {
compatible = "regulator-fixed";
regulator-name = "+1.05V_RUN_AVDD_HDMI_PLL";
regulator-min-microvolt = <1050000>;
@@ -1984,7 +1984,7 @@ vdd_hdmi_pll: regulator@8 {
vin-supply = <&vdd_1v05_run>;
};
- vdd_5v0_hdmi: regulator@9 {
+ vdd_5v0_hdmi: regulator-hdmicon {
compatible = "regulator-fixed";
regulator-name = "+5V_HDMI_CON";
regulator-min-microvolt = <5000000>;
@@ -1995,7 +1995,7 @@ vdd_5v0_hdmi: regulator@9 {
};
/* Molex power connector */
- vdd_5v0_sata: regulator@10 {
+ vdd_5v0_sata: regulator-5v0sata {
compatible = "regulator-fixed";
regulator-name = "+5V_SATA";
regulator-min-microvolt = <5000000>;
@@ -2005,7 +2005,7 @@ vdd_5v0_sata: regulator@10 {
vin-supply = <&vdd_5v0_sys>;
};
- vdd_12v0_sata: regulator@11 {
+ vdd_12v0_sata: regulator-12v0sata {
compatible = "regulator-fixed";
regulator-name = "+12V_SATA";
regulator-min-microvolt = <12000000>;
diff --git a/arch/arm/boot/dts/tegra124-nyan.dtsi b/arch/arm/boot/dts/tegra124-nyan.dtsi
index 63a81270300a..2d045d1a9bcf 100644
--- a/arch/arm/boot/dts/tegra124-nyan.dtsi
+++ b/arch/arm/boot/dts/tegra124-nyan.dtsi
@@ -582,7 +582,7 @@ backlight: backlight {
256>;
};
- clk32k_in: clock@0 {
+ clk32k_in: clock-32k {
compatible = "fixed-clock";
clock-frequency = <32768>;
#clock-cells = <0>;
@@ -615,7 +615,7 @@ power {
};
};
- vdd_mux: regulator@0 {
+ vdd_mux: regulator-mux {
compatible = "regulator-fixed";
regulator-name = "+VDD_MUX";
regulator-min-microvolt = <12000000>;
@@ -624,7 +624,7 @@ vdd_mux: regulator@0 {
regulator-boot-on;
};
- vdd_5v0_sys: regulator@1 {
+ vdd_5v0_sys: regulator-5v0sys {
compatible = "regulator-fixed";
regulator-name = "+5V_SYS";
regulator-min-microvolt = <5000000>;
@@ -634,7 +634,7 @@ vdd_5v0_sys: regulator@1 {
vin-supply = <&vdd_mux>;
};
- vdd_3v3_sys: regulator@2 {
+ vdd_3v3_sys: regulator-3v3sys {
compatible = "regulator-fixed";
regulator-name = "+3.3V_SYS";
regulator-min-microvolt = <3300000>;
@@ -644,7 +644,7 @@ vdd_3v3_sys: regulator@2 {
vin-supply = <&vdd_mux>;
};
- vdd_3v3_run: regulator@3 {
+ vdd_3v3_run: regulator-3v3run {
compatible = "regulator-fixed";
regulator-name = "+3.3V_RUN";
regulator-min-microvolt = <3300000>;
@@ -656,7 +656,7 @@ vdd_3v3_run: regulator@3 {
vin-supply = <&vdd_3v3_sys>;
};
- vdd_3v3_hdmi: regulator@4 {
+ vdd_3v3_hdmi: regulator-3v3hdmi {
compatible = "regulator-fixed";
regulator-name = "+3.3V_AVDD_HDMI_AP_GATED";
regulator-min-microvolt = <3300000>;
@@ -664,7 +664,7 @@ vdd_3v3_hdmi: regulator@4 {
vin-supply = <&vdd_3v3_run>;
};
- vdd_led: regulator@5 {
+ vdd_led: regulator-led {
compatible = "regulator-fixed";
regulator-name = "+VDD_LED";
gpio = <&gpio TEGRA_GPIO(P, 2) GPIO_ACTIVE_HIGH>;
@@ -672,7 +672,7 @@ vdd_led: regulator@5 {
vin-supply = <&vdd_mux>;
};
- vdd_5v0_ts: regulator@6 {
+ vdd_5v0_ts: regulator-ts {
compatible = "regulator-fixed";
regulator-name = "+5V_VDD_TS_SW";
regulator-min-microvolt = <5000000>;
@@ -683,7 +683,7 @@ vdd_5v0_ts: regulator@6 {
vin-supply = <&vdd_5v0_sys>;
};
- vdd_usb1_vbus: regulator@7 {
+ vdd_usb1_vbus: regulator-usb1 {
compatible = "regulator-fixed";
regulator-name = "+5V_USB_HS";
regulator-min-microvolt = <5000000>;
@@ -694,7 +694,7 @@ vdd_usb1_vbus: regulator@7 {
vin-supply = <&vdd_5v0_sys>;
};
- vdd_usb3_vbus: regulator@8 {
+ vdd_usb3_vbus: regulator-usb3 {
compatible = "regulator-fixed";
regulator-name = "+5V_USB_SS";
regulator-min-microvolt = <5000000>;
@@ -705,7 +705,7 @@ vdd_usb3_vbus: regulator@8 {
vin-supply = <&vdd_5v0_sys>;
};
- vdd_3v3_panel: regulator@9 {
+ vdd_3v3_panel: regulator-panel {
compatible = "regulator-fixed";
regulator-name = "+3.3V_PANEL";
regulator-min-microvolt = <3300000>;
@@ -715,7 +715,7 @@ vdd_3v3_panel: regulator@9 {
vin-supply = <&vdd_3v3_run>;
};
- vdd_3v3_lp0: regulator@10 {
+ vdd_3v3_lp0: regulator-lp0 {
compatible = "regulator-fixed";
regulator-name = "+3.3V_LP0";
regulator-min-microvolt = <3300000>;
@@ -730,7 +730,7 @@ vdd_3v3_lp0: regulator@10 {
vin-supply = <&vdd_3v3_sys>;
};
- vdd_hdmi_pll: regulator@11 {
+ vdd_hdmi_pll: regulator-hdmipll {
compatible = "regulator-fixed";
regulator-name = "+1.05V_RUN_AVDD_HDMI_PLL";
regulator-min-microvolt = <1050000>;
@@ -739,7 +739,7 @@ vdd_hdmi_pll: regulator@11 {
vin-supply = <&vdd_1v05_run>;
};
- vdd_5v0_hdmi: regulator@12 {
+ vdd_5v0_hdmi: regulator-hdmicon {
compatible = "regulator-fixed";
regulator-name = "+5V_HDMI_CON";
regulator-min-microvolt = <5000000>;
diff --git a/arch/arm/boot/dts/tegra124-venice2.dts b/arch/arm/boot/dts/tegra124-venice2.dts
index e6b54ac1ebd1..457b8aada1f8 100644
--- a/arch/arm/boot/dts/tegra124-venice2.dts
+++ b/arch/arm/boot/dts/tegra124-venice2.dts
@@ -1061,7 +1061,7 @@ backlight: backlight {
default-brightness-level = <6>;
};
- clk32k_in: clock@0 {
+ clk32k_in: clock-32k {
compatible = "fixed-clock";
clock-frequency = <32768>;
#clock-cells = <0>;
@@ -1086,7 +1086,7 @@ panel: panel {
ddc-i2c-bus = <&dpaux>;
};
- vdd_mux: regulator@0 {
+ vdd_mux: regulator-mux {
compatible = "regulator-fixed";
regulator-name = "+VDD_MUX";
regulator-min-microvolt = <12000000>;
@@ -1095,7 +1095,7 @@ vdd_mux: regulator@0 {
regulator-boot-on;
};
- vdd_5v0_sys: regulator@1 {
+ vdd_5v0_sys: regulator-5v0sys {
compatible = "regulator-fixed";
regulator-name = "+5V_SYS";
regulator-min-microvolt = <5000000>;
@@ -1105,7 +1105,7 @@ vdd_5v0_sys: regulator@1 {
vin-supply = <&vdd_mux>;
};
- vdd_3v3_sys: regulator@2 {
+ vdd_3v3_sys: regulator-3v3sys {
compatible = "regulator-fixed";
regulator-name = "+3.3V_SYS";
regulator-min-microvolt = <3300000>;
@@ -1115,7 +1115,7 @@ vdd_3v3_sys: regulator@2 {
vin-supply = <&vdd_mux>;
};
- vdd_3v3_run: regulator@3 {
+ vdd_3v3_run: regulator-3v3run {
compatible = "regulator-fixed";
regulator-name = "+3.3V_RUN";
regulator-min-microvolt = <3300000>;
@@ -1127,7 +1127,7 @@ vdd_3v3_run: regulator@3 {
vin-supply = <&vdd_3v3_sys>;
};
- vdd_3v3_hdmi: regulator@4 {
+ vdd_3v3_hdmi: regulator-hdmi {
compatible = "regulator-fixed";
regulator-name = "+3.3V_AVDD_HDMI_AP_GATED";
regulator-min-microvolt = <3300000>;
@@ -1135,7 +1135,7 @@ vdd_3v3_hdmi: regulator@4 {
vin-supply = <&vdd_3v3_run>;
};
- vdd_led: regulator@5 {
+ vdd_led: regulator-led {
compatible = "regulator-fixed";
regulator-name = "+VDD_LED";
regulator-min-microvolt = <3300000>;
@@ -1145,7 +1145,7 @@ vdd_led: regulator@5 {
vin-supply = <&vdd_mux>;
};
- vdd_5v0_ts: regulator@6 {
+ vdd_5v0_ts: regulator-ts {
compatible = "regulator-fixed";
regulator-name = "+5V_VDD_TS_SW";
regulator-min-microvolt = <5000000>;
@@ -1156,7 +1156,7 @@ vdd_5v0_ts: regulator@6 {
vin-supply = <&vdd_5v0_sys>;
};
- vdd_usb1_vbus: regulator@7 {
+ vdd_usb1_vbus: regulator-usb1 {
compatible = "regulator-fixed";
regulator-name = "+5V_USB_HS";
regulator-min-microvolt = <5000000>;
@@ -1167,7 +1167,7 @@ vdd_usb1_vbus: regulator@7 {
vin-supply = <&vdd_5v0_sys>;
};
- vdd_usb3_vbus: regulator@8 {
+ vdd_usb3_vbus: regulator-usb3 {
compatible = "regulator-fixed";
regulator-name = "+5V_USB_SS";
regulator-min-microvolt = <5000000>;
@@ -1178,7 +1178,7 @@ vdd_usb3_vbus: regulator@8 {
vin-supply = <&vdd_5v0_sys>;
};
- vdd_3v3_panel: regulator@9 {
+ vdd_3v3_panel: regulator-panel {
compatible = "regulator-fixed";
regulator-name = "+3.3V_PANEL";
regulator-min-microvolt = <3300000>;
@@ -1188,7 +1188,7 @@ vdd_3v3_panel: regulator@9 {
vin-supply = <&vdd_3v3_run>;
};
- vdd_3v3_lp0: regulator@10 {
+ vdd_3v3_lp0: regulator-lp0 {
compatible = "regulator-fixed";
regulator-name = "+3.3V_LP0";
regulator-min-microvolt = <3300000>;
@@ -1203,7 +1203,7 @@ vdd_3v3_lp0: regulator@10 {
vin-supply = <&vdd_3v3_sys>;
};
- vdd_hdmi_pll: regulator@11 {
+ vdd_hdmi_pll: regulator-hdmipll {
compatible = "regulator-fixed";
regulator-name = "+1.05V_RUN_AVDD_HDMI_PLL";
regulator-min-microvolt = <1050000>;
@@ -1212,7 +1212,7 @@ vdd_hdmi_pll: regulator@11 {
vin-supply = <&vdd_1v05_run>;
};
- vdd_5v0_hdmi: regulator@12 {
+ vdd_5v0_hdmi: regulator-hdmicon {
compatible = "regulator-fixed";
regulator-name = "+5V_HDMI_CON";
regulator-min-microvolt = <5000000>;
diff --git a/arch/arm/boot/dts/tegra20-acer-a500-picasso.dts b/arch/arm/boot/dts/tegra20-acer-a500-picasso.dts
index 5ac288fa9c5e..0abfd6215016 100644
--- a/arch/arm/boot/dts/tegra20-acer-a500-picasso.dts
+++ b/arch/arm/boot/dts/tegra20-acer-a500-picasso.dts
@@ -824,7 +824,7 @@ bat1010: battery-2s1p {
};
/* PMIC has a built-in 32KHz oscillator which is used by PMC */
- clk32k_in: clock@0 {
+ clk32k_in: clock-32k-in {
compatible = "fixed-clock";
#clock-cells = <0>;
clock-frequency = <32768>;
@@ -836,7 +836,7 @@ clk32k_in: clock@0 {
* oscillator is used as a reference clock-source by the
* Azurewave WiFi/BT module.
*/
- rtc_32k_wifi: clock@1 {
+ rtc_32k_wifi: clock-32k-wifi {
compatible = "fixed-clock";
#clock-cells = <0>;
clock-frequency = <32768>;
@@ -961,7 +961,7 @@ lvds_encoder_output: endpoint {
};
};
- vdd_5v0_sys: regulator@0 {
+ vdd_5v0_sys: regulator-5v0 {
compatible = "regulator-fixed";
regulator-name = "vdd_5v0";
regulator-min-microvolt = <5000000>;
@@ -969,7 +969,7 @@ vdd_5v0_sys: regulator@0 {
regulator-always-on;
};
- vdd_3v3_sys: regulator@1 {
+ vdd_3v3_sys: regulator-3v3 {
compatible = "regulator-fixed";
regulator-name = "vdd_3v3_vs";
regulator-min-microvolt = <3300000>;
@@ -978,7 +978,7 @@ vdd_3v3_sys: regulator@1 {
vin-supply = <&vdd_5v0_sys>;
};
- vdd_1v8_sys: regulator@2 {
+ vdd_1v8_sys: regulator-1v8 {
compatible = "regulator-fixed";
regulator-name = "vdd_1v8_vs";
regulator-min-microvolt = <1800000>;
@@ -987,7 +987,7 @@ vdd_1v8_sys: regulator@2 {
vin-supply = <&vdd_5v0_sys>;
};
- vdd_pnl: regulator@3 {
+ vdd_pnl: regulator-panel {
compatible = "regulator-fixed";
regulator-name = "vdd_panel";
regulator-min-microvolt = <3300000>;
diff --git a/arch/arm/boot/dts/tegra20-harmony.dts b/arch/arm/boot/dts/tegra20-harmony.dts
index ae4312eedcbd..a8494378c382 100644
--- a/arch/arm/boot/dts/tegra20-harmony.dts
+++ b/arch/arm/boot/dts/tegra20-harmony.dts
@@ -640,7 +640,7 @@ backlight: backlight {
default-brightness-level = <6>;
};
- clk32k_in: clock@0 {
+ clk32k_in: clock-32k {
compatible = "fixed-clock";
clock-frequency = <32768>;
#clock-cells = <0>;
@@ -666,7 +666,7 @@ panel: panel {
backlight = <&backlight>;
};
- vdd_5v0_reg: regulator@0 {
+ vdd_5v0_reg: regulator-5v0 {
compatible = "regulator-fixed";
regulator-name = "vdd_5v0";
regulator-min-microvolt = <5000000>;
@@ -674,7 +674,7 @@ vdd_5v0_reg: regulator@0 {
regulator-always-on;
};
- regulator@1 {
+ regulator-1v5 {
compatible = "regulator-fixed";
regulator-name = "vdd_1v5";
regulator-min-microvolt = <1500000>;
@@ -682,7 +682,7 @@ regulator@1 {
gpio = <&pmic 0 GPIO_ACTIVE_HIGH>;
};
- regulator@2 {
+ regulator-1v2 {
compatible = "regulator-fixed";
regulator-name = "vdd_1v2";
regulator-min-microvolt = <1200000>;
@@ -691,7 +691,7 @@ regulator@2 {
enable-active-high;
};
- pci_vdd_reg: regulator@3 {
+ pci_vdd_reg: regulator-1v05 {
compatible = "regulator-fixed";
regulator-name = "vdd_1v05";
regulator-min-microvolt = <1050000>;
@@ -700,7 +700,7 @@ pci_vdd_reg: regulator@3 {
enable-active-high;
};
- vdd_pnl_reg: regulator@4 {
+ vdd_pnl_reg: regulator-pn1 {
compatible = "regulator-fixed";
regulator-name = "vdd_pnl";
regulator-min-microvolt = <2800000>;
@@ -709,7 +709,7 @@ vdd_pnl_reg: regulator@4 {
enable-active-high;
};
- vdd_bl_reg: regulator@5 {
+ vdd_bl_reg: regulator-bl {
compatible = "regulator-fixed";
regulator-name = "vdd_bl";
regulator-min-microvolt = <2800000>;
@@ -718,7 +718,7 @@ vdd_bl_reg: regulator@5 {
enable-active-high;
};
- vdd_5v0_hdmi: regulator@6 {
+ vdd_5v0_hdmi: regulator-hdmi {
compatible = "regulator-fixed";
regulator-name = "VDDIO_HDMI";
regulator-min-microvolt = <5000000>;
diff --git a/arch/arm/boot/dts/tegra20-medcom-wide.dts b/arch/arm/boot/dts/tegra20-medcom-wide.dts
index b31c9bca16e6..cb6199d7d224 100644
--- a/arch/arm/boot/dts/tegra20-medcom-wide.dts
+++ b/arch/arm/boot/dts/tegra20-medcom-wide.dts
@@ -92,7 +92,7 @@ sound {
clock-names = "pll_a", "pll_a_out0", "mclk";
};
- vcc_24v_reg: regulator@100 {
+ vcc_24v_reg: regulator-24v0 {
compatible = "regulator-fixed";
regulator-name = "vcc_24v";
regulator-min-microvolt = <24000000>;
@@ -100,7 +100,7 @@ vcc_24v_reg: regulator@100 {
regulator-always-on;
};
- vdd_5v0_reg: regulator@101 {
+ vdd_5v0_reg: regulator-5v0 {
compatible = "regulator-fixed";
regulator-name = "vdd_5v0";
vin-supply = <&vcc_24v_reg>;
@@ -109,7 +109,7 @@ vdd_5v0_reg: regulator@101 {
regulator-always-on;
};
- vdd_3v3_reg: regulator@102 {
+ vdd_3v3_reg: regulator-3v3 {
compatible = "regulator-fixed";
regulator-name = "vdd_3v3";
vin-supply = <&vcc_24v_reg>;
@@ -118,7 +118,7 @@ vdd_3v3_reg: regulator@102 {
regulator-always-on;
};
- vdd_1v8_reg: regulator@103 {
+ vdd_1v8_reg: regulator-1v8 {
compatible = "regulator-fixed";
regulator-name = "vdd_1v8";
vin-supply = <&vdd_3v3_reg>;
diff --git a/arch/arm/boot/dts/tegra20-paz00.dts b/arch/arm/boot/dts/tegra20-paz00.dts
index acc816bfd233..61c830f57683 100644
--- a/arch/arm/boot/dts/tegra20-paz00.dts
+++ b/arch/arm/boot/dts/tegra20-paz00.dts
@@ -577,7 +577,7 @@ backlight: backlight {
backlight-boot-off;
};
- clk32k_in: clock@0 {
+ clk32k_in: clock-32k {
compatible = "fixed-clock";
clock-frequency = <32768>;
#clock-cells = <0>;
@@ -614,7 +614,7 @@ panel: panel {
backlight = <&backlight>;
};
- p5valw_reg: regulator@0 {
+ p5valw_reg: regulator-5v0alw {
compatible = "regulator-fixed";
regulator-name = "+5valw";
regulator-min-microvolt = <5000000>;
@@ -622,7 +622,7 @@ p5valw_reg: regulator@0 {
regulator-always-on;
};
- vdd_pnl_reg: regulator@1 {
+ vdd_pnl_reg: regulator-3v0 {
compatible = "regulator-fixed";
regulator-name = "+3VS,vdd_pnl";
regulator-min-microvolt = <3300000>;
diff --git a/arch/arm/boot/dts/tegra20-plutux.dts b/arch/arm/boot/dts/tegra20-plutux.dts
index 5811b7006a9b..71a8236491df 100644
--- a/arch/arm/boot/dts/tegra20-plutux.dts
+++ b/arch/arm/boot/dts/tegra20-plutux.dts
@@ -60,7 +60,7 @@ sound {
clock-names = "pll_a", "pll_a_out0", "mclk";
};
- vcc_24v_reg: regulator@100 {
+ vcc_24v_reg: regulator-24v0 {
compatible = "regulator-fixed";
regulator-name = "vcc_24v";
regulator-min-microvolt = <24000000>;
@@ -68,7 +68,7 @@ vcc_24v_reg: regulator@100 {
regulator-always-on;
};
- vdd_5v0_reg: regulator@101 {
+ vdd_5v0_reg: regulator-5v0 {
compatible = "regulator-fixed";
regulator-name = "vdd_5v0";
vin-supply = <&vcc_24v_reg>;
@@ -77,7 +77,7 @@ vdd_5v0_reg: regulator@101 {
regulator-always-on;
};
- vdd_3v3_reg: regulator@102 {
+ vdd_3v3_reg: regulator-3v3 {
compatible = "regulator-fixed";
regulator-name = "vdd_3v3";
vin-supply = <&vcc_24v_reg>;
@@ -86,7 +86,7 @@ vdd_3v3_reg: regulator@102 {
regulator-always-on;
};
- vdd_1v8_reg: regulator@103 {
+ vdd_1v8_reg: regulator-1v8 {
compatible = "regulator-fixed";
regulator-name = "vdd_1v8";
vin-supply = <&vdd_3v3_reg>;
diff --git a/arch/arm/boot/dts/tegra20-seaboard.dts b/arch/arm/boot/dts/tegra20-seaboard.dts
index 92d494b8c3d2..ca7adeb1fdd9 100644
--- a/arch/arm/boot/dts/tegra20-seaboard.dts
+++ b/arch/arm/boot/dts/tegra20-seaboard.dts
@@ -792,7 +792,7 @@ backlight: backlight {
default-brightness-level = <6>;
};
- clk32k_in: clock@0 {
+ clk32k_in: clock-32k {
compatible = "fixed-clock";
clock-frequency = <32768>;
#clock-cells = <0>;
@@ -828,7 +828,7 @@ panel: panel {
ddc-i2c-bus = <&lvds_ddc>;
};
- vdd_5v0_reg: regulator@0 {
+ vdd_5v0_reg: regulator-5v0 {
compatible = "regulator-fixed";
regulator-name = "vdd_5v0";
regulator-min-microvolt = <5000000>;
@@ -836,7 +836,7 @@ vdd_5v0_reg: regulator@0 {
regulator-always-on;
};
- regulator@1 {
+ regulator-1v5 {
compatible = "regulator-fixed";
regulator-name = "vdd_1v5";
regulator-min-microvolt = <1500000>;
@@ -844,7 +844,7 @@ regulator@1 {
gpio = <&pmic 0 GPIO_ACTIVE_HIGH>;
};
- regulator@2 {
+ regulator-1v2 {
compatible = "regulator-fixed";
regulator-name = "vdd_1v2";
regulator-min-microvolt = <1200000>;
@@ -853,7 +853,7 @@ regulator@2 {
enable-active-high;
};
- vbus_reg: regulator@3 {
+ vbus_reg: regulator-vbus {
compatible = "regulator-fixed";
regulator-name = "vdd_vbus_wup1";
regulator-min-microvolt = <5000000>;
@@ -864,7 +864,7 @@ vbus_reg: regulator@3 {
regulator-boot-on;
};
- vdd_pnl_reg: regulator@4 {
+ vdd_pnl_reg: regulator-pnl {
compatible = "regulator-fixed";
regulator-name = "vdd_pnl";
regulator-min-microvolt = <2800000>;
@@ -873,7 +873,7 @@ vdd_pnl_reg: regulator@4 {
enable-active-high;
};
- vdd_bl_reg: regulator@5 {
+ vdd_bl_reg: regulator-bl {
compatible = "regulator-fixed";
regulator-name = "vdd_bl";
regulator-min-microvolt = <2800000>;
@@ -882,7 +882,7 @@ vdd_bl_reg: regulator@5 {
enable-active-high;
};
- vdd_hdmi: regulator@6 {
+ vdd_hdmi: regulator-hdmi {
compatible = "regulator-fixed";
regulator-name = "VDDIO_HDMI";
regulator-min-microvolt = <5000000>;
diff --git a/arch/arm/boot/dts/tegra20-tamonten.dtsi b/arch/arm/boot/dts/tegra20-tamonten.dtsi
index dd4d506683de..2ad56f84bb62 100644
--- a/arch/arm/boot/dts/tegra20-tamonten.dtsi
+++ b/arch/arm/boot/dts/tegra20-tamonten.dtsi
@@ -502,13 +502,13 @@ mmc@c8000600 {
status = "okay";
};
- clk32k_in: clock@0 {
+ clk32k_in: clock-32k {
compatible = "fixed-clock";
clock-frequency = <32768>;
#clock-cells = <0>;
};
- pci_vdd_reg: regulator@1 {
+ pci_vdd_reg: regulator-1v05 {
compatible = "regulator-fixed";
regulator-name = "vdd_1v05";
regulator-min-microvolt = <1050000>;
diff --git a/arch/arm/boot/dts/tegra20-tec.dts b/arch/arm/boot/dts/tegra20-tec.dts
index 10ff09d86efa..4f41c74384b2 100644
--- a/arch/arm/boot/dts/tegra20-tec.dts
+++ b/arch/arm/boot/dts/tegra20-tec.dts
@@ -69,7 +69,7 @@ sound {
clock-names = "pll_a", "pll_a_out0", "mclk";
};
- vcc_24v_reg: regulator@100 {
+ vcc_24v_reg: regulator-24v {
compatible = "regulator-fixed";
regulator-name = "vcc_24v";
regulator-min-microvolt = <24000000>;
@@ -77,7 +77,7 @@ vcc_24v_reg: regulator@100 {
regulator-always-on;
};
- vdd_5v0_reg: regulator@101 {
+ vdd_5v0_reg: regulator-5v0 {
compatible = "regulator-fixed";
regulator-name = "vdd_5v0";
vin-supply = <&vcc_24v_reg>;
@@ -86,7 +86,7 @@ vdd_5v0_reg: regulator@101 {
regulator-always-on;
};
- vdd_3v3_reg: regulator@102 {
+ vdd_3v3_reg: regulator-3v3 {
compatible = "regulator-fixed";
regulator-name = "vdd_3v3";
vin-supply = <&vcc_24v_reg>;
@@ -95,7 +95,7 @@ vdd_3v3_reg: regulator@102 {
regulator-always-on;
};
- vdd_1v8_reg: regulator@103 {
+ vdd_1v8_reg: regulator-1v8 {
compatible = "regulator-fixed";
regulator-name = "vdd_1v8";
vin-supply = <&vdd_3v3_reg>;
diff --git a/arch/arm/boot/dts/tegra20-trimslice.dts b/arch/arm/boot/dts/tegra20-trimslice.dts
index 4bc87bc0c2a4..1f4893699614 100644
--- a/arch/arm/boot/dts/tegra20-trimslice.dts
+++ b/arch/arm/boot/dts/tegra20-trimslice.dts
@@ -379,7 +379,7 @@ mmc@c8000600 {
bus-width = <4>;
};
- clk32k_in: clock@0 {
+ clk32k_in: clock-32k {
compatible = "fixed-clock";
clock-frequency = <32768>;
#clock-cells = <0>;
@@ -401,7 +401,7 @@ poweroff {
gpios = <&gpio TEGRA_GPIO(X, 7) GPIO_ACTIVE_LOW>;
};
- hdmi_vdd_reg: regulator@0 {
+ hdmi_vdd_reg: regulator-hdmi {
compatible = "regulator-fixed";
regulator-name = "avdd_hdmi";
regulator-min-microvolt = <3300000>;
@@ -409,7 +409,7 @@ hdmi_vdd_reg: regulator@0 {
regulator-always-on;
};
- hdmi_pll_reg: regulator@1 {
+ hdmi_pll_reg: regulator-hdmipll {
compatible = "regulator-fixed";
regulator-name = "avdd_hdmi_pll";
regulator-min-microvolt = <1800000>;
@@ -417,7 +417,7 @@ hdmi_pll_reg: regulator@1 {
regulator-always-on;
};
- vbus_reg: regulator@2 {
+ vbus_reg: regulator-vbus {
compatible = "regulator-fixed";
regulator-name = "usb1_vbus";
regulator-min-microvolt = <5000000>;
@@ -428,7 +428,7 @@ vbus_reg: regulator@2 {
regulator-boot-on;
};
- pci_clk_reg: regulator@3 {
+ pci_clk_reg: regulator-pciclk {
compatible = "regulator-fixed";
regulator-name = "pci_clk";
regulator-min-microvolt = <3300000>;
@@ -436,7 +436,7 @@ pci_clk_reg: regulator@3 {
regulator-always-on;
};
- pci_vdd_reg: regulator@4 {
+ pci_vdd_reg: regulator-pcivdd {
compatible = "regulator-fixed";
regulator-name = "pci_vdd";
regulator-min-microvolt = <1050000>;
diff --git a/arch/arm/boot/dts/tegra20-ventana.dts b/arch/arm/boot/dts/tegra20-ventana.dts
index 5a2578b3707f..4284054ebd2c 100644
--- a/arch/arm/boot/dts/tegra20-ventana.dts
+++ b/arch/arm/boot/dts/tegra20-ventana.dts
@@ -606,7 +606,7 @@ backlight: backlight {
default-brightness-level = <6>;
};
- clk32k_in: clock@0 {
+ clk32k_in: clock-32k {
compatible = "fixed-clock";
clock-frequency = <32768>;
#clock-cells = <0>;
@@ -647,7 +647,7 @@ panel: panel {
ddc-i2c-bus = <&lvds_ddc>;
};
- vdd_5v0_reg: regulator@0 {
+ vdd_5v0_reg: regulator-5v0 {
compatible = "regulator-fixed";
regulator-name = "vdd_5v0";
regulator-min-microvolt = <5000000>;
@@ -655,7 +655,7 @@ vdd_5v0_reg: regulator@0 {
regulator-always-on;
};
- regulator@1 {
+ regulator-1v5 {
compatible = "regulator-fixed";
regulator-name = "vdd_1v5";
regulator-min-microvolt = <1500000>;
@@ -663,7 +663,7 @@ regulator@1 {
gpio = <&pmic 0 GPIO_ACTIVE_HIGH>;
};
- regulator@2 {
+ regulator-1v2 {
compatible = "regulator-fixed";
regulator-name = "vdd_1v2";
regulator-min-microvolt = <1200000>;
@@ -672,7 +672,7 @@ regulator@2 {
enable-active-high;
};
- vdd_pnl_reg: regulator@3 {
+ vdd_pnl_reg: regulator-pnl {
compatible = "regulator-fixed";
regulator-name = "vdd_pnl";
regulator-min-microvolt = <2800000>;
@@ -681,7 +681,7 @@ vdd_pnl_reg: regulator@3 {
enable-active-high;
};
- vdd_bl_reg: regulator@4 {
+ vdd_bl_reg: regulator-bl {
compatible = "regulator-fixed";
regulator-name = "vdd_bl";
regulator-min-microvolt = <2800000>;
diff --git a/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-common.dtsi b/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-common.dtsi
index 9732cd6f20b7..6433fde6db02 100644
--- a/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-common.dtsi
+++ b/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-common.dtsi
@@ -1045,7 +1045,7 @@ battery_cell: battery-cell {
};
/* PMIC has a built-in 32KHz oscillator which is used by PMC */
- clk32k_in: clock@0 {
+ clk32k_in: clock-32k {
compatible = "fixed-clock";
#clock-cells = <0>;
clock-frequency = <32768>;
@@ -1183,7 +1183,7 @@ lvds_encoder_output: endpoint {
};
};
- vdd_5v0_sys: regulator@0 {
+ vdd_5v0_sys: regulator-5v0 {
compatible = "regulator-fixed";
regulator-name = "vdd_5v0";
regulator-min-microvolt = <5000000>;
@@ -1192,7 +1192,7 @@ vdd_5v0_sys: regulator@0 {
regulator-boot-on;
};
- vdd_3v3_sys: regulator@1 {
+ vdd_3v3_sys: regulator-3v3 {
compatible = "regulator-fixed";
regulator-name = "vdd_3v3";
regulator-min-microvolt = <3300000>;
@@ -1202,7 +1202,7 @@ vdd_3v3_sys: regulator@1 {
vin-supply = <&vdd_5v0_sys>;
};
- vdd_pnl: regulator@2 {
+ vdd_pnl: regulator-panel {
compatible = "regulator-fixed";
regulator-name = "vdd_panel";
regulator-min-microvolt = <3300000>;
@@ -1213,7 +1213,7 @@ vdd_pnl: regulator@2 {
vin-supply = <&vdd_3v3_sys>;
};
- vcc_3v3_ts: regulator@3 {
+ vcc_3v3_ts: regulator-ts {
compatible = "regulator-fixed";
regulator-name = "ldo_s-1167_3v3";
regulator-min-microvolt = <3300000>;
diff --git a/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-maxim-pmic.dtsi b/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-maxim-pmic.dtsi
index 53966fa4eef2..cd28e8782f7d 100644
--- a/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-maxim-pmic.dtsi
+++ b/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-maxim-pmic.dtsi
@@ -166,12 +166,12 @@ ldo8 {
};
};
- vdd_3v3_sys: regulator@1 {
+ vdd_3v3_sys: regulator-3v3 {
gpio = <&pmic 3 GPIO_ACTIVE_HIGH>;
enable-active-high;
};
- regulator@4 {
+ regulator-usb {
compatible = "regulator-fixed";
regulator-name = "avdd_usb";
regulator-min-microvolt = <3300000>;
diff --git a/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-ti-pmic.dtsi b/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-ti-pmic.dtsi
index 9365ae607239..ee4a3f482769 100644
--- a/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-ti-pmic.dtsi
+++ b/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-ti-pmic.dtsi
@@ -143,7 +143,7 @@ vdd_core: core-regulator@60 {
};
};
- vdd_3v3_sys: regulator@1 {
+ vdd_3v3_sys: regulator-3v3 {
gpio = <&pmic 6 GPIO_ACTIVE_HIGH>;
enable-active-high;
};
diff --git a/arch/arm/boot/dts/tegra30-beaver.dts b/arch/arm/boot/dts/tegra30-beaver.dts
index e159feeedef7..2ec5e47d9623 100644
--- a/arch/arm/boot/dts/tegra30-beaver.dts
+++ b/arch/arm/boot/dts/tegra30-beaver.dts
@@ -1966,7 +1966,7 @@ usb-phy@7d008000 {
status = "okay";
};
- clk32k_in: clock@0 {
+ clk32k_in: clock-32k {
compatible = "fixed-clock";
clock-frequency = <32768>;
#clock-cells = <0>;
@@ -1985,7 +1985,7 @@ gpled2 {
};
};
- vdd_5v_in_reg: regulator@0 {
+ vdd_5v_in_reg: regulator-5v0 {
compatible = "regulator-fixed";
regulator-name = "vdd_5v_in";
regulator-min-microvolt = <5000000>;
@@ -1993,7 +1993,7 @@ vdd_5v_in_reg: regulator@0 {
regulator-always-on;
};
- chargepump_5v_reg: regulator@1 {
+ chargepump_5v_reg: regulator-chargepump {
compatible = "regulator-fixed";
regulator-name = "chargepump_5v";
regulator-min-microvolt = <5000000>;
@@ -2004,7 +2004,7 @@ chargepump_5v_reg: regulator@1 {
gpio = <&pmic 0 GPIO_ACTIVE_HIGH>;
};
- ddr_reg: regulator@2 {
+ ddr_reg: regulator-ddr {
compatible = "regulator-fixed";
regulator-name = "vdd_ddr";
regulator-min-microvolt = <1500000>;
@@ -2016,7 +2016,7 @@ ddr_reg: regulator@2 {
vin-supply = <&vdd_5v_in_reg>;
};
- vdd_5v_sata_reg: regulator@3 {
+ vdd_5v_sata_reg: regulator-sata {
compatible = "regulator-fixed";
regulator-name = "vdd_5v_sata";
regulator-min-microvolt = <5000000>;
@@ -2028,7 +2028,7 @@ vdd_5v_sata_reg: regulator@3 {
vin-supply = <&vdd_5v_in_reg>;
};
- usb1_vbus_reg: regulator@4 {
+ usb1_vbus_reg: regulator-usb1 {
compatible = "regulator-fixed";
regulator-name = "usb1_vbus";
regulator-min-microvolt = <5000000>;
@@ -2039,7 +2039,7 @@ usb1_vbus_reg: regulator@4 {
vin-supply = <&vdd_5v_in_reg>;
};
- usb3_vbus_reg: regulator@5 {
+ usb3_vbus_reg: regulator-usb3 {
compatible = "regulator-fixed";
regulator-name = "usb3_vbus";
regulator-min-microvolt = <5000000>;
@@ -2050,7 +2050,7 @@ usb3_vbus_reg: regulator@5 {
vin-supply = <&vdd_5v_in_reg>;
};
- sys_3v3_reg: regulator@6 {
+ sys_3v3_reg: regulator-3v3 {
compatible = "regulator-fixed";
regulator-name = "sys_3v3,vdd_3v3_alw";
regulator-min-microvolt = <3300000>;
@@ -2062,7 +2062,7 @@ sys_3v3_reg: regulator@6 {
vin-supply = <&vdd_5v_in_reg>;
};
- sys_3v3_pexs_reg: regulator@7 {
+ sys_3v3_pexs_reg: regulator-pexs {
compatible = "regulator-fixed";
regulator-name = "sys_3v3_pexs";
regulator-min-microvolt = <3300000>;
@@ -2074,7 +2074,7 @@ sys_3v3_pexs_reg: regulator@7 {
vin-supply = <&sys_3v3_reg>;
};
- vdd_5v0_hdmi: regulator@8 {
+ vdd_5v0_hdmi: regulator-hdmi {
compatible = "regulator-fixed";
regulator-name = "+VDD_5V_HDMI";
regulator-min-microvolt = <5000000>;
diff --git a/arch/arm/boot/dts/tegra30-cardhu-a02.dts b/arch/arm/boot/dts/tegra30-cardhu-a02.dts
index 4899e05a0d9c..247185314f46 100644
--- a/arch/arm/boot/dts/tegra30-cardhu-a02.dts
+++ b/arch/arm/boot/dts/tegra30-cardhu-a02.dts
@@ -16,7 +16,7 @@ mmc@78000400 {
keep-power-in-suspend;
};
- ddr_reg: regulator@100 {
+ ddr_reg: regulator-ddr {
compatible = "regulator-fixed";
regulator-name = "vdd_ddr";
regulator-min-microvolt = <1500000>;
@@ -27,7 +27,7 @@ ddr_reg: regulator@100 {
gpio = <&pmic 6 GPIO_ACTIVE_HIGH>;
};
- sys_3v3_reg: regulator@101 {
+ sys_3v3_reg: regulator-3v3 {
compatible = "regulator-fixed";
regulator-name = "sys_3v3";
regulator-min-microvolt = <3300000>;
@@ -38,7 +38,7 @@ sys_3v3_reg: regulator@101 {
gpio = <&pmic 7 GPIO_ACTIVE_HIGH>;
};
- usb1_vbus_reg: regulator@102 {
+ usb1_vbus_reg: regulator-usb1 {
compatible = "regulator-fixed";
regulator-name = "usb1_vbus";
regulator-min-microvolt = <5000000>;
@@ -49,7 +49,7 @@ usb1_vbus_reg: regulator@102 {
vin-supply = <&vdd_5v0_reg>;
};
- usb3_vbus_reg: regulator@103 {
+ usb3_vbus_reg: regulator-usb3 {
compatible = "regulator-fixed";
regulator-name = "usb3_vbus";
regulator-min-microvolt = <5000000>;
@@ -60,7 +60,7 @@ usb3_vbus_reg: regulator@103 {
vin-supply = <&vdd_5v0_reg>;
};
- vdd_5v0_reg: regulator@104 {
+ vdd_5v0_reg: regulator-5v0 {
compatible = "regulator-fixed";
regulator-name = "5v0";
regulator-min-microvolt = <5000000>;
@@ -69,7 +69,7 @@ vdd_5v0_reg: regulator@104 {
gpio = <&pmic 2 GPIO_ACTIVE_HIGH>;
};
- vdd_bl_reg: regulator@105 {
+ vdd_bl_reg: regulator-bl {
compatible = "regulator-fixed";
regulator-name = "vdd_bl";
regulator-min-microvolt = <5000000>;
diff --git a/arch/arm/boot/dts/tegra30-cardhu-a04.dts b/arch/arm/boot/dts/tegra30-cardhu-a04.dts
index a11028b8b67b..2911f08863a0 100644
--- a/arch/arm/boot/dts/tegra30-cardhu-a04.dts
+++ b/arch/arm/boot/dts/tegra30-cardhu-a04.dts
@@ -16,7 +16,7 @@ mmc@78000400 {
keep-power-in-suspend;
};
- ddr_reg: regulator@100 {
+ ddr_reg: regulator-ddr {
compatible = "regulator-fixed";
regulator-name = "ddr";
regulator-min-microvolt = <1500000>;
@@ -27,7 +27,7 @@ ddr_reg: regulator@100 {
gpio = <&pmic 7 GPIO_ACTIVE_HIGH>;
};
- sys_3v3_reg: regulator@101 {
+ sys_3v3_reg: regulator-3v3 {
compatible = "regulator-fixed";
regulator-name = "sys_3v3";
regulator-min-microvolt = <3300000>;
@@ -38,7 +38,7 @@ sys_3v3_reg: regulator@101 {
gpio = <&pmic 6 GPIO_ACTIVE_HIGH>;
};
- usb1_vbus_reg: regulator@102 {
+ usb1_vbus_reg: regulator-usb1 {
compatible = "regulator-fixed";
regulator-name = "usb1_vbus";
regulator-min-microvolt = <5000000>;
@@ -49,7 +49,7 @@ usb1_vbus_reg: regulator@102 {
vin-supply = <&vdd_5v0_reg>;
};
- usb3_vbus_reg: regulator@103 {
+ usb3_vbus_reg: regulator-usb3 {
compatible = "regulator-fixed";
regulator-name = "usb3_vbus";
regulator-min-microvolt = <5000000>;
@@ -60,7 +60,7 @@ usb3_vbus_reg: regulator@103 {
vin-supply = <&vdd_5v0_reg>;
};
- vdd_5v0_reg: regulator@104 {
+ vdd_5v0_reg: regulator-5v0 {
compatible = "regulator-fixed";
regulator-name = "5v0";
regulator-min-microvolt = <5000000>;
@@ -69,7 +69,7 @@ vdd_5v0_reg: regulator@104 {
gpio = <&pmic 8 GPIO_ACTIVE_HIGH>;
};
- vdd_bl_reg: regulator@105 {
+ vdd_bl_reg: regulator-bl {
compatible = "regulator-fixed";
regulator-name = "vdd_bl";
regulator-min-microvolt = <5000000>;
@@ -80,7 +80,7 @@ vdd_bl_reg: regulator@105 {
gpio = <&gpio TEGRA_GPIO(DD, 2) GPIO_ACTIVE_HIGH>;
};
- vdd_bl2_reg: regulator@106 {
+ vdd_bl2_reg: regulator-bl2 {
compatible = "regulator-fixed";
regulator-name = "vdd_bl2";
regulator-min-microvolt = <5000000>;
diff --git a/arch/arm/boot/dts/tegra30-cardhu.dtsi b/arch/arm/boot/dts/tegra30-cardhu.dtsi
index 448f1397e64a..d2f6121049a4 100644
--- a/arch/arm/boot/dts/tegra30-cardhu.dtsi
+++ b/arch/arm/boot/dts/tegra30-cardhu.dtsi
@@ -433,7 +433,7 @@ backlight: backlight {
default-brightness-level = <6>;
};
- clk32k_in: clock@0 {
+ clk32k_in: clock-32k {
compatible = "fixed-clock";
clock-frequency = <32768>;
#clock-cells = <0>;
@@ -475,7 +475,7 @@ panel: panel {
backlight = <&backlight>;
};
- vdd_ac_bat_reg: regulator@0 {
+ vdd_ac_bat_reg: regulator-acbat {
compatible = "regulator-fixed";
regulator-name = "vdd_ac_bat";
regulator-min-microvolt = <5000000>;
@@ -483,7 +483,7 @@ vdd_ac_bat_reg: regulator@0 {
regulator-always-on;
};
- cam_1v8_reg: regulator@1 {
+ cam_1v8_reg: regulator-cam {
compatible = "regulator-fixed";
regulator-name = "cam_1v8";
regulator-min-microvolt = <1800000>;
@@ -493,7 +493,7 @@ cam_1v8_reg: regulator@1 {
vin-supply = <&vio_reg>;
};
- cp_5v_reg: regulator@2 {
+ cp_5v_reg: regulator-5v0cp {
compatible = "regulator-fixed";
regulator-name = "cp_5v";
regulator-min-microvolt = <5000000>;
@@ -504,7 +504,7 @@ cp_5v_reg: regulator@2 {
gpio = <&pmic 0 GPIO_ACTIVE_HIGH>;
};
- emmc_3v3_reg: regulator@3 {
+ emmc_3v3_reg: regulator-emmc {
compatible = "regulator-fixed";
regulator-name = "emmc_3v3";
regulator-min-microvolt = <3300000>;
@@ -516,7 +516,7 @@ emmc_3v3_reg: regulator@3 {
vin-supply = <&sys_3v3_reg>;
};
- modem_3v3_reg: regulator@4 {
+ modem_3v3_reg: regulator-modem {
compatible = "regulator-fixed";
regulator-name = "modem_3v3";
regulator-min-microvolt = <3300000>;
@@ -525,7 +525,7 @@ modem_3v3_reg: regulator@4 {
gpio = <&gpio TEGRA_GPIO(D, 6) GPIO_ACTIVE_HIGH>;
};
- pex_hvdd_3v3_reg: regulator@5 {
+ pex_hvdd_3v3_reg: regulator-pex {
compatible = "regulator-fixed";
regulator-name = "pex_hvdd_3v3";
regulator-min-microvolt = <3300000>;
@@ -535,7 +535,7 @@ pex_hvdd_3v3_reg: regulator@5 {
vin-supply = <&sys_3v3_reg>;
};
- vdd_cam1_ldo_reg: regulator@6 {
+ vdd_cam1_ldo_reg: regulator-cam1 {
compatible = "regulator-fixed";
regulator-name = "vdd_cam1_ldo";
regulator-min-microvolt = <2800000>;
@@ -545,7 +545,7 @@ vdd_cam1_ldo_reg: regulator@6 {
vin-supply = <&sys_3v3_reg>;
};
- vdd_cam2_ldo_reg: regulator@7 {
+ vdd_cam2_ldo_reg: regulator-cam2 {
compatible = "regulator-fixed";
regulator-name = "vdd_cam2_ldo";
regulator-min-microvolt = <2800000>;
@@ -555,7 +555,7 @@ vdd_cam2_ldo_reg: regulator@7 {
vin-supply = <&sys_3v3_reg>;
};
- vdd_cam3_ldo_reg: regulator@8 {
+ vdd_cam3_ldo_reg: regulator-cam3 {
compatible = "regulator-fixed";
regulator-name = "vdd_cam3_ldo";
regulator-min-microvolt = <3300000>;
@@ -565,7 +565,7 @@ vdd_cam3_ldo_reg: regulator@8 {
vin-supply = <&sys_3v3_reg>;
};
- vdd_com_reg: regulator@9 {
+ vdd_com_reg: regulator-com {
compatible = "regulator-fixed";
regulator-name = "vdd_com";
regulator-min-microvolt = <3300000>;
@@ -577,7 +577,7 @@ vdd_com_reg: regulator@9 {
vin-supply = <&sys_3v3_reg>;
};
- vdd_fuse_3v3_reg: regulator@10 {
+ vdd_fuse_3v3_reg: regulator-fuse {
compatible = "regulator-fixed";
regulator-name = "vdd_fuse_3v3";
regulator-min-microvolt = <3300000>;
@@ -587,7 +587,7 @@ vdd_fuse_3v3_reg: regulator@10 {
vin-supply = <&sys_3v3_reg>;
};
- vdd_pnl1_reg: regulator@11 {
+ vdd_pnl1_reg: regulator-pnl1 {
compatible = "regulator-fixed";
regulator-name = "vdd_pnl1";
regulator-min-microvolt = <3300000>;
@@ -599,7 +599,7 @@ vdd_pnl1_reg: regulator@11 {
vin-supply = <&sys_3v3_reg>;
};
- vdd_vid_reg: regulator@12 {
+ vdd_vid_reg: regulator-vid {
compatible = "regulator-fixed";
regulator-name = "vddio_vid";
regulator-min-microvolt = <5000000>;
--
2.32.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH v1 3/3] ARM: tegra: Update Broadcom Bluetooth device-tree nodes
2021-09-26 23:37 [PATCH v1 0/3] Tegra ARM32 device-tree improvements for 5.16 Dmitry Osipenko
2021-09-26 23:37 ` [PATCH v1 1/3] ARM: tegra: acer-a500: Correct compatible of ak8975 magnetometer Dmitry Osipenko
2021-09-26 23:37 ` [PATCH v1 2/3] ARM: tegra: Name clock and regulator nodes according to DT-schema Dmitry Osipenko
@ 2021-09-26 23:37 ` Dmitry Osipenko
2021-10-04 20:41 ` Thierry Reding
2 siblings, 1 reply; 8+ messages in thread
From: Dmitry Osipenko @ 2021-09-26 23:37 UTC (permalink / raw)
To: Thierry Reding, Jonathan Hunter, David Heidelberg
Cc: devicetree, linux-kernel, linux-tegra
The host-wakeup GPIO is now marked as deprecated in the broadcom-bluetooth
device-tree binding, it's replaced with the host-wakeup interrupt. Update
Tegra device-trees to the recent version of the Bluetooth binding.
Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
---
arch/arm/boot/dts/tegra20-acer-a500-picasso.dts | 5 ++++-
arch/arm/boot/dts/tegra30-asus-nexus7-grouper-common.dtsi | 5 ++++-
arch/arm/boot/dts/tegra30-ouya.dts | 5 ++++-
3 files changed, 12 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/tegra20-acer-a500-picasso.dts b/arch/arm/boot/dts/tegra20-acer-a500-picasso.dts
index 0abfd6215016..2e6340e92fe1 100644
--- a/arch/arm/boot/dts/tegra20-acer-a500-picasso.dts
+++ b/arch/arm/boot/dts/tegra20-acer-a500-picasso.dts
@@ -393,6 +393,10 @@ uartc: serial@70006200 {
bluetooth {
compatible = "brcm,bcm4329-bt";
+ interrupt-parent = <&gpio>;
+ interrupts = <TEGRA_GPIO(U, 6) IRQ_TYPE_EDGE_RISING>;
+ interrupt-names = "host-wakeup";
+
/* PLLP 216MHz / 16 / 4 */
max-speed = <3375000>;
@@ -403,7 +407,6 @@ bluetooth {
vddio-supply = <&vdd_1v8_sys>;
device-wakeup-gpios = <&gpio TEGRA_GPIO(U, 1) GPIO_ACTIVE_HIGH>;
- host-wakeup-gpios = <&gpio TEGRA_GPIO(U, 6) GPIO_ACTIVE_HIGH>;
shutdown-gpios = <&gpio TEGRA_GPIO(U, 0) GPIO_ACTIVE_HIGH>;
};
};
diff --git a/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-common.dtsi b/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-common.dtsi
index 6433fde6db02..56e566fb922b 100644
--- a/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-common.dtsi
+++ b/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-common.dtsi
@@ -819,6 +819,10 @@ uartc: serial@70006200 {
bluetooth {
compatible = "brcm,bcm4330-bt";
+ interrupt-parent = <&gpio>;
+ interrupts = <TEGRA_GPIO(U, 6) IRQ_TYPE_EDGE_RISING>;
+ interrupt-names = "host-wakeup";
+
max-speed = <4000000>;
clocks = <&tegra_pmc TEGRA_PMC_CLK_BLINK>;
@@ -828,7 +832,6 @@ bluetooth {
vddio-supply = <&vdd_1v8>;
device-wakeup-gpios = <&gpio TEGRA_GPIO(U, 1) GPIO_ACTIVE_HIGH>;
- host-wakeup-gpios = <&gpio TEGRA_GPIO(U, 6) GPIO_ACTIVE_HIGH>;
shutdown-gpios = <&gpio TEGRA_GPIO(U, 0) GPIO_ACTIVE_HIGH>;
};
};
diff --git a/arch/arm/boot/dts/tegra30-ouya.dts b/arch/arm/boot/dts/tegra30-ouya.dts
index 90db5ff72537..4259871b76c9 100644
--- a/arch/arm/boot/dts/tegra30-ouya.dts
+++ b/arch/arm/boot/dts/tegra30-ouya.dts
@@ -93,6 +93,10 @@ uartc: serial@70006200 {
bluetooth {
compatible = "brcm,bcm4330-bt";
+ interrupt-parent = <&gpio>;
+ interrupts = <TEGRA_GPIO(U, 6) IRQ_TYPE_EDGE_RISING>;
+ interrupt-names = "host-wakeup";
+
max-speed = <4000000>;
clocks = <&tegra_pmc TEGRA_PMC_CLK_BLINK>;
@@ -103,7 +107,6 @@ bluetooth {
shutdown-gpios = <&gpio TEGRA_GPIO(U, 0) GPIO_ACTIVE_HIGH>;
device-wakeup-gpios = <&gpio TEGRA_GPIO(U, 1) GPIO_ACTIVE_HIGH>;
- host-wakeup-gpios = <&gpio TEGRA_GPIO(U, 6) GPIO_ACTIVE_HIGH>;
};
};
--
2.32.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v1 1/3] ARM: tegra: acer-a500: Correct compatible of ak8975 magnetometer
2021-09-26 23:37 ` [PATCH v1 1/3] ARM: tegra: acer-a500: Correct compatible of ak8975 magnetometer Dmitry Osipenko
@ 2021-10-04 20:40 ` Thierry Reding
0 siblings, 0 replies; 8+ messages in thread
From: Thierry Reding @ 2021-10-04 20:40 UTC (permalink / raw)
To: Dmitry Osipenko
Cc: Thierry Reding, Jonathan Hunter, David Heidelberg, devicetree,
linux-kernel, linux-tegra
[-- Attachment #1: Type: text/plain, Size: 445 bytes --]
On Mon, Sep 27, 2021 at 02:37:02AM +0300, Dmitry Osipenko wrote:
> From: David Heidelberg <david@ixit.cz>
>
> The "ak,ak8975" compatible is not recognized by dt-bindings, it's
> deprecated. Use supported "asahi-kasei,ak8975" compatible.
>
> Signed-off-by: David Heidelberg <david@ixit.cz>
> ---
> arch/arm/boot/dts/tegra20-acer-a500-picasso.dts | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Applied, thanks.
Thierry
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v1 3/3] ARM: tegra: Update Broadcom Bluetooth device-tree nodes
2021-09-26 23:37 ` [PATCH v1 3/3] ARM: tegra: Update Broadcom Bluetooth device-tree nodes Dmitry Osipenko
@ 2021-10-04 20:41 ` Thierry Reding
0 siblings, 0 replies; 8+ messages in thread
From: Thierry Reding @ 2021-10-04 20:41 UTC (permalink / raw)
To: Dmitry Osipenko
Cc: Thierry Reding, Jonathan Hunter, David Heidelberg, devicetree,
linux-kernel, linux-tegra
[-- Attachment #1: Type: text/plain, Size: 656 bytes --]
On Mon, Sep 27, 2021 at 02:37:04AM +0300, Dmitry Osipenko wrote:
> The host-wakeup GPIO is now marked as deprecated in the broadcom-bluetooth
> device-tree binding, it's replaced with the host-wakeup interrupt. Update
> Tegra device-trees to the recent version of the Bluetooth binding.
>
> Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
> ---
> arch/arm/boot/dts/tegra20-acer-a500-picasso.dts | 5 ++++-
> arch/arm/boot/dts/tegra30-asus-nexus7-grouper-common.dtsi | 5 ++++-
> arch/arm/boot/dts/tegra30-ouya.dts | 5 ++++-
> 3 files changed, 12 insertions(+), 3 deletions(-)
Applied, thanks.
Thierry
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v1 2/3] ARM: tegra: Name clock and regulator nodes according to DT-schema
2021-09-26 23:37 ` [PATCH v1 2/3] ARM: tegra: Name clock and regulator nodes according to DT-schema Dmitry Osipenko
@ 2021-10-04 20:56 ` Thierry Reding
2021-10-24 19:08 ` Dmitry Osipenko
0 siblings, 1 reply; 8+ messages in thread
From: Thierry Reding @ 2021-10-04 20:56 UTC (permalink / raw)
To: Dmitry Osipenko, Rob Herring
Cc: Thierry Reding, Jonathan Hunter, David Heidelberg, devicetree,
linux-kernel, linux-tegra
[-- Attachment #1: Type: text/plain, Size: 51930 bytes --]
On Mon, Sep 27, 2021 at 02:37:03AM +0300, Dmitry Osipenko wrote:
> From: David Heidelberg <david@ixit.cz>
>
> Name clocks and regulators according to DT-schema to fix warnings such as:
>
> arch/arm/boot/dts/tegra20-acer-a500-picasso.dt.yaml: /: clock@0: 'anyOf' conditional failed, one must be fixed:
> 'reg' is a required property
> 'ranges' is a required property
> From schema: /home/runner/.local/lib/python3.8/site-packages/dtschema/schemas/root-node.yaml
>
> Signed-off-by: David Heidelberg <david@ixit.cz>
> ---
> arch/arm/boot/dts/tegra114-dalmore.dts | 16 +++++------
> arch/arm/boot/dts/tegra114-roth.dts | 14 +++++-----
> arch/arm/boot/dts/tegra114-tn7.dts | 8 +++---
> arch/arm/boot/dts/tegra124-jetson-tk1.dts | 26 ++++++++---------
> arch/arm/boot/dts/tegra124-nyan.dtsi | 28 +++++++++----------
> arch/arm/boot/dts/tegra124-venice2.dts | 28 +++++++++----------
> .../boot/dts/tegra20-acer-a500-picasso.dts | 12 ++++----
> arch/arm/boot/dts/tegra20-harmony.dts | 16 +++++------
> arch/arm/boot/dts/tegra20-medcom-wide.dts | 8 +++---
> arch/arm/boot/dts/tegra20-paz00.dts | 6 ++--
> arch/arm/boot/dts/tegra20-plutux.dts | 8 +++---
> arch/arm/boot/dts/tegra20-seaboard.dts | 16 +++++------
> arch/arm/boot/dts/tegra20-tamonten.dtsi | 4 +--
> arch/arm/boot/dts/tegra20-tec.dts | 8 +++---
> arch/arm/boot/dts/tegra20-trimslice.dts | 12 ++++----
> arch/arm/boot/dts/tegra20-ventana.dts | 12 ++++----
> .../tegra30-asus-nexus7-grouper-common.dtsi | 10 +++----
> ...egra30-asus-nexus7-grouper-maxim-pmic.dtsi | 4 +--
> .../tegra30-asus-nexus7-grouper-ti-pmic.dtsi | 2 +-
> arch/arm/boot/dts/tegra30-beaver.dts | 20 ++++++-------
> arch/arm/boot/dts/tegra30-cardhu-a02.dts | 12 ++++----
> arch/arm/boot/dts/tegra30-cardhu-a04.dts | 14 +++++-----
> arch/arm/boot/dts/tegra30-cardhu.dtsi | 28 +++++++++----------
> 23 files changed, 156 insertions(+), 156 deletions(-)
I'm not sure about this one. Last time I talked to Rob about this, he
seemed to suggest that reintroducing the top-level clocks and regulators
nodes was preferable to doing these renamed.
Rob, which way do we want to decide this? If we do want to reintroduce
the clocks and regulators top-level "bus" nodes, we probably need the
tooling to learn to accept that, because I /think/ the reason why they
were originally dropped a while ago was because the tooling complained
about the "bus" nodes not being named "bus" (I think that was because
they traditionally have had "simple-bus" in their compatible and hence
the "simple-bus" schema was applied along with the schema for the node
name.
So I think we have a couple of options here:
* Make it so that these nodes no longer need the "simple-bus"
compatible string: this would need new code to parse these nodes
and instantiate platform device for them like "simple-bus" would
do.
* Relax the rules for the "simple-bus" compatible string. It looks
like this would be the simplest, because we could just extend the
pattern in the "simple-bus" schema. There's a slight catch, though,
because we'd also need to allow "#size-cells = <0>" for these. And
maybe also make "ranges" not required because it doesn't make sense
for #size-cells = <0>.
* Create a new compatible string specifically for these. That would be
similar to the first option, except that it's perhaps a bit more
formal. We could basically create a separate schema for this type of
bus, so that simple-bus could remain as-is.
Perhaps there's also yet another option that I haven't thought of that's
better than the above.
Quoting in full below for you to get a better picture of what kinds of
changes we're talking about here.
Thierry
>
> diff --git a/arch/arm/boot/dts/tegra114-dalmore.dts b/arch/arm/boot/dts/tegra114-dalmore.dts
> index 7fd901f8d39a..d23050bfaba6 100644
> --- a/arch/arm/boot/dts/tegra114-dalmore.dts
> +++ b/arch/arm/boot/dts/tegra114-dalmore.dts
> @@ -1151,7 +1151,7 @@ backlight: backlight {
> default-brightness-level = <6>;
> };
>
> - clk32k_in: clock@0 {
> + clk32k_in: clock-32k {
> compatible = "fixed-clock";
> clock-frequency = <32768>;
> #clock-cells = <0>;
> @@ -1186,7 +1186,7 @@ volume_up {
> };
> };
>
> - vdd_ac_bat_reg: regulator@0 {
> + vdd_ac_bat_reg: regulator-acbat {
> compatible = "regulator-fixed";
> regulator-name = "vdd_ac_bat";
> regulator-min-microvolt = <5000000>;
> @@ -1194,7 +1194,7 @@ vdd_ac_bat_reg: regulator@0 {
> regulator-always-on;
> };
>
> - dvdd_ts_reg: regulator@1 {
> + dvdd_ts_reg: regulator-ts {
> compatible = "regulator-fixed";
> regulator-name = "dvdd_ts";
> regulator-min-microvolt = <1800000>;
> @@ -1203,7 +1203,7 @@ dvdd_ts_reg: regulator@1 {
> gpio = <&gpio TEGRA_GPIO(H, 5) GPIO_ACTIVE_HIGH>;
> };
>
> - usb1_vbus_reg: regulator@3 {
> + usb1_vbus_reg: regulator-usb1 {
> compatible = "regulator-fixed";
> regulator-name = "usb1_vbus";
> regulator-min-microvolt = <5000000>;
> @@ -1214,7 +1214,7 @@ usb1_vbus_reg: regulator@3 {
> vin-supply = <&tps65090_dcdc1_reg>;
> };
>
> - usb3_vbus_reg: regulator@4 {
> + usb3_vbus_reg: regulator-usb3 {
> compatible = "regulator-fixed";
> regulator-name = "usb2_vbus";
> regulator-min-microvolt = <5000000>;
> @@ -1225,7 +1225,7 @@ usb3_vbus_reg: regulator@4 {
> vin-supply = <&tps65090_dcdc1_reg>;
> };
>
> - vdd_hdmi_reg: regulator@5 {
> + vdd_hdmi_reg: regulator-hdmi {
> compatible = "regulator-fixed";
> regulator-name = "vdd_hdmi_5v0";
> regulator-min-microvolt = <5000000>;
> @@ -1233,7 +1233,7 @@ vdd_hdmi_reg: regulator@5 {
> vin-supply = <&tps65090_dcdc1_reg>;
> };
>
> - vdd_cam_1v8_reg: regulator@6 {
> + vdd_cam_1v8_reg: regulator-cam {
> compatible = "regulator-fixed";
> regulator-name = "vdd_cam_1v8_reg";
> regulator-min-microvolt = <1800000>;
> @@ -1242,7 +1242,7 @@ vdd_cam_1v8_reg: regulator@6 {
> gpio = <&palmas_gpio 6 0>;
> };
>
> - vdd_5v0_hdmi: regulator@7 {
> + vdd_5v0_hdmi: regulator-hdmicon {
> compatible = "regulator-fixed";
> regulator-name = "VDD_5V0_HDMI_CON";
> regulator-min-microvolt = <5000000>;
> diff --git a/arch/arm/boot/dts/tegra114-roth.dts b/arch/arm/boot/dts/tegra114-roth.dts
> index 07960171fabe..18521707ff5b 100644
> --- a/arch/arm/boot/dts/tegra114-roth.dts
> +++ b/arch/arm/boot/dts/tegra114-roth.dts
> @@ -1016,7 +1016,7 @@ backlight: backlight {
> enable-gpios = <&gpio TEGRA_GPIO(H, 2) GPIO_ACTIVE_HIGH>;
> };
>
> - clk32k_in: clock@0 {
> + clk32k_in: clock-32k {
> compatible = "fixed-clock";
> clock-frequency = <32768>;
> #clock-cells = <0>;
> @@ -1045,7 +1045,7 @@ power {
> };
> };
>
> - lcd_bl_en: regulator@0 {
> + lcd_bl_en: regulator-lcden {
> compatible = "regulator-fixed";
> regulator-name = "lcd_bl_en";
> regulator-min-microvolt = <5000000>;
> @@ -1053,7 +1053,7 @@ lcd_bl_en: regulator@0 {
> regulator-boot-on;
> };
>
> - vdd_lcd: regulator@1 {
> + vdd_lcd: regulator-lcd {
> compatible = "regulator-fixed";
> regulator-name = "vdd_lcd_1v8";
> regulator-min-microvolt = <1800000>;
> @@ -1064,7 +1064,7 @@ vdd_lcd: regulator@1 {
> regulator-boot-on;
> };
>
> - regulator@2 {
> + regulator-1v8ts {
> compatible = "regulator-fixed";
> regulator-name = "vdd_1v8_ts";
> regulator-min-microvolt = <1800000>;
> @@ -1073,7 +1073,7 @@ regulator@2 {
> regulator-boot-on;
> };
>
> - regulator@3 {
> + regulator-3v3ts {
> compatible = "regulator-fixed";
> regulator-name = "vdd_3v3_ts";
> regulator-min-microvolt = <3300000>;
> @@ -1083,7 +1083,7 @@ regulator@3 {
> regulator-boot-on;
> };
>
> - regulator@4 {
> + regulator-1v8com {
> compatible = "regulator-fixed";
> regulator-name = "vdd_1v8_com";
> regulator-min-microvolt = <1800000>;
> @@ -1094,7 +1094,7 @@ regulator@4 {
> regulator-boot-on;
> };
>
> - regulator@5 {
> + regulator-3v3com {
> compatible = "regulator-fixed";
> regulator-name = "vdd_3v3_com";
> regulator-min-microvolt = <3300000>;
> diff --git a/arch/arm/boot/dts/tegra114-tn7.dts b/arch/arm/boot/dts/tegra114-tn7.dts
> index 745d234b105b..0534247458f0 100644
> --- a/arch/arm/boot/dts/tegra114-tn7.dts
> +++ b/arch/arm/boot/dts/tegra114-tn7.dts
> @@ -273,7 +273,7 @@ backlight: backlight {
> power-supply = <&lcd_bl_en>;
> };
>
> - clk32k_in: clock@0 {
> + clk32k_in: clock-32k {
> compatible = "fixed-clock";
> clock-frequency = <32768>;
> #clock-cells = <0>;
> @@ -303,7 +303,7 @@ volume_up {
> };
>
> /* FIXME: output of BQ24192 */
> - vs_sys: regulator@0 {
> + vs_sys: regulator-vs {
> compatible = "regulator-fixed";
> regulator-name = "VS_SYS";
> regulator-min-microvolt = <4200000>;
> @@ -312,7 +312,7 @@ vs_sys: regulator@0 {
> regulator-boot-on;
> };
>
> - lcd_bl_en: regulator@1 {
> + lcd_bl_en: regulator-lcden {
> compatible = "regulator-fixed";
> regulator-name = "VDD_LCD_BL";
> regulator-min-microvolt = <16500000>;
> @@ -323,7 +323,7 @@ lcd_bl_en: regulator@1 {
> regulator-boot-on;
> };
>
> - vdd_lcd: regulator@2 {
> + vdd_lcd: regulator-lcd {
> compatible = "regulator-fixed";
> regulator-name = "VD_LCD_1V8";
> regulator-min-microvolt = <1800000>;
> diff --git a/arch/arm/boot/dts/tegra124-jetson-tk1.dts b/arch/arm/boot/dts/tegra124-jetson-tk1.dts
> index 35ab296408e1..094f94a627b2 100644
> --- a/arch/arm/boot/dts/tegra124-jetson-tk1.dts
> +++ b/arch/arm/boot/dts/tegra124-jetson-tk1.dts
> @@ -1868,7 +1868,7 @@ usb-phy@7d008000 {
> vbus-supply = <&vdd_usb3_vbus>;
> };
>
> - clk32k_in: clock@0 {
> + clk32k_in: clock-32k {
> compatible = "fixed-clock";
> clock-frequency = <32768>;
> #clock-cells = <0>;
> @@ -1892,7 +1892,7 @@ power {
> };
> };
>
> - vdd_mux: regulator@0 {
> + vdd_mux: regulator-mux {
> compatible = "regulator-fixed";
> regulator-name = "+VDD_MUX";
> regulator-min-microvolt = <12000000>;
> @@ -1901,7 +1901,7 @@ vdd_mux: regulator@0 {
> regulator-boot-on;
> };
>
> - vdd_5v0_sys: regulator@1 {
> + vdd_5v0_sys: regulator-5v0sys {
> compatible = "regulator-fixed";
> regulator-name = "+5V_SYS";
> regulator-min-microvolt = <5000000>;
> @@ -1911,7 +1911,7 @@ vdd_5v0_sys: regulator@1 {
> vin-supply = <&vdd_mux>;
> };
>
> - vdd_3v3_sys: regulator@2 {
> + vdd_3v3_sys: regulator-3v3sys {
> compatible = "regulator-fixed";
> regulator-name = "+3.3V_SYS";
> regulator-min-microvolt = <3300000>;
> @@ -1921,7 +1921,7 @@ vdd_3v3_sys: regulator@2 {
> vin-supply = <&vdd_mux>;
> };
>
> - vdd_3v3_run: regulator@3 {
> + vdd_3v3_run: regulator-3v3run {
> compatible = "regulator-fixed";
> regulator-name = "+3.3V_RUN";
> regulator-min-microvolt = <3300000>;
> @@ -1933,7 +1933,7 @@ vdd_3v3_run: regulator@3 {
> vin-supply = <&vdd_3v3_sys>;
> };
>
> - vdd_3v3_hdmi: regulator@4 {
> + vdd_3v3_hdmi: regulator-3v3hdmi {
> compatible = "regulator-fixed";
> regulator-name = "+3.3V_AVDD_HDMI_AP_GATED";
> regulator-min-microvolt = <3300000>;
> @@ -1941,7 +1941,7 @@ vdd_3v3_hdmi: regulator@4 {
> vin-supply = <&vdd_3v3_run>;
> };
>
> - vdd_usb1_vbus: regulator@5 {
> + vdd_usb1_vbus: regulator-usb1 {
> compatible = "regulator-fixed";
> regulator-name = "+USB0_VBUS_SW";
> regulator-min-microvolt = <5000000>;
> @@ -1952,7 +1952,7 @@ vdd_usb1_vbus: regulator@5 {
> vin-supply = <&vdd_5v0_sys>;
> };
>
> - vdd_usb3_vbus: regulator@6 {
> + vdd_usb3_vbus: regulator-usb3 {
> compatible = "regulator-fixed";
> regulator-name = "+5V_USB_HS";
> regulator-min-microvolt = <5000000>;
> @@ -1963,7 +1963,7 @@ vdd_usb3_vbus: regulator@6 {
> vin-supply = <&vdd_5v0_sys>;
> };
>
> - vdd_3v3_lp0: regulator@7 {
> + vdd_3v3_lp0: regulator-lp0 {
> compatible = "regulator-fixed";
> regulator-name = "+3.3V_LP0";
> regulator-min-microvolt = <3300000>;
> @@ -1975,7 +1975,7 @@ vdd_3v3_lp0: regulator@7 {
> vin-supply = <&vdd_3v3_sys>;
> };
>
> - vdd_hdmi_pll: regulator@8 {
> + vdd_hdmi_pll: regulator-hdmipll {
> compatible = "regulator-fixed";
> regulator-name = "+1.05V_RUN_AVDD_HDMI_PLL";
> regulator-min-microvolt = <1050000>;
> @@ -1984,7 +1984,7 @@ vdd_hdmi_pll: regulator@8 {
> vin-supply = <&vdd_1v05_run>;
> };
>
> - vdd_5v0_hdmi: regulator@9 {
> + vdd_5v0_hdmi: regulator-hdmicon {
> compatible = "regulator-fixed";
> regulator-name = "+5V_HDMI_CON";
> regulator-min-microvolt = <5000000>;
> @@ -1995,7 +1995,7 @@ vdd_5v0_hdmi: regulator@9 {
> };
>
> /* Molex power connector */
> - vdd_5v0_sata: regulator@10 {
> + vdd_5v0_sata: regulator-5v0sata {
> compatible = "regulator-fixed";
> regulator-name = "+5V_SATA";
> regulator-min-microvolt = <5000000>;
> @@ -2005,7 +2005,7 @@ vdd_5v0_sata: regulator@10 {
> vin-supply = <&vdd_5v0_sys>;
> };
>
> - vdd_12v0_sata: regulator@11 {
> + vdd_12v0_sata: regulator-12v0sata {
> compatible = "regulator-fixed";
> regulator-name = "+12V_SATA";
> regulator-min-microvolt = <12000000>;
> diff --git a/arch/arm/boot/dts/tegra124-nyan.dtsi b/arch/arm/boot/dts/tegra124-nyan.dtsi
> index 63a81270300a..2d045d1a9bcf 100644
> --- a/arch/arm/boot/dts/tegra124-nyan.dtsi
> +++ b/arch/arm/boot/dts/tegra124-nyan.dtsi
> @@ -582,7 +582,7 @@ backlight: backlight {
> 256>;
> };
>
> - clk32k_in: clock@0 {
> + clk32k_in: clock-32k {
> compatible = "fixed-clock";
> clock-frequency = <32768>;
> #clock-cells = <0>;
> @@ -615,7 +615,7 @@ power {
> };
> };
>
> - vdd_mux: regulator@0 {
> + vdd_mux: regulator-mux {
> compatible = "regulator-fixed";
> regulator-name = "+VDD_MUX";
> regulator-min-microvolt = <12000000>;
> @@ -624,7 +624,7 @@ vdd_mux: regulator@0 {
> regulator-boot-on;
> };
>
> - vdd_5v0_sys: regulator@1 {
> + vdd_5v0_sys: regulator-5v0sys {
> compatible = "regulator-fixed";
> regulator-name = "+5V_SYS";
> regulator-min-microvolt = <5000000>;
> @@ -634,7 +634,7 @@ vdd_5v0_sys: regulator@1 {
> vin-supply = <&vdd_mux>;
> };
>
> - vdd_3v3_sys: regulator@2 {
> + vdd_3v3_sys: regulator-3v3sys {
> compatible = "regulator-fixed";
> regulator-name = "+3.3V_SYS";
> regulator-min-microvolt = <3300000>;
> @@ -644,7 +644,7 @@ vdd_3v3_sys: regulator@2 {
> vin-supply = <&vdd_mux>;
> };
>
> - vdd_3v3_run: regulator@3 {
> + vdd_3v3_run: regulator-3v3run {
> compatible = "regulator-fixed";
> regulator-name = "+3.3V_RUN";
> regulator-min-microvolt = <3300000>;
> @@ -656,7 +656,7 @@ vdd_3v3_run: regulator@3 {
> vin-supply = <&vdd_3v3_sys>;
> };
>
> - vdd_3v3_hdmi: regulator@4 {
> + vdd_3v3_hdmi: regulator-3v3hdmi {
> compatible = "regulator-fixed";
> regulator-name = "+3.3V_AVDD_HDMI_AP_GATED";
> regulator-min-microvolt = <3300000>;
> @@ -664,7 +664,7 @@ vdd_3v3_hdmi: regulator@4 {
> vin-supply = <&vdd_3v3_run>;
> };
>
> - vdd_led: regulator@5 {
> + vdd_led: regulator-led {
> compatible = "regulator-fixed";
> regulator-name = "+VDD_LED";
> gpio = <&gpio TEGRA_GPIO(P, 2) GPIO_ACTIVE_HIGH>;
> @@ -672,7 +672,7 @@ vdd_led: regulator@5 {
> vin-supply = <&vdd_mux>;
> };
>
> - vdd_5v0_ts: regulator@6 {
> + vdd_5v0_ts: regulator-ts {
> compatible = "regulator-fixed";
> regulator-name = "+5V_VDD_TS_SW";
> regulator-min-microvolt = <5000000>;
> @@ -683,7 +683,7 @@ vdd_5v0_ts: regulator@6 {
> vin-supply = <&vdd_5v0_sys>;
> };
>
> - vdd_usb1_vbus: regulator@7 {
> + vdd_usb1_vbus: regulator-usb1 {
> compatible = "regulator-fixed";
> regulator-name = "+5V_USB_HS";
> regulator-min-microvolt = <5000000>;
> @@ -694,7 +694,7 @@ vdd_usb1_vbus: regulator@7 {
> vin-supply = <&vdd_5v0_sys>;
> };
>
> - vdd_usb3_vbus: regulator@8 {
> + vdd_usb3_vbus: regulator-usb3 {
> compatible = "regulator-fixed";
> regulator-name = "+5V_USB_SS";
> regulator-min-microvolt = <5000000>;
> @@ -705,7 +705,7 @@ vdd_usb3_vbus: regulator@8 {
> vin-supply = <&vdd_5v0_sys>;
> };
>
> - vdd_3v3_panel: regulator@9 {
> + vdd_3v3_panel: regulator-panel {
> compatible = "regulator-fixed";
> regulator-name = "+3.3V_PANEL";
> regulator-min-microvolt = <3300000>;
> @@ -715,7 +715,7 @@ vdd_3v3_panel: regulator@9 {
> vin-supply = <&vdd_3v3_run>;
> };
>
> - vdd_3v3_lp0: regulator@10 {
> + vdd_3v3_lp0: regulator-lp0 {
> compatible = "regulator-fixed";
> regulator-name = "+3.3V_LP0";
> regulator-min-microvolt = <3300000>;
> @@ -730,7 +730,7 @@ vdd_3v3_lp0: regulator@10 {
> vin-supply = <&vdd_3v3_sys>;
> };
>
> - vdd_hdmi_pll: regulator@11 {
> + vdd_hdmi_pll: regulator-hdmipll {
> compatible = "regulator-fixed";
> regulator-name = "+1.05V_RUN_AVDD_HDMI_PLL";
> regulator-min-microvolt = <1050000>;
> @@ -739,7 +739,7 @@ vdd_hdmi_pll: regulator@11 {
> vin-supply = <&vdd_1v05_run>;
> };
>
> - vdd_5v0_hdmi: regulator@12 {
> + vdd_5v0_hdmi: regulator-hdmicon {
> compatible = "regulator-fixed";
> regulator-name = "+5V_HDMI_CON";
> regulator-min-microvolt = <5000000>;
> diff --git a/arch/arm/boot/dts/tegra124-venice2.dts b/arch/arm/boot/dts/tegra124-venice2.dts
> index e6b54ac1ebd1..457b8aada1f8 100644
> --- a/arch/arm/boot/dts/tegra124-venice2.dts
> +++ b/arch/arm/boot/dts/tegra124-venice2.dts
> @@ -1061,7 +1061,7 @@ backlight: backlight {
> default-brightness-level = <6>;
> };
>
> - clk32k_in: clock@0 {
> + clk32k_in: clock-32k {
> compatible = "fixed-clock";
> clock-frequency = <32768>;
> #clock-cells = <0>;
> @@ -1086,7 +1086,7 @@ panel: panel {
> ddc-i2c-bus = <&dpaux>;
> };
>
> - vdd_mux: regulator@0 {
> + vdd_mux: regulator-mux {
> compatible = "regulator-fixed";
> regulator-name = "+VDD_MUX";
> regulator-min-microvolt = <12000000>;
> @@ -1095,7 +1095,7 @@ vdd_mux: regulator@0 {
> regulator-boot-on;
> };
>
> - vdd_5v0_sys: regulator@1 {
> + vdd_5v0_sys: regulator-5v0sys {
> compatible = "regulator-fixed";
> regulator-name = "+5V_SYS";
> regulator-min-microvolt = <5000000>;
> @@ -1105,7 +1105,7 @@ vdd_5v0_sys: regulator@1 {
> vin-supply = <&vdd_mux>;
> };
>
> - vdd_3v3_sys: regulator@2 {
> + vdd_3v3_sys: regulator-3v3sys {
> compatible = "regulator-fixed";
> regulator-name = "+3.3V_SYS";
> regulator-min-microvolt = <3300000>;
> @@ -1115,7 +1115,7 @@ vdd_3v3_sys: regulator@2 {
> vin-supply = <&vdd_mux>;
> };
>
> - vdd_3v3_run: regulator@3 {
> + vdd_3v3_run: regulator-3v3run {
> compatible = "regulator-fixed";
> regulator-name = "+3.3V_RUN";
> regulator-min-microvolt = <3300000>;
> @@ -1127,7 +1127,7 @@ vdd_3v3_run: regulator@3 {
> vin-supply = <&vdd_3v3_sys>;
> };
>
> - vdd_3v3_hdmi: regulator@4 {
> + vdd_3v3_hdmi: regulator-hdmi {
> compatible = "regulator-fixed";
> regulator-name = "+3.3V_AVDD_HDMI_AP_GATED";
> regulator-min-microvolt = <3300000>;
> @@ -1135,7 +1135,7 @@ vdd_3v3_hdmi: regulator@4 {
> vin-supply = <&vdd_3v3_run>;
> };
>
> - vdd_led: regulator@5 {
> + vdd_led: regulator-led {
> compatible = "regulator-fixed";
> regulator-name = "+VDD_LED";
> regulator-min-microvolt = <3300000>;
> @@ -1145,7 +1145,7 @@ vdd_led: regulator@5 {
> vin-supply = <&vdd_mux>;
> };
>
> - vdd_5v0_ts: regulator@6 {
> + vdd_5v0_ts: regulator-ts {
> compatible = "regulator-fixed";
> regulator-name = "+5V_VDD_TS_SW";
> regulator-min-microvolt = <5000000>;
> @@ -1156,7 +1156,7 @@ vdd_5v0_ts: regulator@6 {
> vin-supply = <&vdd_5v0_sys>;
> };
>
> - vdd_usb1_vbus: regulator@7 {
> + vdd_usb1_vbus: regulator-usb1 {
> compatible = "regulator-fixed";
> regulator-name = "+5V_USB_HS";
> regulator-min-microvolt = <5000000>;
> @@ -1167,7 +1167,7 @@ vdd_usb1_vbus: regulator@7 {
> vin-supply = <&vdd_5v0_sys>;
> };
>
> - vdd_usb3_vbus: regulator@8 {
> + vdd_usb3_vbus: regulator-usb3 {
> compatible = "regulator-fixed";
> regulator-name = "+5V_USB_SS";
> regulator-min-microvolt = <5000000>;
> @@ -1178,7 +1178,7 @@ vdd_usb3_vbus: regulator@8 {
> vin-supply = <&vdd_5v0_sys>;
> };
>
> - vdd_3v3_panel: regulator@9 {
> + vdd_3v3_panel: regulator-panel {
> compatible = "regulator-fixed";
> regulator-name = "+3.3V_PANEL";
> regulator-min-microvolt = <3300000>;
> @@ -1188,7 +1188,7 @@ vdd_3v3_panel: regulator@9 {
> vin-supply = <&vdd_3v3_run>;
> };
>
> - vdd_3v3_lp0: regulator@10 {
> + vdd_3v3_lp0: regulator-lp0 {
> compatible = "regulator-fixed";
> regulator-name = "+3.3V_LP0";
> regulator-min-microvolt = <3300000>;
> @@ -1203,7 +1203,7 @@ vdd_3v3_lp0: regulator@10 {
> vin-supply = <&vdd_3v3_sys>;
> };
>
> - vdd_hdmi_pll: regulator@11 {
> + vdd_hdmi_pll: regulator-hdmipll {
> compatible = "regulator-fixed";
> regulator-name = "+1.05V_RUN_AVDD_HDMI_PLL";
> regulator-min-microvolt = <1050000>;
> @@ -1212,7 +1212,7 @@ vdd_hdmi_pll: regulator@11 {
> vin-supply = <&vdd_1v05_run>;
> };
>
> - vdd_5v0_hdmi: regulator@12 {
> + vdd_5v0_hdmi: regulator-hdmicon {
> compatible = "regulator-fixed";
> regulator-name = "+5V_HDMI_CON";
> regulator-min-microvolt = <5000000>;
> diff --git a/arch/arm/boot/dts/tegra20-acer-a500-picasso.dts b/arch/arm/boot/dts/tegra20-acer-a500-picasso.dts
> index 5ac288fa9c5e..0abfd6215016 100644
> --- a/arch/arm/boot/dts/tegra20-acer-a500-picasso.dts
> +++ b/arch/arm/boot/dts/tegra20-acer-a500-picasso.dts
> @@ -824,7 +824,7 @@ bat1010: battery-2s1p {
> };
>
> /* PMIC has a built-in 32KHz oscillator which is used by PMC */
> - clk32k_in: clock@0 {
> + clk32k_in: clock-32k-in {
> compatible = "fixed-clock";
> #clock-cells = <0>;
> clock-frequency = <32768>;
> @@ -836,7 +836,7 @@ clk32k_in: clock@0 {
> * oscillator is used as a reference clock-source by the
> * Azurewave WiFi/BT module.
> */
> - rtc_32k_wifi: clock@1 {
> + rtc_32k_wifi: clock-32k-wifi {
> compatible = "fixed-clock";
> #clock-cells = <0>;
> clock-frequency = <32768>;
> @@ -961,7 +961,7 @@ lvds_encoder_output: endpoint {
> };
> };
>
> - vdd_5v0_sys: regulator@0 {
> + vdd_5v0_sys: regulator-5v0 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_5v0";
> regulator-min-microvolt = <5000000>;
> @@ -969,7 +969,7 @@ vdd_5v0_sys: regulator@0 {
> regulator-always-on;
> };
>
> - vdd_3v3_sys: regulator@1 {
> + vdd_3v3_sys: regulator-3v3 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_3v3_vs";
> regulator-min-microvolt = <3300000>;
> @@ -978,7 +978,7 @@ vdd_3v3_sys: regulator@1 {
> vin-supply = <&vdd_5v0_sys>;
> };
>
> - vdd_1v8_sys: regulator@2 {
> + vdd_1v8_sys: regulator-1v8 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_1v8_vs";
> regulator-min-microvolt = <1800000>;
> @@ -987,7 +987,7 @@ vdd_1v8_sys: regulator@2 {
> vin-supply = <&vdd_5v0_sys>;
> };
>
> - vdd_pnl: regulator@3 {
> + vdd_pnl: regulator-panel {
> compatible = "regulator-fixed";
> regulator-name = "vdd_panel";
> regulator-min-microvolt = <3300000>;
> diff --git a/arch/arm/boot/dts/tegra20-harmony.dts b/arch/arm/boot/dts/tegra20-harmony.dts
> index ae4312eedcbd..a8494378c382 100644
> --- a/arch/arm/boot/dts/tegra20-harmony.dts
> +++ b/arch/arm/boot/dts/tegra20-harmony.dts
> @@ -640,7 +640,7 @@ backlight: backlight {
> default-brightness-level = <6>;
> };
>
> - clk32k_in: clock@0 {
> + clk32k_in: clock-32k {
> compatible = "fixed-clock";
> clock-frequency = <32768>;
> #clock-cells = <0>;
> @@ -666,7 +666,7 @@ panel: panel {
> backlight = <&backlight>;
> };
>
> - vdd_5v0_reg: regulator@0 {
> + vdd_5v0_reg: regulator-5v0 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_5v0";
> regulator-min-microvolt = <5000000>;
> @@ -674,7 +674,7 @@ vdd_5v0_reg: regulator@0 {
> regulator-always-on;
> };
>
> - regulator@1 {
> + regulator-1v5 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_1v5";
> regulator-min-microvolt = <1500000>;
> @@ -682,7 +682,7 @@ regulator@1 {
> gpio = <&pmic 0 GPIO_ACTIVE_HIGH>;
> };
>
> - regulator@2 {
> + regulator-1v2 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_1v2";
> regulator-min-microvolt = <1200000>;
> @@ -691,7 +691,7 @@ regulator@2 {
> enable-active-high;
> };
>
> - pci_vdd_reg: regulator@3 {
> + pci_vdd_reg: regulator-1v05 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_1v05";
> regulator-min-microvolt = <1050000>;
> @@ -700,7 +700,7 @@ pci_vdd_reg: regulator@3 {
> enable-active-high;
> };
>
> - vdd_pnl_reg: regulator@4 {
> + vdd_pnl_reg: regulator-pn1 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_pnl";
> regulator-min-microvolt = <2800000>;
> @@ -709,7 +709,7 @@ vdd_pnl_reg: regulator@4 {
> enable-active-high;
> };
>
> - vdd_bl_reg: regulator@5 {
> + vdd_bl_reg: regulator-bl {
> compatible = "regulator-fixed";
> regulator-name = "vdd_bl";
> regulator-min-microvolt = <2800000>;
> @@ -718,7 +718,7 @@ vdd_bl_reg: regulator@5 {
> enable-active-high;
> };
>
> - vdd_5v0_hdmi: regulator@6 {
> + vdd_5v0_hdmi: regulator-hdmi {
> compatible = "regulator-fixed";
> regulator-name = "VDDIO_HDMI";
> regulator-min-microvolt = <5000000>;
> diff --git a/arch/arm/boot/dts/tegra20-medcom-wide.dts b/arch/arm/boot/dts/tegra20-medcom-wide.dts
> index b31c9bca16e6..cb6199d7d224 100644
> --- a/arch/arm/boot/dts/tegra20-medcom-wide.dts
> +++ b/arch/arm/boot/dts/tegra20-medcom-wide.dts
> @@ -92,7 +92,7 @@ sound {
> clock-names = "pll_a", "pll_a_out0", "mclk";
> };
>
> - vcc_24v_reg: regulator@100 {
> + vcc_24v_reg: regulator-24v0 {
> compatible = "regulator-fixed";
> regulator-name = "vcc_24v";
> regulator-min-microvolt = <24000000>;
> @@ -100,7 +100,7 @@ vcc_24v_reg: regulator@100 {
> regulator-always-on;
> };
>
> - vdd_5v0_reg: regulator@101 {
> + vdd_5v0_reg: regulator-5v0 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_5v0";
> vin-supply = <&vcc_24v_reg>;
> @@ -109,7 +109,7 @@ vdd_5v0_reg: regulator@101 {
> regulator-always-on;
> };
>
> - vdd_3v3_reg: regulator@102 {
> + vdd_3v3_reg: regulator-3v3 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_3v3";
> vin-supply = <&vcc_24v_reg>;
> @@ -118,7 +118,7 @@ vdd_3v3_reg: regulator@102 {
> regulator-always-on;
> };
>
> - vdd_1v8_reg: regulator@103 {
> + vdd_1v8_reg: regulator-1v8 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_1v8";
> vin-supply = <&vdd_3v3_reg>;
> diff --git a/arch/arm/boot/dts/tegra20-paz00.dts b/arch/arm/boot/dts/tegra20-paz00.dts
> index acc816bfd233..61c830f57683 100644
> --- a/arch/arm/boot/dts/tegra20-paz00.dts
> +++ b/arch/arm/boot/dts/tegra20-paz00.dts
> @@ -577,7 +577,7 @@ backlight: backlight {
> backlight-boot-off;
> };
>
> - clk32k_in: clock@0 {
> + clk32k_in: clock-32k {
> compatible = "fixed-clock";
> clock-frequency = <32768>;
> #clock-cells = <0>;
> @@ -614,7 +614,7 @@ panel: panel {
> backlight = <&backlight>;
> };
>
> - p5valw_reg: regulator@0 {
> + p5valw_reg: regulator-5v0alw {
> compatible = "regulator-fixed";
> regulator-name = "+5valw";
> regulator-min-microvolt = <5000000>;
> @@ -622,7 +622,7 @@ p5valw_reg: regulator@0 {
> regulator-always-on;
> };
>
> - vdd_pnl_reg: regulator@1 {
> + vdd_pnl_reg: regulator-3v0 {
> compatible = "regulator-fixed";
> regulator-name = "+3VS,vdd_pnl";
> regulator-min-microvolt = <3300000>;
> diff --git a/arch/arm/boot/dts/tegra20-plutux.dts b/arch/arm/boot/dts/tegra20-plutux.dts
> index 5811b7006a9b..71a8236491df 100644
> --- a/arch/arm/boot/dts/tegra20-plutux.dts
> +++ b/arch/arm/boot/dts/tegra20-plutux.dts
> @@ -60,7 +60,7 @@ sound {
> clock-names = "pll_a", "pll_a_out0", "mclk";
> };
>
> - vcc_24v_reg: regulator@100 {
> + vcc_24v_reg: regulator-24v0 {
> compatible = "regulator-fixed";
> regulator-name = "vcc_24v";
> regulator-min-microvolt = <24000000>;
> @@ -68,7 +68,7 @@ vcc_24v_reg: regulator@100 {
> regulator-always-on;
> };
>
> - vdd_5v0_reg: regulator@101 {
> + vdd_5v0_reg: regulator-5v0 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_5v0";
> vin-supply = <&vcc_24v_reg>;
> @@ -77,7 +77,7 @@ vdd_5v0_reg: regulator@101 {
> regulator-always-on;
> };
>
> - vdd_3v3_reg: regulator@102 {
> + vdd_3v3_reg: regulator-3v3 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_3v3";
> vin-supply = <&vcc_24v_reg>;
> @@ -86,7 +86,7 @@ vdd_3v3_reg: regulator@102 {
> regulator-always-on;
> };
>
> - vdd_1v8_reg: regulator@103 {
> + vdd_1v8_reg: regulator-1v8 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_1v8";
> vin-supply = <&vdd_3v3_reg>;
> diff --git a/arch/arm/boot/dts/tegra20-seaboard.dts b/arch/arm/boot/dts/tegra20-seaboard.dts
> index 92d494b8c3d2..ca7adeb1fdd9 100644
> --- a/arch/arm/boot/dts/tegra20-seaboard.dts
> +++ b/arch/arm/boot/dts/tegra20-seaboard.dts
> @@ -792,7 +792,7 @@ backlight: backlight {
> default-brightness-level = <6>;
> };
>
> - clk32k_in: clock@0 {
> + clk32k_in: clock-32k {
> compatible = "fixed-clock";
> clock-frequency = <32768>;
> #clock-cells = <0>;
> @@ -828,7 +828,7 @@ panel: panel {
> ddc-i2c-bus = <&lvds_ddc>;
> };
>
> - vdd_5v0_reg: regulator@0 {
> + vdd_5v0_reg: regulator-5v0 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_5v0";
> regulator-min-microvolt = <5000000>;
> @@ -836,7 +836,7 @@ vdd_5v0_reg: regulator@0 {
> regulator-always-on;
> };
>
> - regulator@1 {
> + regulator-1v5 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_1v5";
> regulator-min-microvolt = <1500000>;
> @@ -844,7 +844,7 @@ regulator@1 {
> gpio = <&pmic 0 GPIO_ACTIVE_HIGH>;
> };
>
> - regulator@2 {
> + regulator-1v2 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_1v2";
> regulator-min-microvolt = <1200000>;
> @@ -853,7 +853,7 @@ regulator@2 {
> enable-active-high;
> };
>
> - vbus_reg: regulator@3 {
> + vbus_reg: regulator-vbus {
> compatible = "regulator-fixed";
> regulator-name = "vdd_vbus_wup1";
> regulator-min-microvolt = <5000000>;
> @@ -864,7 +864,7 @@ vbus_reg: regulator@3 {
> regulator-boot-on;
> };
>
> - vdd_pnl_reg: regulator@4 {
> + vdd_pnl_reg: regulator-pnl {
> compatible = "regulator-fixed";
> regulator-name = "vdd_pnl";
> regulator-min-microvolt = <2800000>;
> @@ -873,7 +873,7 @@ vdd_pnl_reg: regulator@4 {
> enable-active-high;
> };
>
> - vdd_bl_reg: regulator@5 {
> + vdd_bl_reg: regulator-bl {
> compatible = "regulator-fixed";
> regulator-name = "vdd_bl";
> regulator-min-microvolt = <2800000>;
> @@ -882,7 +882,7 @@ vdd_bl_reg: regulator@5 {
> enable-active-high;
> };
>
> - vdd_hdmi: regulator@6 {
> + vdd_hdmi: regulator-hdmi {
> compatible = "regulator-fixed";
> regulator-name = "VDDIO_HDMI";
> regulator-min-microvolt = <5000000>;
> diff --git a/arch/arm/boot/dts/tegra20-tamonten.dtsi b/arch/arm/boot/dts/tegra20-tamonten.dtsi
> index dd4d506683de..2ad56f84bb62 100644
> --- a/arch/arm/boot/dts/tegra20-tamonten.dtsi
> +++ b/arch/arm/boot/dts/tegra20-tamonten.dtsi
> @@ -502,13 +502,13 @@ mmc@c8000600 {
> status = "okay";
> };
>
> - clk32k_in: clock@0 {
> + clk32k_in: clock-32k {
> compatible = "fixed-clock";
> clock-frequency = <32768>;
> #clock-cells = <0>;
> };
>
> - pci_vdd_reg: regulator@1 {
> + pci_vdd_reg: regulator-1v05 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_1v05";
> regulator-min-microvolt = <1050000>;
> diff --git a/arch/arm/boot/dts/tegra20-tec.dts b/arch/arm/boot/dts/tegra20-tec.dts
> index 10ff09d86efa..4f41c74384b2 100644
> --- a/arch/arm/boot/dts/tegra20-tec.dts
> +++ b/arch/arm/boot/dts/tegra20-tec.dts
> @@ -69,7 +69,7 @@ sound {
> clock-names = "pll_a", "pll_a_out0", "mclk";
> };
>
> - vcc_24v_reg: regulator@100 {
> + vcc_24v_reg: regulator-24v {
> compatible = "regulator-fixed";
> regulator-name = "vcc_24v";
> regulator-min-microvolt = <24000000>;
> @@ -77,7 +77,7 @@ vcc_24v_reg: regulator@100 {
> regulator-always-on;
> };
>
> - vdd_5v0_reg: regulator@101 {
> + vdd_5v0_reg: regulator-5v0 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_5v0";
> vin-supply = <&vcc_24v_reg>;
> @@ -86,7 +86,7 @@ vdd_5v0_reg: regulator@101 {
> regulator-always-on;
> };
>
> - vdd_3v3_reg: regulator@102 {
> + vdd_3v3_reg: regulator-3v3 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_3v3";
> vin-supply = <&vcc_24v_reg>;
> @@ -95,7 +95,7 @@ vdd_3v3_reg: regulator@102 {
> regulator-always-on;
> };
>
> - vdd_1v8_reg: regulator@103 {
> + vdd_1v8_reg: regulator-1v8 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_1v8";
> vin-supply = <&vdd_3v3_reg>;
> diff --git a/arch/arm/boot/dts/tegra20-trimslice.dts b/arch/arm/boot/dts/tegra20-trimslice.dts
> index 4bc87bc0c2a4..1f4893699614 100644
> --- a/arch/arm/boot/dts/tegra20-trimslice.dts
> +++ b/arch/arm/boot/dts/tegra20-trimslice.dts
> @@ -379,7 +379,7 @@ mmc@c8000600 {
> bus-width = <4>;
> };
>
> - clk32k_in: clock@0 {
> + clk32k_in: clock-32k {
> compatible = "fixed-clock";
> clock-frequency = <32768>;
> #clock-cells = <0>;
> @@ -401,7 +401,7 @@ poweroff {
> gpios = <&gpio TEGRA_GPIO(X, 7) GPIO_ACTIVE_LOW>;
> };
>
> - hdmi_vdd_reg: regulator@0 {
> + hdmi_vdd_reg: regulator-hdmi {
> compatible = "regulator-fixed";
> regulator-name = "avdd_hdmi";
> regulator-min-microvolt = <3300000>;
> @@ -409,7 +409,7 @@ hdmi_vdd_reg: regulator@0 {
> regulator-always-on;
> };
>
> - hdmi_pll_reg: regulator@1 {
> + hdmi_pll_reg: regulator-hdmipll {
> compatible = "regulator-fixed";
> regulator-name = "avdd_hdmi_pll";
> regulator-min-microvolt = <1800000>;
> @@ -417,7 +417,7 @@ hdmi_pll_reg: regulator@1 {
> regulator-always-on;
> };
>
> - vbus_reg: regulator@2 {
> + vbus_reg: regulator-vbus {
> compatible = "regulator-fixed";
> regulator-name = "usb1_vbus";
> regulator-min-microvolt = <5000000>;
> @@ -428,7 +428,7 @@ vbus_reg: regulator@2 {
> regulator-boot-on;
> };
>
> - pci_clk_reg: regulator@3 {
> + pci_clk_reg: regulator-pciclk {
> compatible = "regulator-fixed";
> regulator-name = "pci_clk";
> regulator-min-microvolt = <3300000>;
> @@ -436,7 +436,7 @@ pci_clk_reg: regulator@3 {
> regulator-always-on;
> };
>
> - pci_vdd_reg: regulator@4 {
> + pci_vdd_reg: regulator-pcivdd {
> compatible = "regulator-fixed";
> regulator-name = "pci_vdd";
> regulator-min-microvolt = <1050000>;
> diff --git a/arch/arm/boot/dts/tegra20-ventana.dts b/arch/arm/boot/dts/tegra20-ventana.dts
> index 5a2578b3707f..4284054ebd2c 100644
> --- a/arch/arm/boot/dts/tegra20-ventana.dts
> +++ b/arch/arm/boot/dts/tegra20-ventana.dts
> @@ -606,7 +606,7 @@ backlight: backlight {
> default-brightness-level = <6>;
> };
>
> - clk32k_in: clock@0 {
> + clk32k_in: clock-32k {
> compatible = "fixed-clock";
> clock-frequency = <32768>;
> #clock-cells = <0>;
> @@ -647,7 +647,7 @@ panel: panel {
> ddc-i2c-bus = <&lvds_ddc>;
> };
>
> - vdd_5v0_reg: regulator@0 {
> + vdd_5v0_reg: regulator-5v0 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_5v0";
> regulator-min-microvolt = <5000000>;
> @@ -655,7 +655,7 @@ vdd_5v0_reg: regulator@0 {
> regulator-always-on;
> };
>
> - regulator@1 {
> + regulator-1v5 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_1v5";
> regulator-min-microvolt = <1500000>;
> @@ -663,7 +663,7 @@ regulator@1 {
> gpio = <&pmic 0 GPIO_ACTIVE_HIGH>;
> };
>
> - regulator@2 {
> + regulator-1v2 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_1v2";
> regulator-min-microvolt = <1200000>;
> @@ -672,7 +672,7 @@ regulator@2 {
> enable-active-high;
> };
>
> - vdd_pnl_reg: regulator@3 {
> + vdd_pnl_reg: regulator-pnl {
> compatible = "regulator-fixed";
> regulator-name = "vdd_pnl";
> regulator-min-microvolt = <2800000>;
> @@ -681,7 +681,7 @@ vdd_pnl_reg: regulator@3 {
> enable-active-high;
> };
>
> - vdd_bl_reg: regulator@4 {
> + vdd_bl_reg: regulator-bl {
> compatible = "regulator-fixed";
> regulator-name = "vdd_bl";
> regulator-min-microvolt = <2800000>;
> diff --git a/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-common.dtsi b/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-common.dtsi
> index 9732cd6f20b7..6433fde6db02 100644
> --- a/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-common.dtsi
> +++ b/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-common.dtsi
> @@ -1045,7 +1045,7 @@ battery_cell: battery-cell {
> };
>
> /* PMIC has a built-in 32KHz oscillator which is used by PMC */
> - clk32k_in: clock@0 {
> + clk32k_in: clock-32k {
> compatible = "fixed-clock";
> #clock-cells = <0>;
> clock-frequency = <32768>;
> @@ -1183,7 +1183,7 @@ lvds_encoder_output: endpoint {
> };
> };
>
> - vdd_5v0_sys: regulator@0 {
> + vdd_5v0_sys: regulator-5v0 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_5v0";
> regulator-min-microvolt = <5000000>;
> @@ -1192,7 +1192,7 @@ vdd_5v0_sys: regulator@0 {
> regulator-boot-on;
> };
>
> - vdd_3v3_sys: regulator@1 {
> + vdd_3v3_sys: regulator-3v3 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_3v3";
> regulator-min-microvolt = <3300000>;
> @@ -1202,7 +1202,7 @@ vdd_3v3_sys: regulator@1 {
> vin-supply = <&vdd_5v0_sys>;
> };
>
> - vdd_pnl: regulator@2 {
> + vdd_pnl: regulator-panel {
> compatible = "regulator-fixed";
> regulator-name = "vdd_panel";
> regulator-min-microvolt = <3300000>;
> @@ -1213,7 +1213,7 @@ vdd_pnl: regulator@2 {
> vin-supply = <&vdd_3v3_sys>;
> };
>
> - vcc_3v3_ts: regulator@3 {
> + vcc_3v3_ts: regulator-ts {
> compatible = "regulator-fixed";
> regulator-name = "ldo_s-1167_3v3";
> regulator-min-microvolt = <3300000>;
> diff --git a/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-maxim-pmic.dtsi b/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-maxim-pmic.dtsi
> index 53966fa4eef2..cd28e8782f7d 100644
> --- a/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-maxim-pmic.dtsi
> +++ b/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-maxim-pmic.dtsi
> @@ -166,12 +166,12 @@ ldo8 {
> };
> };
>
> - vdd_3v3_sys: regulator@1 {
> + vdd_3v3_sys: regulator-3v3 {
> gpio = <&pmic 3 GPIO_ACTIVE_HIGH>;
> enable-active-high;
> };
>
> - regulator@4 {
> + regulator-usb {
> compatible = "regulator-fixed";
> regulator-name = "avdd_usb";
> regulator-min-microvolt = <3300000>;
> diff --git a/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-ti-pmic.dtsi b/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-ti-pmic.dtsi
> index 9365ae607239..ee4a3f482769 100644
> --- a/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-ti-pmic.dtsi
> +++ b/arch/arm/boot/dts/tegra30-asus-nexus7-grouper-ti-pmic.dtsi
> @@ -143,7 +143,7 @@ vdd_core: core-regulator@60 {
> };
> };
>
> - vdd_3v3_sys: regulator@1 {
> + vdd_3v3_sys: regulator-3v3 {
> gpio = <&pmic 6 GPIO_ACTIVE_HIGH>;
> enable-active-high;
> };
> diff --git a/arch/arm/boot/dts/tegra30-beaver.dts b/arch/arm/boot/dts/tegra30-beaver.dts
> index e159feeedef7..2ec5e47d9623 100644
> --- a/arch/arm/boot/dts/tegra30-beaver.dts
> +++ b/arch/arm/boot/dts/tegra30-beaver.dts
> @@ -1966,7 +1966,7 @@ usb-phy@7d008000 {
> status = "okay";
> };
>
> - clk32k_in: clock@0 {
> + clk32k_in: clock-32k {
> compatible = "fixed-clock";
> clock-frequency = <32768>;
> #clock-cells = <0>;
> @@ -1985,7 +1985,7 @@ gpled2 {
> };
> };
>
> - vdd_5v_in_reg: regulator@0 {
> + vdd_5v_in_reg: regulator-5v0 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_5v_in";
> regulator-min-microvolt = <5000000>;
> @@ -1993,7 +1993,7 @@ vdd_5v_in_reg: regulator@0 {
> regulator-always-on;
> };
>
> - chargepump_5v_reg: regulator@1 {
> + chargepump_5v_reg: regulator-chargepump {
> compatible = "regulator-fixed";
> regulator-name = "chargepump_5v";
> regulator-min-microvolt = <5000000>;
> @@ -2004,7 +2004,7 @@ chargepump_5v_reg: regulator@1 {
> gpio = <&pmic 0 GPIO_ACTIVE_HIGH>;
> };
>
> - ddr_reg: regulator@2 {
> + ddr_reg: regulator-ddr {
> compatible = "regulator-fixed";
> regulator-name = "vdd_ddr";
> regulator-min-microvolt = <1500000>;
> @@ -2016,7 +2016,7 @@ ddr_reg: regulator@2 {
> vin-supply = <&vdd_5v_in_reg>;
> };
>
> - vdd_5v_sata_reg: regulator@3 {
> + vdd_5v_sata_reg: regulator-sata {
> compatible = "regulator-fixed";
> regulator-name = "vdd_5v_sata";
> regulator-min-microvolt = <5000000>;
> @@ -2028,7 +2028,7 @@ vdd_5v_sata_reg: regulator@3 {
> vin-supply = <&vdd_5v_in_reg>;
> };
>
> - usb1_vbus_reg: regulator@4 {
> + usb1_vbus_reg: regulator-usb1 {
> compatible = "regulator-fixed";
> regulator-name = "usb1_vbus";
> regulator-min-microvolt = <5000000>;
> @@ -2039,7 +2039,7 @@ usb1_vbus_reg: regulator@4 {
> vin-supply = <&vdd_5v_in_reg>;
> };
>
> - usb3_vbus_reg: regulator@5 {
> + usb3_vbus_reg: regulator-usb3 {
> compatible = "regulator-fixed";
> regulator-name = "usb3_vbus";
> regulator-min-microvolt = <5000000>;
> @@ -2050,7 +2050,7 @@ usb3_vbus_reg: regulator@5 {
> vin-supply = <&vdd_5v_in_reg>;
> };
>
> - sys_3v3_reg: regulator@6 {
> + sys_3v3_reg: regulator-3v3 {
> compatible = "regulator-fixed";
> regulator-name = "sys_3v3,vdd_3v3_alw";
> regulator-min-microvolt = <3300000>;
> @@ -2062,7 +2062,7 @@ sys_3v3_reg: regulator@6 {
> vin-supply = <&vdd_5v_in_reg>;
> };
>
> - sys_3v3_pexs_reg: regulator@7 {
> + sys_3v3_pexs_reg: regulator-pexs {
> compatible = "regulator-fixed";
> regulator-name = "sys_3v3_pexs";
> regulator-min-microvolt = <3300000>;
> @@ -2074,7 +2074,7 @@ sys_3v3_pexs_reg: regulator@7 {
> vin-supply = <&sys_3v3_reg>;
> };
>
> - vdd_5v0_hdmi: regulator@8 {
> + vdd_5v0_hdmi: regulator-hdmi {
> compatible = "regulator-fixed";
> regulator-name = "+VDD_5V_HDMI";
> regulator-min-microvolt = <5000000>;
> diff --git a/arch/arm/boot/dts/tegra30-cardhu-a02.dts b/arch/arm/boot/dts/tegra30-cardhu-a02.dts
> index 4899e05a0d9c..247185314f46 100644
> --- a/arch/arm/boot/dts/tegra30-cardhu-a02.dts
> +++ b/arch/arm/boot/dts/tegra30-cardhu-a02.dts
> @@ -16,7 +16,7 @@ mmc@78000400 {
> keep-power-in-suspend;
> };
>
> - ddr_reg: regulator@100 {
> + ddr_reg: regulator-ddr {
> compatible = "regulator-fixed";
> regulator-name = "vdd_ddr";
> regulator-min-microvolt = <1500000>;
> @@ -27,7 +27,7 @@ ddr_reg: regulator@100 {
> gpio = <&pmic 6 GPIO_ACTIVE_HIGH>;
> };
>
> - sys_3v3_reg: regulator@101 {
> + sys_3v3_reg: regulator-3v3 {
> compatible = "regulator-fixed";
> regulator-name = "sys_3v3";
> regulator-min-microvolt = <3300000>;
> @@ -38,7 +38,7 @@ sys_3v3_reg: regulator@101 {
> gpio = <&pmic 7 GPIO_ACTIVE_HIGH>;
> };
>
> - usb1_vbus_reg: regulator@102 {
> + usb1_vbus_reg: regulator-usb1 {
> compatible = "regulator-fixed";
> regulator-name = "usb1_vbus";
> regulator-min-microvolt = <5000000>;
> @@ -49,7 +49,7 @@ usb1_vbus_reg: regulator@102 {
> vin-supply = <&vdd_5v0_reg>;
> };
>
> - usb3_vbus_reg: regulator@103 {
> + usb3_vbus_reg: regulator-usb3 {
> compatible = "regulator-fixed";
> regulator-name = "usb3_vbus";
> regulator-min-microvolt = <5000000>;
> @@ -60,7 +60,7 @@ usb3_vbus_reg: regulator@103 {
> vin-supply = <&vdd_5v0_reg>;
> };
>
> - vdd_5v0_reg: regulator@104 {
> + vdd_5v0_reg: regulator-5v0 {
> compatible = "regulator-fixed";
> regulator-name = "5v0";
> regulator-min-microvolt = <5000000>;
> @@ -69,7 +69,7 @@ vdd_5v0_reg: regulator@104 {
> gpio = <&pmic 2 GPIO_ACTIVE_HIGH>;
> };
>
> - vdd_bl_reg: regulator@105 {
> + vdd_bl_reg: regulator-bl {
> compatible = "regulator-fixed";
> regulator-name = "vdd_bl";
> regulator-min-microvolt = <5000000>;
> diff --git a/arch/arm/boot/dts/tegra30-cardhu-a04.dts b/arch/arm/boot/dts/tegra30-cardhu-a04.dts
> index a11028b8b67b..2911f08863a0 100644
> --- a/arch/arm/boot/dts/tegra30-cardhu-a04.dts
> +++ b/arch/arm/boot/dts/tegra30-cardhu-a04.dts
> @@ -16,7 +16,7 @@ mmc@78000400 {
> keep-power-in-suspend;
> };
>
> - ddr_reg: regulator@100 {
> + ddr_reg: regulator-ddr {
> compatible = "regulator-fixed";
> regulator-name = "ddr";
> regulator-min-microvolt = <1500000>;
> @@ -27,7 +27,7 @@ ddr_reg: regulator@100 {
> gpio = <&pmic 7 GPIO_ACTIVE_HIGH>;
> };
>
> - sys_3v3_reg: regulator@101 {
> + sys_3v3_reg: regulator-3v3 {
> compatible = "regulator-fixed";
> regulator-name = "sys_3v3";
> regulator-min-microvolt = <3300000>;
> @@ -38,7 +38,7 @@ sys_3v3_reg: regulator@101 {
> gpio = <&pmic 6 GPIO_ACTIVE_HIGH>;
> };
>
> - usb1_vbus_reg: regulator@102 {
> + usb1_vbus_reg: regulator-usb1 {
> compatible = "regulator-fixed";
> regulator-name = "usb1_vbus";
> regulator-min-microvolt = <5000000>;
> @@ -49,7 +49,7 @@ usb1_vbus_reg: regulator@102 {
> vin-supply = <&vdd_5v0_reg>;
> };
>
> - usb3_vbus_reg: regulator@103 {
> + usb3_vbus_reg: regulator-usb3 {
> compatible = "regulator-fixed";
> regulator-name = "usb3_vbus";
> regulator-min-microvolt = <5000000>;
> @@ -60,7 +60,7 @@ usb3_vbus_reg: regulator@103 {
> vin-supply = <&vdd_5v0_reg>;
> };
>
> - vdd_5v0_reg: regulator@104 {
> + vdd_5v0_reg: regulator-5v0 {
> compatible = "regulator-fixed";
> regulator-name = "5v0";
> regulator-min-microvolt = <5000000>;
> @@ -69,7 +69,7 @@ vdd_5v0_reg: regulator@104 {
> gpio = <&pmic 8 GPIO_ACTIVE_HIGH>;
> };
>
> - vdd_bl_reg: regulator@105 {
> + vdd_bl_reg: regulator-bl {
> compatible = "regulator-fixed";
> regulator-name = "vdd_bl";
> regulator-min-microvolt = <5000000>;
> @@ -80,7 +80,7 @@ vdd_bl_reg: regulator@105 {
> gpio = <&gpio TEGRA_GPIO(DD, 2) GPIO_ACTIVE_HIGH>;
> };
>
> - vdd_bl2_reg: regulator@106 {
> + vdd_bl2_reg: regulator-bl2 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_bl2";
> regulator-min-microvolt = <5000000>;
> diff --git a/arch/arm/boot/dts/tegra30-cardhu.dtsi b/arch/arm/boot/dts/tegra30-cardhu.dtsi
> index 448f1397e64a..d2f6121049a4 100644
> --- a/arch/arm/boot/dts/tegra30-cardhu.dtsi
> +++ b/arch/arm/boot/dts/tegra30-cardhu.dtsi
> @@ -433,7 +433,7 @@ backlight: backlight {
> default-brightness-level = <6>;
> };
>
> - clk32k_in: clock@0 {
> + clk32k_in: clock-32k {
> compatible = "fixed-clock";
> clock-frequency = <32768>;
> #clock-cells = <0>;
> @@ -475,7 +475,7 @@ panel: panel {
> backlight = <&backlight>;
> };
>
> - vdd_ac_bat_reg: regulator@0 {
> + vdd_ac_bat_reg: regulator-acbat {
> compatible = "regulator-fixed";
> regulator-name = "vdd_ac_bat";
> regulator-min-microvolt = <5000000>;
> @@ -483,7 +483,7 @@ vdd_ac_bat_reg: regulator@0 {
> regulator-always-on;
> };
>
> - cam_1v8_reg: regulator@1 {
> + cam_1v8_reg: regulator-cam {
> compatible = "regulator-fixed";
> regulator-name = "cam_1v8";
> regulator-min-microvolt = <1800000>;
> @@ -493,7 +493,7 @@ cam_1v8_reg: regulator@1 {
> vin-supply = <&vio_reg>;
> };
>
> - cp_5v_reg: regulator@2 {
> + cp_5v_reg: regulator-5v0cp {
> compatible = "regulator-fixed";
> regulator-name = "cp_5v";
> regulator-min-microvolt = <5000000>;
> @@ -504,7 +504,7 @@ cp_5v_reg: regulator@2 {
> gpio = <&pmic 0 GPIO_ACTIVE_HIGH>;
> };
>
> - emmc_3v3_reg: regulator@3 {
> + emmc_3v3_reg: regulator-emmc {
> compatible = "regulator-fixed";
> regulator-name = "emmc_3v3";
> regulator-min-microvolt = <3300000>;
> @@ -516,7 +516,7 @@ emmc_3v3_reg: regulator@3 {
> vin-supply = <&sys_3v3_reg>;
> };
>
> - modem_3v3_reg: regulator@4 {
> + modem_3v3_reg: regulator-modem {
> compatible = "regulator-fixed";
> regulator-name = "modem_3v3";
> regulator-min-microvolt = <3300000>;
> @@ -525,7 +525,7 @@ modem_3v3_reg: regulator@4 {
> gpio = <&gpio TEGRA_GPIO(D, 6) GPIO_ACTIVE_HIGH>;
> };
>
> - pex_hvdd_3v3_reg: regulator@5 {
> + pex_hvdd_3v3_reg: regulator-pex {
> compatible = "regulator-fixed";
> regulator-name = "pex_hvdd_3v3";
> regulator-min-microvolt = <3300000>;
> @@ -535,7 +535,7 @@ pex_hvdd_3v3_reg: regulator@5 {
> vin-supply = <&sys_3v3_reg>;
> };
>
> - vdd_cam1_ldo_reg: regulator@6 {
> + vdd_cam1_ldo_reg: regulator-cam1 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_cam1_ldo";
> regulator-min-microvolt = <2800000>;
> @@ -545,7 +545,7 @@ vdd_cam1_ldo_reg: regulator@6 {
> vin-supply = <&sys_3v3_reg>;
> };
>
> - vdd_cam2_ldo_reg: regulator@7 {
> + vdd_cam2_ldo_reg: regulator-cam2 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_cam2_ldo";
> regulator-min-microvolt = <2800000>;
> @@ -555,7 +555,7 @@ vdd_cam2_ldo_reg: regulator@7 {
> vin-supply = <&sys_3v3_reg>;
> };
>
> - vdd_cam3_ldo_reg: regulator@8 {
> + vdd_cam3_ldo_reg: regulator-cam3 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_cam3_ldo";
> regulator-min-microvolt = <3300000>;
> @@ -565,7 +565,7 @@ vdd_cam3_ldo_reg: regulator@8 {
> vin-supply = <&sys_3v3_reg>;
> };
>
> - vdd_com_reg: regulator@9 {
> + vdd_com_reg: regulator-com {
> compatible = "regulator-fixed";
> regulator-name = "vdd_com";
> regulator-min-microvolt = <3300000>;
> @@ -577,7 +577,7 @@ vdd_com_reg: regulator@9 {
> vin-supply = <&sys_3v3_reg>;
> };
>
> - vdd_fuse_3v3_reg: regulator@10 {
> + vdd_fuse_3v3_reg: regulator-fuse {
> compatible = "regulator-fixed";
> regulator-name = "vdd_fuse_3v3";
> regulator-min-microvolt = <3300000>;
> @@ -587,7 +587,7 @@ vdd_fuse_3v3_reg: regulator@10 {
> vin-supply = <&sys_3v3_reg>;
> };
>
> - vdd_pnl1_reg: regulator@11 {
> + vdd_pnl1_reg: regulator-pnl1 {
> compatible = "regulator-fixed";
> regulator-name = "vdd_pnl1";
> regulator-min-microvolt = <3300000>;
> @@ -599,7 +599,7 @@ vdd_pnl1_reg: regulator@11 {
> vin-supply = <&sys_3v3_reg>;
> };
>
> - vdd_vid_reg: regulator@12 {
> + vdd_vid_reg: regulator-vid {
> compatible = "regulator-fixed";
> regulator-name = "vddio_vid";
> regulator-min-microvolt = <5000000>;
> --
> 2.32.0
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v1 2/3] ARM: tegra: Name clock and regulator nodes according to DT-schema
2021-10-04 20:56 ` Thierry Reding
@ 2021-10-24 19:08 ` Dmitry Osipenko
0 siblings, 0 replies; 8+ messages in thread
From: Dmitry Osipenko @ 2021-10-24 19:08 UTC (permalink / raw)
To: Thierry Reding, Rob Herring
Cc: Thierry Reding, Jonathan Hunter, David Heidelberg, devicetree,
linux-kernel, linux-tegra
04.10.2021 23:56, Thierry Reding пишет:
>> From: David Heidelberg <david@ixit.cz>
>>
>> Name clocks and regulators according to DT-schema to fix warnings such as:
>>
>> arch/arm/boot/dts/tegra20-acer-a500-picasso.dt.yaml: /: clock@0: 'anyOf' conditional failed, one must be fixed:
>> 'reg' is a required property
>> 'ranges' is a required property
>> From schema: /home/runner/.local/lib/python3.8/site-packages/dtschema/schemas/root-node.yaml
>>
>> Signed-off-by: David Heidelberg <david@ixit.cz>
>> ---
>> arch/arm/boot/dts/tegra114-dalmore.dts | 16 +++++------
>> arch/arm/boot/dts/tegra114-roth.dts | 14 +++++-----
>> arch/arm/boot/dts/tegra114-tn7.dts | 8 +++---
>> arch/arm/boot/dts/tegra124-jetson-tk1.dts | 26 ++++++++---------
>> arch/arm/boot/dts/tegra124-nyan.dtsi | 28 +++++++++----------
>> arch/arm/boot/dts/tegra124-venice2.dts | 28 +++++++++----------
>> .../boot/dts/tegra20-acer-a500-picasso.dts | 12 ++++----
>> arch/arm/boot/dts/tegra20-harmony.dts | 16 +++++------
>> arch/arm/boot/dts/tegra20-medcom-wide.dts | 8 +++---
>> arch/arm/boot/dts/tegra20-paz00.dts | 6 ++--
>> arch/arm/boot/dts/tegra20-plutux.dts | 8 +++---
>> arch/arm/boot/dts/tegra20-seaboard.dts | 16 +++++------
>> arch/arm/boot/dts/tegra20-tamonten.dtsi | 4 +--
>> arch/arm/boot/dts/tegra20-tec.dts | 8 +++---
>> arch/arm/boot/dts/tegra20-trimslice.dts | 12 ++++----
>> arch/arm/boot/dts/tegra20-ventana.dts | 12 ++++----
>> .../tegra30-asus-nexus7-grouper-common.dtsi | 10 +++----
>> ...egra30-asus-nexus7-grouper-maxim-pmic.dtsi | 4 +--
>> .../tegra30-asus-nexus7-grouper-ti-pmic.dtsi | 2 +-
>> arch/arm/boot/dts/tegra30-beaver.dts | 20 ++++++-------
>> arch/arm/boot/dts/tegra30-cardhu-a02.dts | 12 ++++----
>> arch/arm/boot/dts/tegra30-cardhu-a04.dts | 14 +++++-----
>> arch/arm/boot/dts/tegra30-cardhu.dtsi | 28 +++++++++----------
>> 23 files changed, 156 insertions(+), 156 deletions(-)
> I'm not sure about this one. Last time I talked to Rob about this, he
> seemed to suggest that reintroducing the top-level clocks and regulators
> nodes was preferable to doing these renamed.
>
> Rob, which way do we want to decide this? If we do want to reintroduce
> the clocks and regulators top-level "bus" nodes, we probably need the
> tooling to learn to accept that, because I /think/ the reason why they
> were originally dropped a while ago was because the tooling complained
> about the "bus" nodes not being named "bus" (I think that was because
> they traditionally have had "simple-bus" in their compatible and hence
> the "simple-bus" schema was applied along with the schema for the node
> name.
>
> So I think we have a couple of options here:
>
> * Make it so that these nodes no longer need the "simple-bus"
> compatible string: this would need new code to parse these nodes
> and instantiate platform device for them like "simple-bus" would
> do.
>
> * Relax the rules for the "simple-bus" compatible string. It looks
> like this would be the simplest, because we could just extend the
> pattern in the "simple-bus" schema. There's a slight catch, though,
> because we'd also need to allow "#size-cells = <0>" for these. And
> maybe also make "ranges" not required because it doesn't make sense
> for #size-cells = <0>.
>
> * Create a new compatible string specifically for these. That would be
> similar to the first option, except that it's perhaps a bit more
> formal. We could basically create a separate schema for this type of
> bus, so that simple-bus could remain as-is.
>
> Perhaps there's also yet another option that I haven't thought of that's
> better than the above.
>
> Quoting in full below for you to get a better picture of what kinds of
> changes we're talking about here.
Could you please explain what's the point in having a top-level node?
It's also not clear how it will help fixing the warning messages since
reg property still should be required.
If there is no good point, then let's rename nodes and move on.
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2021-10-24 19:08 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-26 23:37 [PATCH v1 0/3] Tegra ARM32 device-tree improvements for 5.16 Dmitry Osipenko
2021-09-26 23:37 ` [PATCH v1 1/3] ARM: tegra: acer-a500: Correct compatible of ak8975 magnetometer Dmitry Osipenko
2021-10-04 20:40 ` Thierry Reding
2021-09-26 23:37 ` [PATCH v1 2/3] ARM: tegra: Name clock and regulator nodes according to DT-schema Dmitry Osipenko
2021-10-04 20:56 ` Thierry Reding
2021-10-24 19:08 ` Dmitry Osipenko
2021-09-26 23:37 ` [PATCH v1 3/3] ARM: tegra: Update Broadcom Bluetooth device-tree nodes Dmitry Osipenko
2021-10-04 20:41 ` Thierry Reding
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).