From patchwork Thu Oct 27 11:57:45 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ulf Hansson X-Patchwork-Id: 13021992 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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id C5B67ECAAA1 for ; Thu, 27 Oct 2022 11:59:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-Id:Date:Subject:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=rKq5+YHxZxJ0fg0kTQzy36yU8IBtwZCD97dVrLrcIpQ=; b=d3tlzQWAVnRC/O G/NTw1Mil4s1k5887rhXRqIbyxNTivi6rw/buidrowRgcORn9cl6DPv572lIprkoIMR6akUlZffMW 64ar1tJXFpFvN4KKeT2Khhej3gshBYGbpcZWLXn2rdAkXgyaFyixIn8V8Z87104MrzpvWRDsRoSvZ GappLdePy0AgcbWu5xIrPcxMkdHh1lwVI/3BtI7iKKp2r0CvqtDIfbINHyAKrLpVt76r2vQ7UTBB3 +KmWVeQZGAoBq4dXvdOvoPq56JsWJCxl09OhxwQdZMVyl1i29DL/zameCNB0SnJA5FhjH4fYCP4jS M/fWLtKK1lVFR0lJqJlA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oo1WF-00D6Qw-KM; Thu, 27 Oct 2022 11:57:55 +0000 Received: from mail-lj1-x235.google.com ([2a00:1450:4864:20::235]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oo1WC-00D6Pz-CA for linux-arm-kernel@lists.infradead.org; Thu, 27 Oct 2022 11:57:53 +0000 Received: by mail-lj1-x235.google.com with SMTP id s24so2041090ljs.11 for ; Thu, 27 Oct 2022 04:57:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=+haLpTJY+Yq4kbp0eK3uqAJ8H9p8YmI5+8lTBRSO0DU=; b=bijkMGDpdA8sgHlIOQ7XcM13fnEgWD1TwSsfE8gS2+pLPiPPkIaJpHqo2uUDOHcUDT SQeBsLBxgJoMoQH/iViYEVJ4JHW/NIOvD5ap6jPuFLzcRC3b2TNvWbxiFbUgW9oIBtp+ fCYQEa7piPcxHHe/z49q5Kz/g1C7LpGXrv2JCZboM/yz0wOKk6r4rz34OxaMqoSGfeTB xHAyn8+OirvuDeHPlMAViArRlcpuefrGkWajpoPHUkILhWMVtkx7bswh41qrq7lpRyW4 Xxp42e0kTnoDop/zizBZ4bA4mO95z6xPjLixrmaOkCC7zmGbzZK4YAykuqG5WD4pc4dD cqUg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=+haLpTJY+Yq4kbp0eK3uqAJ8H9p8YmI5+8lTBRSO0DU=; b=WnOeS63JQUYHsxC7w3jP5gUS/n2MF8Alg8VitJuUGCU470nhDgvTSipEEZYgdQkuBT Q584jG81Q8jWDx3iKDkEVg5GGyKWHDlMeAJvIoKEV78hOrOgElJB20jYvixZgtLUGvFS sd+ojRzsOttUOLzLVuEAToYT/ifScV+I5SAiyFOFu6ZF1DYaZRbNvwE3VUswzqmTBKbj Ik3iCtk2E/pJkrZxI5vpYMTQcd3mLiPwlyH1OzpbKAWUVcTzO2cnNmaAUEIiSKCHo9nS LddeYvH0qtM8cmo6C31fGeILQ/MEmfOvyuV8li9D+lV9t/ZOFoEvbmnAy8H7RHMQv76E axQA== X-Gm-Message-State: ACrzQf2/PJwzfNIoofWMRi25UyrKMOEGjiE8r1mdy2FxUJ1YmzLbG7b9 yPksWlWv+Qu+YSBR3ovvkHTFnQ== X-Google-Smtp-Source: AMsMyM48/wQq/Jkx50F3FMnAFPj14OpIn9u3fQj2o8IH2V7AQCvr7JRv/tlWtDMIFblkHzUaum92EA== X-Received: by 2002:a2e:a554:0:b0:26f:ea5d:a075 with SMTP id e20-20020a2ea554000000b0026fea5da075mr18884164ljn.322.1666871869632; Thu, 27 Oct 2022 04:57:49 -0700 (PDT) Received: from uffe-XPS13.. (h-94-254-63-18.NA.cust.bahnhof.se. [94.254.63.18]) by smtp.gmail.com with ESMTPSA id z13-20020a056512370d00b0049f53b65790sm156841lfr.228.2022.10.27.04.57.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 27 Oct 2022 04:57:48 -0700 (PDT) From: Ulf Hansson To: Andy Gross , Bjorn Andersson , Konrad Dybcio Cc: Dmitry Baryshkov , Maulik Shah , Rajendra Nayak , Sudeep Holla , Amit Pundir , Ulf Hansson , linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: [PATCH] arm64: dts: qcom: sm8250: Disable the not yet supported cluster idle state Date: Thu, 27 Oct 2022 13:57:45 +0200 Message-Id: <20221027115745.240516-1-ulf.hansson@linaro.org> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221027_045752_439982_2E58A22A X-CRM114-Status: GOOD ( 14.47 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org To support the deeper cluster idle state for sm8250 platforms, some additional synchronization is needed between the rpmh-rsc device and the CPU cluster PM domain. Until that is supported, let's disable the cluster idle state. This fixes a problem that has been reported for the Qcom RB5 platform (see below), but most likely other sm8250 platforms suffers from similar issues, so let's make the fix generic for sm8250. vreg_l11c_3p3: failed to enable: -ETIMEDOUT qcom-rpmh-regulator 18200000.rsc:pm8150l-rpmh-regulators: ldo11: devm_regulator_register() failed, ret=-110 qcom-rpmh-regulator: probe of 18200000.rsc:pm8150l-rpmh-regulators failed with error -110 Reported-by: Amit Pundir Fixes: 32bc936d7321 ("arm64: dts: qcom: sm8250: Add cpuidle states") Signed-off-by: Ulf Hansson Reviewed-by: Sudeep Holla Tested-by: Amit Pundir --- This problem was reported by Amit [1] together with an attempt to fix it. It turned, that we wanted a more generic fix, hence I posted $subject patch. Also note that, $subject patch is also depending (from functionality point of view) on a another for genpd [2]. Although, that should soon reach v6.1-rc[n] and stable kernels. Bjorn, can you pick this up as a fix for v6.1-rc and tag it for stable kernels? Kind regards Ulf Hansson [1] https://lore.kernel.org/lkml/20221018145348.4051809-1-amit.pundir@linaro.org/ [2] https://lore.kernel.org/lkml/CAJZ5v0hJxRiL03XDFpU8FuabsHn5i6JMksJ=dfj8B+Dm=s3LYA@mail.gmail.com/ --- arch/arm64/boot/dts/qcom/sm8250.dtsi | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm64/boot/dts/qcom/sm8250.dtsi b/arch/arm64/boot/dts/qcom/sm8250.dtsi index a5b62cadb129..e276eed1f8e2 100644 --- a/arch/arm64/boot/dts/qcom/sm8250.dtsi +++ b/arch/arm64/boot/dts/qcom/sm8250.dtsi @@ -334,6 +334,7 @@ CLUSTER_SLEEP_0: cluster-sleep-0 { exit-latency-us = <6562>; min-residency-us = <9987>; local-timer-stop; + status = "disabled"; }; }; };