From patchwork Fri Feb 15 02:25:51 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yangbo Lu X-Patchwork-Id: 1042574 X-Patchwork-Delegate: van.freenix@gmail.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="OL5Qe7Xu"; dkim-atps=neutral Received: from lists.denx.de (dione.denx.de [81.169.180.215]) by ozlabs.org (Postfix) with ESMTP id 440xwS3QSxz9sCh for ; Fri, 15 Feb 2019 13:26:48 +1100 (AEDT) Received: by lists.denx.de (Postfix, from userid 105) id 13973C220CB; Fri, 15 Feb 2019 02:26: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 41396C220B2; Fri, 15 Feb 2019 02:26:04 +0000 (UTC) Received: by lists.denx.de (Postfix, from userid 105) id 62CB8C220D8; Fri, 15 Feb 2019 02:25:56 +0000 (UTC) Received: from EUR04-DB3-obe.outbound.protection.outlook.com (mail-eopbgr60062.outbound.protection.outlook.com [40.107.6.62]) by lists.denx.de (Postfix) with ESMTPS id 0D06CC220EC for ; Fri, 15 Feb 2019 02:25:53 +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=/GmYeL7klzl/KHv8DkLIeWSOAQOTmZneHN8+rSU6uRQ=; b=OL5Qe7Xup5YmxJX33bIgMpI/0CwMJszXeRtX6NjENFPMfjUNWVeaRDVqC8okK7/prnUljh0J5m9Rlx0gdZyeBTD+9iFVaYNxH6ehjg4jvPPRsbA576yw6A9R/oMuj3TbgXL1EmNECPVQo60YuQ5AoTJIL7fSQcfPsk4Utdd/P18= Received: from VI1PR0401MB2237.eurprd04.prod.outlook.com (10.169.133.18) by VI1PR0401MB1984.eurprd04.prod.outlook.com (10.166.141.12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1622.16; Fri, 15 Feb 2019 02:25:51 +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; Fri, 15 Feb 2019 02:25:51 +0000 From: "Y.b. Lu" To: "u-boot@lists.denx.de" Thread-Topic: [v2, 3/3] mmc: fsl_esdhc: clarify i.MX eSDHC specific functions Thread-Index: AQHUxNXF0pctzXTKpUyVk7cyV9CDhQ== Date: Fri, 15 Feb 2019 02:25:51 +0000 Message-ID: <20190215022656.19310-3-yangbo.lu@nxp.com> References: <20190215022656.19310-1-yangbo.lu@nxp.com> In-Reply-To: <20190215022656.19310-1-yangbo.lu@nxp.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-clientproxiedby: HK2PR0401CA0008.apcprd04.prod.outlook.com (2603:1096:202:2::18) 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: 0d772c40-d35a-42d8-ad65-08d692ece78a 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:VI1PR0401MB1984; x-ms-traffictypediagnostic: VI1PR0401MB1984: x-microsoft-exchange-diagnostics: =?iso-8859-1?Q?1; VI1PR0401MB1984; 23:DEsFi0nhRyg+bIf2YWOFJ91iCr72iM8OYrYF/?= =?iso-8859-1?q?6cObb251FgDrsWrs1HqpSyc57?= =?iso-8859-1?q?iZDt57D0+PygIfYOIfHvidJQmIhGlg4Au+Aa+PpFGYuWNdbtCOF?= =?iso-8859-1?q?9niLp+SrCbD1EmjvbmxC9izaRZRkFqYuReLdMYNe8uo2KZM1B0U?= =?iso-8859-1?q?6V5LacpA37GBySfDmkVJ3wzZ7yJ9/rD9ALFL9bvR0mnjOlB3B2N?= =?iso-8859-1?q?kX03HCMaawNKA8VyU9GPwIYIokwsbZsJQ0zj0pMVeqEzm2kc042?= =?iso-8859-1?q?idrGzI/J+JnowGjai4cwQw+2N7aCexmi7y708j6yVoQyvahhBUE?= =?iso-8859-1?q?GAzNF1mToNeVchyxqinHOjODcpSDPmATtS/TLDsL1H+k2AytCam?= =?iso-8859-1?q?cBuIojAThVK7n4ttnX7Z88voTr0fmQDsZEYWBMRq17oAimcRLkj?= =?iso-8859-1?q?faD8Bkj77KonHfRfhA1J/BnYjaIToXyTCWNGQz41/W6PkUxYmDY?= =?iso-8859-1?q?nm+6LOtvIUxQSr4HKqEidge9vqdXti1iMfPiZ4fUVguhE31rQoy?= =?iso-8859-1?q?BGWI2FielPAcv849xerFW7I04lYLGsEp8uFFBdaYNTB3nobjus7?= =?iso-8859-1?q?syblepGB3+IFajA9Q0tAYKWjvyKoRGe+fDUykk+i0zYFQxRPWKD?= =?iso-8859-1?q?EzyxJbiRVBYgrJhIP0mfhVPV2/QiXLJWTwK1WgQ3VVwskRslb75?= =?iso-8859-1?q?tS/vBUSKvClgws7Nn4wnmDuZnkBODXwgJZF7PVlzq5N1AacfxVY?= =?iso-8859-1?q?7Cvv5LlaL6WsxjNNeGezAj7Bsr9aArfHt5mKtfqCQ1FTY6A6Tpk?= =?iso-8859-1?q?Z9fiN+teiiPrwTWuWkLcyLaOnU1H0DGBuHMpziB22Ro/YhVTl24?= =?iso-8859-1?q?C7exJJ7LRQJIW2Rino9AQJdM8R3z3j4LNRLalT4Ik1uI3pkAe2m?= =?iso-8859-1?q?iKWmnpiJKpGbipDSndKH1OkhWdsLjdPpGmVUbdTQiWfVRZRpEqy?= =?iso-8859-1?q?KV6UAks3Slzlc+XL8a16cgX/rFmrMG5pwGcY7fR18O/pzzQhET2?= =?iso-8859-1?q?FwmMioPnRx2PeiZkBODaxJCVRpFhYU91fK7LBIvVkHt0SjNs6sx?= =?iso-8859-1?q?f1mwuTx74lRUU1ldEiCWK7WrF1uIxNDfrful5bIfSE6vGklMjfa?= =?iso-8859-1?q?MKLukDxZIxm79y0a2S5pDwjAYFd+sVBB9MOqDXFcfaCL9AmOhbR?= =?iso-8859-1?q?Qu6+DMIA1ublYaSZVw1a6CiIFGoouXkaF9q2GdNAS6iatuFfAS5?= =?iso-8859-1?q?Rb0LUZ9/cQ77kLSYhoI1lZ+IidFW3GTUffmOZ3K1RlL1sFo6KKR?= =?iso-8859-1?q?vLHWmKialgWm4jNSIBKXLgik=3D?= x-microsoft-antispam-prvs: x-forefront-prvs: 09497C15EB x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(136003)(39860400002)(366004)(396003)(376002)(346002)(189003)(199004)(36756003)(446003)(11346002)(6116002)(102836004)(386003)(6506007)(3846002)(476003)(6916009)(66066001)(186003)(486006)(2616005)(86362001)(26005)(25786009)(71190400001)(53936002)(4326008)(71200400001)(2351001)(1076003)(106356001)(105586002)(99286004)(8936002)(8676002)(81156014)(81166006)(54906003)(50226002)(52116002)(76176011)(316002)(6486002)(305945005)(14454004)(6512007)(5640700003)(256004)(2906002)(2501003)(7736002)(68736007)(97736004)(478600001)(6436002); DIR:OUT; SFP:1101; SCL:1; SRVR:VI1PR0401MB1984; 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: iyBPIcovMVo6t8Nb06YZTjrauTT20zt+2HTyct0nzkfc8QhdWr9kTWvuIFqmiLAw0DVX/EmzUJjlciCEBApPInDlp6LSN85uSnw8cAzqFK15WEUW3C3jGBf/Auo9la4QM/5B7wrij3eQ5aP+9o/vwDxKsaAnAMp/Li2AdZF2Qmcwj38Vk1Js5vIIjYeR1fh981aEtfZXqGgAtTO39q2Xb1Kd3TgfL6aFBpO8y1t86p3tf/V0eHPooJYGpVvhhfw3fDx7/ioJiA/WnaOANaLlHb/kAMmqw8YxbO6T1ra3AaWBg8dyXWgty9DabI3/09CWcT2+5TCjseNskf18vphHWhvgwXvUdDE7Isv/c2mP3ZtrYlBZT1eQCjqCzDz47KXpL/NyYfvtF+5/W+D1fmKSXZNHfsJouZ0+pTrG/jZ8Yno= MIME-Version: 1.0 X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 0d772c40-d35a-42d8-ad65-08d692ece78a X-MS-Exchange-CrossTenant-originalarrivaltime: 15 Feb 2019 02:25:49.3959 (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: VI1PR0401MB1984 Subject: [U-Boot] [v2, 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 --- Changes for v2: - Added "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 e9622ce9a4..92924b63ac 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, };