Received: by 10.223.185.116 with SMTP id b49csp2239574wrg; Sat, 24 Feb 2018 14:44:02 -0800 (PST) X-Google-Smtp-Source: AH8x225jmuLIQGVrBG6SPt1l+HcqboSkIKfPNUicaasgDJKTht6adIs0cyEDpyd3g5zBrkk63Jgk X-Received: by 2002:a17:902:6e0f:: with SMTP id u15-v6mr5879694plk.78.1519512242261; Sat, 24 Feb 2018 14:44:02 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1519512242; cv=none; d=google.com; s=arc-20160816; b=oleTq80yLL7dtBjnVLc7yJLhqrJkBLSaXsSyBjna8E70YlH75odm1jQCM59x2y6+lH aHvLMgmYiayv0THlSMKBGn7WBuI3ljjdHcO4rS2kMQ7uVHaumGtk1bacqY4K0aPhowZq LDTP23r1q83Yi8Pn4BB1hvqu13dew97jdIyhRcEMxTmcLvaz0mXeibGJmGCcclIMnxQd nsUrFxssrwyd8COqHPXNE9GF4Wctqi1Kj/hy1McoDNvr2y1E/x/LeN20iTa4GuFF0Q7l ysdQ+poVF1odgIyypBlMZQUeTPkt2ykF/DHyYT+mYvZD8iO0pfWtG3ZoCzA7v8fx0HVm jWYA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:content-disposition :mime-version:message-id:subject:cc:to:from:date :arc-authentication-results; bh=yqRbx7T8reYBxtNXnmai+S35TedIpriuKInNQyecS4I=; b=RWYpCFQ0DC5c06NQBCCO390h1XCwPDJPnOUmMi0aQyg/eTFOZYibftrJGW95LVAswj hIifObNQC3B4WaMw13QHgBcxH6egSe610vWvC63VHxs2El7Ob2xaAOf+QU8g9h8jnBqR SaGNzNFS+b0gXb3WS022Mcx3HseiWa2Q+f6xu7wgejRU6JytAsDn3UQfqmUJcGx5KWIy r5ZRgU2U0oeELMnddZOVf/WmX8JmTiXnowlPR+/FXWZ54SXC8mBte5Wa3qswBXF/cpbe vYppf+NJ3Nv2K1IYkiVTm8+NbP08+wavjw0r1YSFs9NO+aj/CJLfHyqHmg23gM35crcz ee8w== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id bh5-v6si4100173plb.763.2018.02.24.14.43.36; Sat, 24 Feb 2018 14:44:02 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751508AbeBXWml (ORCPT + 99 others); Sat, 24 Feb 2018 17:42:41 -0500 Received: from agrajag.zerfleddert.de ([88.198.237.222]:33020 "EHLO agrajag.zerfleddert.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751449AbeBXWmk (ORCPT ); Sat, 24 Feb 2018 17:42:40 -0500 Received: by agrajag.zerfleddert.de (Postfix, from userid 1000) id 813325B20165; Sat, 24 Feb 2018 23:42:38 +0100 (CET) Date: Sat, 24 Feb 2018 23:42:38 +0100 From: Tobias Jordan To: linux-arm-kernel@lists.infradead.org, linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Michal Simek , Wolfram Sang Subject: [PATCH] i2c: cadence: Fix PM device usage count Message-ID: <20180224224238.nlr3mrifwngjzkrz@agrajag.zerfleddert.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org pm_runtime_get_sync() increases the device's usage count even when reporting an error, so add a call to pm_runtime_put_noidle() in the error branch. Fixes: 7fa32329ca03 ("i2c: cadence: Move to sensible power management") Signed-off-by: Tobias Jordan --- This is one of a number of patches for problems found using coccinelle scripting in the SIL2LinuxMP project. The patch has been compile-tested; it's based on linux-next-20180223. For a discussion of the corresponding issue, see https://marc.info/?l=linux-pm&m=151904483924999&w=2 drivers/i2c/busses/i2c-cadence.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/i2c/busses/i2c-cadence.c b/drivers/i2c/busses/i2c-cadence.c index b13605718291..898faa7fbac5 100644 --- a/drivers/i2c/busses/i2c-cadence.c +++ b/drivers/i2c/busses/i2c-cadence.c @@ -574,8 +574,10 @@ static int cdns_i2c_master_xfer(struct i2c_adapter *adap, struct i2c_msg *msgs, bool hold_quirk; ret = pm_runtime_get_sync(id->dev); - if (ret < 0) + if (ret < 0) { + pm_runtime_put_noidle(id->dev); return ret; + } /* Check if the bus is free */ if (cdns_i2c_readreg(CDNS_I2C_SR_OFFSET) & CDNS_I2C_SR_BA) { ret = -EAGAIN; -- 2.11.0