From patchwork Mon Jul 19 14:52:02 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Greg Kroah-Hartman X-Patchwork-Id: 479969 Delivered-To: patch@linaro.org Received: by 2002:a02:c94a:0:0:0:0:0 with SMTP id u10csp4250565jao; Mon, 19 Jul 2021 08:48:38 -0700 (PDT) X-Google-Smtp-Source: ABdhPJy7/2MhCYF5VrDhlpEoCm8Y3cU8Pbmi9V3O6JNifskPJcroQUAnCCjZSpLtcelqm3PhJjrN X-Received: by 2002:a05:6402:3192:: with SMTP id di18mr36172922edb.186.1626709718113; Mon, 19 Jul 2021 08:48:38 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1626709718; cv=none; d=google.com; s=arc-20160816; b=tLBaptXdfAFVpifC9BYRR6nibRO6wmdRHaS0UXoaJqGNGxK+MIQXh7Nu8DvFTCG0aK h0vJScmgjTbeYitq2e6Zwk1PcgSIPepeK7P3MzpaDl1TUhCcZ/d2tWs5toVPr1JkdFro XU+nfmoosXagqxwkhDnL4x57dsGQgYuRxByCN5YufC9y/c4wr+fbf+83mux1XoMflbd7 bR5X6bBleyHK5eNMTjbjadH2RghTUNhRCM+LCFjLeSMfsKcLmA5BTjxrbt9bQwaEfSXh l1TMAgs8DWC4+fCVfEPyE/DA3+5Ks1986Itoqld8dnliNKOq39Vd2hfr4J6SJ3Q2lcJe erHw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :user-agent:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=+ccKkUlr4GQy7pUOUEufwW6btBYtSJBLQCXwX46zl38=; b=f4CTBqG2LnNnOD1HpAovLPfW7dS4C+t/rjvVEzHOFSnqMc1jATMRoqQz0dbJT4vWVD 2YJxpPjDtn8RkFUgosmQs4FahjSujQ6PMl9LT9FUIEZ6vmAtTk1xejHKr5MD5LXkmVaM jUQeYMnjTjyrCitGqfzfOJjuYrMbWHK0lw6oid7gjilP5DbDcU9Bxp2+9c1PBB8GxkUR rw2QirHjSnC9rrDdlYiEUk6nL30luqZ8uiySBCaFTtsYqprN1sh4BwfamneAaISPMcZh /LpuiZrCuD0cZHUpHZ0Za/+kbq0P2CPsgOzE4wihfU/MOxSb3Zty7fh4ThZi3WrPBFsm iWKA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=FUOaKhNo; spf=pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=stable-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id x22si21782145edd.466.2021.07.19.08.48.37; Mon, 19 Jul 2021 08:48:38 -0700 (PDT) Received-SPF: pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=FUOaKhNo; spf=pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=stable-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1345084AbhGSPHy (ORCPT + 12 others); Mon, 19 Jul 2021 11:07:54 -0400 Received: from mail.kernel.org ([198.145.29.99]:38494 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1346360AbhGSPFb (ORCPT ); Mon, 19 Jul 2021 11:05:31 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id C21AC601FD; Mon, 19 Jul 2021 15:46:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1626709570; bh=r8nFwJZzDXYJwpPPYt+7eTkcuqPC3tJF8xULDi0IxX0=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=FUOaKhNoirAfFhahTYtjzEa57zf6EOT/IhbEu6Mb+FsAJ4VF5euUeoIdrgMUN73Uq ZkXGi13Rlz0EmLhgm6alSiZyfD7cRneWJlxkCxNVVxFsz+k7b9gtZY057UL6tohHvS Bjvi9pz+51Hk/yB3QrrOPPWDE4filZmN22LHMgbk= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Jonathan Cameron , Rui Miguel Silva , Mauro Carvalho Chehab , Sasha Levin Subject: [PATCH 5.4 014/149] iio: gyro: fxa21002c: Balance runtime pm + use pm_runtime_resume_and_get(). Date: Mon, 19 Jul 2021 16:52:02 +0200 Message-Id: <20210719144904.796647817@linuxfoundation.org> X-Mailer: git-send-email 2.32.0 In-Reply-To: <20210719144901.370365147@linuxfoundation.org> References: <20210719144901.370365147@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org From: Jonathan Cameron [ Upstream commit 41120ebbb1eb5e9dec93320e259d5b2c93226073 ] In both the probe() error path and remove() pm_runtime_put_noidle() is called which will decrement the runtime pm reference count. However, there is no matching function to have raised the reference count. Not this isn't a fix as the runtime pm core will stop the reference count going negative anyway. An alternative would have been to raise the count in these paths, but it is not clear why that would be necessary. Whilst we are here replace some boilerplate with pm_runtime_resume_and_get() Found using coccicheck script under review at: https://lore.kernel.org/lkml/20210427141946.2478411-1-Julia.Lawall@inria.fr/ Signed-off-by: Jonathan Cameron Reviewed-by: Rui Miguel Silva Reviewed-by: Mauro Carvalho Chehab Link: https://lore.kernel.org/r/20210509113354.660190-2-jic23@kernel.org Signed-off-by: Sasha Levin --- drivers/iio/gyro/fxas21002c_core.c | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) -- 2.30.2 diff --git a/drivers/iio/gyro/fxas21002c_core.c b/drivers/iio/gyro/fxas21002c_core.c index 958cf8b6002c..45e2b5b33072 100644 --- a/drivers/iio/gyro/fxas21002c_core.c +++ b/drivers/iio/gyro/fxas21002c_core.c @@ -300,14 +300,7 @@ out_unlock: static int fxas21002c_pm_get(struct fxas21002c_data *data) { - struct device *dev = regmap_get_device(data->regmap); - int ret; - - ret = pm_runtime_get_sync(dev); - if (ret < 0) - pm_runtime_put_noidle(dev); - - return ret; + return pm_runtime_resume_and_get(regmap_get_device(data->regmap)); } static int fxas21002c_pm_put(struct fxas21002c_data *data) @@ -940,7 +933,6 @@ int fxas21002c_core_probe(struct device *dev, struct regmap *regmap, int irq, pm_disable: pm_runtime_disable(dev); pm_runtime_set_suspended(dev); - pm_runtime_put_noidle(dev); return ret; } @@ -954,7 +946,6 @@ void fxas21002c_core_remove(struct device *dev) pm_runtime_disable(dev); pm_runtime_set_suspended(dev); - pm_runtime_put_noidle(dev); } EXPORT_SYMBOL_GPL(fxas21002c_core_remove);