From patchwork Fri Nov 1 11:42:26 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Josua Mayer X-Patchwork-Id: 13859283 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id DFA5CE6B243 for ; Fri, 1 Nov 2024 11:51:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:MIME-Version:Cc:To: In-Reply-To:References:Message-Id:Content-Transfer-Encoding:Content-Type: Subject:Date:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=+9ANHnSWP27iHbc+a8ou4FqQNkq16ExJMF8UFj1ohL4=; b=duV2Qzxj7lo48Crsj3bnnz2HLq lkZZljen0qdR2jM6LsK+97ZzD1LmInPz33XA5TzgHWT7Brs+nxqCj3xJ82cEe1sTYpw1u4ovGTbxF dSkcDeUEXKI/GLMArvSAaAn3PhamYhrAe3q2v1DGTC/vYVEU8g+lN8iL2OVzIlEL/2Ln30RoKBc3N iTzroKrW0vSUf1ybptPGTiX/CV4WNL9jPlb3zNaQIiCV5MZO2qRMJaIB/Xo3OJnmRWytDzn8VeCBi iWdjgdZ96kvIPe/jaP4LLAYmwQMLAdR4d2AEHtNOgRwbjZ/rI2UeBTAT+tbyMLcqotwvlTTlqXNKo DmZS/MgA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1t6qBk-00000006oHG-3Jmi; Fri, 01 Nov 2024 11:51:36 +0000 Received: from mail-db8eur05on20725.outbound.protection.outlook.com ([2a01:111:f403:2614::725] helo=EUR05-DB8-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1t6q3F-00000006mdv-1eHO for linux-arm-kernel@lists.infradead.org; Fri, 01 Nov 2024 11:42:51 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=RvHJMw07XP9iTT3tC9oAyLX/pnAqdskA6vaCewpo8TER1Z0+LL0LEk40VBEEiR77TuVZ5+7jvW4ujMGRbNBCbuzvjHVl9fgnsd05YZXL5dWDyrxemULf+jLXRpBLGkEpdS6oby5FgMabroUN0By3ximIqL0BJVp0U4FozjZI9Cr2pTYZuphqAmWiXinhPuIfY2Pq450UGmz+pf3sBkI7WymyjHLRJP9x3X32wz/kjdeDUSspYavphLX+ww2IenZYFMVaYZwV4dMJq1aqKvmclEUItocG1maEdD6gArnN/zBxqFhLYpFknMAKIyBHnySoOkcbOmutPf0Qp/xABPQYnA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=+9ANHnSWP27iHbc+a8ou4FqQNkq16ExJMF8UFj1ohL4=; b=F7FUPjWVafc+2YvvUfU06x6yLqZnGNiefUbuZd8W9Sh8BlCjZ9ZCzfxBJ7Rpjjj3fALLSRiIXUuJacS5SupD5PA/fc8cYJF0L8JYWv/X16z78KUNRf/5Wt9J3MVOr3jtIHd4L7zQlawXf05JtnkqaCaMFtbUmqVVZR4HFHDYVn0cL5vYkCtqZZWSiucWldtUC5Z1rIwQqnoLErlfdoRNebIYhNKdef/VDH5JLmS9UsPy4jUl5xCO82Exl3pmSlbxkNuO+t8QBjeHR6nCztoEKGjdQTgl5KNcuqcL789mr0O9Hpvk7ieJrtLA1SF04p1YFjZBh7o08K/kPfmAe51q9Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=solid-run.com; dmarc=pass action=none header.from=solid-run.com; dkim=pass header.d=solid-run.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=solidrn.onmicrosoft.com; s=selector1-solidrn-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=+9ANHnSWP27iHbc+a8ou4FqQNkq16ExJMF8UFj1ohL4=; b=fJxT86KyseNYwA5ki6LoqEbxYNG88HzmFQNe+owi3lF4oe0cN5W6cJ4k+IXf0uHp68qgyVD3mAhw+i2CbL7fTrrOo58ecRTpbCZ08kzh3gWLJ+4aJ+2Baduc3QkC/ei0P/bTNclcQ9M9fc4/cu0NgXCKarB7xOgh/LOBA95yFEQ= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=solid-run.com; Received: from AM9PR04MB7586.eurprd04.prod.outlook.com (2603:10a6:20b:2d5::17) by GVXPR04MB10519.eurprd04.prod.outlook.com (2603:10a6:150:1dc::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8093.32; Fri, 1 Nov 2024 11:42:29 +0000 Received: from AM9PR04MB7586.eurprd04.prod.outlook.com ([fe80::c04e:8a97:516c:5529]) by AM9PR04MB7586.eurprd04.prod.outlook.com ([fe80::c04e:8a97:516c:5529%4]) with mapi id 15.20.8093.027; Fri, 1 Nov 2024 11:42:29 +0000 From: Josua Mayer Date: Fri, 01 Nov 2024 12:42:26 +0100 Subject: [PATCH v3 2/2] mmc: host: sdhci-esdhc-imx: update esdhc sysctl dtocv bitmask Message-Id: <20241101-imx-emmc-reset-v3-2-184965eed476@solid-run.com> References: <20241101-imx-emmc-reset-v3-0-184965eed476@solid-run.com> In-Reply-To: <20241101-imx-emmc-reset-v3-0-184965eed476@solid-run.com> To: Adrian Hunter , Haibo Chen , Ulf Hansson , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam Cc: Mikhail Anikin , Jon Nettleton , Yazan Shhady , Rabeeh Khoury , imx@lists.linux.dev, linux-mmc@vger.kernel.org, s32@nxp.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Josua Mayer X-Mailer: b4 0.14.2 X-ClientProxiedBy: FR0P281CA0198.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:ad::8) To AM9PR04MB7586.eurprd04.prod.outlook.com (2603:10a6:20b:2d5::17) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM9PR04MB7586:EE_|GVXPR04MB10519:EE_ X-MS-Office365-Filtering-Correlation-Id: c715eac6-49d5-42cb-6696-08dcfa6a4419 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|7416014|366016|52116014|1800799024|38350700014; X-Microsoft-Antispam-Message-Info: =?utf-8?q?5CzhgpyadDWz3yrdUN/1k+PYmnd7tzM?= =?utf-8?q?PuSQ9+ibqvycxRBf1glqykrpF/J8zXDDHCqXAnx/p22g2Eqhd8b6BdxRBTFsbo4EU?= =?utf-8?q?OweGfoJs61kLWRmIkIHAG7A3frtD2wCwTkfBHb6mG/r3xqeTTjEl4TncsvedITUcR?= =?utf-8?q?tFehKmtBVpXoAeMYsLTjPWV29pRMk2ZsM9vylk2sU+VFRVFFjto3ReO3cGa+oJIsu?= =?utf-8?q?S4JATpiI+GiXGxafiNZbt+d3MLPWF5QP+cHaviXp61RM1Ar/F+5IPiPr5DaVjjAVN?= =?utf-8?q?p518SrYmdWPImubPnvG5PFqS3mGWsVySC/TjT0kLXI9d7qYorYoX5tYVpKXWdE0MI?= =?utf-8?q?EQCyWL6IKJFMviWzNUVAHrIO1MzSPbavED/rhN6G0cpw5LgKtYV12AQGRuxyBZa3s?= =?utf-8?q?mNLazY5YWpQHvXr3EY3e3vVfuuFsVqVSqY1pBHDol/dU6ep//dhn9s8OZtG1Rq4/d?= =?utf-8?q?LZ33frBmDkIHwEO0ogHPCtEhGzd3yoRjqsRHerleI/WWzIWtZ0HaVEwz8Fj14FhTX?= =?utf-8?q?q/xHQvUodytNJf/xikdAMELQckE+1/XG+4w9mnsCiC2ni+TNbU2tm7cL1CcxxIOFX?= =?utf-8?q?sXVf4gTQQzBCfklRGUwyQpoUOrGVMq7GYwILujwd6m988sSq4nrLFQ3jfUrIr3X85?= =?utf-8?q?zzIIunavT1pqrXpQCciXqP9K1rQ+uZJ3aoRzcPdPwS63MssIQc32K+eyoRhgQqmRi?= =?utf-8?q?11oHXKdFlt5HeP4472JZXV9i2KD6Cjc+01Fsw/LuO6SIi+7zWcyWfgSYV0vZu4zFz?= =?utf-8?q?HsA/0mG35cjUB/pVPOCl04yTz9d2YJG9GE9xeUoygIIqCUiU3XCox41yno4c6MPVl?= =?utf-8?q?rWclE0QOQTv3faXOwZd90ZQRLUmCU3FIXeQcfzITqtyaJ2NRjt/lqpdZuw9jEut3S?= =?utf-8?q?XIITUYfIU0p6f0jSFU0huFOuUgx4YrFC+r2yeloy2ADQhyqtrwTZWzYRWgRzil+0B?= =?utf-8?q?qwYvuEOiumfvkgVDEezo+waIBhAJgOsyYC4LBQCbsMkmvkTR5Zc92jRQrBS7vSWGk?= =?utf-8?q?NVvt7u3eugcLuuNfe9FbFpoQVn0Fvy45dFiLI5625bntG1+jMtZhomih4nVQ6JTH1?= =?utf-8?q?JYbetJbJcejYPwJYr2Zsg7RIYC2xSOvrNvhYAVOt0dj4WWO89/2Oz4dlo2Yzx+Bry?= =?utf-8?q?mj4rP1KvL9hDnRPegTtckaKf4ifIRBiYWY6TCwk52KnS42FZZ9T6esVaRRgmNosxz?= =?utf-8?q?7QHod4iXjlo2TWVubeHCxHiBFLaT3jkoJbN32vcv1IKxvgP5jqNiuXxXhosMW36ET?= =?utf-8?q?YtoEttgL3OpXMte0f/WxAKB9yzpjItKBDm9JkS7J1qTBErYeibYtnscw=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM9PR04MB7586.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(376014)(7416014)(366016)(52116014)(1800799024)(38350700014);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?boxD53OpQvvouRCvTwNaAfOfdXU4?= =?utf-8?q?C8qx4yq+w2Slveb5Q0bA9FyqpQvV9gFQADhMU+rjBlk6rCd4WlbBASa7CRDJEHqua?= =?utf-8?q?kpyrV/utTF2SbMUdQ682KcEBwg/Dx0OR/i3BPtA3HQmoplCoWVLNjb4GF2jqdyH2X?= =?utf-8?q?TZBP8M9DRECzM4asdVbNKERlaOBx/H/d2LFc5CgPuzgmmwY/aO6OYT7R8a7DSQ0bC?= =?utf-8?q?JDKBEAjUjj4TfSX6PxeIBLZTHWdRUKAaywzAfclyMyxem6tOxYdz+PBNkALj3vHd7?= =?utf-8?q?acaFjmvnYvuSVgTvbcyXennbdaYNvJZSx10MBF/ArDWt3joIbKzlVlAIDSS03qEjL?= =?utf-8?q?pOryOig5/prBdCem/PEoTx30NLJjNUXiy1i9df4m/22e9kLmDIxr0Hx6IYQF4NzPI?= =?utf-8?q?etrOWgcQR+VeOciJ5nJlu87VCqh9/VU6hc7xMKRjK19QriIB/agKyppWqZqvlwtE8?= =?utf-8?q?LEM/8vqlUO3grg5xtTh7PMDpHim0z3BKvtHSbdwqCA0LOolToOT2pDb7STYNLrKhv?= =?utf-8?q?T1r0RBHgo0lXWjFatChPp3bpIOSV9UDtfK8iwsiik0lzzTNA/cSljfPDlrjHctrkH?= =?utf-8?q?qJpf5Hliyj1gf37F3qaASP1v7w4YiJbj1+IrElbG1JSVhB261scg7lTJcpKTr/Agj?= =?utf-8?q?IhKU6gO3MzJd+s8ja7Im1aqgSGe6PMQP6J9xoR5wN2q2XjI9lDxal/Bv93BriSQJs?= =?utf-8?q?TmNlZJCcMDEe8HvIQ3DhZ5GWC9uGRbBURKeeDCToEcRUydIG9gM1986bCLF4OhGGu?= =?utf-8?q?mC5uIBO65oaW6fp7CmKE+6ultv/oTucZrTQ/KmqnZnE5q6ER3QuIsxMOb4lMFI5ZZ?= =?utf-8?q?R2utmEyNBQHsnHIMe1t2m94+rvcWk4EPGUw0AVROdhEn7U+ousC8O+yXAZFE8IslW?= =?utf-8?q?4y4UVCh77QDMewXC9yE8lHkrXUNCVwXvHEieY0f579hqp+ea7/z4fNKVSxSyNwUj1?= =?utf-8?q?ofX5ewAYjqxmwvqP7tbaEHks5XOPzfJK2M+iDBCVtxG0rcIbhOdD8yFXEmt6+l5+5?= =?utf-8?q?yjy99CLgSzLNp1vW2176XDka5y8UJINWJIx9shKb2of1e+G10Lebrobrt9KM8/KPZ?= =?utf-8?q?4yo39U37ldBIaxTehFWbQMwc1L0gkYUSNvszTBntNtYFsSnDMa7QHLUF1bk0ID+zA?= =?utf-8?q?Wt9O5wJAnHylBSZCnDGgNXWm3id9iUT+Mnukr+dXEcvV5uDSawWX8OeIf31+5cTJS?= =?utf-8?q?DdZ5uF4NRVd7+6wweLzpnj6owde2rXaamhLoHvu0fd7YTl8bp07MDgvBX4UI/emK/?= =?utf-8?q?dNU5yUwrtt2T/MKdsCxSHL6z9+oI2Nrb8ydMwMxD6TYALtoZ4yJ2Ef1PCSc9y0LMt?= =?utf-8?q?plI6SCTNmOENiFGCznrs5nu78aY+aomFHbFt0yGoGgUWGh2QfiC6IA4SAGDXIvDFc?= =?utf-8?q?EdrlDlp2568vdYLwSC8wj41eynw29P1j+guV633yevXdZjEChDDAP4RbURoPOSehP?= =?utf-8?q?6vJdSz+tnGU0Jq3bN7e4Z/LzGvdgaXLhoeh5ZnJPhxw1BKX1A9MPiftgEBHDnTR63?= =?utf-8?q?sdUGqP9WYMsU?= X-OriginatorOrg: solid-run.com X-MS-Exchange-CrossTenant-Network-Message-Id: c715eac6-49d5-42cb-6696-08dcfa6a4419 X-MS-Exchange-CrossTenant-AuthSource: AM9PR04MB7586.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Nov 2024 11:42:29.6750 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: a4a8aaf3-fd27-4e27-add2-604707ce5b82 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: POYOjLJUggeu5eT94CLjylHqTV54mUOO2GbJMtywXP887PX6rot5v0KKxQmV4lmanj8GrUPcgFwLOdzbnL3UJA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: GVXPR04MB10519 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241101_044249_578204_3EA7E360 X-CRM114-Status: GOOD ( 12.20 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org NXP ESDHC supports setting data timeout using uSDHCx_SYS_CTRL register DTOCV bits (bits 16-19). Currently the driver accesses those bits by 32-bit write using SDHCI_TIMEOUT_CONTROL (0x2E) defined in drivers/mmc/host/sdhci.h. This is offset by two bytes relative to uSDHCx_SYS_CTRL (0x2C). The driver also defines ESDHC_SYS_CTRL_DTOCV_MASK as first 4 bits, which is correct relative to SDHCI_TIMEOUT_CONTROL but not relative to uSDHCx_SYS_CTRL. The definition carrying control register in its name is therefore inconsistent. Update the bitmask definition for bits 16-19 to be correct relative to control register base. Update the esdhc_set_timeout function to set timeout value at control register base, not timeout offset. This solves a purely cosmetic problem. Signed-off-by: Josua Mayer Reviewed-by: Haibo Chen --- drivers/mmc/host/sdhci-esdhc-imx.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/mmc/host/sdhci-esdhc-imx.c b/drivers/mmc/host/sdhci-esdhc-imx.c index f106e291c276d0c8063e9ac59a126acf5e9e239e..cda3cc4cc22cfa214369f40f097ca50937898604 100644 --- a/drivers/mmc/host/sdhci-esdhc-imx.c +++ b/drivers/mmc/host/sdhci-esdhc-imx.c @@ -30,7 +30,7 @@ #include "sdhci-esdhc.h" #include "cqhci.h" -#define ESDHC_SYS_CTRL_DTOCV_MASK 0x0f +#define ESDHC_SYS_CTRL_DTOCV_MASK GENMASK(19, 16) #define ESDHC_SYS_CTRL_IPP_RST_N BIT(23) #define ESDHC_CTRL_D3CD 0x08 #define ESDHC_BURST_LEN_EN_INCR (1 << 27) @@ -1386,8 +1386,8 @@ static void esdhc_set_timeout(struct sdhci_host *host, struct mmc_command *cmd) /* use maximum timeout counter */ esdhc_clrset_le(host, ESDHC_SYS_CTRL_DTOCV_MASK, - esdhc_is_usdhc(imx_data) ? 0xF : 0xE, - SDHCI_TIMEOUT_CONTROL); + esdhc_is_usdhc(imx_data) ? 0xF0000 : 0xE0000, + ESDHC_SYSTEM_CONTROL); } static u32 esdhc_cqhci_irq(struct sdhci_host *host, u32 intmask)