Thread (12 messages) 12 messages, 3 authors, 2023-07-12

Re: [PATCH 2/2] ARM: dts: imx6: pfla02: Fix SD card reboot problem

From: Andrej Picej <hidden>
Date: 2023-07-05 08:28:32
Also in: linux-devicetree, lkml

Hi Marco,

On 4. 07. 23 10:17, Marco Felsch wrote:
On 23-07-04, Andrej Picej wrote:
quoted
If regulator is not marked as always-on the regulator gets disabled on
reboot breaking the next boot.

Signed-off-by: Andrej Picej <redacted>
---
  arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi | 1 +
  1 file changed, 1 insertion(+)
diff --git a/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi b/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi
index 80adb2a02cc9..25d6a036d5b8 100644
--- a/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi
@@ -192,6 +192,7 @@ vdd_3v3_pmic_io_reg: ldo6 {
  			vdd_sd0_reg: ldo9 {
  				regulator-min-microvolt = <3300000>;
  				regulator-max-microvolt = <3300000>;
+				regulator-always-on;
I think this is the supply for the sd-card, so you can make use of
'vmmc-supply'.
This is already the case:
&usdhc3 {
	pinctrl-names = "default";
	pinctrl-0 = <&pinctrl_usdhc3
		     &pinctrl_usdhc3_cdwp>;
	cd-gpios = <&gpio1 27 GPIO_ACTIVE_LOW>;
	wp-gpios = <&gpio1 29 GPIO_ACTIVE_HIGH>;
	vmmc-supply = <&vdd_sd0_reg>;
	status = "disabled";
};
I think the main reason for a failed boot is that the PMIC doesn't get 
reset and that the bootloader doesn't specifically enable the SD card 
regulator.

Could this patch still be applied or should we put the fix in reset 
routine/bootloader?

Best regards,
Andrej
Regards,
   Marco
quoted
  			};
  
  			vdd_sd1_reg: ldo10 {
-- 
2.25.1

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help