From patchwork Fri May 20 10:09:58 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yangbo Lu X-Patchwork-Id: 624442 X-Patchwork-Delegate: yorksun@freescale.com Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from theia.denx.de (theia.denx.de [85.214.87.163]) by ozlabs.org (Postfix) with ESMTP id 3rB3pk0h9Jz9s9Z for ; Fri, 20 May 2016 20:20:13 +1000 (AEST) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 5CC244BFB2; Fri, 20 May 2016 12:20:09 +0200 (CEST) Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id iZPPcB_pFZo7; Fri, 20 May 2016 12:20:09 +0200 (CEST) Received: from theia.denx.de (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id C2E5A4BE95; Fri, 20 May 2016 12:20:08 +0200 (CEST) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id F20944BE95 for ; Fri, 20 May 2016 12:20:03 +0200 (CEST) Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id I_GRGbzUG7Pg for ; Fri, 20 May 2016 12:20:03 +0200 (CEST) X-policyd-weight: NOT_IN_SBL_XBL_SPAMHAUS=-1.5 NOT_IN_SPAMCOP=-1.5 NOT_IN_BL_NJABL=-1.5 (only DNSBL check requested) Received: from na01-by2-obe.outbound.protection.outlook.com (mail-by2on0061.outbound.protection.outlook.com [207.46.100.61]) by theia.denx.de (Postfix) with ESMTPS id 3032F4B801 for ; Fri, 20 May 2016 12:19:59 +0200 (CEST) Received: from BY2PR03CA005.namprd03.prod.outlook.com (10.255.93.22) by BL2PR03MB465.namprd03.prod.outlook.com (10.141.92.14) with Microsoft SMTP Server (TLS) id 15.1.497.12; Fri, 20 May 2016 10:19:57 +0000 Received: from BN1BFFO11FD026.protection.gbl (10.255.93.4) by BY2PR03CA005.outlook.office365.com (10.255.93.22) with Microsoft SMTP Server (TLS) id 15.1.497.12 via Frontend Transport; Fri, 20 May 2016 10:19:56 +0000 Authentication-Results: spf=fail (sender IP is 192.88.168.50) smtp.mailfrom=nxp.com; nxp.com; dkim=none (message not signed) header.d=none;nxp.com; dmarc=none action=none header.from=nxp.com; Received-SPF: Fail (protection.outlook.com: domain of nxp.com does not designate 192.88.168.50 as permitted sender) receiver=protection.outlook.com; client-ip=192.88.168.50; helo=tx30smr01.am.freescale.net; Received: from tx30smr01.am.freescale.net (192.88.168.50) by BN1BFFO11FD026.mail.protection.outlook.com (10.58.144.89) with Microsoft SMTP Server (TLS) id 15.1.497.8 via Frontend Transport; Fri, 20 May 2016 10:19:56 +0000 Received: from titan.ap.freescale.net ([10.192.208.233]) by tx30smr01.am.freescale.net (8.14.3/8.14.0) with ESMTP id u4KAJqQo030375; Fri, 20 May 2016 03:19:54 -0700 From: Yangbo Lu To: Date: Fri, 20 May 2016 18:09:58 +0800 Message-ID: <1463739001-25905-1-git-send-email-yangbo.lu@nxp.com> X-Mailer: git-send-email 2.1.0.27.g96db324 X-EOPAttributedMessage: 0 X-Matching-Connectors: 131082131965368550; (91ab9b29-cfa4-454e-5278-08d120cd25b8); () X-Forefront-Antispam-Report: CIP:192.88.168.50; IPV:NLI; CTRY:US; EFV:NLI; SFV:NSPM; SFS:(10009020)(6009001)(2980300002)(1109001)(1110001)(339900001)(189002)(199003)(9170700003)(6806005)(105606002)(106466001)(2351001)(229853001)(1220700001)(586003)(85426001)(5008740100001)(50986999)(110136002)(86362001)(48376002)(50466002)(81166006)(8936002)(189998001)(87936001)(92566002)(11100500001)(19580405001)(5003940100001)(19580395003)(77096005)(33646002)(50226002)(8676002)(47776003)(4326007)(36756003)(2906002)(7059030)(15760500001); DIR:OUT; SFP:1101; SCL:1; SRVR:BL2PR03MB465; H:tx30smr01.am.freescale.net; FPR:; SPF:Fail; MLV:sfv; MX:1; A:1; LANG:en; X-Microsoft-Exchange-Diagnostics: 1; BN1BFFO11FD026; 1:aH/UfBVqiVGbuSV+O62nHYWEBXbVjDamHqf54zh8B16OF8qfqABATS7RG3MXP4yL6ZvZOGwwCgj0kpTrSDo/NsNNxLy8farQvMJyhajMOSls1kUy6p8BzFj7Wwu4opAX/luC2Jm7BhvPDbOlZ69DrAnfHXBfK6FtH3X5/fDa4ajD593gDroKz+hemGskkjIovz9WavEHu/WMoMQ6oZSOVaEAYWLUY42nq0B1/UtfqzVdf9nnySB17EkrbI1vlHxrS9tqJstABtumMmxKnDk2QqV+0TFz4qhcDlC0LtjHnrQeiWILqH53plVDxdzFo8KYRWTpnoFz/nuW4V7Gp8EDRo0ANexhCu7LhLtHNJ22sgcDySQ9+NrXEvq1CVAQXCz1oii5uSTxmwN4S+Fvh4UCUZ1VSkWmWlOOkP8NItzOzeoflshDTTG/FieivisxzyksZtYXUWWn3s7VlzEVzQbJvuxL4cPt6vJ611xZJzF7vexgtfB4+vXUlPZSHbs1PxABxKy4ouLJF8oBGVU6M4+ArDltLPRYkIlESYhUQ30xxHyNo0ZWkDVBbifzVnkbv7K72aUDpsFpYDM9TKJz3rJHdxNE5QyICdd2pU4hcpaoEmwl/xLI3OTd98Z+yrceq45vv4mFFNlUezcHtVyNbkJdH8M3TAbCc5YD33ph80hdHMk= MIME-Version: 1.0 X-MS-Office365-Filtering-Correlation-Id: a1b5b933-2ea7-4437-f785-08d380984aa6 X-Microsoft-Exchange-Diagnostics: 1; BL2PR03MB465; 2:wB5G0S9tiHeJ07Ua0N6lTC8PvN9cw9aHyAYZAKBJIsecxyouX5RpTyNoje+iSzzb8iZB2XhdUAlParFANO/G1T3ljaZmU3Vakd81lap5kea5ZNQYJjFdcetQFf5I1L/Wz803gVc/qDd3TSRZowIGU82tX7euGS5Z9bNwvCFgKTA1iAi1F8Ol08ooxOwpkf4b; 3:saHypgwAoCgLjZjldyqjkkycgJOJ/EtAm0KRA4+xhmjRIS+B8TqaePJ8dhhJpPo8rjx5CuNUjinZ/ALXqhsCMAIzWwFhkuDUj2DVSWCKa6eQKHKQCeh7wpXAHgoRysB8biXVs5hn19PMgh3Fqbf0ddip+slredjgnTt3raEZdukiq5tTS6zmVncXkhSYJVsWb46FqMRbF1YerosSwDT2GXaHMA/ChmLf9EiJjE79yPk=; 25:gyxvIgz3SiIufvkXjsQYOiFOGFMw838LD8mpk7zY4tFuBB+T4Q6DFW9GYpCdcXqDl/TxYKoYvps0FL0+26K7ZZFQaQ0Xc/xX6rSfi/GvHN/n849/+vEvxfJuDddvMIqF8jo9RtaF2PvW6/0j8FH3dENGVr0ZXX8ECXQa5Md0vgoyp6Ap/3kYg0ScJ/dM0NIwoWwuAfAv8hdecosB0oi5xKHVZgaYtuiFgZWMZTHA8R55luPFOOu6kmA29v37eIEj9HwNOLO+LzJJk8jXVMqpjvFY/li1062eNc74jBBuXxIRohXh9CXHBKyLlGIHs7fgIXVJ6o3ngoHThl3vaMoWdCobfRJi21fcca4p8D8PuE5LXSMolwAzBbxd7NFUYFPaOoYMIY1bGfqr9T2G8LpesueLW95q++iX1J1lUYy/wXA= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BL2PR03MB465; X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(601004)(2401047)(13023025)(13015025)(13024025)(5005006)(8121501046)(13017025)(13018025)(3002001)(10201501046)(6055026); SRVR:BL2PR03MB465; BCL:0; PCL:0; RULEID:(400006); SRVR:BL2PR03MB465; X-Microsoft-Exchange-Diagnostics: 1; BL2PR03MB465; 4:QjvHU02HTCICOmrqgaSt0QfqioGXE3vs4DCEmDi3w6xPPh21eoeEovCIlmeDZlF4HXBcKRVHxHowAPEyKhJoCz/QfvUd78+9pmIo7+2w+ZukeEKbAeZSeohUDX9UZOZGNpY9Em9O11bDu+FhRwdwZkoSWdM99jNwLcLZa8MgBXO5cwuUnPwdcbtf9qvtprip3RakFLcTxQWLOJ+T1baosEcbPBMwW+5tSYPPsiNTgr2BzNQstRGeSlWqyRfGPrhc0i2P9iANb/cRofB1OAZl8MGtRtTqMfcVO616ulH/wubYmpF4kRWNHP1q7FskZ7updOfAL/LzknfLkdIlu9qbSwLpBofRaFnHe9ahthw9voyoqYHcJgQkKBsd169QRU8ltechetHMUMp+THseh+PLQkuA3bDeCgwGlRrgj666lhPYUySNj2L9BzdkHyuuyj+ApHjnEVNuSIQ1fnRfLTK+r8HosB0jmLYBG9Xy5/W8tIs= X-Forefront-PRVS: 09480768F8 X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; BL2PR03MB465; 23:CjHJpxaJpmktDP4ccFSdipuWpb+xsBoVlf/JOV5+ZU?= =?us-ascii?Q?I9Nd6uoHtu9m0zXKzQNfvA4BOrdRpObGrGJgL2DtABWhRanObTn6Nq5y772+?= =?us-ascii?Q?P2yRx4/fKxi+T6QaRtERgbLgxXGgt2i99/A0Ph+pr/HZrWz9P5RpBA1a0IuK?= =?us-ascii?Q?GJazzra62cBzOvr/JOiyvXngVZW4gWt211GFlNiIcL7Fl8ZjXKkGz3qvt/rX?= =?us-ascii?Q?zmEjcShwkxNqPs/lAEBLfv2J19L8hoZmJA/aUwdNHgiaL74qdR+sqkzdhY9q?= =?us-ascii?Q?5cLi8S+xg7/IpKGdoJUrowvsbpD/q3Az9S0TXRNwpREh1GLicxnXvr39P1Hd?= =?us-ascii?Q?5ZBY8rVed05hzN4hb4s7xFZIbK2gptvJUwkLKmdWy/JU/iI++ruCorrKqZyg?= =?us-ascii?Q?J9DbYf4WV97kUIPfoH6BIrW+QP+1FoRGtkDM1C2UmNwSJS3wtJHXdmeI2MHi?= =?us-ascii?Q?t1NvhjsrjkcEVfBdxUe6zR3dNS9Z5igImZ2Hfsz7fLqn/z3ixPMKhebozNH9?= =?us-ascii?Q?JhMMMVkg/rTImzQ2I9tYkkPx8/GKFHupJDskvGCnoNEsUBX60DT3BgH+A/yh?= =?us-ascii?Q?VnmTmKDPh+7BEyg3fzRkdeL9+vPEo+XbiIQ6P8I4i0ZMp4xwc4tFOOaE56if?= =?us-ascii?Q?ZxdGdYY95rgTwHRMyxdnXLQ6MkIZhM6Uup5ahmDE4T32tj5CwTMF2ieo0SeL?= =?us-ascii?Q?9mTe3w4SZ5TEzRrAsZ2T8CkasTQz4NoYDWkFVp06fJuzzPY71rFCrz3u01+L?= =?us-ascii?Q?HyjNiLpHiaSE3b5HtQqVjQ2iNzph35l0NCcMdLOj3JMAMMSsbL4j5r1dmY03?= =?us-ascii?Q?gUlAEUQm2lJp6K/IfelT9Dqmuf2SyWzQvk43ZkgQJkAl+4X1PBEsK4NWzi0V?= =?us-ascii?Q?AtxRtlNbGzEJqXDLIw/46WSQk7JWdSRvI91pc13hmtW/IjQfu1iM3WQG7dGj?= =?us-ascii?Q?1NdZCU2bSYnZIEpa4o57D8VO6BinLBm5rianCj6lv/+Ozlt240lKtMIjgl7y?= =?us-ascii?Q?Jc/X87fMhDC+PsEzKTlKEXaBNtzJEasmLMt52bFrsn8rF1JS86Rm28mdEVlK?= =?us-ascii?Q?BEjFTgWoUtWUG2faUcJqk8yIme?= X-Microsoft-Exchange-Diagnostics: 1; BL2PR03MB465; 5:XHUceec1pIvIFQk0eH4UGyqtcDZL3g2a8wMPkFDkMVHygwzljNl7M9w4wQfc/i4iIPe05BIiY7OtsrhYzdN4f3afpNzoK2DKwFz4JH+mjPoWZqMTgBH17hf6RGzJO/JvH1APcL/VHlJ9BZY1M+aGDypJvD3Ks8QRiN3rDMwDuOY=; 24:kIhTIQNEddhVfx660H3zwxu6ClzR5tS2Xwde3CJGMsgbWmMv2bZFq6w/NHc6Psffk0ZAnwnqetT7UvtrgExxMwvWxG940JdJ17r5xutblHg=; 7:rqqganBSEc5cMCOAWYye2chMNwoTSuDJ033YpUj1ppMuyeAbCnvyCmz6Wu8KwrWC5YeDVzqVuoBLPSYSfkeEhofEhm1XSJigUrn/hjKFtq1HRbLaN4XnLJFwBMCK1p40tb+6BX99FBRXcmsJ7dpe8Zi2NWdn3mg9BJcui8UYNx7B5GAAJ+FeoMa0tkOGiusZ SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 May 2016 10:19:56.3340 (UTC) X-MS-Exchange-CrossTenant-Id: 5afe0b00-7697-4969-b663-5eab37d5f47e X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=5afe0b00-7697-4969-b663-5eab37d5f47e; Ip=[192.88.168.50]; Helo=[tx30smr01.am.freescale.net] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL2PR03MB465 Subject: [U-Boot] [PATCH 1/4] mmc: fsl_esdhc: don't set XFERTYP_RSPTYP_48_BUSY for CMD with busy response X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.15 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" For CMD with busy response, the eSDHC driver would poll DAT0 until CMD completion rather than polling IRQSTAT. So, don't set XFERTYP_RSPTYP_48_BUSY to avoid interrupts (DTOE or TC) in IRQSTAT. Signed-off-by: Yangbo Lu --- drivers/mmc/fsl_esdhc.c | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/fsl_esdhc.c b/drivers/mmc/fsl_esdhc.c index 3acf9e8..b06dd69 100644 --- a/drivers/mmc/fsl_esdhc.c +++ b/drivers/mmc/fsl_esdhc.c @@ -126,8 +126,16 @@ static uint esdhc_xfertyp(struct mmc_cmd *cmd, struct mmc_data *data) xfertyp |= XFERTYP_CICEN; if (cmd->resp_type & MMC_RSP_136) xfertyp |= XFERTYP_RSPTYP_136; - else if (cmd->resp_type & MMC_RSP_BUSY) - xfertyp |= XFERTYP_RSPTYP_48_BUSY; + /* + * For CMD with busy response, the eSDHC driver would poll DAT0 + * until CMD completion rather than polling IRQSTAT. So, don't + * set XFERTYP_RSPTYP_48_BUSY to avoid interrupts (DTOE or TC) + * in IRQSTAT. + * + * Remove: + * else if (cmd->resp_type & MMC_RSP_BUSY) + * xfertyp |= XFERTYP_RSPTYP_48_BUSY; + */ else if (cmd->resp_type & MMC_RSP_PRESENT) xfertyp |= XFERTYP_RSPTYP_48;