From patchwork Mon Mar 13 08:49:52 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Johan Hovold X-Patchwork-Id: 13172085 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0894DC74A5B for ; Mon, 13 Mar 2023 08:54:06 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230247AbjCMIyE (ORCPT ); Mon, 13 Mar 2023 04:54:04 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33370 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230230AbjCMIxi (ORCPT ); Mon, 13 Mar 2023 04:53:38 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6D77B4FF2D; Mon, 13 Mar 2023 01:51:04 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id D518561149; Mon, 13 Mar 2023 08:50:56 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 2BFB5C433EF; Mon, 13 Mar 2023 08:50:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1678697456; bh=UTxAK37bDA3EVcUJ2I3cJfHTWjh0McAXEajETjzOVq8=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=d2yww4OelgDzAjGUZXFWezscNGRWs+NxhWiIpHVNVTJqzgh1hChaDrJTlEHIfsvfW WxpiTngKuBp/IYlB5ACOY8Sz7/SjSOCQjL3Z+RMoZchNNKkepo6S2ghlug2zfJyzWG 6R8XoC8gI5RHt9gyKAKv8IukPPrI0jNwfLS5f9hNXWOFSji+vwzHHrr5iRahFOyME7 8tutOqENoqBo6Hn4spl53Qlae7nDXLKvmWQkbevu/mc29XcMrSPzwShdhoDDc6IEhK K65J2mMPGUoxxNv9EugoTqwFKyokhrFmU4QoCD75EQrMzeLXSkpKmn4cRCqRAm9U2U KmWqwRhP/e1Dg== Received: from johan by xi.lan with local (Exim 4.94.2) (envelope-from ) id 1pbduO-0006Hu-LH; Mon, 13 Mar 2023 09:51:56 +0100 From: Johan Hovold To: Georgi Djakov Cc: Bjorn Andersson , Andy Gross , Konrad Dybcio , Dmitry Baryshkov , Yassine Oudjana , Christophe JAILLET , linux-arm-msm@vger.kernel.org, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, Johan Hovold , stable@vger.kernel.org Subject: [PATCH 1/2] interconnect: qcom: rpm: fix msm8996 interconnect registration Date: Mon, 13 Mar 2023 09:49:52 +0100 Message-Id: <20230313084953.24088-2-johan+linaro@kernel.org> X-Mailer: git-send-email 2.39.2 In-Reply-To: <20230313084953.24088-1-johan+linaro@kernel.org> References: <20230313084953.24088-1-johan+linaro@kernel.org> MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org A recent commit broke interconnect provider registration for the msm8996 platform by accidentally removing a conditional when adding the missing clock disable in the power-domain lookup error path. Fixes: b6edcc7570b2 ("interconnect: qcom: rpm: fix probe PM domain error handling") Reported-by: Christophe JAILLET Link: https://lore.kernel.org/r/641d04a3-9236-fe76-a20f-11466a01460e@wanadoo.fr Cc: stable@vger.kernel.org # 5.17 Signed-off-by: Johan Hovold Reviewed-by: Konrad Dybcio --- drivers/interconnect/qcom/icc-rpm.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/interconnect/qcom/icc-rpm.c b/drivers/interconnect/qcom/icc-rpm.c index 4d0997b210f7..3b055cd893ea 100644 --- a/drivers/interconnect/qcom/icc-rpm.c +++ b/drivers/interconnect/qcom/icc-rpm.c @@ -498,7 +498,8 @@ int qnoc_probe(struct platform_device *pdev) if (desc->has_bus_pd) { ret = dev_pm_domain_attach(dev, true); - goto err_disable_clks; + if (ret) + goto err_disable_clks; } provider = &qp->provider; From patchwork Mon Mar 13 08:49:53 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Johan Hovold X-Patchwork-Id: 13172084 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 120E7C6FD19 for ; Mon, 13 Mar 2023 08:53:57 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229656AbjCMIxz (ORCPT ); Mon, 13 Mar 2023 04:53:55 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35088 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230371AbjCMIxa (ORCPT ); Mon, 13 Mar 2023 04:53:30 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0F01C2BEDB; Mon, 13 Mar 2023 01:50:58 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id DC9116116D; Mon, 13 Mar 2023 08:50:56 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 2F056C4339B; Mon, 13 Mar 2023 08:50:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1678697456; bh=aGmBJTLihmn6NlY3ZdQbfuZ8mhBDbOUV15yYyJq3Nq4=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=P3MiTsciJBrc4TsZhq68bs7oHbpEJ07tEltt2TZojiU+CULxKjs4hz9em8jFMQxRA bs+sRiiSz+1vtjs5mzQKvhjInbkj13HjJ0OH3JijgidZpmD1ewQjhORIkU/wufdhFz u2tuLa8IPfPufQMLhngCqVTuueEj9wL37U9jhwUNimDnlskgnRr5XwbiCNwjKBD0QK kH7ftGLDC7ajdOwzdBPn9GaLmNjXTsIz+45ubrg2rTt01UOG0sMdNnfTpFOdZ3NlBg mgL7ioJURhOutkqE788NZ0oL9oo0P/bKZmMFKcVvLbiuQSJB0CVbemxDfeX5nKJuHG lk51fCW5YSemg== Received: from johan by xi.lan with local (Exim 4.94.2) (envelope-from ) id 1pbduO-0006Hw-OJ; Mon, 13 Mar 2023 09:51:56 +0100 From: Johan Hovold To: Georgi Djakov Cc: Bjorn Andersson , Andy Gross , Konrad Dybcio , Dmitry Baryshkov , Yassine Oudjana , Christophe JAILLET , linux-arm-msm@vger.kernel.org, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, Johan Hovold Subject: [PATCH 2/2] interconnect: qcom: rpm: drop bogus pm domain attach Date: Mon, 13 Mar 2023 09:49:53 +0100 Message-Id: <20230313084953.24088-3-johan+linaro@kernel.org> X-Mailer: git-send-email 2.39.2 In-Reply-To: <20230313084953.24088-1-johan+linaro@kernel.org> References: <20230313084953.24088-1-johan+linaro@kernel.org> MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org Any power domain would already have been attached by the platform bus code so drop the bogus power domain attach which always succeeds from probe. This effectively reverts commit 7de109c0abe9 ("interconnect: icc-rpm: Add support for bus power domain"). Fixes: 7de109c0abe9 ("interconnect: icc-rpm: Add support for bus power domain") Cc: Yassine Oudjana Signed-off-by: Johan Hovold Tested-by: Konrad Dybcio # MSM8996 Sony Kagura Reviewed-by: Konrad Dybcio --- drivers/interconnect/qcom/icc-rpm.c | 6 ------ drivers/interconnect/qcom/icc-rpm.h | 1 - drivers/interconnect/qcom/msm8996.c | 1 - 3 files changed, 8 deletions(-) diff --git a/drivers/interconnect/qcom/icc-rpm.c b/drivers/interconnect/qcom/icc-rpm.c index 3b055cd893ea..c303ce22a7cd 100644 --- a/drivers/interconnect/qcom/icc-rpm.c +++ b/drivers/interconnect/qcom/icc-rpm.c @@ -496,12 +496,6 @@ int qnoc_probe(struct platform_device *pdev) if (ret) return ret; - if (desc->has_bus_pd) { - ret = dev_pm_domain_attach(dev, true); - if (ret) - goto err_disable_clks; - } - provider = &qp->provider; provider->dev = dev; provider->set = qcom_icc_set; diff --git a/drivers/interconnect/qcom/icc-rpm.h b/drivers/interconnect/qcom/icc-rpm.h index a49af844ab13..02257b0d3d5c 100644 --- a/drivers/interconnect/qcom/icc-rpm.h +++ b/drivers/interconnect/qcom/icc-rpm.h @@ -91,7 +91,6 @@ struct qcom_icc_desc { size_t num_nodes; const char * const *clocks; size_t num_clocks; - bool has_bus_pd; enum qcom_icc_type type; const struct regmap_config *regmap_cfg; unsigned int qos_offset; diff --git a/drivers/interconnect/qcom/msm8996.c b/drivers/interconnect/qcom/msm8996.c index 25a1a32bc611..14efd2761b7a 100644 --- a/drivers/interconnect/qcom/msm8996.c +++ b/drivers/interconnect/qcom/msm8996.c @@ -1823,7 +1823,6 @@ static const struct qcom_icc_desc msm8996_a0noc = { .num_nodes = ARRAY_SIZE(a0noc_nodes), .clocks = bus_a0noc_clocks, .num_clocks = ARRAY_SIZE(bus_a0noc_clocks), - .has_bus_pd = true, .regmap_cfg = &msm8996_a0noc_regmap_config };