* [PATCH] ARM: dts: microchip: at91-sam9x75_curiosity: Add PMIC suspend voltage configuration
@ 2025-01-13 7:17 Andrei Simion
2025-01-13 7:40 ` Mihai.Sain
2025-01-18 14:38 ` Claudiu Beznea
0 siblings, 2 replies; 3+ messages in thread
From: Andrei Simion @ 2025-01-13 7:17 UTC (permalink / raw)
To: nicolas.ferre, claudiu.beznea, alexandre.belloni, robh, krzk+dt,
conor+dt
Cc: devicetree, linux-arm-kernel, linux-kernel, Andrei Simion
Add missing essential configuration for the PMIC rails, which is necessary
for proper low-power mode operation. This patch adds the required settings
to ensure that the regulators behave correctly during Suspend-to-RAM and
Standby states. Otherwise, after resuming, it receives: "No configuration"
message.
Our driver implements the set_suspend_voltage and set_suspend_mode
callbacks, which require the `regulator-suspend-microvolt` property to be
specified in the device tree for each regulator node. This property defines
the voltage level that the regulator should maintain during suspend mode.
Additionally, according to the datasheet, some regulators need to be turned
on or off during suspend mode. This patch addresses these requirements by
adding the `regulator-on-in-suspend` and `regulator-off-in-suspend`
properties where appropriate.
Fixes: 371a47c9a58a ("ARM: dts: microchip: sam9x75_curiosity: add sam9x75 curiosity board")
Signed-off-by: Andrei Simion <andrei.simion@microchip.com>
---
.../dts/microchip/at91-sam9x75_curiosity.dts | 18 ++++++++++++++++++
1 file changed, 18 insertions(+)
diff --git a/arch/arm/boot/dts/microchip/at91-sam9x75_curiosity.dts b/arch/arm/boot/dts/microchip/at91-sam9x75_curiosity.dts
index 1a6a909a5043..5514ad10cda5 100644
--- a/arch/arm/boot/dts/microchip/at91-sam9x75_curiosity.dts
+++ b/arch/arm/boot/dts/microchip/at91-sam9x75_curiosity.dts
@@ -110,10 +110,12 @@ vdd_3v3: VDD_IO {
regulator-state-standby {
regulator-on-in-suspend;
+ regulator-suspend-microvolt = <3300000>;
regulator-mode = <4>;
};
regulator-state-mem {
+ regulator-off-in-suspend;
regulator-mode = <4>;
};
};
@@ -128,11 +130,13 @@ vddioddr: VDD_DDR {
regulator-state-standby {
regulator-on-in-suspend;
+ regulator-suspend-microvolt = <1350000>;
regulator-mode = <4>;
};
regulator-state-mem {
regulator-on-in-suspend;
+ regulator-suspend-microvolt = <1350000>;
regulator-mode = <4>;
};
};
@@ -147,10 +151,12 @@ vddcore: VDD_CORE {
regulator-state-standby {
regulator-on-in-suspend;
+ regulator-suspend-microvolt = <1150000>;
regulator-mode = <4>;
};
regulator-state-mem {
+ regulator-off-in-suspend;
regulator-mode = <4>;
};
};
@@ -166,10 +172,12 @@ dcdc4: VDD_OTHER {
regulator-state-standby {
regulator-on-in-suspend;
+ regulator-suspend-microvolt = <1150000>;
regulator-mode = <4>;
};
regulator-state-mem {
+ regulator-off-in-suspend;
regulator-mode = <4>;
};
};
@@ -182,6 +190,11 @@ vldo1: LDO1 {
regulator-state-standby {
regulator-on-in-suspend;
+ regulator-suspend-microvolt = <3300000>;
+ };
+
+ regulator-state-mem {
+ regulator-off-in-suspend;
};
};
@@ -192,6 +205,11 @@ vldo2: LDO2 {
regulator-state-standby {
regulator-on-in-suspend;
+ regulator-suspend-microvolt = <1800000>;
+ };
+
+ regulator-state-mem {
+ regulator-off-in-suspend;
};
};
};
base-commit: 37136bf5c3a6f6b686d74f41837a6406bec6b7bc
--
2.34.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* RE: [PATCH] ARM: dts: microchip: at91-sam9x75_curiosity: Add PMIC suspend voltage configuration
2025-01-13 7:17 [PATCH] ARM: dts: microchip: at91-sam9x75_curiosity: Add PMIC suspend voltage configuration Andrei Simion
@ 2025-01-13 7:40 ` Mihai.Sain
2025-01-18 14:38 ` Claudiu Beznea
1 sibling, 0 replies; 3+ messages in thread
From: Mihai.Sain @ 2025-01-13 7:40 UTC (permalink / raw)
To: Andrei.Simion, Nicolas.Ferre, claudiu.beznea, alexandre.belloni,
robh, krzk+dt, conor+dt
Cc: devicetree, linux-arm-kernel, linux-kernel
> Add missing essential configuration for the PMIC rails, which is necessary for
> proper low-power mode operation. This patch adds the required settings to
> ensure that the regulators behave correctly during Suspend-to-RAM and Standby
> states. Otherwise, after resuming, it receives: "No configuration"
> message.
>
> Our driver implements the set_suspend_voltage and set_suspend_mode callbacks,
> which require the `regulator-suspend-microvolt` property to be specified in the
> device tree for each regulator node. This property defines the voltage level
> that the regulator should maintain during suspend mode.
>
> Additionally, according to the datasheet, some regulators need to be turned on
> or off during suspend mode. This patch addresses these requirements by adding
> the `regulator-on-in-suspend` and `regulator-off-in-suspend` properties where
> appropriate.
>
> Fixes: 371a47c9a58a ("ARM: dts: microchip: sam9x75_curiosity: add sam9x75
> curiosity board")
> Signed-off-by: Andrei Simion <andrei.simion@microchip.com>
Hi Andrei,
Tested-by: Mihai Sain <mihai.sain@microchip.com>
Best regards,
Mihai Sain
> ---
> .../dts/microchip/at91-sam9x75_curiosity.dts | 18 ++++++++++++++++++
> 1 file changed, 18 insertions(+)
>
> diff --git a/arch/arm/boot/dts/microchip/at91-sam9x75_curiosity.dts
> b/arch/arm/boot/dts/microchip/at91-sam9x75_curiosity.dts
> index 1a6a909a5043..5514ad10cda5 100644
> --- a/arch/arm/boot/dts/microchip/at91-sam9x75_curiosity.dts
> +++ b/arch/arm/boot/dts/microchip/at91-sam9x75_curiosity.dts
> @@ -110,10 +110,12 @@ vdd_3v3: VDD_IO {
>
> regulator-state-standby {
> regulator-on-in-suspend;
> + regulator-suspend-microvolt = <3300000>;
> regulator-mode = <4>;
> };
>
> regulator-state-mem {
> + regulator-off-in-suspend;
> regulator-mode = <4>;
> };
> };
> @@ -128,11 +130,13 @@ vddioddr: VDD_DDR {
>
> regulator-state-standby {
> regulator-on-in-suspend;
> + regulator-suspend-microvolt = <1350000>;
> regulator-mode = <4>;
> };
>
> regulator-state-mem {
> regulator-on-in-suspend;
> + regulator-suspend-microvolt = <1350000>;
> regulator-mode = <4>;
> };
> };
> @@ -147,10 +151,12 @@ vddcore: VDD_CORE {
>
> regulator-state-standby {
> regulator-on-in-suspend;
> + regulator-suspend-microvolt = <1150000>;
> regulator-mode = <4>;
> };
>
> regulator-state-mem {
> + regulator-off-in-suspend;
> regulator-mode = <4>;
> };
> };
> @@ -166,10 +172,12 @@ dcdc4: VDD_OTHER {
>
> regulator-state-standby {
> regulator-on-in-suspend;
> + regulator-suspend-microvolt = <1150000>;
> regulator-mode = <4>;
> };
>
> regulator-state-mem {
> + regulator-off-in-suspend;
> regulator-mode = <4>;
> };
> };
> @@ -182,6 +190,11 @@ vldo1: LDO1 {
>
> regulator-state-standby {
> regulator-on-in-suspend;
> + regulator-suspend-microvolt = <3300000>;
> + };
> +
> + regulator-state-mem {
> + regulator-off-in-suspend;
> };
> };
>
> @@ -192,6 +205,11 @@ vldo2: LDO2 {
>
> regulator-state-standby {
> regulator-on-in-suspend;
> + regulator-suspend-microvolt = <1800000>;
> + };
> +
> + regulator-state-mem {
> + regulator-off-in-suspend;
> };
> };
> };
>
> base-commit: 37136bf5c3a6f6b686d74f41837a6406bec6b7bc
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] ARM: dts: microchip: at91-sam9x75_curiosity: Add PMIC suspend voltage configuration
2025-01-13 7:17 [PATCH] ARM: dts: microchip: at91-sam9x75_curiosity: Add PMIC suspend voltage configuration Andrei Simion
2025-01-13 7:40 ` Mihai.Sain
@ 2025-01-18 14:38 ` Claudiu Beznea
1 sibling, 0 replies; 3+ messages in thread
From: Claudiu Beznea @ 2025-01-18 14:38 UTC (permalink / raw)
To: Andrei Simion, nicolas.ferre, alexandre.belloni, robh, krzk+dt,
conor+dt
Cc: devicetree, linux-arm-kernel, linux-kernel
Hi, Andrei,
On 13.01.2025 09:17, Andrei Simion wrote:
> Add missing essential configuration for the PMIC rails, which is necessary
> for proper low-power mode operation. This patch adds the required settings
> to ensure that the regulators behave correctly during Suspend-to-RAM and
> Standby states. Otherwise, after resuming, it receives: "No configuration"
> message.
>
> Our driver implements the set_suspend_voltage and set_suspend_mode
> callbacks, which require the `regulator-suspend-microvolt` property to be
> specified in the device tree for each regulator node. This property defines
> the voltage level that the regulator should maintain during suspend mode.
>
> Additionally, according to the datasheet, some regulators need to be turned
> on or off during suspend mode. This patch addresses these requirements by
> adding the `regulator-on-in-suspend` and `regulator-off-in-suspend`
> properties where appropriate.
>
> Fixes: 371a47c9a58a ("ARM: dts: microchip: sam9x75_curiosity: add sam9x75 curiosity board")
> Signed-off-by: Andrei Simion <andrei.simion@microchip.com>
> ---
> .../dts/microchip/at91-sam9x75_curiosity.dts | 18 ++++++++++++++++++
> 1 file changed, 18 insertions(+)
>
> diff --git a/arch/arm/boot/dts/microchip/at91-sam9x75_curiosity.dts b/arch/arm/boot/dts/microchip/at91-sam9x75_curiosity.dts
> index 1a6a909a5043..5514ad10cda5 100644
> --- a/arch/arm/boot/dts/microchip/at91-sam9x75_curiosity.dts
> +++ b/arch/arm/boot/dts/microchip/at91-sam9x75_curiosity.dts
> @@ -110,10 +110,12 @@ vdd_3v3: VDD_IO {
>
> regulator-state-standby {
> regulator-on-in-suspend;
> + regulator-suspend-microvolt = <3300000>;
> regulator-mode = <4>;
> };
>
> regulator-state-mem {
> + regulator-off-in-suspend;
> regulator-mode = <4>;
> };
> };
> @@ -128,11 +130,13 @@ vddioddr: VDD_DDR {
>
> regulator-state-standby {
> regulator-on-in-suspend;
> + regulator-suspend-microvolt = <1350000>;
> regulator-mode = <4>;
> };
>
> regulator-state-mem {
> regulator-on-in-suspend;
> + regulator-suspend-microvolt = <1350000>;
> regulator-mode = <4>;
> };
> };
> @@ -147,10 +151,12 @@ vddcore: VDD_CORE {
>
> regulator-state-standby {
> regulator-on-in-suspend;
> + regulator-suspend-microvolt = <1150000>;
> regulator-mode = <4>;
> };
>
> regulator-state-mem {
> + regulator-off-in-suspend;
Is this correct? In the upstream kernel the SAM9X7 supports only standby
and ULP0 (see sam9x7_pm_init()). I don't think it is correct to turn off
the CPU while in ULP0.
Moreover, AFAICT, the MCP16502 will not transition out of active state
according to the FIGURE 4-2: Finite State Machine (FSM) States Diagram for
MCP16502AC from [1] and the schematics from Figure 3-6. Power Management
Integrated Circuit described in [2]. The LPM, HPM are tied to the ground
and SHDN is currently controlled (in the upstream kernel) only for backup
and self-refresh mode (see at91_backup_mode). Am I wrong?
Thank you,
Claudiu
[1]
https://ww1.microchip.com/downloads/aemDocuments/documents/APID/ProductDocuments/DataSheets/MCP16502-Data-Sheet-DS20006275.pdf
[2]
https://ww1.microchip.com/downloads/aemDocuments/documents/MPU32/ProductDocuments/UserGuides/SAM9X75-Curiosity-User-Guide-DS60001859.pdf
> regulator-mode = <4>;
> };
> };
> @@ -166,10 +172,12 @@ dcdc4: VDD_OTHER {
>
> regulator-state-standby {
> regulator-on-in-suspend;
> + regulator-suspend-microvolt = <1150000>;
> regulator-mode = <4>;
> };
>
> regulator-state-mem {
> + regulator-off-in-suspend;
> regulator-mode = <4>;
> };
> };
> @@ -182,6 +190,11 @@ vldo1: LDO1 {
>
> regulator-state-standby {
> regulator-on-in-suspend;
> + regulator-suspend-microvolt = <3300000>;
> + };
> +
> + regulator-state-mem {
> + regulator-off-in-suspend;
> };
> };
>
> @@ -192,6 +205,11 @@ vldo2: LDO2 {
>
> regulator-state-standby {
> regulator-on-in-suspend;
> + regulator-suspend-microvolt = <1800000>;
> + };
> +
> + regulator-state-mem {
> + regulator-off-in-suspend;
> };
> };
> };
>
> base-commit: 37136bf5c3a6f6b686d74f41837a6406bec6b7bc
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-01-18 14:38 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-01-13 7:17 [PATCH] ARM: dts: microchip: at91-sam9x75_curiosity: Add PMIC suspend voltage configuration Andrei Simion
2025-01-13 7:40 ` Mihai.Sain
2025-01-18 14:38 ` Claudiu Beznea
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox