From patchwork Thu Oct 10 10:30:52 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Viresh Kumar X-Patchwork-Id: 175742 Delivered-To: patch@linaro.org Received: by 2002:a92:7e96:0:0:0:0:0 with SMTP id q22csp2087419ill; Thu, 10 Oct 2019 03:31:06 -0700 (PDT) X-Google-Smtp-Source: APXvYqyhF6OtYOJmDFPYF/ijlKl4LKcjtWXtkSRu9gYSBGiWVGvOMf76maBLz9EtV58CysfemMlm X-Received: by 2002:a17:906:c4f:: with SMTP id t15mr7275787ejf.293.1570703466656; Thu, 10 Oct 2019 03:31:06 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1570703466; cv=none; d=google.com; s=arc-20160816; b=n+0RBg7fcx4S/j1XehYYeUDleCg1jp2Xky1iHcQeThMmfHrJaikme2k8D12kSnAWDh Ibz9YJBNo4HKmU+rhMlfpfWSy0Q/2qqTytIquTySco1NbkAopvmiPmwmpUVv0+klh7jr spDGfpuOKAK2ndjtLUUTG1FZlqManbThOVSLRl3ZEmAvLJeB5qK+ue/zC4yARJ+UGsMX wXgKm0TfUTeKVe9L/uZKvHqf/zV/JImOybTCw5LjxkRHRG19VtBfz7ASvkC2ARiBOk4F E6icfimnkJWBZTMtLyMzRdQtfc5cchFfCgCGDHrV7fHckRITchXStRZN/1OpYhNMDol8 t3tA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=HkxBDeI13PS/unHL1Nr+Sh+9tXf4QbZkcMWKpQOyoZs=; b=SZp9jYk1wkDtaqV4pgW1qOKMLZOvyaTcrRdMNIAzkoIRhd7LuwL/3zyazCOctMkjZ+ ksFWAdPzyC3+U8qjLudOxVHBWSBQjH6ccuyugUZYiuJCFIk2qJVfIPCSghFJZBc4DNUB lmBM06zfAxpyuUVFYOMKsW7E22/ROHwaewkikte4dHoeQGgjDjsooK2Ux6Gg6A/CzaWD DX0VsXEY79RSdF5SuP15FUv5iBWMu88bmBOHCHuyNoj1Iiq723hmhyAfCLhi1xmqi5b3 QcgTU0dxpOK4DckZyNUGMRvt/2zANw/QmmEoTXT+eL3t1MLp2diRcfxBGQ1znokRi/tG BYbg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=A3Ccp5m0; spf=pass (google.com: best guess record for domain of linux-pm-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-pm-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id sb7si2919644ejb.321.2019.10.10.03.31.06; Thu, 10 Oct 2019 03:31:06 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-pm-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=A3Ccp5m0; spf=pass (google.com: best guess record for domain of linux-pm-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-pm-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726523AbfJJKbC (ORCPT + 10 others); Thu, 10 Oct 2019 06:31:02 -0400 Received: from mail-pl1-f195.google.com ([209.85.214.195]:45441 "EHLO mail-pl1-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725601AbfJJKbC (ORCPT ); Thu, 10 Oct 2019 06:31:02 -0400 Received: by mail-pl1-f195.google.com with SMTP id u12so2554651pls.12 for ; Thu, 10 Oct 2019 03:31:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=HkxBDeI13PS/unHL1Nr+Sh+9tXf4QbZkcMWKpQOyoZs=; b=A3Ccp5m0GI0vf9vINUCaNgrXVyXgcdJaG3xHrIHMVAavr6PGkGIk2Rnv8iPDi14niy DeIrUZ5K3Q1T7ubCWBcWkWX8JmKwbimhiuLo0lm4549GDL1CKjT06RlAALVfrRDR5u7n FJ+AGVq+D276MDy6CMC86mTU+GOmxPFdikGggsEMOufzYdp5kj2OID/tuyMnSKV8Iikj c4hhdJW6rUhKvSG89tn4nMcBIbT54aJbpI5/7GZeJCjtchMX7zI0INRAYVWCOtTBxQob ZScd8nGDewPTjXDz4bzQ9r8eT/lyHZ7WcKtDsJiRlvjMmq8NMc2aTO+NmaEbxMel79G1 aycw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=HkxBDeI13PS/unHL1Nr+Sh+9tXf4QbZkcMWKpQOyoZs=; b=qWgAYrWmon6a1/060NrTSwrRnnqa7edJgaBmx3lToPVWJjB5ScVf1Q6DI6y+fPogcq WmRm3QWveZ5S9blWA2Z2RvqG2T9RlQhoVCVgMgFZH0ePYr89uaR76/hkvWYdX7rVQBfz WLKUF6eFRyxNkCJ+/VMDiuFx0kisCRn776yCFM9Dj9A8fHhyOoffMcLrAEz3EIJs1lNC tFCcMtXjGS/zLVJJTgCfLpWE8GwMWwA1PrSpVfoX+nCgCKbtu8fCazrJQZBSTLHWhyYe IwlSdtHxhwMnSO/bHVwC+RLxPW0uxe4muMhtflzFIGEMCdwmPVlN/iuDf09QdXG/Fs0K hWMg== X-Gm-Message-State: APjAAAUG5WmSNEKKllRoNLK9vZvBNYNdOLBzz3YMaShlyV4NpkK9hlcE C5KXvCI8mDqCmNPg24VgYijtbw== X-Received: by 2002:a17:902:9a01:: with SMTP id v1mr8899294plp.132.1570703461455; Thu, 10 Oct 2019 03:31:01 -0700 (PDT) Received: from localhost ([122.172.151.112]) by smtp.gmail.com with ESMTPSA id q13sm10056498pjq.0.2019.10.10.03.30.59 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 10 Oct 2019 03:31:00 -0700 (PDT) From: Viresh Kumar To: Viresh Kumar , Nishanth Menon , Stephen Boyd Cc: Viresh Kumar , linux-pm@vger.kernel.org, Vincent Guittot , Rafael Wysocki , "v5 . 0+" , Niklas Cassel , linux-kernel@vger.kernel.org Subject: [PATCH] opp: of: drop incorrect lockdep_assert_held() Date: Thu, 10 Oct 2019 16:00:52 +0530 Message-Id: <6306e18beab9deff6ee6b32f489390908495fe14.1570703431.git.viresh.kumar@linaro.org> X-Mailer: git-send-email 2.21.0.rc0.269.g1a574e7a288b MIME-Version: 1.0 Sender: linux-pm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org _find_opp_of_np() doesn't traverse the list of OPP tables but instead just the entries within an OPP table and so only requires to lock the OPP table itself. The lockdep_assert_held() was added there by mistake and isn't really required. Fixes: 5d6d106fa455 ("OPP: Populate required opp tables from "required-opps" property") Cc: v5.0+ # v5.0+ Reported-by: Niklas Cassel Signed-off-by: Viresh Kumar --- drivers/opp/of.c | 2 -- 1 file changed, 2 deletions(-) -- 2.21.0.rc0.269.g1a574e7a288b Tested-by: Niklas Cassel diff --git a/drivers/opp/of.c b/drivers/opp/of.c index 1813f5ad5fa2..6dc41faf74b5 100644 --- a/drivers/opp/of.c +++ b/drivers/opp/of.c @@ -77,8 +77,6 @@ static struct dev_pm_opp *_find_opp_of_np(struct opp_table *opp_table, { struct dev_pm_opp *opp; - lockdep_assert_held(&opp_table_lock); - mutex_lock(&opp_table->lock); list_for_each_entry(opp, &opp_table->opp_list, node) {