If the tuning step is not set, the tuning step is set to 1.
For some sd cards, the following Tuning timeout will occur,
so set the default tuning step.
Tuning failed, falling back to fixed sampling clock
mmc0: Tuning failed, falling back to fixed sampling clock
Fixes: 1e336aa0c025 ("mmc: sdhci-esdhc-imx: correct the tuning start tap and step setting")
Signed-off-by: Xiaolei Wang <[email protected]>
---
arch/arm/boot/dts/nxp/imx/imx7d-sdb.dts | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/boot/dts/nxp/imx/imx7d-sdb.dts b/arch/arm/boot/dts/nxp/imx/imx7d-sdb.dts
index 75f1cd14bea1..d7396fbfcaf8 100644
--- a/arch/arm/boot/dts/nxp/imx/imx7d-sdb.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx7d-sdb.dts
@@ -493,6 +493,7 @@ &usdhc1 {
vmmc-supply = <®_sd1_vmmc>;
wakeup-source;
keep-power-in-suspend;
+ fsl,tuning-step = <2>;
status = "okay";
};
--
2.25.1
Hi Xiaolei,
On Fri, Jul 21, 2023 at 8:21 PM Xiaolei Wang <[email protected]> wrote:
>
> If the tuning step is not set, the tuning step is set to 1.
> For some sd cards, the following Tuning timeout will occur,
> so set the default tuning step.
>
> Tuning failed, falling back to fixed sampling clock
> mmc0: Tuning failed, falling back to fixed sampling clock
>
> Fixes: 1e336aa0c025 ("mmc: sdhci-esdhc-imx: correct the tuning start tap and step setting")
> Signed-off-by: Xiaolei Wang <[email protected]>
> ---
> arch/arm/boot/dts/nxp/imx/imx7d-sdb.dts | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/boot/dts/nxp/imx/imx7d-sdb.dts b/arch/arm/boot/dts/nxp/imx/imx7d-sdb.dts
> index 75f1cd14bea1..d7396fbfcaf8 100644
> --- a/arch/arm/boot/dts/nxp/imx/imx7d-sdb.dts
> +++ b/arch/arm/boot/dts/nxp/imx/imx7d-sdb.dts
> @@ -493,6 +493,7 @@ &usdhc1 {
> vmmc-supply = <®_sd1_vmmc>;
> wakeup-source;
> keep-power-in-suspend;
> + fsl,tuning-step = <2>;
I wondering if this should be a imx7s.dtsi property instead.
NXP vendor kernel passes:
fsl,tuning-step = <2>;
fsl,tuning-start-tap = <20>;
https://github.com/nxp-imx/linux-imx/blob/lf-6.1.y/arch/arm/boot/dts/imx7s.dtsi#L1216-L1217
Bough? Any suggestions?
Thanks
On 7/22/23 8:16 AM, Fabio Estevam wrote:
> CAUTION: This email comes from a non Wind River email account!
> Do not click links or open attachments unless you recognize the sender and know the content is safe.
>
> Hi Xiaolei,
>
> On Fri, Jul 21, 2023 at 8:21 PM Xiaolei Wang <[email protected]> wrote:
>> If the tuning step is not set, the tuning step is set to 1.
>> For some sd cards, the following Tuning timeout will occur,
>> so set the default tuning step.
>>
>> Tuning failed, falling back to fixed sampling clock
>> mmc0: Tuning failed, falling back to fixed sampling clock
>>
>> Fixes: 1e336aa0c025 ("mmc: sdhci-esdhc-imx: correct the tuning start tap and step setting")
>> Signed-off-by: Xiaolei Wang <[email protected]>
>> ---
>> arch/arm/boot/dts/nxp/imx/imx7d-sdb.dts | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm/boot/dts/nxp/imx/imx7d-sdb.dts b/arch/arm/boot/dts/nxp/imx/imx7d-sdb.dts
>> index 75f1cd14bea1..d7396fbfcaf8 100644
>> --- a/arch/arm/boot/dts/nxp/imx/imx7d-sdb.dts
>> +++ b/arch/arm/boot/dts/nxp/imx/imx7d-sdb.dts
>> @@ -493,6 +493,7 @@ &usdhc1 {
>> vmmc-supply = <®_sd1_vmmc>;
>> wakeup-source;
>> keep-power-in-suspend;
>> + fsl,tuning-step = <2>;
> I wondering if this should be a imx7s.dtsi property instead.
>
> NXP vendor kernel passes:
>
> fsl,tuning-step = <2>;
> fsl,tuning-start-tap = <20>;
>
> https://github.com/nxp-imx/linux-imx/blob/lf-6.1.y/arch/arm/boot/dts/imx7s.dtsi#L1216-L1217
>
> Bough? Any suggestions?
Oh, I didn't notice this patch on linux-imx, I refer to the settings in
https://github.com/nxp-imx/uboot-imx.git,
but it should be the same, I think it is better to bring this patch in
thanks
xiaolei
>
> Thanks
On Sat, Jul 22, 2023 at 4:29 AM wangxiaolei <[email protected]> wrote:
> Oh, I didn't notice this patch on linux-imx, I refer to the settings in
> https://github.com/nxp-imx/uboot-imx.git,
>
> but it should be the same, I think it is better to bring this patch in
Please send a patch that passes:
fsl,tuning-step = <2>;
fsl,tuning-start-tap = <20>;
to all usdhc instances in imx7s.dtsi.
Other imx dtsi files do this, but imx7s.dtsi currently misses these properties.
Thanks