devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: nvidia: Remove unused and undocumented "regulator-ramp-delay-scale" property
@ 2024-11-15 19:34 Rob Herring (Arm)
  2025-01-09 22:54 ` Rob Herring
  0 siblings, 1 reply; 3+ messages in thread
From: Rob Herring (Arm) @ 2024-11-15 19:34 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Conor Dooley, Thierry Reding,
	Jonathan Hunter
  Cc: devicetree, linux-tegra, linux-kernel

Remove "regulator-ramp-delay-scale" property which is both unused in the
kernel and undocumented. Most likely they are leftovers from downstream.

Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
 arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dts | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dts b/arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dts
index c56824d7f4d8..0ecdd7243b2e 100644
--- a/arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dts
+++ b/arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dts
@@ -266,7 +266,6 @@ vdd_soc: sd0 {
 					regulator-max-microvolt = <1170000>;
 					regulator-enable-ramp-delay = <146>;
 					regulator-ramp-delay = <27500>;
-					regulator-ramp-delay-scale = <300>;
 					regulator-always-on;
 					regulator-boot-on;
 
@@ -281,7 +280,6 @@ vdd_ddr: sd1 {
 					regulator-max-microvolt = <1150000>;
 					regulator-enable-ramp-delay = <176>;
 					regulator-ramp-delay = <27500>;
-					regulator-ramp-delay-scale = <300>;
 					regulator-always-on;
 					regulator-boot-on;
 
@@ -296,7 +294,6 @@ vdd_pre: sd2 {
 					regulator-max-microvolt = <1350000>;
 					regulator-enable-ramp-delay = <176>;
 					regulator-ramp-delay = <27500>;
-					regulator-ramp-delay-scale = <350>;
 					regulator-always-on;
 					regulator-boot-on;
 
@@ -311,7 +308,6 @@ vdd_1v8: sd3 {
 					regulator-max-microvolt = <1800000>;
 					regulator-enable-ramp-delay = <242>;
 					regulator-ramp-delay = <27500>;
-					regulator-ramp-delay-scale = <360>;
 					regulator-always-on;
 					regulator-boot-on;
 
@@ -326,7 +322,6 @@ vdd_sys_1v2: ldo0 {
 					regulator-max-microvolt = <1200000>;
 					regulator-enable-ramp-delay = <26>;
 					regulator-ramp-delay = <100000>;
-					regulator-ramp-delay-scale = <200>;
 					regulator-always-on;
 					regulator-boot-on;
 
@@ -341,7 +336,6 @@ vdd_pex_1v05: ldo1 {
 					regulator-max-microvolt = <1050000>;
 					regulator-enable-ramp-delay = <22>;
 					regulator-ramp-delay = <100000>;
-					regulator-ramp-delay-scale = <200>;
 
 					maxim,active-fps-source = <MAX77620_FPS_SRC_NONE>;
 					maxim,active-fps-power-up-slot = <0>;
@@ -354,7 +348,6 @@ vddio_sdmmc: ldo2 {
 					regulator-max-microvolt = <3300000>;
 					regulator-enable-ramp-delay = <62>;
 					regulator-ramp-delay = <100000>;
-					regulator-ramp-delay-scale = <200>;
 
 					maxim,active-fps-source = <MAX77620_FPS_SRC_NONE>;
 					maxim,active-fps-power-up-slot = <0>;
@@ -371,7 +364,6 @@ vdd_rtc: ldo4 {
 					regulator-max-microvolt = <1100000>;
 					regulator-enable-ramp-delay = <22>;
 					regulator-ramp-delay = <100000>;
-					regulator-ramp-delay-scale = <200>;
 					regulator-disable-active-discharge;
 					regulator-always-on;
 					regulator-boot-on;
@@ -395,7 +387,6 @@ avdd_1v05_pll: ldo7 {
 					regulator-max-microvolt = <1050000>;
 					regulator-enable-ramp-delay = <24>;
 					regulator-ramp-delay = <100000>;
-					regulator-ramp-delay-scale = <200>;
 
 					maxim,active-fps-source = <MAX77620_FPS_SRC_1>;
 					maxim,active-fps-power-up-slot = <3>;
@@ -408,7 +399,6 @@ avdd_1v05: ldo8 {
 					regulator-max-microvolt = <1050000>;
 					regulator-enable-ramp-delay = <22>;
 					regulator-ramp-delay = <100000>;
-					regulator-ramp-delay-scale = <200>;
 
 					maxim,active-fps-source = <MAX77620_FPS_SRC_1>;
 					maxim,active-fps-power-up-slot = <6>;
-- 
2.45.2


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] arm64: dts: nvidia: Remove unused and undocumented "regulator-ramp-delay-scale" property
  2024-11-15 19:34 [PATCH] arm64: dts: nvidia: Remove unused and undocumented "regulator-ramp-delay-scale" property Rob Herring (Arm)
@ 2025-01-09 22:54 ` Rob Herring
  2025-02-13 20:29   ` Rob Herring
  0 siblings, 1 reply; 3+ messages in thread
From: Rob Herring @ 2025-01-09 22:54 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Conor Dooley, Thierry Reding,
	Jonathan Hunter
  Cc: devicetree, linux-tegra, linux-kernel

On Fri, Nov 15, 2024 at 1:34 PM Rob Herring (Arm) <robh@kernel.org> wrote:
>
> Remove "regulator-ramp-delay-scale" property which is both unused in the
> kernel and undocumented. Most likely they are leftovers from downstream.
>
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---
>  arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dts | 10 ----------
>  1 file changed, 10 deletions(-)

Ping!

>
> diff --git a/arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dts b/arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dts
> index c56824d7f4d8..0ecdd7243b2e 100644
> --- a/arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dts
> +++ b/arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dts
> @@ -266,7 +266,6 @@ vdd_soc: sd0 {
>                                         regulator-max-microvolt = <1170000>;
>                                         regulator-enable-ramp-delay = <146>;
>                                         regulator-ramp-delay = <27500>;
> -                                       regulator-ramp-delay-scale = <300>;
>                                         regulator-always-on;
>                                         regulator-boot-on;
>
> @@ -281,7 +280,6 @@ vdd_ddr: sd1 {
>                                         regulator-max-microvolt = <1150000>;
>                                         regulator-enable-ramp-delay = <176>;
>                                         regulator-ramp-delay = <27500>;
> -                                       regulator-ramp-delay-scale = <300>;
>                                         regulator-always-on;
>                                         regulator-boot-on;
>
> @@ -296,7 +294,6 @@ vdd_pre: sd2 {
>                                         regulator-max-microvolt = <1350000>;
>                                         regulator-enable-ramp-delay = <176>;
>                                         regulator-ramp-delay = <27500>;
> -                                       regulator-ramp-delay-scale = <350>;
>                                         regulator-always-on;
>                                         regulator-boot-on;
>
> @@ -311,7 +308,6 @@ vdd_1v8: sd3 {
>                                         regulator-max-microvolt = <1800000>;
>                                         regulator-enable-ramp-delay = <242>;
>                                         regulator-ramp-delay = <27500>;
> -                                       regulator-ramp-delay-scale = <360>;
>                                         regulator-always-on;
>                                         regulator-boot-on;
>
> @@ -326,7 +322,6 @@ vdd_sys_1v2: ldo0 {
>                                         regulator-max-microvolt = <1200000>;
>                                         regulator-enable-ramp-delay = <26>;
>                                         regulator-ramp-delay = <100000>;
> -                                       regulator-ramp-delay-scale = <200>;
>                                         regulator-always-on;
>                                         regulator-boot-on;
>
> @@ -341,7 +336,6 @@ vdd_pex_1v05: ldo1 {
>                                         regulator-max-microvolt = <1050000>;
>                                         regulator-enable-ramp-delay = <22>;
>                                         regulator-ramp-delay = <100000>;
> -                                       regulator-ramp-delay-scale = <200>;
>
>                                         maxim,active-fps-source = <MAX77620_FPS_SRC_NONE>;
>                                         maxim,active-fps-power-up-slot = <0>;
> @@ -354,7 +348,6 @@ vddio_sdmmc: ldo2 {
>                                         regulator-max-microvolt = <3300000>;
>                                         regulator-enable-ramp-delay = <62>;
>                                         regulator-ramp-delay = <100000>;
> -                                       regulator-ramp-delay-scale = <200>;
>
>                                         maxim,active-fps-source = <MAX77620_FPS_SRC_NONE>;
>                                         maxim,active-fps-power-up-slot = <0>;
> @@ -371,7 +364,6 @@ vdd_rtc: ldo4 {
>                                         regulator-max-microvolt = <1100000>;
>                                         regulator-enable-ramp-delay = <22>;
>                                         regulator-ramp-delay = <100000>;
> -                                       regulator-ramp-delay-scale = <200>;
>                                         regulator-disable-active-discharge;
>                                         regulator-always-on;
>                                         regulator-boot-on;
> @@ -395,7 +387,6 @@ avdd_1v05_pll: ldo7 {
>                                         regulator-max-microvolt = <1050000>;
>                                         regulator-enable-ramp-delay = <24>;
>                                         regulator-ramp-delay = <100000>;
> -                                       regulator-ramp-delay-scale = <200>;
>
>                                         maxim,active-fps-source = <MAX77620_FPS_SRC_1>;
>                                         maxim,active-fps-power-up-slot = <3>;
> @@ -408,7 +399,6 @@ avdd_1v05: ldo8 {
>                                         regulator-max-microvolt = <1050000>;
>                                         regulator-enable-ramp-delay = <22>;
>                                         regulator-ramp-delay = <100000>;
> -                                       regulator-ramp-delay-scale = <200>;
>
>                                         maxim,active-fps-source = <MAX77620_FPS_SRC_1>;
>                                         maxim,active-fps-power-up-slot = <6>;
> --
> 2.45.2
>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] arm64: dts: nvidia: Remove unused and undocumented "regulator-ramp-delay-scale" property
  2025-01-09 22:54 ` Rob Herring
