| Message ID | 1434554299-23443-10-git-send-email-shubhraj@xilinx.com |
|---|---|
| State | Accepted |
| Headers | show
Return-Path: <linux-i2c-owner@vger.kernel.org> X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 2F4D01401EF for <incoming@patchwork.ozlabs.org>; Thu, 18 Jun 2015 01:18:43 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932531AbbFQPSl (ORCPT <rfc822;incoming@patchwork.ozlabs.org>); Wed, 17 Jun 2015 11:18:41 -0400 Received: from mail-bn1bon0058.outbound.protection.outlook.com ([157.56.111.58]:4010 "EHLO na01-bn1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S932529AbbFQPSl (ORCPT <rfc822;linux-i2c@vger.kernel.org>); Wed, 17 Jun 2015 11:18:41 -0400 Received: from BY2FFO11FD032.protection.gbl (10.1.14.34) by BY2FFO11HUB002.protection.gbl (10.1.14.144) with Microsoft SMTP Server (TLS) id 15.1.190.9; Wed, 17 Jun 2015 15:18:39 +0000 Authentication-Results: spf=pass (sender IP is 149.199.60.83) smtp.mailfrom=xilinx.com; vger.kernel.org; dkim=none (message not signed) header.d=none; Received-SPF: Pass (protection.outlook.com: domain of xilinx.com designates 149.199.60.83 as permitted sender) receiver=protection.outlook.com; client-ip=149.199.60.83; helo=xsj-pvapsmtpgw01; Received: from xsj-pvapsmtpgw01 (149.199.60.83) by BY2FFO11FD032.mail.protection.outlook.com (10.1.14.210) with Microsoft SMTP Server (TLS) id 15.1.190.9 via Frontend Transport; Wed, 17 Jun 2015 15:18:38 +0000 Received: from unknown-38-66.xilinx.com ([149.199.38.66] helo=xsj-pvapsmtp01) by xsj-pvapsmtpgw01 with esmtp (Exim 4.63) (envelope-from <shubhrajyoti.datta@xilinx.com>) id 1Z5F6s-0005lV-KL for linux-i2c@vger.kernel.org; Wed, 17 Jun 2015 08:18:38 -0700 Received: from [127.0.0.1] (helo=localhost) by xsj-pvapsmtp01 with smtp (Exim 4.63) (envelope-from <shubhrajyoti.datta@xilinx.com>) id 1Z5F6s-0002Gf-Eq for linux-i2c@vger.kernel.org; Wed, 17 Jun 2015 08:18:38 -0700 Received: from xsj-pvapsmtp01 (mailman.xilinx.com [149.199.38.66]) by xsj-smtp-dlp2.xlnx.xilinx.com (8.13.8/8.13.1) with ESMTP id t5HFI6Ht022785; Wed, 17 Jun 2015 08:18:06 -0700 Received: from [172.23.146.171] (helo=xhdl3763.xilinx.com) by xsj-pvapsmtp01 with esmtp (Exim 4.63) (envelope-from <shubhraj@xilinx.com>) id 1Z5F6g-0002Dc-St; Wed, 17 Jun 2015 08:18:27 -0700 Received: by xhdl3763.xilinx.com (Postfix, from userid 9049) id 862202CE03CE; Wed, 17 Jun 2015 20:48:26 +0530 (IST) From: Shubhrajyoti Datta <shubhrajyoti.datta@xilinx.com> To: <linux-i2c@vger.kernel.org> CC: Shubhrajyoti Datta <shubhraj@xilinx.com> Subject: [PATCHv2 9/9] i2c: xiic: Do not continue in case of errors in Rx Date: Wed, 17 Jun 2015 20:48:19 +0530 Message-ID: <1434554299-23443-10-git-send-email-shubhraj@xilinx.com> X-Mailer: git-send-email 1.7.4 In-Reply-To: <1434554299-23443-1-git-send-email-shubhraj@xilinx.com> References: <1434554299-23443-1-git-send-email-shubhraj@xilinx.com> X-RCIS-Action: ALLOW X-TM-AS-Product-Ver: IMSS-7.1.0.1224-8.0.0.1202-21618.005 X-TM-AS-User-Approved-Sender: Yes;Yes X-EOPAttributedMessage: 0 X-Microsoft-Exchange-Diagnostics: 1; BY2FFO11FD032; 1:rqkLuehChQjzdX1AWG1baRHeN3dQQCgINkAs86ZIbDHdSTGazBztdrgHkWk+0od9LI2ooMmv2Svbm/UgMmpQACjYRy3Z+aMljVQ8ypEYIeLaa38NC4F4rJyDcGnGY5lb+3WNfalqookrvAakuKE+HgQdfaAMB5bAd4GikpA7GhrY2e89OYMdJJ3VGuPr4kqgiWdMLSwMnrHgBs90tSkGWT7pFinKfheemOA1A1K1UM48BPBp/t4guRiyAS4qwAYyPS1oSRw9ivnA4r0KhB+tMaAIBqw0jJ5LYZxArisdnRk= X-Forefront-Antispam-Report: CIP:149.199.60.83; CTRY:US; IPV:NLI; EFV:NLI; SFV:NSPM; SFS:(10009020)(6009001)(2980300002)(438002)(199003)(189002)(110136002)(107886002)(5001960100002)(189998001)(63266004)(92566002)(2950100001)(36756003)(103686003)(450100001)(77156002)(62966003)(36386004)(50226001)(81156007)(42186005)(46386002)(6806004)(19580405001)(19580395003)(48376002)(47776003)(33646002)(50466002)(50986999)(76176999)(229853001)(2351001)(106466001)(87936001)(46102003)(45336002)(86362001)(52956003)(83323001)(107986001)(90966001)(4001430100001); DIR:OUT; SFP:1101; SCL:1; SRVR:BY2FFO11HUB002; H:xsj-pvapsmtpgw01; FPR:; SPF:Pass; MLV:ovrnspm; A:1; MX:1; PTR:unknown-60-83.xilinx.com; LANG:en; MIME-Version: 1.0 Content-Type: text/plain X-Microsoft-Exchange-Diagnostics: 1; BY2FFO11HUB002; 2:IrC/ATLE8EgFuA4kFVpbfy9KeGiepj26+OjHhwjlVtioEoWk8R7wFzlByl+eQUqC; 2:FIZdcs72W74BLbrqS5fhND2RISzH8HtGNEPb2FhZETtxE8fdVvrFsqTprIAQ3Bhd6I2SbpbDvlIy976/em2/sVY2AbB0YkVOaA4I3/3x8ALQpaYe0tXlc5VyhdIMJIKjNwLEGRbhhV4Ss9fYqILnle06glG7ZswMkh3+V4g7QWrbofTXa9D45NcuFm11GqduasQoiNHY9n1l1IpgM+/Zc5Bl5Tn+cQI3Ap514eGR77U=; 6:2ZFVeHzMWzIijsYarsjZxbZhV3hpsSPjYT61Ccrtv7YrrLHHhd7Si/tAoqK0iS55zr+DvvEhGdW2JDLmDhXHgLSz+hREGVN3rCm93J5JpV1Hy//mfbF5ZdPnocflWlXzkiMVxJfvc4inr4ykFpIREOncXvmTnBWXhekBNWllHhDLzaqHTozs8MId3Ir1zNEp00+VyeObZMOpOZrQmCf+whFen1kTi6iOxL369df3csL72ZP76cUPSfA1rtATXMLh/yz5vu4Dzov5/uHqGb2Yxzy+mgkLspKkazXEpPJBvLWuH8nuXGqXC4ZzFJErcdQYPALaRuqQh/Yl7MR1tuGhSGHQ2DkuNQpSE0SIlu9uQxokclVvc/oEI7Rq8WoCnqPxXjzuRXJ7ah5Aw8asgGfW2g6BtBrrGFJerVb/TUIeo2DbvP2WbVbAGQT7MQEsIZItkDusLv7drWyj3DgjVIvwZjCiGcWHJC7CuuOKLHl2DGBOUfc41KL80ctGCiEGu+HZ X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BY2FFO11HUB002; X-Microsoft-Antispam-PRVS: <BY2FFO11HUB002202937C92B1338093F01EBA60@BY2FFO11HUB002.protection.gbl> X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(601004)(520003)(5005006)(3002001); SRVR:BY2FFO11HUB002; BCL:0; PCL:0; RULEID:; SRVR:BY2FFO11HUB002; X-Microsoft-Exchange-Diagnostics: 1; BY2FFO11HUB002; 3:RBvXF7LMtLu0S+JnMSD2mRU9CcjUs6ZYuqIDViZnQOBULJYYTkS6hRm5cqk78oJdpCdn9Y3m/aMYG8P+dNLbk0VaxC8xofPoG4aRFydpZccG3tpMQdKc+U3+UOq6Thf/B1D+NegJrSa00XnhdYhsDx5gFFwr2R7gQVX2pmj1979Epl4KWZHh+IB0qLmSk2DPxeYiEl/IM/RKGyY5fYHo/WKiXeXDyYyeFpxFH/l2W3qWVfd/4KHr2PiNOE54lZjH2bT1mHa3QX2RDGF3jqVZ2Aaq8Ukqwa1rQ1P19+HneHGC58NEhzFkiPs9+hMRHJ80 X-Forefront-PRVS: 0610D16BBE X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; BY2FFO11HUB002; 9:2o4AvclAt490rUT/dU2n3na6ioKVTHn/8aekfNJSo?= =?us-ascii?Q?2xU5VmYVQNaMv7wymYvRKc0Vzub414BtlqM73+pydarvGAuaYvcmcSMiRj4K?= =?us-ascii?Q?4TYQRzkspMXRDb2a3H7DQdQY/f7B7N9/BX0DTvAje9gmnDS5JptkjVpiBWnX?= =?us-ascii?Q?jmFBDqo7/C0ZyTlY2QL+9h+JYwbhUwIrL6zA3e3yt0rSefRebiiNyLXXYLBe?= =?us-ascii?Q?w9SKkVFSe1DiNwXI/6S2CsvLiIu7WdkcJ3AMUP4g8ULeYj63jHswxTV1hKS5?= =?us-ascii?Q?iDSZFZ5102IKE1mDzr2k8hf1jLBM7H3JYrUXvgzCn1qGLjozsPWxTIva1kyY?= =?us-ascii?Q?bRmp9rTOCIbAaSZb4zgzu2p1D9KFkFGQ3RCiKB/hJD93bduuCMIad6RpCObg?= =?us-ascii?Q?fo3H6HL6qbAIhWi0gsvPKQsxO0iABxfQGQPKmAxU4FQi6rGI7foE8plCK+7/?= =?us-ascii?Q?bEbW5+TtpJLXftfqLQQeztGAoExqwYFyQYMEoAT81VdMpqKQdXem0NMSBAdB?= =?us-ascii?Q?bHxwGJg06ukV7CDBErESyO8xMqijOZpNyO76J19MaEI+Rd3hKn/LfziPSXk6?= =?us-ascii?Q?/zzruz78EwAyAeOxjQAFvcokKeUv7PX9ZluaGRZgqeiVkEy93s2J75QSA10q?= =?us-ascii?Q?/3WEQsAz8imTqn6lNbeaau0YBjUaTLv/vUZKGxOO7bt4GYewTDrgI3PQ6BkT?= =?us-ascii?Q?XBqSKabHhx2Y9ZKw/a7MHtxCBO4bUvZ/OjULugEsggk9qrT8GBqU7wDEY+mc?= =?us-ascii?Q?gRHJq0qxghLw20b1tsfNvmAkeRh2xKvAY1SIKllbQswIrvXVKkLxGhEb8P+x?= =?us-ascii?Q?m/Ii8f3pmKC74PNhLPcDOibavjlvwhXi8qKlGIMj/jS0sVHqg2U9MO08tCtv?= =?us-ascii?Q?gq+gwu8/8uzdCvCQSsp0fmxbsM8VGYT0Q3zAoNKYSzJiRKBnUQAXrr57WXb5?= =?us-ascii?Q?b9QmhQk++oTAkVxzI4B8gZ5wHvCyBmCynBNiGin8/MtsQaecV4DmvcGvVlfN?= =?us-ascii?Q?70KMlPwn7iwrNi1N1htdV200oV72s8cXbCx1pp9qXfSmLtxGrgwnPC9WbugQ?= =?us-ascii?Q?V8ljYh+zbamexGleyy7t4cKRvCjEPZzvNxpDzVIZAe+YYgsDA=3D=3D?= X-Microsoft-Exchange-Diagnostics: 1; BY2FFO11HUB002; 3:q2mGQaXWobJFsRa9K/ytMV4dmUNzCpaZGaGM2BGc6os4dhnCSVNzAo2hF3H5EraNILX8mhMQa3mKmnzfSotEn0EXaJ8A8CGZWDgi9c7ucCMi9v3X1WH4jc8JN0JB5csyjQIcbZAOik5D1frJ/y/oHg==; 10:OVA5BeMbTmycjHvilEv3KkqdmRTzCYuJltYiuczdj9wlRWXS8Zv2SBJPLvOcgz5m+RyW1AU6xtrmnTpv7gj2Am32pyy6h66OiQsueuMycKM= X-OriginatorOrg: xilinx.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Jun 2015 15:18:38.9504 (UTC) X-MS-Exchange-CrossTenant-Id: 657af505-d5df-48d0-8300-c31994686c5c X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=657af505-d5df-48d0-8300-c31994686c5c; Ip=[149.199.60.83]; Helo=[xsj-pvapsmtpgw01] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY2FFO11HUB002 Sender: linux-i2c-owner@vger.kernel.org Precedence: bulk List-ID: <linux-i2c.vger.kernel.org> X-Mailing-List: linux-i2c@vger.kernel.org |
diff --git a/drivers/i2c/busses/i2c-xiic.c b/drivers/i2c/busses/i2c-xiic.c index 182ea68..c071897 100644 --- a/drivers/i2c/busses/i2c-xiic.c +++ b/drivers/i2c/busses/i2c-xiic.c @@ -399,6 +399,8 @@ static irqreturn_t xiic_process(int irq, void *dev_id) */ xiic_reinit(i2c); + if (i2c->rx_msg) + xiic_wakeup(i2c, STATE_ERROR); if (i2c->tx_msg) xiic_wakeup(i2c, STATE_ERROR); }
In case of error conditions like Arbitration lost or NACK lets signal the waiting process. Handle error cases in the Rx path Signed-off-by: Shubhrajyoti Datta <shubhraj@xilinx.com> --- drivers/i2c/busses/i2c-xiic.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-)