Commit 9438f970 authored by Francesco Dolcini's avatar Francesco Dolcini Committed by Nishanth Menon
Browse files

arm64: dts: ti: k3-am62-verdin-dahlia: Keep CTRL_SLEEP_MOCI# regulator on 

This reverts commit 3935fbc8.

CTRL_SLEEP_MOCI# is a signal that is defined for all the SoM
implementing the Verdin family specification, this signal is supposed to
control the power enable in the carrier board when the system is in deep
sleep mode. However this is not possible with Texas Instruments AM62
SoC, IOs output buffer is disabled in deep sleep and IOs are in
tri-state mode.

Given that we cannot properly control this pin, force it to be always
high to minimize potential issues.

Fixes: 3935fbc8 ("arm64: dts: ti: k3-am62-verdin-dahlia: support sleep-moci")
Cc:  <stable@vger.kernel.org>
Link: https://e2e.ti.com/support/processors-group/processors/f/processors-forum/1361669/am625-gpio-output-state-in-deep-sleep/5244802


Signed-off-by: default avatarFrancesco Dolcini <francesco.dolcini@toradex.com>
Link: https://lore.kernel.org/r/20240731054804.6061-1-francesco@dolcini.it


Signed-off-by: default avatarNishanth Menon <nm@ti.com>
parent 137d9e76

arch/arm64/boot/dts/ti/k3-am62-verdin-dahlia.dtsi

+0 −22
Original line number Diff line number Diff line
@@ -43,15 +43,6 @@ simple-audio-card,cpu { 
			sound-dai = <&mcasp0>;

		};

	};



	reg_usb_hub: regulator-usb-hub {

		compatible = "regulator-fixed";

		enable-active-high;

		/* Verdin CTRL_SLEEP_MOCI# (SODIMM 256) */

		gpio = <&main_gpio0 31 GPIO_ACTIVE_HIGH>;

		regulator-boot-on;

		regulator-name = "HUB_PWR_EN";

	};

};



/* Verdin ETHs */
@@ -193,11 +184,6 @@ &ospi0 { 
	status = "okay";

};



/* Do not force CTRL_SLEEP_MOCI# always enabled */

&reg_force_sleep_moci {

	status = "disabled";

};



/* Verdin SD_1 */

&sdhci1 {

	status = "okay";
@@ -218,15 +204,7 @@ &usbss1 { 
};



&usb1 {

	#address-cells = <1>;

	#size-cells = <0>;

	status = "okay";



	usb-hub@1 {

		compatible = "usb424,2744";

		reg = <1>;

		vdd-supply = <&reg_usb_hub>;

	};

};



/* Verdin CTRL_WAKE1_MICO# */

arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi

+0 −6
Original line number Diff line number Diff line
@@ -138,12 +138,6 @@ reg_1v8_eth: regulator-1v8-eth { 
		vin-supply = <&reg_1v8>;

	};



	/*

	 * By default we enable CTRL_SLEEP_MOCI#, this is required to have

	 * peripherals on the carrier board powered.

	 * If more granularity or power saving is required this can be disabled

	 * in the carrier board device tree files.

	 */

	reg_force_sleep_moci: regulator-force-sleep-moci {

		compatible = "regulator-fixed";

		enable-active-high;