@ 2025-02-13 20:29   ` Rob Herring
  0 siblings, 0 replies; 3+ messages in thread
From: Rob Herring @ 2025-02-13 20:29 UTC (permalink / raw)
  To: Thierry Reding, Jonathan Hunter
  Cc: devicetree, linux-tegra, linux-kernel, Krzysztof Kozlowski,
	Conor Dooley

On Thu, Jan 9, 2025 at 4:54 PM Rob Herring <robh@kernel.org> wrote:
>
> On Fri, Nov 15, 2024 at 1:34 PM Rob Herring (Arm) <robh@kernel.org> wrote:
> >
> > Remove "regulator-ramp-delay-scale" property which is both unused in the
> > kernel and undocumented. Most likely they are leftovers from downstream.
> >
> > Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> > ---
> >  arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dts | 10 ----------
> >  1 file changed, 10 deletions(-)
>
> Ping!

Ping again...

>
> >
> > diff --git a/arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dts b/arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dts
> > index c56824d7f4d8..0ecdd7243b2e 100644
> > --- a/arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dts
> > +++ b/arch/arm64/boot/dts/nvidia/tegra210-p3450-0000.dts
> > @@ -266,7 +266,6 @@ vdd_soc: sd0 {
> >                                         regulator-max-microvolt = <1170000>;
> >                                         regulator-enable-ramp-delay = <146>;
> >                                         regulator-ramp-delay = <27500>;
> > -                                       regulator-ramp-delay-scale = <300>;
> >                                         regulator-always-on;
> >                                         regulator-boot-on;
> >
> > @@ -281,7 +280,6 @@ vdd_ddr: sd1 {
> >                                         regulator-max-microvolt = <1150000>;
> >                                         regulator-enable-ramp-delay = <176>;
> >                                         regulator-ramp-delay = <27500>;
> > -                                       regulator-ramp-delay-scale = <300>;
> >                                         regulator-always-on;
> >                                         regulator-boot-on;
> >
> > @@ -296,7 +294,6 @@ vdd_pre: sd2 {
> >                                         regulator-max-microvolt = <1350000>;
> >                                         regulator-enable-ramp-delay = <176>;
> >                                         regulator-ramp-delay = <27500>;
> > -                                       regulator-ramp-delay-scale = <350>;
> >                                         regulator-always-on;
> >                                         regulator-boot-on;
> >
> > @@ -311,7 +308,6 @@ vdd_1v8: sd3 {
> >                                         regulator-max-microvolt = <1800000>;
> >                                         regulator-enable-ramp-delay = <242>;
> >                                         regulator-ramp-delay = <27500>;
> > -                                       regulator-ramp-delay-scale = <360>;
> >                                         regulator-always-on;
> >                                         regulator-boot-on;
> >
> > @@ -326,7 +322,6 @@ vdd_sys_1v2: ldo0 {
> >                                         regulator-max-microvolt = <1200000>;
> >                                         regulator-enable-ramp-delay = <26>;
> >                                         regulator-ramp-delay = <100000>;
> > -                                       regulator-ramp-delay-scale = <200>;
> >                                         regulator-always-on;
> >                                         regulator-boot-on;
> >
> > @@ -341,7 +336,6 @@ vdd_pex_1v05: ldo1 {
> >                                         regulator-max-microvolt = <1050000>;
> >                                         regulator-enable-ramp-delay = <22>;
> >                                         regulator-ramp-delay = <100000>;
> > -                                       regulator-ramp-delay-scale = <200>;
> >
> >                                         maxim,active-fps-source = <MAX77620_FPS_SRC_NONE>;
> >                                         maxim,active-fps-power-up-slot = <0>;
> > @@ -354,7 +348,6 @@ vddio_sdmmc: ldo2 {
> >                                         regulator-max-microvolt = <3300000>;
> >                                         regulator-enable-ramp-delay = <62>;
> >                                         regulator-ramp-delay = <100000>;
> > -                                       regulator-ramp-delay-scale = <200>;
> >
> >                                         maxim,active-fps-source = <MAX77620_FPS_SRC_NONE>;
> >                                         maxim,active-fps-power-up-slot = <0>;
> > @@ -371,7 +364,6 @@ vdd_rtc: ldo4 {
> >                                         regulator-max-microvolt = <1100000>;
> >                                         regulator-enable-ramp-delay = <22>;
> >                                         regulator-ramp-delay = <100000>;
> > -                                       regulator-ramp-delay-scale = <200>;
> >                                         regulator-disable-active-discharge;
> >                                         regulator-always-on;
> >                                         regulator-boot-on;
> > @@ -395,7 +387,6 @@ avdd_1v05_pll: ldo7 {
> >                                         regulator-max-microvolt = <1050000>;
> >                                         regulator-enable-ramp-delay = <24>;
> >                                         regulator-ramp-delay = <100000>;
> > -                                       regulator-ramp-delay-scale = <200>;
> >
> >                                         maxim,active-fps-source = <MAX77620_FPS_SRC_1>;
> >                                         maxim,active-fps-power-up-slot = <3>;
> > @@ -408,7 +399,6 @@ avdd_1v05: ldo8 {
> >                                         regulator-max-microvolt = <1050000>;
> >                                         regulator-enable-ramp-delay = <22>;
> >                                         regulator-ramp-delay = <100000>;
> > -                                       regulator-ramp-delay-scale = <200>;
> >
> >                                         maxim,active-fps-source = <MAX77620_FPS_SRC_1>;
> >                                         maxim,active-fps-power-up-slot = <6>;
> > --
> > 2.45.2
> >

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2025-02-13 20:29 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-11-15 19:34 [PATCH] arm64: dts: nvidia: Remove unused and undocumented "regulator-ramp-delay-scale" property Rob Herring (Arm)
2025-01-09 22:54 ` Rob Herring
2025-02-13 20:29   ` Rob Herring

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).