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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <7ed6fde0-37a4-a667-2629-0a6b8a202a69@norik.com>
Date:   Wed, 5 Jul 2023 10:28:22 +0200
From:   Andrej Picej <andrej.picej@...ik.com>
To:     Marco Felsch <m.felsch@...gutronix.de>
Cc:     robh+dt@...nel.org, krzysztof.kozlowski+dt@...aro.org,
        conor+dt@...nel.org, shawnguo@...nel.org, s.hauer@...gutronix.de,
        kernel@...gutronix.de, festevam@...il.com, linux-imx@....com,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org, upstream@...tec.de
Subject: Re: [PATCH 2/2] ARM: dts: imx6: pfla02: Fix SD card reboot problem

Hi Marco,

On 4. 07. 23 10:17, Marco Felsch wrote:
> On 23-07-04, Andrej Picej wrote:
>> If regulator is not marked as always-on the regulator gets disabled on
>> reboot breaking the next boot.
>>
>> Signed-off-by: Andrej Picej <andrej.picej@...ik.com>
>> ---
>>   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
> 
>>   			};
>>   
>>   			vdd_sd1_reg: ldo10 {
>> -- 
>> 2.25.1
>>
>>
>>