Skip to content

Commit

Permalink
arm64: dts: ti: k3-am62-verdin-dahlia: Keep CTRL_SLEEP_MOCI# regulato…
Browse files Browse the repository at this point in the history
…r on

commit 9438f97 upstream.

This reverts commit 3935fbc.

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: 3935fbc ("arm64: dts: ti: k3-am62-verdin-dahlia: support sleep-moci")
Cc:  <[email protected]>
Link: https://e2e.ti.com/support/processors-group/processors/f/processors-forum/1361669/am625-gpio-output-state-in-deep-sleep/5244802
Signed-off-by: Francesco Dolcini <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Nishanth Menon <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
  • Loading branch information
dolcini authored and gregkh committed Aug 14, 2024
1 parent cd490a2 commit aa6b80a
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 28 deletions.
22 changes: 0 additions & 22 deletions arch/arm64/boot/dts/ti/k3-am62-verdin-dahlia.dtsi
Original file line number Diff line number Diff line change
Expand Up @@ -43,15 +43,6 @@
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 */
Expand Down Expand Up @@ -193,11 +184,6 @@
status = "okay";
};

/* Do not force CTRL_SLEEP_MOCI# always enabled */
&reg_force_sleep_moci {
status = "disabled";
};

/* Verdin SD_1 */
&sdhci1 {
status = "okay";
Expand All @@ -218,15 +204,7 @@
};

&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# */
Expand Down
6 changes: 0 additions & 6 deletions arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi
Original file line number Diff line number Diff line change
Expand Up @@ -138,12 +138,6 @@
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;
Expand Down

0 comments on commit aa6b80a

Please sign in to comment.