From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f173.google.com (mail-pl1-f173.google.com [209.85.214.173]) (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 09B4B209689 for ; Mon, 2 Dec 2024 12:59:01 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.173 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733144344; cv=none; b=MIvZtnaypuUqlA5vFblx3/C6BrZWTNAjx03p8te1Hw7sGZHH4tC7zPoVr7e0DN8FuP1ZTXprbZ8dRMNutM9SKcBLUWT2sCjMRSKAJS/yNKz40Qnhi2mj8TXYvJMTGyleB6FhpRIAT6T8/tdAl5VAmJcSTXrM0VOBqo0kE1OHpMU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733144344; c=relaxed/simple; bh=84F50RDLSxwvaiL/Ut5/RJLgaAhNDidEJM6Aelx6NOw=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=SoGsfGzqlnznxaJKpVSY/nVzYmF+IBarEVf58nrFrjIMYpJTE4SP1tZGcAFdQkjfIgEthCkc5UpM5D0H90ybcPCKcIAvMZid2q/P0qpblL+hcfWTNjTRLzfP1clekI1f4w+GEMTZeBr+1X5AyFPzb7Mbw75udGesBE+Ix03LYew= 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=DKyK0hww; arc=none smtp.client-ip=209.85.214.173 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="DKyK0hww" Received: by mail-pl1-f173.google.com with SMTP id d9443c01a7336-21583cf5748so9462265ad.1 for ; Mon, 02 Dec 2024 04:59:01 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1733144341; x=1733749141; 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=6CCvtQVXQrLvAOM+4VRXFfpLmG3wrstufKddBuVhAJQ=; b=DKyK0hwwQNncIv2W+kktnDsm+uwGi61NVsPGfzI7wRLdfQ3Ka06Bz5srnrjKpCnLN5 2vBDGzgB4LEll0nb7jnCWNlmaUxAEPbSBgTDoGA7A316P+8f9B3GuYx+WfJ2oH8bi51z JUdDipAhSc/o+zOrFYarR7G3a/EkeY3zFAyIG9pHwDE2SoefVa107fA1iheDoY6FOGM6 p6ek52PQAGAnJySRRmfaBdQQQktJV4gtRdlcIUmqY6JG5BS1+1B/DwtoXBJtsTiSJW6V HDDYwwwFxkTVUFck4r8P4zxAW5DJDOqJ/xH6D/g33C0zG8Q4VXCOv0iRf8Gy7G3Folys yJow== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1733144341; x=1733749141; 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=6CCvtQVXQrLvAOM+4VRXFfpLmG3wrstufKddBuVhAJQ=; b=THtKJlFKjaASR/nQNIH6+R2yRlJTxJ/nlpc03sU2Gn3BoceDJ/XGi1K8lIdQznF5aC bf93KEeVravd5j6ZF2CHE4fwvBOTqi9Pbu3BjfYrbiZBAnSmU7QNBfnv7FeY8HEFjqz8 J95xq3+c8F59yva75lFIUMCFMbCw5WEDKk2QHeL21yWKOrf9SrOx9Xj0ZnvoRAYUdblt O71/qXwqOFgSZoWVNR1xQbwLtd5n7JGXyOKQw+HN2Gvyml051D7hZmuQK30zHkMBuhrd yHnJD2yhLK0yTmgE1It9oKasOZlq2/MIdVQmzncEQrm6yOiLGenBI3EUxpLhTZDBnvYD JUjw== X-Forwarded-Encrypted: i=1; AJvYcCVVhs6HG0BnEHJlfUzqgVQ6DvMcBNHCQ75+/ijQEhS12x219h84gh8etqSk17/arjwM2D5UH4jMv6ykI7wS@vger.kernel.org X-Gm-Message-State: AOJu0YwjT5IkUFv4G/tTooS/DiswFXf5COtnOAAKHQhKaDnzLd9gKysg VMthWaL/7UvgSd66uFt6+8GDZv+OgpAAXHpJ552vXQAh0NHh5hGbSLSZ+U1OcQ== X-Gm-Gg: ASbGncuSg3hppe9FKuPeYdp9wAzoEvVvDKcn9L3E9c5/ur/1W1TSKniikzsqfoautil cHLGpiCsnEMVR4P9kABN3KKtFTBi7ntSiFICiFy3QI8U6y80gb9Xf7B4r2FnZ1BSDpKR7tRvn0i /Jyc7OGrHCd5W55G6iAgGcZRTlY+IQ2GODbW6XJDmaSRMBji3E0tgKB0lbwrNLG750YmnsRYgmx 4yP3BAyZ/+JHfsXc3C5b3N2Cf497UYo0OuC75aV6NtKiGbtnr+kv7vjECIgKw== X-Google-Smtp-Source: AGHT+IGb8lymZBMlKvEv3D6XKmMObg2pfnQBEi8/oeAOIIzR3QMSbRXUP7cxeVvRLLDRh2EmtlIy1g== X-Received: by 2002:a17:902:e5c5:b0:215:6cb2:787e with SMTP id d9443c01a7336-2156cb27b96mr105778365ad.9.1733144341390; Mon, 02 Dec 2024 04:59:01 -0800 (PST) Received: from thinkpad ([120.60.140.110]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2156c99a827sm34935945ad.166.2024.12.02.04.58.56 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 02 Dec 2024 04:59:00 -0800 (PST) Date: Mon, 2 Dec 2024 18:28:45 +0530 From: Manivannan Sadhasivam To: Bjorn Helgaas Cc: kw@linux.com, gregkh@linuxfoundation.org, arnd@arndb.de, lpieralisi@kernel.org, shuah@kernel.org, kishon@kernel.org, aman1.gupta@samsung.com, p.rajanbabu@samsung.com, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, bhelgaas@google.com, linux-arm-msm@vger.kernel.org, robh@kernel.org, linux-kselftest@vger.kernel.org, stable+noautosel@kernel.org Subject: Re: [PATCH v2 1/4] PCI: qcom-ep: Mark BAR0/BAR2 as 64bit BARs and BAR1/BAR3 as RESERVED Message-ID: <20241202125845.rp4vc7ape52v4bwd@thinkpad> References: <20241129092415.29437-2-manivannan.sadhasivam@linaro.org> <20241129195537.GA2770926@bhelgaas> 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: <20241129195537.GA2770926@bhelgaas> On Fri, Nov 29, 2024 at 01:55:37PM -0600, Bjorn Helgaas wrote: > On Fri, Nov 29, 2024 at 02:54:12PM +0530, Manivannan Sadhasivam wrote: > > On all Qcom endpoint SoCs, BAR0/BAR2 are 64bit BARs by default and software > > cannot change the type. So mark the those BARs as 64bit BARs and also mark > > the successive BAR1/BAR3 as RESERVED BARs so that the EPF drivers cannot > > use them. > > "Default" implies an initial setting that can be changed, but you say > "by default" and also "software cannot change the type." Can they be > anything *other* than 64-bit BARs? > > If they're hardwired to be 64-bit BARs, I would just say that. > > > Cc: stable+noautosel@kernel.org # depends on patch introducing only_64bit flag > > If stable maintainers need to act on this, do they need to search for > the patch introducing only_64bit flag? That seems onerous; is there a > SHA1 that would make it easier? > But that's not the point of having noautosel tag, AFAIK. Documentation/process/stable-kernel-rules.rst clearly says that this tag is to be used when we do not want the stable team to backport the commit due to a missing dependency. If we really want stable team to backport the change with dependencies, then the dependencies should be mentioned using the SHAs: >From Documentation/process/stable-kernel-rules.rst: ``` * Specify any additional patch prerequisites for cherry picking:: Cc: # 3.3.x: a1f84a3: sched: Check for idle Cc: # 3.3.x: 1b9508f: sched: Rate-limit newidle Cc: # 3.3.x: fd21073: sched: Fix affinity logic Cc: # 3.3.x Signed-off-by: Ingo Molnar The tag sequence has the meaning of:: git cherry-pick a1f84a3 git cherry-pick 1b9508f git cherry-pick fd21073 git cherry-pick ``` Here I did not intend to backport this change with commit adding only_64bit flag because, I'm not sure if that dependency alone would be sufficient. If someone really cares about backporting this change, then they should figure out the dependencies, test the functionality and then ask the stable team. - Mani > > Fixes: f55fee56a631 ("PCI: qcom-ep: Add Qualcomm PCIe Endpoint controller driver") > > Signed-off-by: Manivannan Sadhasivam > > --- > > drivers/pci/controller/dwc/pcie-qcom-ep.c | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/drivers/pci/controller/dwc/pcie-qcom-ep.c b/drivers/pci/controller/dwc/pcie-qcom-ep.c > > index e588fcc54589..f925c4ad4294 100644 > > --- a/drivers/pci/controller/dwc/pcie-qcom-ep.c > > +++ b/drivers/pci/controller/dwc/pcie-qcom-ep.c > > @@ -823,6 +823,10 @@ static const struct pci_epc_features qcom_pcie_epc_features = { > > .msi_capable = true, > > .msix_capable = false, > > .align = SZ_4K, > > + .bar[BAR_0] = { .only_64bit = true, }, > > + .bar[BAR_1] = { .type = BAR_RESERVED, }, > > + .bar[BAR_2] = { .only_64bit = true, }, > > + .bar[BAR_3] = { .type = BAR_RESERVED, }, > > }; > > > > static const struct pci_epc_features * > > -- > > 2.25.1 > > -- மணிவண்ணன் சதாசிவம்