From patchwork Thu Feb 14 08:21:09 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yangbo Lu X-Patchwork-Id: 1041897 X-Patchwork-Delegate: prabhakar@freescale.com Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=lists.denx.de (client-ip=81.169.180.215; helo=lists.denx.de; envelope-from=u-boot-bounces@lists.denx.de; receiver=) Authentication-Results: ozlabs.org; dmarc=fail (p=none dis=none) header.from=nxp.com Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=nxp.com header.i=@nxp.com header.b="CIGbGoCp"; dkim-atps=neutral Received: from lists.denx.de (dione.denx.de [81.169.180.215]) by ozlabs.org (Postfix) with ESMTP id 440Tr51pDQz9sMp for ; Thu, 14 Feb 2019 19:21:20 +1100 (AEDT) Received: by lists.denx.de (Postfix, from userid 105) id 7C907C220C6; Thu, 14 Feb 2019 08:21:15 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de X-Spam-Level: X-Spam-Status: No, score=0.0 required=5.0 tests=BAD_ENC_HEADER, SPF_HELO_PASS, T_DKIM_INVALID autolearn=unavailable autolearn_force=no version=3.4.0 Received: from lists.denx.de (localhost [IPv6:::1]) by lists.denx.de (Postfix) with ESMTP id 4EB4AC21EB1; Thu, 14 Feb 2019 08:21:13 +0000 (UTC) Received: by lists.denx.de (Postfix, from userid 105) id 9AFA5C21EB1; Thu, 14 Feb 2019 08:21:11 +0000 (UTC) Received: from EUR01-HE1-obe.outbound.protection.outlook.com (mail-eopbgr130053.outbound.protection.outlook.com [40.107.13.53]) by lists.denx.de (Postfix) with ESMTPS id ED735C21DA6 for ; Thu, 14 Feb 2019 08:21:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=wWq+WQbW5qYzQ9jdMGwQeqKzEH1ADWGCRw4NLYUEekE=; b=CIGbGoCp+WPAkaQj0n77mEr0qQ+4HT4d6FSW5I1FLuNfr1o9cFopR41RF5kZGOdQFC4mFhmRds5LT77jOFt+QpwbnDisuc3/SRePRzdSDd0rAryuu8D0oYbvzlKjHHEbKls/IS7hegOBAXEQipCQoXQKEhYselWkvpzS5gYwlKc= Received: from VI1PR0401MB2237.eurprd04.prod.outlook.com (10.169.133.18) by VI1PR0401MB1887.eurprd04.prod.outlook.com (10.166.140.137) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1622.16; Thu, 14 Feb 2019 08:21:09 +0000 Received: from VI1PR0401MB2237.eurprd04.prod.outlook.com ([fe80::78f9:21ec:83e4:bc72]) by VI1PR0401MB2237.eurprd04.prod.outlook.com ([fe80::78f9:21ec:83e4:bc72%3]) with mapi id 15.20.1601.023; Thu, 14 Feb 2019 08:21:09 +0000 From: "Y.b. Lu" To: "u-boot@lists.denx.de" Thread-Topic: [PATCH 1/3] mmc: fsl_esdhc: add esdhc_imx flag Thread-Index: AQHUxD49ZpJOTD79PEW6Ck+BTNk1Qg== Date: Thu, 14 Feb 2019 08:21:09 +0000 Message-ID: <20190214082207.12082-1-yangbo.lu@nxp.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-clientproxiedby: HK2PR02CA0185.apcprd02.prod.outlook.com (2603:1096:201:21::21) To VI1PR0401MB2237.eurprd04.prod.outlook.com (2603:10a6:800:28::18) authentication-results: spf=none (sender IP is ) smtp.mailfrom=yangbo.lu@nxp.com; x-ms-exchange-messagesentrepresentingtype: 1 x-mailer: git-send-email 2.17.1 x-originating-ip: [119.31.174.73] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: a29c4d90-b9b0-4741-97cf-08d692555f82 x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600110)(711020)(4605077)(4618075)(2017052603328)(7153060)(7193020); SRVR:VI1PR0401MB1887; x-ms-traffictypediagnostic: VI1PR0401MB1887: x-microsoft-exchange-diagnostics: =?iso-8859-1?Q?1; VI1PR0401MB1887; 23:MeE/opQRhBby5vnsAavexJ8SJb12zSpnw/kYU?= =?iso-8859-1?q?gkfUBIbu/rip5qXNkBp1cmeid?= =?iso-8859-1?q?gJLOPR9qg17hy77ozLzxNCo3DQNGQushCto6DDSawsasGW69FFp?= =?iso-8859-1?q?hEUzhtIAs7y7JNYNsh8aV3ePppQJBHvfAGP8PAJU4K8L6SNACyV?= =?iso-8859-1?q?/4q6RfJdUf2sssRlhAmLv3IAHTwaQRnGvv3XyatuQv5NAr1CYxH?= =?iso-8859-1?q?4aUXtQeyXCQJ6Jvrp5Xxo8GKPKnMIqGBJsBEK4Q2oZ6DzwI8Urr?= =?iso-8859-1?q?STj+nv+F6vYHG/cCjbVtLvwch9vc+R8slFdRyBDtGzA5q3OjrSk?= =?iso-8859-1?q?8Dmbw+v2SOPkMO6dABfLm4rc04wDaor4F1aMzMajAkKEErqCRYd?= =?iso-8859-1?q?Gc1d6di5ReqGDsRtPP1hb4sJ/qaGe/Bspj2AQDYzBqmmS1Rg+ph?= =?iso-8859-1?q?BF5nuYEbRgevcb6N6XlUkiEtCFJEs2J/Hd77kpebLjb25EoYjsy?= =?iso-8859-1?q?Rt7kl+YudVt2Uzm+wkXzt0P56dbV6IClmGRAtPxIFT2Ff2DtOpm?= =?iso-8859-1?q?iwEpRV2r/DzpjDfTKdTLHAbFqmx7Vq6/ia3CWZWRk4gIg3Yr2bz?= =?iso-8859-1?q?K0oRxeSrKulcuSPTBeE+xOoWra3oGPTZZbyBLTp78oUhwjfW3Xy?= =?iso-8859-1?q?6kbpUaGvZ6JvHm1uDdMk9GEYLsWSEinr4f7yXY0xoD6njdCIp8J?= =?iso-8859-1?q?YgGHD2YY11y7awhFqJBx08UQwe0BB3jNsDO5hNJ4HRxNfddwcJw?= =?iso-8859-1?q?7cyu/V/o7Aar/OihDUunWi1QtMsLc2Yduc7QwHudbNLz4MxgfLm?= =?iso-8859-1?q?s3k0kMtOfa5HT65Am33JPBhvSnmykrjRq3vBsOYkK3GD8gcA4n7?= =?iso-8859-1?q?A5O6tuApT6oe5lH/S9RYv+IZwQpQeBygIN3vaBrlgSRzJ2Cxedo?= =?iso-8859-1?q?GnsXfMxQUW7K/hGzk6sKKOqbXwLAXrGBK9BULPmljI0HC0oQdcZ?= =?iso-8859-1?q?3TyqaWs0GKLBn272ZPTSnVn+Esq/kwprVYUpakd65cM8qEPvUJl?= =?iso-8859-1?q?nREgj6t6PmDPJwVkhmomculyizZgsZKeXASt1U8TkgBv7JQN01V?= =?iso-8859-1?q?lsgXu08arysavjo8/wyPi/NER3yAreCUPfGdK7f2YZFdKAONUbO?= =?iso-8859-1?q?Cw8ifWjoCqy2wZhGb4WSY0vfxNzzH3jY4SqsnG0h4DE9J2F8mMJ?= =?iso-8859-1?q?or8PKspR52pkPQM84Aw7ckxFR7G5yvGnnfC7hlBOmN5CgCtV16J?= =?iso-8859-1?q?USt4eBecVYA=3D?= x-microsoft-antispam-prvs: x-forefront-prvs: 09480768F8 x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(39860400002)(366004)(136003)(376002)(346002)(396003)(189003)(199004)(97736004)(1076003)(106356001)(3846002)(6116002)(81156014)(81166006)(68736007)(86362001)(186003)(5640700003)(105586002)(99286004)(2351001)(486006)(50226002)(102836004)(6506007)(386003)(66066001)(14454004)(71200400001)(71190400001)(478600001)(476003)(6916009)(7736002)(4326008)(8936002)(26005)(36756003)(316002)(54906003)(25786009)(256004)(6486002)(2616005)(53936002)(8676002)(2501003)(6512007)(2906002)(305945005)(52116002)(6436002); DIR:OUT; SFP:1101; SCL:1; SRVR:VI1PR0401MB1887; H:VI1PR0401MB2237.eurprd04.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1; received-spf: None (protection.outlook.com: nxp.com does not designate permitted sender hosts) x-ms-exchange-senderadcheck: 1 x-microsoft-antispam-message-info: uJpN/dc6CM/ggGvt+Ljjs+jIjQUvtgnBfoaBmfOJp7U+sk8Z6GJj4lKIHRnWNgUtFgbhI5YnYec76azYKg3R92qpARsYuf/4PWaNUbzRjEIBwp1MUWn0GyDH7bxf8/87Ld5b6SsRniiCOXH8pBH2Zczx3Jp7FiyUcDdckNxqnF6AFDPtCFkTeTz/4VgDYDJUH4DpBxpIDfflYAk24wHvlYqp9vaHqEN9i5htDNC0AoP/Zy15VlDlT1bE80jnCYSKXTJHgfwqrV+7K9/Z2lGLC9xLB/PxR8gCz+Pa5HLVbREsAV2fSjC/d7zPCdX4Zts8d4nHW4gJWAaijME8hOGBgqtrdA3ZKuQ+I/roj0u9sk9LBK236gabVpqfDn8fW1j9fWKKyAOs4q5TngeY8k9QzMYQ6JNHtB+ehdKylU4jcn8= MIME-Version: 1.0 X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: a29c4d90-b9b0-4741-97cf-08d692555f82 X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Feb 2019 08:21:06.8363 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0401MB1887 Subject: [U-Boot] [PATCH 1/3] mmc: fsl_esdhc: add esdhc_imx flag X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.18 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" The fsl_esdhc driver was for Freescale eSDHC on MPC83XX/MPC85XX initially. The later QoriQ series processors (which are evolutions of MPC83XX/MPC85XX) and i.MX series processors were using this driver for their eSDHCs too. So there are two evolution directions for eSDHC now. For the two series processors, the eSDHCs are becoming more and more different. We should have split it into two drivers, like them (sdhci-of-esdhc.c/sdhci-esdhc-imx.c) in linux kernel. But it seemed to be a lot of work now. So let's keep as it is. Be very careful to change the driver if the changes are not common for all eSDHCs, and clarify i.MX eSDHC specific things to distingush them with QorIQ eSDHC. This patch is to added an esdhc_imx flag for the development of i.MX eSDHC, to distinguish it with QoriQ eSDHC. Signed-off-by: Yangbo Lu --- drivers/mmc/fsl_esdhc.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/drivers/mmc/fsl_esdhc.c b/drivers/mmc/fsl_esdhc.c index 21fa2ab1d4..787a13a8a9 100644 --- a/drivers/mmc/fsl_esdhc.c +++ b/drivers/mmc/fsl_esdhc.c @@ -147,6 +147,7 @@ struct fsl_esdhc_priv { struct gpio_desc cd_gpio; struct gpio_desc wp_gpio; #endif + bool esdhc_imx; }; /* Return the XFERTYP flags for a given command and data packet */ @@ -1462,6 +1463,16 @@ static int fsl_esdhc_probe(struct udevice *dev) priv->caps = data->caps; } + /* + * This is to specify whether current eSDHC is an i.MX eSDHC, + * since the i.MX eSDHC has been becoming more and more different + * with QorIQ eSDHC and initial MPC83XX/MPC85XX. + */ + if (!fdt_node_check_compatible(fdt, node, "fsl,esdhc")) + priv->esdhc_imx = true; + else + priv->esdhc_imx = false; + val = dev_read_u32_default(dev, "bus-width", -1); if (val == 8) priv->bus_width = 8; From patchwork Thu Feb 14 08:21:12 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yangbo Lu X-Patchwork-Id: 1041898 X-Patchwork-Delegate: prabhakar@freescale.com Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=lists.denx.de (client-ip=81.169.180.215; helo=lists.denx.de; envelope-from=u-boot-bounces@lists.denx.de; receiver=) Authentication-Results: ozlabs.org; dmarc=fail (p=none dis=none) header.from=nxp.com Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=nxp.com header.i=@nxp.com header.b="FJIEKqON"; dkim-atps=neutral Received: from lists.denx.de (dione.denx.de [81.169.180.215]) by ozlabs.org (Postfix) with ESMTP id 440TrQ3whQz9sMp for ; Thu, 14 Feb 2019 19:21:42 +1100 (AEDT) Received: by lists.denx.de (Postfix, from userid 105) id E4F3AC22093; Thu, 14 Feb 2019 08:21:32 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de X-Spam-Level: X-Spam-Status: No, score=0.0 required=5.0 tests=BAD_ENC_HEADER, SPF_HELO_PASS, T_DKIM_INVALID autolearn=unavailable autolearn_force=no version=3.4.0 Received: from lists.denx.de (localhost [IPv6:::1]) by lists.denx.de (Postfix) with ESMTP id B0B7EC220D3; Thu, 14 Feb 2019 08:21:21 +0000 (UTC) Received: by lists.denx.de (Postfix, from userid 105) id C4F71C220CE; Thu, 14 Feb 2019 08:21:18 +0000 (UTC) Received: from EUR03-DB5-obe.outbound.protection.outlook.com (mail-eopbgr40071.outbound.protection.outlook.com [40.107.4.71]) by lists.denx.de (Postfix) with ESMTPS id 58918C22093 for ; Thu, 14 Feb 2019 08:21:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=cqqfhxSZi4p2d0wSSWuQOoQ7dnoCO84dz8SpoikzQaU=; b=FJIEKqONGaG2ewidUC2PScBi0Dmajh7vHCeUFuJY/HiOlplB2foXfeo0hyR0AVNdxhdC9DShEL6bowBQg8YOEZRW+tYSvzEu7WB+2gfIotcu/ouo+7CScgMB5OsjtYDjfDlDX40VCQEqOGck13EeDADW3fy3hj6Sf06A88LGrnc= Received: from VI1PR0401MB2237.eurprd04.prod.outlook.com (10.169.133.18) by VI1PR0401MB1887.eurprd04.prod.outlook.com (10.166.140.137) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1622.16; Thu, 14 Feb 2019 08:21:12 +0000 Received: from VI1PR0401MB2237.eurprd04.prod.outlook.com ([fe80::78f9:21ec:83e4:bc72]) by VI1PR0401MB2237.eurprd04.prod.outlook.com ([fe80::78f9:21ec:83e4:bc72%3]) with mapi id 15.20.1601.023; Thu, 14 Feb 2019 08:21:12 +0000 From: "Y.b. Lu" To: "u-boot@lists.denx.de" Thread-Topic: [PATCH 2/3] mmc: fsl_esdhc: clean up register definition Thread-Index: AQHUxD4/nbMGQfsKL0GP+PHfr94rhw== Date: Thu, 14 Feb 2019 08:21:12 +0000 Message-ID: <20190214082207.12082-2-yangbo.lu@nxp.com> References: <20190214082207.12082-1-yangbo.lu@nxp.com> In-Reply-To: <20190214082207.12082-1-yangbo.lu@nxp.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-clientproxiedby: HK2PR02CA0185.apcprd02.prod.outlook.com (2603:1096:201:21::21) To VI1PR0401MB2237.eurprd04.prod.outlook.com (2603:10a6:800:28::18) authentication-results: spf=none (sender IP is ) smtp.mailfrom=yangbo.lu@nxp.com; x-ms-exchange-messagesentrepresentingtype: 1 x-mailer: git-send-email 2.17.1 x-originating-ip: [119.31.174.73] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 4c4b528f-fb70-4b80-b17a-08d692556152 x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600110)(711020)(4605077)(4618075)(2017052603328)(7153060)(7193020); SRVR:VI1PR0401MB1887; x-ms-traffictypediagnostic: VI1PR0401MB1887: x-microsoft-exchange-diagnostics: =?iso-8859-1?Q?1; VI1PR0401MB1887; 23:UiFACarqfEoyomVNurkOcCCXXO6w/XbuleIAe?= =?iso-8859-1?q?vVlc8duWU/AeMICg3IDz2L4uE?= =?iso-8859-1?q?vEvDhLcU5UOuRKj9blE3GhZV/h6AoFjwYoCf3YnvWseJL76BJeA?= =?iso-8859-1?q?EgndI+JOLERs049OxLrYJd0L52H0F0tAzVSRE+qWXkUdUqxtDG2?= =?iso-8859-1?q?g1TnnkHKjQq/bP/7Zy7Wlc8CJsxCV+BKtN+z4cNXC6w5sxDF4br?= =?iso-8859-1?q?5c5D0sR2LbIjK3/OF79KzbI+syKulvWA53H5rwQ7OmJm6SLiuqd?= =?iso-8859-1?q?T2F8xpE/xvv6V8gv+z8d1UJK0Z9F2q4W3mahMf2sse1MSOp/tpl?= =?iso-8859-1?q?W9xxUgrOhRMR14kWVOqlAs6APVdooSNGTNzrZ2wGXO0IvSbORt+?= =?iso-8859-1?q?L3hLof4MRuvjYKACSdrFvR0CQSUoLDfRFg5+irOFSlJCvOXmd6D?= =?iso-8859-1?q?836UWnxCIrt4/5Td2G4g/84uH8c/ZGRuhiNwQY6u+2hOMW5KF2d?= =?iso-8859-1?q?82k4imkF1gAtxftUwjMjzK4LItUoKdKi0o9ILUlFtMhsYf9RIrw?= =?iso-8859-1?q?KOli5ujHcGW+2fNhRGqJoyR4yWvjOojoicRm1IT1oEFQ2U8foMG?= =?iso-8859-1?q?wxDR1EWUN9BjYyL6Izq9T4g0pluUrhb06xS+D3CkwX0IbKoebaM?= =?iso-8859-1?q?+4l/VbhaXl8uGBgZaZZV62i7RR5OpkckOOeHT8JJEFEuGVuzkMx?= =?iso-8859-1?q?4mVSEQ44teTKgW7pdxZIEaomNnTTeBK2llS1cAx5JbpGhWrvyEi?= =?iso-8859-1?q?6XrJ7oDdl31OE7EgllxgwAsuj0u41ip13iRb+LtwA6nCXGB9IMm?= =?iso-8859-1?q?w5qwOQMS3ZJS79PMfaGjpxQ2H+8xgyXa2KqKnj/4IuVyWoECLNo?= =?iso-8859-1?q?CUvrD4JG2ZzwSfTGrL2O+WJA0ATXJnZNf3CXTHLuqJDNq5ELKYP?= =?iso-8859-1?q?J9qo814RHKe6C77EaEI+PekISz8YuneelKeiYPCm83kGJrf86D5?= =?iso-8859-1?q?p+mWkuWxElhQHivSJQRhXW2Hs9SD6HjbTpQWCkNuOul1VehSdwY?= =?iso-8859-1?q?g2eO1T3e8OsSYoEuterZniLtRKOywTOhk/nKuyIHWI+dXNnR+rx?= =?iso-8859-1?q?/LcwJufb6WxeQFmlcmZAXP8gPQ6lguaBBNt01E+U3tlWyAhXWX6?= =?iso-8859-1?q?zhgsHNxWWpr20ukd2TlWz8CBKnhUZamAfhwCW+93Ju7RNNM2uqO?= =?iso-8859-1?q?UO1aIKj9+QeC0ZQAcx4iMXdOLEE+Y81yXIu7r4n+7cVK6QKp1y0?= =?iso-8859-1?q?eX4/bfjYWJWypHKqqi40VpUfvgX+UP/Wj5mjJUhedkRFvFJgD06?= =?iso-8859-1?q?4rUIVPl/s/io1jBPG8NXzYGZMSBRF+ai52G2+ftdXD/fdOq53EF?= =?iso-8859-1?q?pwMUL9krHiFyy7N1KGUaePf2rOCNaoKTVGl7Q=3D?= x-microsoft-antispam-prvs: x-forefront-prvs: 09480768F8 x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(39860400002)(366004)(136003)(376002)(346002)(396003)(189003)(199004)(446003)(97736004)(1076003)(106356001)(30864003)(3846002)(6116002)(81156014)(81166006)(11346002)(68736007)(86362001)(186003)(5640700003)(105586002)(99286004)(2351001)(486006)(50226002)(102836004)(6506007)(76176011)(386003)(66066001)(14454004)(71200400001)(71190400001)(478600001)(476003)(6916009)(7736002)(4326008)(8936002)(26005)(36756003)(316002)(54906003)(25786009)(256004)(6486002)(2616005)(53936002)(14444005)(8676002)(2501003)(6512007)(53946003)(2906002)(305945005)(52116002)(6436002); DIR:OUT; SFP:1101; SCL:1; SRVR:VI1PR0401MB1887; H:VI1PR0401MB2237.eurprd04.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1; received-spf: None (protection.outlook.com: nxp.com does not designate permitted sender hosts) x-ms-exchange-senderadcheck: 1 x-microsoft-antispam-message-info: cppk9WCHocMNWlCWYPBdwydpdbPuc641OtigAwB5SEnhRM0Yrc2nNX/36rJ1nyOekpmOkzOUFSA56fmtbSZOEE5KAEltvkmaWw0FEN++36x8bVKWFQZj/9mZkAjQAPVrckVEkjTbAtN80CHKNkNRWARX6/THusA5k5AagHiSIDF4sir99eK/gIQ255gC1VZ9YI/nF0uylUPj2r0GF1QydzAfPiuO8daFBdPSest6hNOpCKxlSVhM2OzeP4Wv5v6aKyw4fc4pKN2vKb48/dQxGvKk2R6t5fp4Gu9PlAWFAtbYwDfpjGowGwT/MxgkxnsDwui9dA8u6Ku4YDXs4OjV2fSjpL1zJYhNWOrLRFqcHuIx4lmEXlL7rUabTYk5Jl/6F3A3Q11QY3iGFnxVrlNlos82gBg2I1MTcK1o0MUZ3VA= MIME-Version: 1.0 X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 4c4b528f-fb70-4b80-b17a-08d692556152 X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Feb 2019 08:21:09.9269 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0401MB1887 Subject: [U-Boot] [PATCH 2/3] mmc: fsl_esdhc: clean up register definition X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.18 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" The fsl_esdhc driver was for Freescale eSDHC on MPC83XX/MPC85XX initially. The later QoriQ series processors (which are evolutions of MPC83XX/MPC85XX) and i.MX series processors were using this driver for their eSDHCs too. So there are two evolution directions for eSDHC now. For the two series processors, the eSDHCs are becoming more and more different. We should have split it into two drivers, like them (sdhci-of-esdhc.c/sdhci-esdhc-imx.c) in linux kernel. But it seemed to be a lot of work now. So let's keep as it is. Be very careful to change the driver if the changes are not common for all eSDHCs, and clarify i.MX eSDHC specific things to distingush them with QorIQ eSDHC. This patch is to clean up register definition. Besides the QorIQ eSDHC register definition, the i.MX eSDHC specific registers should be clarified. Signed-off-by: Yangbo Lu Acked-by: Peng Fan --- drivers/mmc/fsl_esdhc.c | 66 ++++++------- include/fsl_esdhc.h | 208 ++++++++++++++++++++++------------------ 2 files changed, 149 insertions(+), 125 deletions(-) diff --git a/drivers/mmc/fsl_esdhc.c b/drivers/mmc/fsl_esdhc.c index 787a13a8a9..63f922382f 100644 --- a/drivers/mmc/fsl_esdhc.c +++ b/drivers/mmc/fsl_esdhc.c @@ -54,23 +54,23 @@ struct fsl_esdhc { uint autoc12err; /* Auto CMD error status register */ uint hostcapblt; /* Host controller capabilities register */ uint wml; /* Watermark level register */ - uint mixctrl; /* For USDHC */ + uint mixctrl; /* For i.MX USDHC */ char reserved1[4]; /* reserved */ uint fevt; /* Force event register */ uint admaes; /* ADMA error status register */ uint adsaddr; /* ADMA system address register */ char reserved2[4]; - uint dllctrl; - uint dllstat; - uint clktunectrlstatus; + uint dllctrl; /* For i.MX USDHC */ + uint dllstat; /* For i.MX USDHC */ + uint clktunectrlstatus; /* For i.MX USDHC */ char reserved3[4]; - uint strobe_dllctrl; - uint strobe_dllstat; + uint strobe_dllctrl; /* For i.MX USDHC */ + uint strobe_dllstat; /* For i.MX USDHC */ char reserved4[72]; - uint vendorspec; - uint mmcboot; - uint vendorspec2; - uint tuning_ctrl; /* on i.MX6/7/8 */ + uint vendorspec; /* For i.MX USDHC */ + uint mmcboot; /* For i.MX USDHC */ + uint vendorspec2; /* For i.MX USDHC */ + uint tuning_ctrl; /* For i.MX USDHC, on i.MX6/7/8 */ char reserved5[44]; uint hostver; /* Host controller version register */ char reserved6[4]; /* reserved */ @@ -80,11 +80,11 @@ struct fsl_esdhc { char reserved8[4]; /* reserved */ uint hostcapblt2; /* Host controller capabilities register 2 */ char reserved9[8]; /* reserved */ - uint tcr; /* Tuning control register */ + uint tbctl; /* Tuning block control register */ char reserved10[28]; /* reserved */ uint sddirctl; /* SD direction control register */ char reserved11[712];/* reserved */ - uint scr; /* eSDHC control register */ + uint esdhcctl; /* eSDHC control register */ }; struct fsl_esdhc_plat { @@ -496,7 +496,7 @@ static int esdhc_send_cmd_common(struct fsl_esdhc_priv *priv, struct mmc *mmc, /* Switch voltage to 1.8V if CMD11 succeeded */ if (cmd->cmdidx == SD_CMD_SWITCH_UHS18V) { - esdhc_setbits32(®s->vendorspec, ESDHC_VENDORSPEC_VSELECT); + esdhc_setbits32(®s->vendorspec, VENDORSPEC_VSELECT); printf("Run CMD11 1.8V switch\n"); /* Sleep for 5 ms - max time for card to switch to 1.8V */ @@ -728,23 +728,23 @@ static void esdhc_set_strobe_dll(struct mmc *mmc) struct fsl_esdhc *regs = priv->esdhc_regs; u32 val; - if (priv->clock > ESDHC_STROBE_DLL_CLK_FREQ) { - writel(ESDHC_STROBE_DLL_CTRL_RESET, ®s->strobe_dllctrl); + if (priv->clock > STROBE_DLL_CLK_FREQ) { + writel(STROBE_DLL_CTRL_RESET, ®s->strobe_dllctrl); /* * enable strobe dll ctrl and adjust the delay target * for the uSDHC loopback read clock */ - val = ESDHC_STROBE_DLL_CTRL_ENABLE | + val = STROBE_DLL_CTRL_ENABLE | (priv->strobe_dll_delay_target << - ESDHC_STROBE_DLL_CTRL_SLV_DLY_TARGET_SHIFT); + STROBE_DLL_CTRL_SLV_DLY_TARGET_SHIFT); writel(val, ®s->strobe_dllctrl); /* wait 1us to make sure strobe dll status register stable */ mdelay(1); val = readl(®s->strobe_dllstat); - if (!(val & ESDHC_STROBE_DLL_STS_REF_LOCK)) + if (!(val & STROBE_DLL_STS_REF_LOCK)) pr_warn("HS400 strobe DLL status REF not lock!\n"); - if (!(val & ESDHC_STROBE_DLL_STS_SLV_LOCK)) + if (!(val & STROBE_DLL_STS_SLV_LOCK)) pr_warn("HS400 strobe DLL status SLV not lock!\n"); } } @@ -817,9 +817,9 @@ static int esdhc_set_voltage(struct mmc *mmc) } #endif - esdhc_clrbits32(®s->vendorspec, ESDHC_VENDORSPEC_VSELECT); + esdhc_clrbits32(®s->vendorspec, VENDORSPEC_VSELECT); if (!(esdhc_read32(®s->vendorspec) & - ESDHC_VENDORSPEC_VSELECT)) + VENDORSPEC_VSELECT)) return 0; return -EAGAIN; @@ -833,8 +833,8 @@ static int esdhc_set_voltage(struct mmc *mmc) } } #endif - esdhc_setbits32(®s->vendorspec, ESDHC_VENDORSPEC_VSELECT); - if (esdhc_read32(®s->vendorspec) & ESDHC_VENDORSPEC_VSELECT) + esdhc_setbits32(®s->vendorspec, VENDORSPEC_VSELECT); + if (esdhc_read32(®s->vendorspec) & VENDORSPEC_VSELECT) return 0; return -EAGAIN; @@ -952,7 +952,7 @@ static int esdhc_set_ios_common(struct fsl_esdhc_priv *priv, struct mmc *mmc) #ifdef CONFIG_FSL_ESDHC_USE_PERIPHERAL_CLK /* Select to use peripheral clock */ esdhc_clock_control(priv, false); - esdhc_setbits32(®s->scr, ESDHCCTL_PCS); + esdhc_setbits32(®s->esdhcctl, ESDHCCTL_PCS); esdhc_clock_control(priv, true); #endif /* Set the clock speed */ @@ -1028,7 +1028,7 @@ static int esdhc_init_common(struct fsl_esdhc_priv *priv, struct mmc *mmc) /* Put VEND_SPEC to default value */ if (priv->vs18_enable) esdhc_write32(®s->vendorspec, (VENDORSPEC_INIT | - ESDHC_VENDORSPEC_VSELECT)); + VENDORSPEC_VSELECT)); else esdhc_write32(®s->vendorspec, VENDORSPEC_INIT); @@ -1038,7 +1038,7 @@ static int esdhc_init_common(struct fsl_esdhc_priv *priv, struct mmc *mmc) #ifndef ARCH_MXC /* Enable cache snooping */ - esdhc_write32(®s->scr, 0x00000040); + esdhc_write32(®s->esdhcctl, 0x00000040); #endif #ifndef CONFIG_FSL_USDHC @@ -1184,7 +1184,7 @@ static int fsl_esdhc_init(struct fsl_esdhc_priv *priv, #endif if (priv->vs18_enable) - esdhc_setbits32(®s->vendorspec, ESDHC_VENDORSPEC_VSELECT); + esdhc_setbits32(®s->vendorspec, VENDORSPEC_VSELECT); writel(SDHCI_IRQ_EN_BITS, ®s->irqstaten); cfg = &plat->cfg; @@ -1272,11 +1272,11 @@ static int fsl_esdhc_init(struct fsl_esdhc_priv *priv, if (priv->flags & ESDHC_FLAG_STD_TUNING) { u32 val = readl(®s->tuning_ctrl); - val |= ESDHC_STD_TUNING_EN; - val &= ~ESDHC_TUNING_START_TAP_MASK; + val |= STD_TUNING_EN; + val &= ~TUNING_START_TAP_MASK; val |= priv->tuning_start_tap; - val &= ~ESDHC_TUNING_STEP_MASK; - val |= (priv->tuning_step) << ESDHC_TUNING_STEP_SHIFT; + val &= ~TUNING_STEP_MASK; + val |= (priv->tuning_step) << TUNING_STEP_SHIFT; writel(val, ®s->tuning_ctrl); } } @@ -1484,10 +1484,10 @@ static int fsl_esdhc_probe(struct udevice *dev) val = fdtdec_get_int(fdt, node, "fsl,tuning-step", 1); priv->tuning_step = val; val = fdtdec_get_int(fdt, node, "fsl,tuning-start-tap", - ESDHC_TUNING_START_TAP_DEFAULT); + TUNING_START_TAP_DEFAULT); priv->tuning_start_tap = val; val = fdtdec_get_int(fdt, node, "fsl,strobe-dll-delay-target", - ESDHC_STROBE_DLL_CTRL_SLV_DLY_TARGET_DEFAULT); + STROBE_DLL_CTRL_SLV_DLY_TARGET_DEFAULT); priv->strobe_dll_delay_target = val; if (dev_read_bool(dev, "non-removable")) { diff --git a/include/fsl_esdhc.h b/include/fsl_esdhc.h index 8dbd5249a7..e693a6244e 100644 --- a/include/fsl_esdhc.h +++ b/include/fsl_esdhc.h @@ -20,27 +20,87 @@ #include "../board/freescale/common/qixis.h" #endif -/* FSL eSDHC-specific constants */ +/* + * eSDHC register definition + */ + +/* SDMA system address register */ +#define DSADDR 0x2e000 + +/* Block attributes register */ +#define BLKATTR 0x2e004 +#define BLKATTR_CNT(x) ((x & 0xffff) << 16) +#define BLKATTR_SIZE(x) (x & 0x1fff) +#define MAX_BLK_CNT 0x7fff /* so malloc will have enough room with 32M */ + +/* Command argument register */ +#define CMDARG 0x0002e008 + +/* Transfer type register */ +#define XFERTYP 0x0002e00c +#define XFERTYP_CMD(x) ((x & 0x3f) << 24) +#define XFERTYP_CMDTYP_NORMAL 0x0 +#define XFERTYP_CMDTYP_SUSPEND 0x00400000 +#define XFERTYP_CMDTYP_RESUME 0x00800000 +#define XFERTYP_CMDTYP_ABORT 0x00c00000 +#define XFERTYP_DPSEL 0x00200000 +#define XFERTYP_CICEN 0x00100000 +#define XFERTYP_CCCEN 0x00080000 +#define XFERTYP_RSPTYP_NONE 0 +#define XFERTYP_RSPTYP_136 0x00010000 +#define XFERTYP_RSPTYP_48 0x00020000 +#define XFERTYP_RSPTYP_48_BUSY 0x00030000 +#define XFERTYP_MSBSEL 0x00000020 +#define XFERTYP_DTDSEL 0x00000010 +#define XFERTYP_DDREN 0x00000008 +#define XFERTYP_AC12EN 0x00000004 +#define XFERTYP_BCEN 0x00000002 +#define XFERTYP_DMAEN 0x00000001 + +/* Command response 0/1/2/3 register */ +#define CMDRSP0 0x2e010 +#define CMDRSP1 0x2e014 +#define CMDRSP2 0x2e018 +#define CMDRSP3 0x2e01c + +/* Buffer data port register */ +#define DATPORT 0x2e020 + +/* Present state register */ +#define PRSSTAT 0x0002e024 +#define PRSSTAT_DAT0 (0x01000000) +#define PRSSTAT_CLSL (0x00800000) +#define PRSSTAT_WPSPL (0x00080000) +#define PRSSTAT_CDPL (0x00040000) +#define PRSSTAT_CINS (0x00010000) +#define PRSSTAT_BREN (0x00000800) +#define PRSSTAT_BWEN (0x00000400) +#define PRSSTAT_SDSTB (0X00000008) +#define PRSSTAT_DLA (0x00000004) +#define PRSSTAT_CICHB (0x00000002) +#define PRSSTAT_CIDHB (0x00000001) + +/* Protocol control register */ +#define PROCTL 0x0002e028 +#define PROCTL_INIT 0x00000020 +#define PROCTL_DTW_4 0x00000002 +#define PROCTL_DTW_8 0x00000004 +#define PROCTL_D3CD 0x00000008 + +/* Syetem control register */ #define SYSCTL 0x0002e02c #define SYSCTL_INITA 0x08000000 #define SYSCTL_TIMEOUT_MASK 0x000f0000 #define SYSCTL_CLOCK_MASK 0x0000fff0 -#if !defined(CONFIG_FSL_USDHC) #define SYSCTL_CKEN 0x00000008 #define SYSCTL_PEREN 0x00000004 #define SYSCTL_HCKEN 0x00000002 #define SYSCTL_IPGEN 0x00000001 -#endif #define SYSCTL_RSTA 0x01000000 #define SYSCTL_RSTC 0x02000000 #define SYSCTL_RSTD 0x04000000 -#define VENDORSPEC_CKEN 0x00004000 -#define VENDORSPEC_PEREN 0x00002000 -#define VENDORSPEC_HCKEN 0x00001000 -#define VENDORSPEC_IPGEN 0x00000800 -#define VENDORSPEC_INIT 0x20007809 - +/* Interrupt status register */ #define IRQSTAT 0x0002e030 #define IRQSTAT_DMAE (0x10000000) #define IRQSTAT_AC12E (0x01000000) @@ -63,9 +123,10 @@ #define CMD_ERR (IRQSTAT_CIE | IRQSTAT_CEBE | IRQSTAT_CCE) #define DATA_ERR (IRQSTAT_DEBE | IRQSTAT_DCE | IRQSTAT_DTOE | \ - IRQSTAT_DMAE) + IRQSTAT_DMAE) #define DATA_COMPLETE (IRQSTAT_TC | IRQSTAT_DINT) +/* Interrupt status enable register */ #define IRQSTATEN 0x0002e034 #define IRQSTATEN_DMAE (0x10000000) #define IRQSTATEN_AC12E (0x01000000) @@ -86,62 +147,15 @@ #define IRQSTATEN_TC (0x00000002) #define IRQSTATEN_CC (0x00000001) -#define ESDHCCTL 0x0002e40c -#define ESDHCCTL_PCS (0x00080000) - -#define PRSSTAT 0x0002e024 -#define PRSSTAT_DAT0 (0x01000000) -#define PRSSTAT_CLSL (0x00800000) -#define PRSSTAT_WPSPL (0x00080000) -#define PRSSTAT_CDPL (0x00040000) -#define PRSSTAT_CINS (0x00010000) -#define PRSSTAT_BREN (0x00000800) -#define PRSSTAT_BWEN (0x00000400) -#define PRSSTAT_SDSTB (0X00000008) -#define PRSSTAT_DLA (0x00000004) -#define PRSSTAT_CICHB (0x00000002) -#define PRSSTAT_CIDHB (0x00000001) - -#define PROCTL 0x0002e028 -#define PROCTL_INIT 0x00000020 -#define PROCTL_DTW_4 0x00000002 -#define PROCTL_DTW_8 0x00000004 -#define PROCTL_D3CD 0x00000008 - -#define CMDARG 0x0002e008 - -#define XFERTYP 0x0002e00c -#define XFERTYP_CMD(x) ((x & 0x3f) << 24) -#define XFERTYP_CMDTYP_NORMAL 0x0 -#define XFERTYP_CMDTYP_SUSPEND 0x00400000 -#define XFERTYP_CMDTYP_RESUME 0x00800000 -#define XFERTYP_CMDTYP_ABORT 0x00c00000 -#define XFERTYP_DPSEL 0x00200000 -#define XFERTYP_CICEN 0x00100000 -#define XFERTYP_CCCEN 0x00080000 -#define XFERTYP_RSPTYP_NONE 0 -#define XFERTYP_RSPTYP_136 0x00010000 -#define XFERTYP_RSPTYP_48 0x00020000 -#define XFERTYP_RSPTYP_48_BUSY 0x00030000 -#define XFERTYP_MSBSEL 0x00000020 -#define XFERTYP_DTDSEL 0x00000010 -#define XFERTYP_DDREN 0x00000008 -#define XFERTYP_AC12EN 0x00000004 -#define XFERTYP_BCEN 0x00000002 -#define XFERTYP_DMAEN 0x00000001 - -#define CINS_TIMEOUT 1000 -#define PIO_TIMEOUT 500 - -#define DSADDR 0x2e004 - -#define CMDRSP0 0x2e010 -#define CMDRSP1 0x2e014 -#define CMDRSP2 0x2e018 -#define CMDRSP3 0x2e01c - -#define DATPORT 0x2e020 +/* Host controller capabilities register */ +#define ESDHC_HOSTCAPBLT_VS18 0x04000000 +#define ESDHC_HOSTCAPBLT_VS30 0x02000000 +#define ESDHC_HOSTCAPBLT_VS33 0x01000000 +#define ESDHC_HOSTCAPBLT_SRS 0x00800000 +#define ESDHC_HOSTCAPBLT_DMAS 0x00400000 +#define ESDHC_HOSTCAPBLT_HSS 0x00200000 +/* Watermark level register */ #define WML 0x2e044 #define WML_WRITE 0x00010000 #ifdef CONFIG_FSL_SDHC_V2_3 @@ -160,21 +174,23 @@ #define WML_WR_WML_MASK 0xff0000 #endif -#define BLKATTR 0x2e004 -#define BLKATTR_CNT(x) ((x & 0xffff) << 16) -#define BLKATTR_SIZE(x) (x & 0x1fff) -#define MAX_BLK_CNT 0x7fff /* so malloc will have enough room with 32M */ +/* eSDHC control register */ +#define ESDHCCTL 0x0002e40c +#define ESDHCCTL_PCS (0x00080000) -#define ESDHC_HOSTCAPBLT_VS18 0x04000000 -#define ESDHC_HOSTCAPBLT_VS30 0x02000000 -#define ESDHC_HOSTCAPBLT_VS33 0x01000000 -#define ESDHC_HOSTCAPBLT_SRS 0x00800000 -#define ESDHC_HOSTCAPBLT_DMAS 0x00400000 -#define ESDHC_HOSTCAPBLT_HSS 0x00200000 +/* + * i.MX eSDHC specific register definition + */ -#define ESDHC_VENDORSPEC_VSELECT 0x00000002 /* Use 1.8V */ +/* Vendor specific register */ +#define VENDORSPEC_CKEN 0x00004000 +#define VENDORSPEC_PEREN 0x00002000 +#define VENDORSPEC_HCKEN 0x00001000 +#define VENDORSPEC_IPGEN 0x00000800 +#define VENDORSPEC_INIT 0x20007809 +#define VENDORSPEC_VSELECT 0x00000002 /* Use 1.8V */ -/* Imported from Linux Kernel drivers/mmc/host/sdhci-esdhc-imx.c */ +/* Mixer control register */ #define MIX_CTRL_DDREN BIT(3) #define MIX_CTRL_DTDSEL_READ BIT(4) #define MIX_CTRL_AC23EN BIT(7) @@ -189,24 +205,30 @@ /* Tuning bits */ #define MIX_CTRL_TUNING_MASK 0x03c00000 -/* strobe dll register */ -#define ESDHC_STROBE_DLL_CTRL 0x70 -#define ESDHC_STROBE_DLL_CTRL_ENABLE BIT(0) -#define ESDHC_STROBE_DLL_CTRL_RESET BIT(1) -#define ESDHC_STROBE_DLL_CTRL_SLV_DLY_TARGET_DEFAULT 0x7 -#define ESDHC_STROBE_DLL_CTRL_SLV_DLY_TARGET_SHIFT 3 +/* Strobe dll control register */ +#define STROBE_DLL_CTRL 0x70 +#define STROBE_DLL_CTRL_ENABLE BIT(0) +#define STROBE_DLL_CTRL_RESET BIT(1) +#define STROBE_DLL_CTRL_SLV_DLY_TARGET_DEFAULT 0x7 +#define STROBE_DLL_CTRL_SLV_DLY_TARGET_SHIFT 3 -#define ESDHC_STROBE_DLL_STATUS 0x74 -#define ESDHC_STROBE_DLL_STS_REF_LOCK BIT(1) -#define ESDHC_STROBE_DLL_STS_SLV_LOCK 0x1 -#define ESDHC_STROBE_DLL_CLK_FREQ 100000000 +/* Strobe dll status register */ +#define STROBE_DLL_STATUS 0x74 +#define STROBE_DLL_STS_REF_LOCK BIT(1) +#define STROBE_DLL_STS_SLV_LOCK 0x1 +#define STROBE_DLL_CLK_FREQ 100000000 -#define ESDHC_STD_TUNING_EN BIT(24) +/* Tuning control register */ +#define STD_TUNING_EN BIT(24) /* NOTE: the minimum valid tuning start tap for mx6sl is 1 */ -#define ESDHC_TUNING_START_TAP_DEFAULT 0x1 -#define ESDHC_TUNING_START_TAP_MASK 0xff -#define ESDHC_TUNING_STEP_MASK 0x00070000 -#define ESDHC_TUNING_STEP_SHIFT 16 +#define TUNING_START_TAP_DEFAULT 0x1 +#define TUNING_START_TAP_MASK 0xff +#define TUNING_STEP_MASK 0x00070000 +#define TUNING_STEP_SHIFT 16 + +/* + * Other definition + */ #define ESDHC_FLAG_MULTIBLK_NO_INT BIT(1) #define ESDHC_FLAG_ENGCM07207 BIT(2) @@ -220,6 +242,8 @@ #define ESDHC_FLAG_ERR010450 BIT(10) #define ESDHC_FLAG_HS400_ES BIT(11) +#define PIO_TIMEOUT 500 + struct fsl_esdhc_cfg { phys_addr_t esdhc_base; u32 sdhc_clk; From patchwork Thu Feb 14 08:21:15 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yangbo Lu X-Patchwork-Id: 1041899 X-Patchwork-Delegate: prabhakar@freescale.com Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=lists.denx.de (client-ip=81.169.180.215; helo=lists.denx.de; envelope-from=u-boot-bounces@lists.denx.de; receiver=) Authentication-Results: ozlabs.org; dmarc=fail (p=none dis=none) header.from=nxp.com Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=nxp.com header.i=@nxp.com header.b="jXYVCZk5"; dkim-atps=neutral Received: from lists.denx.de (dione.denx.de [81.169.180.215]) by ozlabs.org (Postfix) with ESMTP id 440TsJ1qvJz9sMp for ; Thu, 14 Feb 2019 19:22:28 +1100 (AEDT) Received: by lists.denx.de (Postfix, from userid 105) id C2899C220AE; Thu, 14 Feb 2019 08:21:57 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de X-Spam-Level: X-Spam-Status: No, score=0.0 required=5.0 tests=BAD_ENC_HEADER, SPF_HELO_PASS, T_DKIM_INVALID autolearn=unavailable autolearn_force=no version=3.4.0 Received: from lists.denx.de (localhost [IPv6:::1]) by lists.denx.de (Postfix) with ESMTP id 51511C220D4; Thu, 14 Feb 2019 08:21:26 +0000 (UTC) Received: by lists.denx.de (Postfix, from userid 105) id 1F481C220BE; Thu, 14 Feb 2019 08:21:21 +0000 (UTC) Received: from EUR03-DB5-obe.outbound.protection.outlook.com (mail-eopbgr40080.outbound.protection.outlook.com [40.107.4.80]) by lists.denx.de (Postfix) with ESMTPS id 112FCC220BD for ; Thu, 14 Feb 2019 08:21:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=eMkn48yLG50K/dqoTCpyxY8hjM2iqtOPdXSc983cSkg=; b=jXYVCZk533ceP/g0DPj9ZFj9A827xeFy3ag9ngAoZBXoB4WM5dKYo56qDDd5psbYDMwL5qpwRPDDIkviAU89cq0VlqGqT4iRtB4ZTRnBI5sWkHN4mxffzjIj1upKUJmIqlaiAfUEfX1xPs1Kn0dnhr4GVvGhkqzSb07YQIqOqjI= Received: from VI1PR0401MB2237.eurprd04.prod.outlook.com (10.169.133.18) by VI1PR0401MB1887.eurprd04.prod.outlook.com (10.166.140.137) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1622.16; Thu, 14 Feb 2019 08:21:15 +0000 Received: from VI1PR0401MB2237.eurprd04.prod.outlook.com ([fe80::78f9:21ec:83e4:bc72]) by VI1PR0401MB2237.eurprd04.prod.outlook.com ([fe80::78f9:21ec:83e4:bc72%3]) with mapi id 15.20.1601.023; Thu, 14 Feb 2019 08:21:15 +0000 From: "Y.b. Lu" To: "u-boot@lists.denx.de" Thread-Topic: [PATCH 3/3] mmc: fsl_esdhc: clarify i.MX eSDHC specific functions Thread-Index: AQHUxD5AXRvVDXAfYkimwtBHPxnloQ== Date: Thu, 14 Feb 2019 08:21:15 +0000 Message-ID: <20190214082207.12082-3-yangbo.lu@nxp.com> References: <20190214082207.12082-1-yangbo.lu@nxp.com> In-Reply-To: <20190214082207.12082-1-yangbo.lu@nxp.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-clientproxiedby: HK2PR02CA0185.apcprd02.prod.outlook.com (2603:1096:201:21::21) To VI1PR0401MB2237.eurprd04.prod.outlook.com (2603:10a6:800:28::18) authentication-results: spf=none (sender IP is ) smtp.mailfrom=yangbo.lu@nxp.com; x-ms-exchange-messagesentrepresentingtype: 1 x-mailer: git-send-email 2.17.1 x-originating-ip: [119.31.174.73] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 867e5db9-297c-4d04-af81-08d69255633b x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600110)(711020)(4605077)(4618075)(2017052603328)(7153060)(7193020); SRVR:VI1PR0401MB1887; x-ms-traffictypediagnostic: VI1PR0401MB1887: x-microsoft-exchange-diagnostics: =?iso-8859-1?Q?1; VI1PR0401MB1887; 23:zB6hOO/mx5AfdgpBH2v6ygktEU7iU3jbAEwRR?= =?iso-8859-1?q?v4Aw2GtbaIyuLXXAV2HpBhAIp?= =?iso-8859-1?q?sI4EN5PRe+noEo0AiGzE1ISSSacNcBrUe1/NwtqpcOJrljZFfb3?= =?iso-8859-1?q?IB8AfirrPDgqxN5KeeOC/zFNkfT3TEMkp/2Q7Bg0BG9d2FanNk+?= =?iso-8859-1?q?ULQaitRfncRnU6nmwRpnMqucOb5nVdv+cG75czwF8sTALGnN5+n?= =?iso-8859-1?q?u3wOCiWKGljVj2VZ52iousMf4RxZxGBiPTuGWZPDwgeg5Mky3ru?= =?iso-8859-1?q?PVkguuQMQrddge6lz/e515zSO8PK7AXdisZdbW9VZpyZreXKnEo?= =?iso-8859-1?q?R56k3k/4EKOhimmMA4/sdUTc9/zIURXkPop89Tmo7G8kxKJZmxn?= =?iso-8859-1?q?00MRU6zuCGsN/wEbr8Px6eyfPdAuJAU8PlNEHxbGIldJwd+0TBp?= =?iso-8859-1?q?K4+qDtYOeLyU+wco35g2uOrO4+1GCj1Ts7M4TS14uhvsWTMouJz?= =?iso-8859-1?q?flSOA/s5md4AYffK3VqCvtbymXFOQAvlfR26l9TaAoZ8kC25DLH?= =?iso-8859-1?q?CAZpAhMjBhk/blBQSCf29ZVDOweJkm29c6v3GPx7w92BySi0h+d?= =?iso-8859-1?q?RT1h4i3q4YaPA/HSd09sDGwyef0QZ1EqP7iupeS1HRO3bxEBgtw?= =?iso-8859-1?q?fqb7c8tujkfBY+kU04Pj4xulEHqfFxEFvncWhkxZS4yUpp4KYUn?= =?iso-8859-1?q?kmukPBHcjIr54DOr5QKCDGN186BA4heiz5LWxTaZTsdd38mPlOD?= =?iso-8859-1?q?+ELtGgmg8p7Q7VGJUrBuuqJq04gQUehEZspmyrujMFRSgcTY+gu?= =?iso-8859-1?q?JYvr1BVe99NHJRicvJ+Vp+KVK+hwnzAsgOn/MValu198Ltc/Eon?= =?iso-8859-1?q?kEwOoEmYZDL0uepjB6gidRUPszbnFAfYH9Pmm2L5phFLbMOPglu?= =?iso-8859-1?q?1KCMbjLFYCPcAVWJSAi9qwbmZeQ2r9QzzxpQTN1ZZhXMe7A1DBK?= =?iso-8859-1?q?p7Ve5MTKoHfrCO8dfdnrD12FzQRz+weUBI9esBDYSwKbjjwoqoU?= =?iso-8859-1?q?vpYAWZ0G5EnNwGegblRgx1R+2ONK2EVjCSBC4WjQzJd/ssa04xa?= =?iso-8859-1?q?iS2WmvuTKr5Jlf6iw3SwgQ46sJ1MidxUuZYhbmDa3PZ4EpTv9ut?= =?iso-8859-1?q?yX9I1y26QQKzPWYx8KQ0QGF9c2bl+2pEkP5+bsbFu03+9U3qvVt?= =?iso-8859-1?q?K/TLLi+Sp3z+gkwzu2h5uwrMWU2BwTFXj0gDThJzxlzDlSEquO1?= =?iso-8859-1?q?HXWiOFP1TvobWqT0boshXZWEPHWU4NSooSsxBLAPIFfba4wDAPp?= =?iso-8859-1?q?hZc2k4zNElRPbdrS+0N03cOk=3D?= x-microsoft-antispam-prvs: x-forefront-prvs: 09480768F8 x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(39860400002)(366004)(136003)(376002)(346002)(396003)(189003)(199004)(446003)(97736004)(1076003)(106356001)(3846002)(6116002)(81156014)(81166006)(11346002)(68736007)(86362001)(186003)(5640700003)(105586002)(99286004)(2351001)(486006)(50226002)(102836004)(6506007)(76176011)(386003)(66066001)(14454004)(71200400001)(71190400001)(478600001)(476003)(6916009)(7736002)(4326008)(8936002)(26005)(36756003)(316002)(54906003)(25786009)(256004)(6486002)(2616005)(53936002)(8676002)(2501003)(6512007)(2906002)(305945005)(52116002)(6436002); DIR:OUT; SFP:1101; SCL:1; SRVR:VI1PR0401MB1887; H:VI1PR0401MB2237.eurprd04.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1; received-spf: None (protection.outlook.com: nxp.com does not designate permitted sender hosts) x-ms-exchange-senderadcheck: 1 x-microsoft-antispam-message-info: sN7Rz53iu5VANH7KBvKaItzA7B+Zx4YZYfaWdacChwx4MyyceLgrWcBe8vsNAAMKe81xMR/6Q0a9zQiEY2OFdRIBQylHd7GnJTPDpp4tPKpUtQpxyEK+xYMWNeFwfxSzgwVZ+9twYhWxLIWq9Dvs3YfN4np4mvlvaLZEZq2j/ih/R6kPja7Es3Zy1+oHULjCENVvNXTls4H2kUD6CMdUwA8jYkI+t51ECrZ3+B76VDybO6ghYId7N+UUekr0sNQhyy+dE3smQBLuZuHlsXjsoUx/xN3Ymk6t39wI9t0PyZPc8C6m8wWaOovTfJDzHg8jyQeFwhWGdv1be7EwLsz/LOpcXxSyuoLC2+3eXHQ3DMcgAbqe6HJ4aTx5DXAdmg7DYAlAYJaIQ8++MxafNzYNQbjPFboIm4k8/aOBmwQk10A= MIME-Version: 1.0 X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 867e5db9-297c-4d04-af81-08d69255633b X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Feb 2019 08:21:13.1166 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0401MB1887 Subject: [U-Boot] [PATCH 3/3] mmc: fsl_esdhc: clarify i.MX eSDHC specific functions X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.18 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" A previous patch had added SDR104/HS200 support for fsl_esdhc. However this was only for i.MX eSDHC, and QorIQ eSDHC used different registers and method. This patch is to clarify i.MX eSDHC specific functions defined in that patch, and to use them only for i.MX eSDHC. The QorIQ eSDHC SDR104/HS200 support will be added in the future. Fixes: 51313b49f2b8 (mmc: fsl_esdhc: support SDR104 and HS200) Signed-off-by: Yangbo Lu Reviewed-by: Peng Fan --- drivers/mmc/fsl_esdhc.c | 79 +++++++++++++++++++++-------------------- 1 file changed, 41 insertions(+), 38 deletions(-) diff --git a/drivers/mmc/fsl_esdhc.c b/drivers/mmc/fsl_esdhc.c index 63f922382f..44c040eb8d 100644 --- a/drivers/mmc/fsl_esdhc.c +++ b/drivers/mmc/fsl_esdhc.c @@ -682,7 +682,7 @@ static void esdhc_clock_control(struct fsl_esdhc_priv *priv, bool enable) #endif #ifdef MMC_SUPPORTS_TUNING -static int esdhc_change_pinstate(struct udevice *dev) +static int esdhc_imx_change_pinstate(struct udevice *dev) { struct fsl_esdhc_priv *priv = dev_get_priv(dev); int ret; @@ -708,7 +708,7 @@ static int esdhc_change_pinstate(struct udevice *dev) return ret; } -static void esdhc_reset_tuning(struct mmc *mmc) +static void esdhc_imx_reset_tuning(struct mmc *mmc) { struct fsl_esdhc_priv *priv = dev_get_priv(mmc->dev); struct fsl_esdhc *regs = priv->esdhc_regs; @@ -722,7 +722,7 @@ static void esdhc_reset_tuning(struct mmc *mmc) } } -static void esdhc_set_strobe_dll(struct mmc *mmc) +static void esdhc_imx_set_strobe_dll(struct mmc *mmc) { struct fsl_esdhc_priv *priv = dev_get_priv(mmc->dev); struct fsl_esdhc *regs = priv->esdhc_regs; @@ -749,7 +749,7 @@ static void esdhc_set_strobe_dll(struct mmc *mmc) } } -static int esdhc_set_timing(struct mmc *mmc) +static int esdhc_imx_set_timing(struct mmc *mmc) { struct fsl_esdhc_priv *priv = dev_get_priv(mmc->dev); struct fsl_esdhc *regs = priv->esdhc_regs; @@ -761,13 +761,13 @@ static int esdhc_set_timing(struct mmc *mmc) switch (mmc->selected_mode) { case MMC_LEGACY: case SD_LEGACY: - esdhc_reset_tuning(mmc); + esdhc_imx_reset_tuning(mmc); writel(mixctrl, ®s->mixctrl); break; case MMC_HS_400: mixctrl |= MIX_CTRL_DDREN | MIX_CTRL_HS400_EN; writel(mixctrl, ®s->mixctrl); - esdhc_set_strobe_dll(mmc); + esdhc_imx_set_strobe_dll(mmc); break; case MMC_HS: case MMC_HS_52: @@ -791,14 +791,13 @@ static int esdhc_set_timing(struct mmc *mmc) priv->mode = mmc->selected_mode; - return esdhc_change_pinstate(mmc->dev); + return esdhc_imx_change_pinstate(mmc->dev); } -static int esdhc_set_voltage(struct mmc *mmc) +static int esdhc_imx_set_voltage(struct mmc *mmc) { struct fsl_esdhc_priv *priv = dev_get_priv(mmc->dev); struct fsl_esdhc *regs = priv->esdhc_regs; - int ret; priv->signal_voltage = mmc->signal_voltage; switch (mmc->signal_voltage) { @@ -975,18 +974,18 @@ static int esdhc_set_ios_common(struct fsl_esdhc_priv *priv, struct mmc *mmc) #endif } - if (priv->mode != mmc->selected_mode) { - ret = esdhc_set_timing(mmc); + if (priv->esdhc_imx && priv->mode != mmc->selected_mode) { + ret = esdhc_imx_set_timing(mmc); if (ret) { - printf("esdhc_set_timing error %d\n", ret); + printf("esdhc_imx_set_timing error %d\n", ret); return ret; } } - if (priv->signal_voltage != mmc->signal_voltage) { - ret = esdhc_set_voltage(mmc); + if (priv->esdhc_imx && priv->signal_voltage != mmc->signal_voltage) { + ret = esdhc_imx_set_voltage(mmc); if (ret) { - printf("esdhc_set_voltage error %d\n", ret); + printf("esdhc_imx_set_voltage error %d\n", ret); return ret; } } @@ -1267,20 +1266,21 @@ static int fsl_esdhc_init(struct fsl_esdhc_priv *priv, cfg->b_max = CONFIG_SYS_MMC_MAX_BLK_COUNT; - writel(0, ®s->dllctrl); - if (priv->flags & ESDHC_FLAG_USDHC) { - if (priv->flags & ESDHC_FLAG_STD_TUNING) { - u32 val = readl(®s->tuning_ctrl); - - val |= STD_TUNING_EN; - val &= ~TUNING_START_TAP_MASK; - val |= priv->tuning_start_tap; - val &= ~TUNING_STEP_MASK; - val |= (priv->tuning_step) << TUNING_STEP_SHIFT; - writel(val, ®s->tuning_ctrl); + if (priv->esdhc_imx) { + writel(0, ®s->dllctrl); + if (priv->flags & ESDHC_FLAG_USDHC) { + if (priv->flags & ESDHC_FLAG_STD_TUNING) { + u32 val = readl(®s->tuning_ctrl); + + val |= STD_TUNING_EN; + val &= ~TUNING_START_TAP_MASK; + val |= priv->tuning_start_tap; + val &= ~TUNING_STEP_MASK; + val |= priv->tuning_step << TUNING_STEP_SHIFT; + writel(val, ®s->tuning_ctrl); + } } } - return 0; } @@ -1481,14 +1481,17 @@ static int fsl_esdhc_probe(struct udevice *dev) else priv->bus_width = 1; - val = fdtdec_get_int(fdt, node, "fsl,tuning-step", 1); - priv->tuning_step = val; - val = fdtdec_get_int(fdt, node, "fsl,tuning-start-tap", - TUNING_START_TAP_DEFAULT); - priv->tuning_start_tap = val; - val = fdtdec_get_int(fdt, node, "fsl,strobe-dll-delay-target", - STROBE_DLL_CTRL_SLV_DLY_TARGET_DEFAULT); - priv->strobe_dll_delay_target = val; + if (priv->esdhc_imx) { + priv->tuning_step = + fdtdec_get_int(fdt, node, "fsl,tuning-step", 1); + priv->tuning_start_tap = + fdtdec_get_int(fdt, node, "fsl,tuning-start-tap", + TUNING_START_TAP_DEFAULT); + priv->strobe_dll_delay_target = + fdtdec_get_int(fdt, node, + "fsl,strobe-dll-delay-target", + STROBE_DLL_CTRL_SLV_DLY_TARGET_DEFAULT); + } if (dev_read_bool(dev, "non-removable")) { priv->non_removable = 1; @@ -1628,9 +1631,9 @@ static const struct dm_mmc_ops fsl_esdhc_ops = { #endif static struct esdhc_soc_data usdhc_imx7d_data = { - .flags = ESDHC_FLAG_USDHC | ESDHC_FLAG_STD_TUNING - | ESDHC_FLAG_HAVE_CAP1 | ESDHC_FLAG_HS200 - | ESDHC_FLAG_HS400, + .flags = ESDHC_FLAG_USDHC | ESDHC_FLAG_STD_TUNING | + ESDHC_FLAG_HAVE_CAP1 | ESDHC_FLAG_HS200 | + ESDHC_FLAG_HS400, .caps = UHS_CAPS | MMC_MODE_HS200 | MMC_MODE_DDR_52MHz | MMC_MODE_HS_52MHz | MMC_MODE_HS, };