| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <YVtqlt4m63WuBHGM@orome.fritz.box>
Date: Mon, 4 Oct 2021 22:56:54 +0200
From: Thierry Reding <thierry.reding@...il.com>
To: Dmitry Osipenko <digetx@...il.com>,
Rob Herring <robh+dt@...nel.org>
Cc: Thierry Reding <treding@...dia.com>,
Jonathan Hunter <jonathanh@...dia.com>,
David Heidelberg <david@...t.cz>, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-tegra@...r.kernel.org
Subject: Re: [PATCH v1 2/3] ARM: tegra: Name clock and regulator nodes
according to DT-schema
On Mon, Sep 27, 2021 at 02:37:03AM +0300, Dmitry Osipenko wrote:
> From: David Heidelberg <david@...t.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@...t.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@...08000 {
> 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@...00600 {
> 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@...00600 {
> 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@...08000 {
> 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@...00400 {
> 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@...00400 {
> 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
>
Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)
Powered by blists - more mailing lists