From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f171.google.com (mail-pl1-f171.google.com [209.85.214.171]) (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 DA8475FBA3 for ; Wed, 6 Mar 2024 09:45:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.171 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709718354; cv=none; b=DIDZlLjeU7arOUeh6HukfrbrlFmEfN4ZPpBjJA75EL0faeMK18I/mzMo0JiJldSpj6dDOLy4oD7gNpcYxLHK7/WXkqDEHxhG2GNlaA77BfiKEdt0ILSKoRZ5oIU4KW2ixXuSeTOzPmYwlwh0Ldz8v3EkAAibthoiC+BI9n7L3e8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709718354; c=relaxed/simple; bh=CMVF7n8dO5vESzb9IowrOlCjHiKoyt3gqfWtMoJTUa8=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=HerEtVjJpIIMqmOq7cjKv1p85+fo5rQTrSgE1OhN6cARQOhbLluhmt+opeytC4/cE6OJnl/OxKww9SLCgOzxkdL7zGUDm5VlwRJITSvsdCkGgk7VJm/Z8NBrd0GzuFDNg/aJ00hnb+Dk+8aRlu6Ptvm5BIL60GYa15Vjb/vsoXg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=aD2KGCDk; arc=none smtp.client-ip=209.85.214.171 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="aD2KGCDk" Received: by mail-pl1-f171.google.com with SMTP id d9443c01a7336-1dd41ed9b74so396565ad.2 for ; Wed, 06 Mar 2024 01:45:52 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1709718352; x=1710323152; 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=onNOJK5dVb7dHDdyyPmx8REkXT8q/MvLJnZknRgJ6mw=; b=aD2KGCDkwwTN4+n3M2IQxusmQv89GQOnCb2vyldhkkZMMNc9eA92HqO2FGHB7l2V6e zrqHPXcSoaW2Fvdw38zLRQroKE0wwStl+DnDAmff9FWnQFi5OB9anLTz7tmueXUrQbOw 7xfz2FsR3Ggnywtk2I5WvW5o0zV5vv1FBEkuRfkbRfH3Xg0uePSAkCZb22R323nRvOgf 1yiGU878gTKUhEmzx/jaDjmWuUbjW+z8xXpTMjHc995u+9hwSUNuyMj5+EuvJHNGY0Yf ggTnMUchI7Smx26RRydFqVh4wiF4QZGldJlDA/PJT/0rM7IFCDn89owgP6kPvefCalQj 6AhA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1709718352; x=1710323152; 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=onNOJK5dVb7dHDdyyPmx8REkXT8q/MvLJnZknRgJ6mw=; b=TWflBIX/bD4N9WWvJCSaXv2r+h+paoB6tPyoS/omBaXDk81p271o1o5Y1N5kM1eubX yTn36buGHAAxrbSd1h6/MWmHzBjE5SQj41k0UK95loxBV8MwUHQmMxJ/aJU5aSaaKToJ RL2ynDXNztV7MA6GJxqpd3GP5z3PCrrMLu42eh880xqR/Od67mH4Y+cFKW3UatFuCEHM 7+13wpKoGA3B2wePSFx26AC7BtvJ3wzCcwVbaAXhXvhAMVw7Tq+ABICCHtoc9qqAubVF K9pgrAwJZL6UHU2d9cY1p6kcyG+yFzqDtGTpOcakBI9dap8nIXG0q/mLYrtYPcoJUqQC MuaQ== X-Forwarded-Encrypted: i=1; AJvYcCV1UMt1wZSo/96gNMdz6wezv5QERJBK1pdpYMrtf+NWXa/HR5ZGLKwuJxSCxkWJob8IDOPHcy7cHCMQgjyMsH7pF5C/tzn8FpzJ0A== X-Gm-Message-State: AOJu0YyEfG9p3r/YUZ8BerSl2GjRkD4c4wEnbAe6cpR4hSHfV2k85Qh/ PdkVcCNkqvmdXpu3vTwdqXXK2MO2utQNUTe+wJRCbpAwAQrnuLMaQMwgZSvziA== X-Google-Smtp-Source: AGHT+IF5LhL5XIw+P5F6Ztyfbld27LtakrR/VOK8wMDEg3ocAgkdsTAQ2fMWpd9fCfHvlu9oM0q57A== X-Received: by 2002:a17:903:24c:b0:1dc:b531:833 with SMTP id j12-20020a170903024c00b001dcb5310833mr4833692plh.63.1709718352039; Wed, 06 Mar 2024 01:45:52 -0800 (PST) Received: from thinkpad ([117.248.1.194]) by smtp.gmail.com with ESMTPSA id c1-20020a170903234100b001db594c9d17sm12099477plh.254.2024.03.06.01.45.47 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 06 Mar 2024 01:45:51 -0800 (PST) Date: Wed, 6 Mar 2024 15:15:45 +0530 From: Manivannan Sadhasivam To: Dmitry Baryshkov Cc: Johan Hovold , Johan Hovold , Bjorn Helgaas , Bjorn Andersson , Konrad Dybcio , Lorenzo Pieralisi , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Rob Herring , Krzysztof Kozlowski , Conor Dooley , linux-arm-msm@vger.kernel.org, linux-pci@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 00/10] arm64: dts: qcom: sc8280xp: PCIe fixes and GICv3 ITS enable Message-ID: <20240306094545.GD4129@thinkpad> References: <20240305081105.11912-1-johan+linaro@kernel.org> <20240306063302.GA4129@thinkpad> <20240306083925.GB4129@thinkpad> Precedence: bulk X-Mailing-List: devicetree@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: On Wed, Mar 06, 2024 at 10:48:30AM +0200, Dmitry Baryshkov wrote: > On Wed, 6 Mar 2024 at 10:39, Manivannan Sadhasivam > wrote: > > > > On Wed, Mar 06, 2024 at 08:20:16AM +0100, Johan Hovold wrote: > > > On Wed, Mar 06, 2024 at 12:03:02PM +0530, Manivannan Sadhasivam wrote: > > > > On Tue, Mar 05, 2024 at 09:10:55AM +0100, Johan Hovold wrote: > > > > > This series addresses a few problems with the sc8280xp PCIe > > > > > implementation. > > > > > > > > > > The DWC PCIe controller can either use its internal MSI controller or an > > > > > external one such as the GICv3 ITS. Enabling the latter allows for > > > > > assigning affinity to individual interrupts, but results in a large > > > > > amount of Correctable Errors being logged on both the Lenovo ThinkPad > > > > > X13s and the sc8280xp-crd reference design. > > > > > > > > > > It turns out that these errors are always generated, but for some yet to > > > > > be determined reason, the AER interrupts are never received when using > > > > > the internal MSI controller, which makes the link errors harder to > > > > > notice. > > > > > > > > Enabling AER error reporting on sc8280xp could similarly also reveal > > > > > existing problems with the related sa8295p and sa8540p platforms as they > > > > > share the base dtsi. > > > > > > > > > > After discussing this with Bjorn Andersson at Qualcomm we have decided > > > > > to go ahead and disable L0s for all controllers on the CRD and the > > > > > X13s. > > > > > > > Just received confirmation from Qcom that L0s is not supported for any of the > > > > PCIe instances in sc8280xp (and its derivatives). Please move the property to > > > > SoC dtsi. > > > > > > Ok, thanks for confirming. But then the devicetree property is not the > > > right way to handle this, and we should disable L0s based on the > > > compatible string instead. > > > > > > > Hmm. I checked further and got the info that there is no change in the IP, but > > the PHY sequence is not tuned correctly for L0s (as I suspected earlier). So > > there will be AERs when L0s is enabled on any controller instance. And there > > will be no updated PHY sequence in the future also for this chipset. > > Why? If it is a bug in the PHY driver, it should be fixed there > instead of adding workarounds. > Fixing the L0s support requires the expertise of the PHY team and they will only do if there is any real demand (like in the case of mobile chipsets). For compute chipsets, they didn't do because most of the NVMe devices out there in the market only support L1 and L1ss. So we have to live with this limitation for now. - Mani > > > > So yeah, let's disable it in the driver instead. > > > > > > > As we are now at 6.8-rc7, I've rebased this series on the Qualcomm PCIe > > > > > binding rework in linux-next so that the whole series can be merged for > > > > > 6.9 (the 'aspm-no-l0s' support and devicetree fixes are all marked for > > > > > stable backport anyway). > > > > > > I'll respin the series. Looks like we've already missed the chance to > > > enable ITS in 6.9 anyway. > > > > > > > Sounds good, thanks! > > > > - Mani > > > > -- > > மணிவண்ணன் சதாசிவம் > > > > > -- > With best wishes > Dmitry -- மணிவண்ணன் சதாசிவம்