Commit c8c59bca authored by Rob Herring (Arm)'s avatar Rob Herring (Arm) Committed by Heiko Stuebner
Browse files

arm64: dts: rockchip: Use "regulator-fixed" for btreg on px30-engicam for vcc3v3-btreg 



The vcc3v3-btreg regulator only has 1 state and no state gpios defined,
so "regulator-gpio" is not the correct binding to use. "regulator-fixed"
is the correct binding to use. It supports an enable GPIO which is
needed in this case.

Signed-off-by: default avatar"Rob Herring (Arm)" <robh@kernel.org>
Link: https://lore.kernel.org/r/20250409205047.1522943-1-robh@kernel.org


Signed-off-by: default avatarHeiko Stuebner <heiko@sntech.de>
parent 7fc027e8

arch/arm64/boot/dts/rockchip/px30-engicam-common.dtsi

+1 −2
Original line number Diff line number Diff line
@@ -31,7 +31,7 @@ sdio_pwrseq: sdio-pwrseq { 
	};



	vcc3v3_btreg: vcc3v3-btreg {

		compatible = "regulator-gpio";

		compatible = "regulator-fixed";

		enable-active-high;

		pinctrl-names = "default";

		pinctrl-0 = <&bt_enable_h>;
@@ -39,7 +39,6 @@ vcc3v3_btreg: vcc3v3-btreg { 
		regulator-min-microvolt = <3300000>;

		regulator-max-microvolt = <3300000>;

		regulator-always-on;

		states = <3300000 0x0>;

	};



	vcc3v3_rf_aux_mod: regulator-vcc3v3-rf-aux-mod {

arch/arm64/boot/dts/rockchip/px30-engicam-ctouch2.dtsi

+1 −1
Original line number Diff line number Diff line
@@ -26,5 +26,5 @@ &sdio_pwrseq { 
};



&vcc3v3_btreg {

	enable-gpios = <&gpio1 RK_PC3 GPIO_ACTIVE_HIGH>;

	gpios = <&gpio1 RK_PC3 GPIO_ACTIVE_HIGH>;

};

arch/arm64/boot/dts/rockchip/px30-engicam-px30-core-edimm2.2.dts

+1 −1
Original line number Diff line number Diff line
@@ -39,5 +39,5 @@ &sdio_pwrseq { 
};



&vcc3v3_btreg {

	enable-gpios = <&gpio1 RK_PC2 GPIO_ACTIVE_HIGH>;

	gpios = <&gpio1 RK_PC2 GPIO_ACTIVE_HIGH>;

};