Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Xiaolei Wang <xiaolei.wang@windriver.com>
To:     robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
        conor+dt@kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de,
        festevam@gmail.com, linux-imx@nxp.com, richardcochran@gmail.com
Cc:     devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-kernel@vger.kernel.org
Subject: [PATCH v2] ARM: dts: imx: Set default tuning step for imx7d usdhc
Date:   Sun, 23 Jul 2023 10:20:50 +0800
Message-Id: <20230723022050.524050-1-xiaolei.wang@windriver.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?TbkvFrEDF1+pn3tfJ7tnZnw/S4sHKgYoSW6dSewru4HtKi0+RavQowiUtlQt?=
 =?us-ascii?Q?fQ2Ewzer6RO8ROZ0dI4D93nGGhYfkHNORkylLTIlOKpIaMoMNNUurHkMOo4+?=
 =?us-ascii?Q?u0oqAL2VCNfUZd17XcglRp1BcZ81g4R0wNSLvTtzsbOWCXfQFE2kLgqTNl4M?=
 =?us-ascii?Q?mdCowZfW2s1lkdSapLFPcNbQntBoENRBJF+kbnkB1Hdr3m7+b7PLI7/N8UYI?=
 =?us-ascii?Q?j2YXHkNhOJUlRdfSS4kPuMVMx/rgARLBHcyovfnHlvO2P1184JdnfUc5DEcx?=
 =?us-ascii?Q?dONUeKQ/wXjtNcDE7+ordNcLYMcRTZLAGD8iQ2g3A7hkdF0DInnrIzH3w0E4?=
 =?us-ascii?Q?pTxwi1KHtY7aKEWNG0Q+znpGRmG4bvxxtzbGHbrH9J8MM7ev/Hxgwp8J8M9d?=
 =?us-ascii?Q?IoT8S6kwh6dCTgJZSgELlaWNKLJ3afpOl+t9+C4n8FdChEU9DUugoVI7vGsM?=
 =?us-ascii?Q?V3hyTMcvCR4vQ+RQTFOf6ye8CYz+Zas7CsZo1j/BRmBQCI08RStrniMbl64H?=
 =?us-ascii?Q?yBmkSHcRD42dHm+8DHH6CTrNWV/pUaeAG+7S4QoVV+MLr9QzRXstms3hNen+?=
 =?us-ascii?Q?y8ZmtrDla2yhlmxb5N32WKr+QkqnM9A3DYv4vwErgvk1nx6lQhTQ0U3mHhRX?=
 =?us-ascii?Q?4Vc0gnD5VY5YNWZoupeSTCRkMVgtg2/VA3CUgS+k6ACgVwUa4DvLtMbUUKjh?=
 =?us-ascii?Q?4bgU9Dv9+D9xsz63pDFZoVwsqMXgouw5TDnGdjg2CUOdg1Jrcl01hGMnaoCI?=
 =?us-ascii?Q?EeB2YUT9A/ha/Wz6MqYjffCetslua8IKsBThgArn8pXyOtXhfS3jNWFcrfdT?=
 =?us-ascii?Q?TqfSbf3jBbIxjeSvItbKMvuuWv02GydsQPcmpY06xkoHiou4yzWb1EWRA4FM?=
 =?us-ascii?Q?UPQxvFrmQ6g0CBcyCylwHnqb8KaPM82xOsCMqoB4QMVpiYzY1FsU2jxY2C0u?=
 =?us-ascii?Q?FLsI20XsRzrCRCgRJ+n3YoAXEnJv9aMhS7M9sUHeBPPTxylYShp22uOQohK0?=
 =?us-ascii?Q?NB/YFAi5F1d0ZjsA8rhbvNlPol4qJe8cxtykJAmcs49yXk+CG2cSePlj2TUt?=
 =?us-ascii?Q?HubQoPc9B8gi/cwfUz05L3cjOnAl6whAeuHYL77PmbU/4HX0sxN++hrtPitx?=
 =?us-ascii?Q?Tk5sAGUCmP+nCswPaM+XQCMof1yRzQWejog47ZsDvUlXXeGXg7wSYaDJJqgI?=
 =?us-ascii?Q?bCOLIBM2RchgwBqAHVJUQ7PHMNoodSFZ61V3k+cVRTbrFQrTBWI88QWeLI1Y?=
 =?us-ascii?Q?mZCL/fHJnPULbnAfr5IbcvKmyz75iw9GycO/ur9my3ujtMNQHG+bb6fkMfpF?=
 =?us-ascii?Q?yiHHDSNmiXpC7evdRsBZ1uSWFIhUxpUl9gTT2dwQpbCGTHUjJYpeKE+eFQuE?=
 =?us-ascii?Q?t4/ZwqwZ78syNlVTf9To4z9Hq8G4scNj/2T8RHYKMoF3EiV+WmO3vmveduFO?=
 =?us-ascii?Q?udnJd9kLcI0Df2G+sJ41EUVN70YKW+DwtIEBxFK5jedsBT9d9htLhQp9X4dD?=
 =?us-ascii?Q?qyjYTaJ8GG4wWCTD76ChTolTLlds03x9Y6LWxSwQW5OX6qaxTWA9cZUJeDYV?=
 =?us-ascii?Q?3omQQaX8qnBQmbDb0fwdH0toJIPnpq7qehLK2+d7Q251F2NqDsrI0GhNFGDb?=
 =?us-ascii?Q?mA=3D=3D?=
