arm64: dts: imx8mp-skov: correct PMIC board limits
authorAhmad Fatoum <a.fatoum@pengutronix.de>
Mon, 6 Jan 2025 16:06:36 +0000 (17:06 +0100)
committerShawn Guo <shawnguo@kernel.org>
Tue, 25 Feb 2025 00:32:56 +0000 (08:32 +0800)
The PMIC voltage constraints in the device tree currently describe the
permissible range of the PMIC. This is unnecessary as this information
already exists in the driver and wrong as it doesn't account for
board-specific constraints, e.g. a 2.1V on VDD_SOC would fry the SoC and
a maximum voltage of 3.4V on the VDD_3V3 rail may be unexpected across
the board.

Fix this by adjusting constraints to reflect the board limits.

Fixes: 6d382d51d979 ("arm64: dts: freescale: Add SKOV IMX8MP CPU revB board")
Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
Signed-off-by: Shawn Guo <shawnguo@kernel.org>
arch/arm64/boot/dts/freescale/imx8mp-skov-reva.dtsi

index 59813ef..ae82166 100644 (file)
 
                        reg_vdd_arm: BUCK2 {
                                regulator-name = "VDD_ARM";
-                               regulator-min-microvolt = <600000>;
-                               regulator-max-microvolt = <2187500>;
+                               regulator-min-microvolt = <850000>;
+                               regulator-max-microvolt = <1000000>;
                                vin-supply = <&reg_5v_p>;
                                regulator-boot-on;
                                regulator-always-on;
 
                        reg_vdd_3v3: BUCK4 {
                                regulator-name = "VDD_3V3";
-                               regulator-min-microvolt = <600000>;
-                               regulator-max-microvolt = <3400000>;
+                               regulator-min-microvolt = <3300000>;
+                               regulator-max-microvolt = <3300000>;
                                vin-supply = <&reg_5v_p>;
                                regulator-boot-on;
                                regulator-always-on;
 
                        reg_vdd_1v8: BUCK5 {
                                regulator-name = "VDD_1V8";
-                               regulator-min-microvolt = <600000>;
-                               regulator-max-microvolt = <3400000>;
+                               regulator-min-microvolt = <1800000>;
+                               regulator-max-microvolt = <1800000>;
                                vin-supply = <&reg_5v_p>;
                                regulator-boot-on;
                                regulator-always-on;
 
                        reg_nvcc_dram_1v1: BUCK6 {
                                regulator-name = "NVCC_DRAM_1V1";
-                               regulator-min-microvolt = <600000>;
-                               regulator-max-microvolt = <3400000>;
+                               regulator-min-microvolt = <1100000>;
+                               regulator-max-microvolt = <1100000>;
                                vin-supply = <&reg_5v_p>;
                                regulator-boot-on;
                                regulator-always-on;
 
                        reg_nvcc_snvs_1v8: LDO1 {
                                regulator-name = "NVCC_SNVS_1V8";
-                               regulator-min-microvolt = <1600000>;
-                               regulator-max-microvolt = <3300000>;
+                               regulator-min-microvolt = <1800000>;
+                               regulator-max-microvolt = <1800000>;
                                vin-supply = <&reg_5v_p>;
                                regulator-boot-on;
                                regulator-always-on;
 
                        reg_vdda_1v8: LDO3 {
                                regulator-name = "VDDA_1V8";
-                               regulator-min-microvolt = <800000>;
-                               regulator-max-microvolt = <3300000>;
+                               regulator-min-microvolt = <1800000>;
+                               regulator-max-microvolt = <1800000>;
                                vin-supply = <&reg_5v_p>;
                                regulator-boot-on;
                                regulator-always-on;