From patchwork Thu Jan 18 05:39:24 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: George Cherian X-Patchwork-Id: 862769 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=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=linux-i2c-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=CAVIUMNETWORKS.onmicrosoft.com header.i=@CAVIUMNETWORKS.onmicrosoft.com header.b="W/TPDTTh"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3zMXqM6jgJz9s72 for ; Thu, 18 Jan 2018 16:40:31 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754616AbeARFkV (ORCPT ); Thu, 18 Jan 2018 00:40:21 -0500 Received: from mail-by2nam03on0043.outbound.protection.outlook.com ([104.47.42.43]:25746 "EHLO NAM03-BY2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754498AbeARFkB (ORCPT ); Thu, 18 Jan 2018 00:40:01 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=Ev5pCY/iLca2lD06Z433lKIfy3uRYGRqfYB7x1G8As0=; b=W/TPDTThvZEkiCokxlcARfafG/HoBqlrf9zsR+eK/SFg3epC1qxGgMR4Knbwn/aP/uMRkg00EBnZyLtqjJzBZb56SPvh1onhyVAvhYA0ZCYGH+tU0hGGgJ3l/usTyXEqjdc5nREHCyOXtenigo7HA08npSHgaLQ+iokpuvwKkbA= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=George.Cherian@cavium.com; Received: from ubuntu.caveonetworks.com (111.93.218.67) by BN3PR0701MB1703.namprd07.prod.outlook.com (10.163.39.157) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.428.17; Thu, 18 Jan 2018 05:39:58 +0000 From: George Cherian To: linux-kernel@vger.kernel.org, linux-i2c@vger.kernel.org Cc: wsa@the-dreams.de, George Cherian Subject: [PATCH 4/4] i2c: xlp9xx: Check for Bus state after every transfer Date: Thu, 18 Jan 2018 05:39:24 +0000 Message-Id: <1516253964-4615-4-git-send-email-george.cherian@cavium.com> X-Mailer: git-send-email 2.1.4 In-Reply-To: <1516253964-4615-1-git-send-email-george.cherian@cavium.com> References: <1516253964-4615-1-git-send-email-george.cherian@cavium.com> MIME-Version: 1.0 X-Originating-IP: [111.93.218.67] X-ClientProxiedBy: BM1PR0101CA0047.INDPRD01.PROD.OUTLOOK.COM (10.174.221.33) To BN3PR0701MB1703.namprd07.prod.outlook.com (10.163.39.157) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 0b7968ad-d3b0-4c6c-5bf1-08d55e35ea32 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(7020095)(4652020)(5600026)(4604075)(4534125)(4602075)(4627221)(201703031133081)(201702281549075)(2017052603307)(7153060)(7193020); SRVR:BN3PR0701MB1703; X-Microsoft-Exchange-Diagnostics: 1; BN3PR0701MB1703; 3:vUpqL/u9jmQwI95j0mbztuLOaAy05gDxRlu6GMy0/XAvMaEML6cYDR4m0Dh7UKnBP/QdXHngX+W4lNuMzFiwrLbbFPJHN637Z5Z+4/EyXKVLnF06lxxMCBfc8ZDV8/VzWNR9Zo2soTqoNunUBYpchDRd3XEq5pc7uamXzyc+whb25tSd5YyCwjuNjL7FzZ4Xn3FcYQFzFDNTTyLXxgwQBmUWSu6hqSkAi+d8zsTWAsiCWAbtqxW0qRsDVGmeEEHc; 25:MTs2uXWl6kTomO7NEisMHpQDCkwbVxEWMQ/o4EcseCaTtiYdlV6miWJS4Ern5gN7tDtsTpmV9RRCydFg6d4pExsThoJEwsuQaCfjCqPv+rAtiMCXIlpSWYYZAuRa2v571YOYxVJ/jmbQ/y/fpcj8ecqVUVUCBUOi4u0TT6/HvySNRoC2vC3H/XRsbXzt3BqHEhFzvGfCPfRDQ/uTHNNbScK+Ijh2G5IVe62DGS0jtqGv7pVEq3CRaeOpvOr9+JLtaFpOpWWOgkOQKCUXMmt+mIe64xW/6xcsCx1bQqSdUi9b9OiJp8m/WsuILy6FmzzlVvVIYytLjfmhkn8SUw93og==; 31:7nhWlTd+C9TaCCMaKk26C9tgIgNgB0yMyWPE+X5nqp5U73g6A9If/cVeY8pdqh6A+qw8+Y02xXhNcg9oZIEUmUUah8/5Gb1bSf/6kLda3+jvetoT6+ZOsaNoxrNtOCcfdiQh51gXtVxMNIHSc16vVw+ca4F1CpKrLq9UEMhsnSoS+nBh/El6WoTkAeg3zeomIuV0hu97qnA9FdEP5zaZaNQ7hR1sw0mhwPRIrPCQKgw= X-MS-TrafficTypeDiagnostic: BN3PR0701MB1703: X-Microsoft-Exchange-Diagnostics: 1; BN3PR0701MB1703; 20:MRLlb1cG4Kz3zR8ckjHk5AgXVSyqEjrCRMHbjQ1m960JlRvcDzlD7gCiyphyzeFrczyy+HWjCbaeozSrWQpTO6KtJ6Hjn/+msLyfddzJaWFA8iWZ4ngkyr4M6x3MbMTNeLQR7ZvO7KAe1Ke6jBZvNXgLl8TzB4MXwVhZH+VHwjFHhzH8ckM03UaetwpMds0WjwyXB4y+Nn+nN+d1n43djWW1Xyer+e274hgl12oJ6IDWCxVHVnrR/QPMT1oXPkhmz1UXTciTYDoQ9NLreEBmGFvuGPkybVuG5yCJ0Je/ChStJHQg4xbZJ1q4rOBiXc+4MrmkcQLINpSET6zv4BoDWcVz811Ot5hpeMvV9C98ZjhDYeP0Z8nKlC6J5jM2w8HQsS5J9sN7UhxLogVD16hsjKFJ2fcupBQgoos6I4sLIZMFhVP5U7T/h+I+MOBXUHBvwpRyz87Mxfv6fabdCLo5eGBIhqihb9P/KEUXtC31VOCR8H1iiMAvAWHqKySX8lWD; 4:1pT/53qpk2kdxPbcwPYRZNFE09kL+1W5L6IfJBh0ZN+bX9BUpPmXIgNjM3x3+2JlFCZV49gl7dC+o7hSR9/y5ibOjEwC5E1SeEKVLFyXttItNnprjKJLaL/YpJIXP2LRPVXXPZxvGRKfgtFLhMp4GFJoZXQfvGOmj8k2SblJJgXj5h6qmmAgXkxjQwqsEhzUeuS0rkp5fVA1biOC+MI6UrjJ69aUiEz/9cjUHCA1hPLTqmN78pkXw8bfHM3U3KOS+ARYosCVImxfOfhjn8uGLw== X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(6040470)(2401047)(5005006)(8121501046)(93006095)(93001095)(3002001)(3231023)(2400056)(944501161)(10201501046)(6041268)(20161123558120)(20161123562045)(20161123560045)(20161123564045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(6072148)(201708071742011); SRVR:BN3PR0701MB1703; BCL:0; PCL:0; RULEID:(100000803101)(100110400095); SRVR:BN3PR0701MB1703; X-Forefront-PRVS: 05568D1FF7 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(366004)(39380400002)(396003)(346002)(376002)(39860400002)(199004)(189003)(107886003)(6512007)(316002)(48376002)(53936002)(97736004)(69596002)(25786009)(86362001)(26005)(7736002)(478600001)(16586007)(6486002)(72206003)(6666003)(2950100002)(106356001)(3846002)(53416004)(8676002)(81156014)(81166006)(105586002)(5660300001)(305945005)(36756003)(6116002)(51416003)(59450400001)(50226002)(6506007)(386003)(47776003)(52116002)(68736007)(50466002)(5009440100003)(66066001)(8936002)(4326008)(2906002)(16526018)(76176011)(551934003); DIR:OUT; SFP:1101; SCL:1; SRVR:BN3PR0701MB1703; H:ubuntu.caveonetworks.com; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en; Received-SPF: None (protection.outlook.com: cavium.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: 1; BN3PR0701MB1703; 23:C50AzawUqgXcwon3ordmaDHahJ/Z0btcA9bHOr4TAbtNGjOGD93LvB2meubbAAg6V3pq9zaIqNkAvaMQNL+WqE3xNK3+x+eCimzogkrnG9or5ARC3utbrury49hvSWiNJvg7wvXNmzNGTn3vNmKPpSV/jw1i/UyWsGbV7MVwwu+1KdB/45c1RCymS5LDCaXyiBb0Gi47H6E2jJ0XjDmRYgYaJDwrSOB9l/F6Yrm8TYsw7MICMlzFkxUGmMZMiqAqGZ1qCYNFIqdhH8xyxlnFjObXR4KEv2p4SnZD9MnCMT2tUEUSkX7KxUIJjx4OK0xRPHGzAT8mYpcbUp560A6MkLBArfyMBS42L4c1/Md5EnLLSJ5tfcKbMrDm8Q4cvVwoiRVdUdndgLzzz6pDHpQa9qFfv0Hh2IYrPFOPZAy5f3OAcOc7PmJHRPjPmim48wlhZai9kNFXHghWez5KfugLd6O3D8ZY8jz1xkRnOY1k0QsrSN4XBHCwyvy2t37nAX/FVO/IOFKS2fEyxRvvucczKQdfawTvz91nqe2J36he5wnrMbSP/fhvMWr4Q3YpCuLIssn5qtr1c1RPy+qSv+xQTmW/YCAM/4TjC+r17XceADqf+U7o0V1ztgLpIFayBP4OsnXm8QiDKqg+kWW7i4TR2uyWYOwtTdgXVAYAtZ7/156+o22T9XFTQjiVYcxU9Bc0a5T4K+z5Z+MNzm2EHXMNeAedN+wL63gUAMFt+viAbBfroBc3hrfFJ48bziD0f361Xsvuxb9DS5aFZxXmUiwpUyrT1po9XCN6lQ0Wwu6TGtN7rj0bl9XFVPi/Xqvcqiyu0v37Jkt1EKZTEeQvJcVKZnS59l0/AmOdcOCT9I73HdSB0JUVl5uogIOu1TdV6x4HashTYDb5UnhPjrzam85/dkXb+FZYh4kPEfNn9qF/38GhHs/eByTC0ADUCJnlIljCsk8/0GYcQAD3VcQC7JcTWh1dYod14KpOkUb0DKPKzWZhmdFenpRQUDAzWEi0LhlTXq2ryGiusE+dLX3Om1BJQUbCzXiV3quxe6TKsXZsIs5OzPJruE5AijKOdIZMu7raa1Z8bLf01zsY7DkE54PCdkqIUUlAS1N6sXqDtfMfRZFgtL+EQpUhmGD4b+ejiVXEYY1jXySgdgfC9omhgQpXLS09oI9NN0QOXmS3QKfI/iFLGg8dihUDt+Cs0BpfoG9tgAr7OCH+tAsHxCFPTKOANQ== X-Microsoft-Exchange-Diagnostics: 1; BN3PR0701MB1703; 6:f+ZktDUpk2g8vY5Bcs1PRFM9gO7PEhn/dbos0UX1AkUWrUHt3kmPOgKRYXogXTkaeZPERFYOgWav2q6pZTOiUIXs5VEDKAMaPNFOAdKUaVSj5EZq2LhfB93cl8zJqdQhlh5zqKpTs2GvEvvAh3CijERnCgxMF235ykBwZZQ77Gt2ZaZBcGqb/Om9lX4vBfguZJn6NMxD7++rMy06tn8dYkLHusIpob603sdRpT5SChzrgc+ZN34MvTn9rLoOluQu9zNwWvEXpVVL0pa359I/p95jsZV+mLusMaHy9YxVlEofZFkqw3XXaG8k9w9Xh4FJ8kM9XnaAg6UP/Bi0iQCTkZ2djh3HiXR2vJxdY7vqWE8=; 5:QA9if+oJZPE980uzo9+6dYuNeqQSFMyHm7cbQVFM8sEJIFRaCSlc/236C1Ca0RasgYUHrdJb2psKNHRIEate010L0HfshWxQK+UcZi/N8/F3f7iMx/jQMswmMt6OMujOs4dXfs0brfetq5a61dLvaKogVmVjI8tOAE31I+55suc=; 24:HBaiTwMrrwV95g7vm8+jgfqRjYv7QEj0/2tcNywJLGqzxC2B86X00G0ET7f/Q6nAhnOp9+NKcNH39GjxOpoJWZ9TU+FRKIvH3XfX/ru/KtY=; 7:RxEDK/O3nkVi3ybeOIcU75omRe9OxClbcm7A0t3mfqpjxrL889lRU7f6Hr5hhGHvrYKkzmpdsX3h4F9WhXekutyMEV9rTp3EzNno6bzV6d2uf4XD710i7jS2DC2wZN8Koth4uPB4U4F6uvXq/H9fYEBsyLXEYcWRsFDh87/Dm4lRbH2kD9XFWSQB+n/x/2GdkgfsVZ2LQDq4fb1WlczpDtHeg5EIcAX9F+e5bnE+oRSpB1qiBLXWqs6vC+wm5KLF SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: cavium.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Jan 2018 05:39:58.4645 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 0b7968ad-d3b0-4c6c-5bf1-08d55e35ea32 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN3PR0701MB1703 Sender: linux-i2c-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-i2c@vger.kernel.org I2C bus enters the STOP condition after the DATA_DONE interrupt is raised. Essentially the driver should be checking the bus state before sending the next transaction. In case the next transaction is initiated while the bus is busy, the prior transactions stop condition is not acheived. Add the check to make sure the bus is not busy before next transaction. Signed-off-by: George Cherian --- drivers/i2c/busses/i2c-xlp9xx.c | 20 +++++++++++++++++++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/drivers/i2c/busses/i2c-xlp9xx.c b/drivers/i2c/busses/i2c-xlp9xx.c index 1f6d780..fa9d5ee 100644 --- a/drivers/i2c/busses/i2c-xlp9xx.c +++ b/drivers/i2c/busses/i2c-xlp9xx.c @@ -16,6 +16,7 @@ #include #include #include +#include #define XLP9XX_I2C_DIV 0x0 #define XLP9XX_I2C_CTRL 0x1 @@ -36,6 +37,8 @@ #define XLP9XX_I2C_TIMEOUT 0X10 #define XLP9XX_I2C_GENCALLADDR 0x11 +#define XLP9XX_I2C_STATUS_BUSY BIT(0) + #define XLP9XX_I2C_CMD_START BIT(7) #define XLP9XX_I2C_CMD_STOP BIT(6) #define XLP9XX_I2C_CMD_READ BIT(5) @@ -71,6 +74,7 @@ #define XLP9XX_I2C_HIGH_FREQ 400000 #define XLP9XX_I2C_FIFO_SIZE 0x80U #define XLP9XX_I2C_TIMEOUT_MS 1000 +#define XLP9XX_I2C_BUSY_TIMEOUT 50 #define XLP9XX_I2C_FIFO_WCNT_MASK 0xff #define XLP9XX_I2C_STATUS_ERRMASK (XLP9XX_I2C_INTEN_ARLOST | \ @@ -264,7 +268,8 @@ static int xlp9xx_i2c_xfer_msg(struct xlp9xx_i2c_dev *priv, struct i2c_msg *msg, int last_msg) { unsigned long timeleft; - u32 intr_mask, cmd, val, len; + u32 intr_mask, cmd, val, len, status; + u32 busy_timeout = XLP9XX_I2C_BUSY_TIMEOUT; priv->msg_buf = msg->buf; priv->msg_buf_remaining = priv->msg_len = msg->len; @@ -351,6 +356,19 @@ static int xlp9xx_i2c_xfer_msg(struct xlp9xx_i2c_dev *priv, struct i2c_msg *msg, return -ETIMEDOUT; } + while (last_msg && busy_timeout) { + status = xlp9xx_read_i2c_reg(priv, XLP9XX_I2C_STATUS); + if ((status & XLP9XX_I2C_STATUS_BUSY) == 0) + break; + + busy_timeout--; + udelay(1); + } + + if (!busy_timeout) { + dev_dbg(priv->dev, "i2c bus busy for too long after transfer\n"); + return -EIO; + } /* update msg->len with actual received length */ if (msg->flags & I2C_M_RECV_LEN) msg->len = priv->msg_len;