X-OriginatorOrg: windriver.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 321f024b-864a-4a5b-9dc1-08db8b237df8
X-MS-Exchange-CrossTenant-AuthSource: MW5PR11MB5764.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Jul 2023 02:21:16.1603
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 8ddb2873-a1ad-4a18-ae4e-4644631433be
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: UM6QJqQmtq6wshKKVKRuyQNDO6Ab7ZN+lGPaglhHVIAsyjNkla9+PYxE+R/T0d0iOW6Gn7/WaL6oyXsAQgA6BxHt5nXZne1pvlv5bmaKfa8=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ2PR11MB8538
X-Proofpoint-ORIG-GUID: EUaetmmOisk2rCWPeHIi4rXl7raqwqto
X-Proofpoint-GUID: EUaetmmOisk2rCWPeHIi4rXl7raqwqto
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.254,Aquarius:18.0.957,Hydra:6.0.591,FMLib:17.11.176.26
 definitions=2023-07-22_12,2023-07-20_01,2023-05-22_02
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 priorityscore=1501
 mlxlogscore=967 lowpriorityscore=0 malwarescore=0 spamscore=0
 impostorscore=0 mlxscore=0 bulkscore=0 phishscore=0 adultscore=0
 clxscore=1011 suspectscore=0 classifier=spam adjust=0 reason=mlx
 scancount=1 engine=8.19.0-2306200000 definitions=main-2307230021
X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,
        RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_PASS,
        T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

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. This refers to the NXP vendor's
commit below:

https://github.com/nxp-imx/linux-imx/blob/lf-6.1.y/
arch/arm/boot/dts/imx7s.dtsi#L1216-L1217

Fixes: 1e336aa0c025 ("mmc: sdhci-esdhc-imx: correct the tuning start tap and step setting")
Signed-off-by: Xiaolei Wang <xiaolei.wang@windriver.com>

v2:
  * Refer to https://github.com/nxp-imx/linux-imx/blob/lf-6.1.y/arch/arm/boot/dts/imx7s.dtsi#L1216-L1217
    and add the following attributes for all usdhc of imx7:
      fsl,tuning-step = <2>;
      fsl,tuning-start-tap = <20>;

---
 arch/arm/boot/dts/nxp/imx/imx7s.dtsi | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/arch/arm/boot/dts/nxp/imx/imx7s.dtsi b/arch/arm/boot/dts/nxp/imx/imx7s.dtsi
index 54026c2c93fa..6ffb428dc939 100644
--- a/arch/arm/boot/dts/nxp/imx/imx7s.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx7s.dtsi
@@ -1184,6 +1184,8 @@ usdhc1: mmc@30b40000 {
 					<&clks IMX7D_USDHC1_ROOT_CLK>;
 				clock-names = "ipg", "ahb", "per";
 				bus-width = <4>;
+				fsl,tuning-step = <2>;
+				fsl,tuning-start-tap = <20>;
 				status = "disabled";
 			};
 
@@ -1196,6 +1198,8 @@ usdhc2: mmc@30b50000 {
 					<&clks IMX7D_USDHC2_ROOT_CLK>;
 				clock-names = "ipg", "ahb", "per";
 				bus-width = <4>;
+				fsl,tuning-step = <2>;
+				fsl,tuning-start-tap = <20>;
 				status = "disabled";
 			};
 
@@ -1208,6 +1212,8 @@ usdhc3: mmc@30b60000 {
 					<&clks IMX7D_USDHC3_ROOT_CLK>;
 				clock-names = "ipg", "ahb", "per";
 				bus-width = <4>;
+				fsl,tuning-step = <2>;
+				fsl,tuning-start-tap = <20>;
 				status = "disabled";
 			};
 
-- 
2.25.1