[v1,4/7] ARM: dts: stm32: osd32: fix ldo1 not required to be always-on

Message ID 20230620151312.1064629-4-sean@geanix.com
State New
Headers
Series [v1,1/7] ARM: dts: stm32: Add alternate pinmux for i2s pins |

Commit Message

Sean Nyekjaer June 20, 2023, 3:13 p.m. UTC
  According to the OSD32MP1 Power System overview[1] there is no hard
requirement for the ldo1 to be always-on.

[1]: https://octavosystems.com/app_notes/osd32mp1-power-system-overview/#connections

Signed-off-by: Sean Nyekjaer <sean@geanix.com>
---
 arch/arm/boot/dts/stm32mp15xx-osd32.dtsi | 2 --
 1 file changed, 2 deletions(-)
  

Comments

Ahmad Fatoum June 20, 2023, 3:31 p.m. UTC | #1
On 20.06.23 17:13, Sean Nyekjaer wrote:
> According to the OSD32MP1 Power System overview[1] there is no hard
> requirement for the ldo1 to be always-on.
> 
> [1]: https://octavosystems.com/app_notes/osd32mp1-power-system-overview/#connections
> 
> Signed-off-by: Sean Nyekjaer <sean@geanix.com>

Acked-by: Ahmad Fatoum <a.fatoum@pengutronix.de>

> ---
>  arch/arm/boot/dts/stm32mp15xx-osd32.dtsi | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/stm32mp15xx-osd32.dtsi b/arch/arm/boot/dts/stm32mp15xx-osd32.dtsi
> index a43965c86fe8..eb43a1e3a0c9 100644
> --- a/arch/arm/boot/dts/stm32mp15xx-osd32.dtsi
> +++ b/arch/arm/boot/dts/stm32mp15xx-osd32.dtsi
> @@ -117,9 +117,7 @@ v1v8_audio: ldo1 {
>  				regulator-name = "v1v8_audio";
>  				regulator-min-microvolt = <1800000>;
>  				regulator-max-microvolt = <1800000>;
> -				regulator-always-on;
>  				interrupts = <IT_CURLIM_LDO1 0>;
> -
>  			};
>  
>  			v3v3_hdmi: ldo2 {
  

Patch

diff --git a/arch/arm/boot/dts/stm32mp15xx-osd32.dtsi b/arch/arm/boot/dts/stm32mp15xx-osd32.dtsi
index a43965c86fe8..eb43a1e3a0c9 100644
--- a/arch/arm/boot/dts/stm32mp15xx-osd32.dtsi
+++ b/arch/arm/boot/dts/stm32mp15xx-osd32.dtsi
@@ -117,9 +117,7 @@  v1v8_audio: ldo1 {
 				regulator-name = "v1v8_audio";
 				regulator-min-microvolt = <1800000>;
 				regulator-max-microvolt = <1800000>;
-				regulator-always-on;
 				interrupts = <IT_CURLIM_LDO1 0>;
-
 			};
 
 			v3v3_hdmi: ldo2 {