From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qt1-f174.google.com (mail-qt1-f174.google.com [209.85.160.174]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3787312B68 for ; Fri, 29 Dec 2023 17:14:36 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="ka4ZNgsS" Received: by mail-qt1-f174.google.com with SMTP id d75a77b69052e-427f000385bso9153031cf.1 for ; Fri, 29 Dec 2023 09:14:35 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1703870075; x=1704474875; darn=vger.kernel.org; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to; bh=zh8fOp0kVPMuIegzy0Ht8ImQGG07PuOEnsybQwGt/AU=; b=ka4ZNgsS4Lrd67+0ixriXiwb3RkMQL1OPj/ZQ/aK0jOcjk41buCZCvEOfMmH1FcK8j 93kMVW24ac44Hg2xz3PhpiVKglRxw2UoenEUYtDZOOLJcupBVYeTcViCGS1DLUtGcQZh EC3iNBHiOyIj2Rin/3t2BUZJsjYLv2RUXW/2V1v+J3ClwQleEHxNDyOX8yr8LDa4UrBR WUcluO+B54Jb2x/p0ZXUNb20Jtd7E1938sYMUR0lPY4lt2SYLKV2cJkgCtF36pTOAEri hAekUNju8IH0Mo9bJudYdUJeu30fw0BjXUqp4J1Ib3NhpioNSuKVriO4STvjgUgsu4hc 5XTw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1703870075; x=1704474875; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=zh8fOp0kVPMuIegzy0Ht8ImQGG07PuOEnsybQwGt/AU=; b=doEe/+Arc1Yv1l9/Qkqq3ifhJBtHEnKGF000GYyeFGN5rtRr0SSgBThD2XHWWObWvk 8KyRxsefrNI13YAbyXklJPcYX5c9AqQ6X8nviFNuv1sfk++FsfEUYPV2hFtG9PAizRSt LFSaTED52+kR+17pGpfugyp9BzrrSMK4mkf+lfQLTNyCgzv2ipObJxq6hLfpSRIkDmui J+gS48Ab9WgBKQgCK/Kkwrfg3SAZcDm6fLmJBJkNAyX0WJ2FwHdEgVOUChnBkVZ1AjXT lll5WEL8Oh6Y4XEwn0VyMPTi/r+mOKP2Cao/4KltSZ0R0NDe5X96ZMj5loA9px+SOETB OKag== X-Gm-Message-State: AOJu0Yz3FdrwM3L2TCqXV46Sp9NUq6m8Rpd1X2YylnjJ0u5oEUBbOOj2 SAmqeZA9/BaLTl0wVLaqmk+7shkVUHJp X-Google-Smtp-Source: AGHT+IH/g21GtNan7HVM4yKKpcfcEVmgcHJK+d54J10TnWBAocH02Sn7SWXgF+RUrLC0tZ9luI8+yw== X-Received: by 2002:a05:622a:1b09:b0:428:9cb:e2d3 with SMTP id bb9-20020a05622a1b0900b0042809cbe2d3mr737431qtb.117.1703870075132; Fri, 29 Dec 2023 09:14:35 -0800 (PST) Received: from thinkpad ([117.207.24.172]) by smtp.gmail.com with ESMTPSA id w2-20020ac86b02000000b00427e7970d38sm3367243qts.51.2023.12.29.09.14.30 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 29 Dec 2023 09:14:34 -0800 (PST) Date: Fri, 29 Dec 2023 22:44:25 +0530 From: Manivannan Sadhasivam To: Konrad Dybcio Cc: Bjorn Andersson , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Johan Hovold , Marijn Suijten , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Krzysztof Kozlowski , Konrad Dybcio Subject: Re: [PATCH 3/3] arm64: dts: qcom: sc8280xp-crd: Add PCIe CLKREQ# sleep state Message-ID: <20231229171425.GC9098@thinkpad> References: <20231227-topic-8280_pcie_dts-v1-0-13d12b1698ff@linaro.org> <20231227-topic-8280_pcie_dts-v1-3-13d12b1698ff@linaro.org> Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20231227-topic-8280_pcie_dts-v1-3-13d12b1698ff@linaro.org> On Wed, Dec 27, 2023 at 11:28:28PM +0100, Konrad Dybcio wrote: > The CLKREQ pin should not be muxed to its active function when the RC > is asleep. Add the missing pin sleep states to resolve that. > I don't understand why it should not be muxed and wondering what is the actual issue you are seeing that lead to this conclusion. - Mani > Fixes: d907fe5acbf1 ("arm64: dts: qcom: sc8280xp-crd: enable WiFi controller") > Fixes: 17e2ccaf65d1 ("arm64: dts: qcom: sc8280xp-crd: enable SDX55 modem") > Fixes: 6a1ec5eca73c ("arm64: dts: qcom: sc8280xp-crd: enable NVMe SSD") > Signed-off-by: Konrad Dybcio > --- > arch/arm64/boot/dts/qcom/sc8280xp-crd.dts | 78 ++++++++++++++++++++----------- > 1 file changed, 51 insertions(+), 27 deletions(-) > > diff --git a/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts b/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts > index ffc4406422ae..58c0c2d10cb3 100644 > --- a/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts > +++ b/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts > @@ -530,8 +530,9 @@ &pcie2a { > > vddpe-3v3-supply = <&vreg_nvme>; > > - pinctrl-names = "default"; > - pinctrl-0 = <&pcie2a_default>; > + pinctrl-0 = <&pcie2a_default>, <&pcie2a_clkreq_default>; > + pinctrl-1 = <&pcie2a_default>, <&pcie2a_clkreq_sleep>; > + pinctrl-names = "default", "sleep"; > > status = "okay"; > }; > @@ -549,8 +550,9 @@ &pcie3a { > > vddpe-3v3-supply = <&vreg_wwan>; > > - pinctrl-names = "default"; > - pinctrl-0 = <&pcie3a_default>; > + pinctrl-0 = <&pcie3a_default>, <&pcie3a_clkreq_default>; > + pinctrl-1 = <&pcie3a_default>, <&pcie3a_clkreq_sleep>; > + pinctrl-names = "default", "sleep"; > > status = "okay"; > }; > @@ -568,8 +570,9 @@ &pcie4 { > > vddpe-3v3-supply = <&vreg_wlan>; > > - pinctrl-names = "default"; > - pinctrl-0 = <&pcie4_default>; > + pinctrl-0 = <&pcie4_default>, <&pcie4_clkreq_default>; > + pinctrl-1 = <&pcie4_default>, <&pcie4_clkreq_sleep>; > + pinctrl-names = "default", "sleep"; > > status = "okay"; > }; > @@ -835,13 +838,6 @@ nvme_reg_en: nvme-reg-en-state { > }; > > pcie2a_default: pcie2a-default-state { > - clkreq-n-pins { > - pins = "gpio142"; > - function = "pcie2a_clkreq"; > - drive-strength = <2>; > - bias-pull-up; > - }; > - > perst-n-pins { > pins = "gpio143"; > function = "gpio"; > @@ -857,14 +853,21 @@ wake-n-pins { > }; > }; > > - pcie3a_default: pcie3a-default-state { > - clkreq-n-pins { > - pins = "gpio150"; > - function = "pcie3a_clkreq"; > - drive-strength = <2>; > - bias-pull-up; > - }; > + pcie2a_clkreq_default: pcie2a-clkreq-default-state { > + pins = "gpio142"; > + function = "pcie2a_clkreq"; > + drive-strength = <2>; > + bias-pull-up; > + }; > + > + pcie2a_clkreq_sleep: pcie2a-clkreq-sleep-state { > + pins = "gpio142"; > + function = "gpio"; > + drive-strength = <2>; > + bias-pull-up; > + }; > > + pcie3a_default: pcie3a-default-state { > perst-n-pins { > pins = "gpio151"; > function = "gpio"; > @@ -880,14 +883,21 @@ wake-n-pins { > }; > }; > > - pcie4_default: pcie4-default-state { > - clkreq-n-pins { > - pins = "gpio140"; > - function = "pcie4_clkreq"; > - drive-strength = <2>; > - bias-pull-up; > - }; > + pcie3a_clkreq_default: pcie3a-clkreq-default-state { > + pins = "gpio150"; > + function = "pcie3a_clkreq"; > + drive-strength = <2>; > + bias-pull-up; > + }; > > + pcie3a_clkreq_sleep: pcie3a-clkreq-sleep-state { > + pins = "gpio150"; > + function = "gpio"; > + drive-strength = <2>; > + bias-pull-up; > + }; > + > + pcie4_default: pcie4-default-state { > perst-n-pins { > pins = "gpio141"; > function = "gpio"; > @@ -903,6 +913,20 @@ wake-n-pins { > }; > }; > > + pcie4_clkreq_default: pcie4-clkreq-default-state { > + pins = "gpio140"; > + function = "pcie4_clkreq"; > + drive-strength = <2>; > + bias-pull-up; > + }; > + > + pcie4_clkreq_sleep: pcie4-clkreq-sleep-state { > + pins = "gpio140"; > + function = "gpio"; > + drive-strength = <2>; > + bias-pull-up; > + }; > + > sdc2_default_state: sdc2-default-state { > clk-pins { > pins = "sdc2_clk"; > > -- > 2.43.0 > > -- மணிவண்ணன் சதாசிவம்