From mboxrd@z Thu Jan 1 00:00:00 1970 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 1BE6FC30653 for ; Wed, 3 Jul 2024 13:03:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=UnTObUWcJ/RaK0SjYoIOKGouNC0KM5T8iGOiMhR/6Gc=; b=ylAv0cUgtjdokb2yv6QJFrFYk8 Ptz8NxULGq5ng7YoZb9gjnKyIm2hi1+YfPJfT3fEMnJUfclJlT/drx1Dy+mb+zmyfWgirE1cOUNmN Mw+wnSlUloUbHdTz20nKfPk+Ng/qznwGbiCrY7xIk0lGjbgy9QHgodGK4Zh1p0VbHalUK4DfLylAO Twkp80k4WTjJdSVCxOmV3/HZDaTmAiCKiChWRCD5RBF5N4XEqzPZj3hRP0yrWUPes1S6Dm62bWkEJ YQhk1q8IMFOBri/0DaU0I0h52C7hvM49svJ21r3FQEDHukNJwKUSjBDWCPt0/2pRO3ZKW7eXqJWHz tME177eQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sOzdO-0000000AFBf-3twh; Wed, 03 Jul 2024 13:02:55 +0000 Received: from sin.source.kernel.org ([2604:1380:40e1:4800::1]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sOzcr-0000000AEx2-1YWb for linux-arm-kernel@lists.infradead.org; Wed, 03 Jul 2024 13:02:23 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sin.source.kernel.org (Postfix) with ESMTP id 1B64CCE198A; Wed, 3 Jul 2024 13:02:17 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id A6208C4AF0B; Wed, 3 Jul 2024 13:02:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1720011736; bh=RmFrPvIZPGCspcDcWKrZAiJYo6HnrUaXiJpp47Mrxl4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=tXJon6l69U6RjRQz46pZmCrfd+qp2EZ6KI6qZrhHIicv/xon1GYx1RhZWBEkbs7Zf 38e+on76S/0BcyKrMCHMysTb2Ws2NQkv4t6QnboRkvODhvUCoOzfUYhaKXAcEcZ0SK zneAl7LGGtMMd3F2kCcwnI1XXNKtWUkC1Sf55m5HyX+InHswiNDPH2ZV+e5PWd9mjV mtPkrI8RQL+HRo01x95byFkgBAWQja9qsi0ZYMyDNWlewWAGQf5ef/BC+M9YmdaOaS mwA9poHdt09SsLiZ++IlA8Eiz4SOOar8sjb0iispsh9z2ARLWQxizmOxfc2eJCmx1G /CcBdk9KL3L9w== Date: Wed, 3 Jul 2024 14:02:10 +0100 From: Will Deacon To: Bibek Kumar Patro Cc: Dmitry Baryshkov , robdclark@gmail.com, robin.murphy@arm.com, joro@8bytes.org, jgg@ziepe.ca, jsnitsel@redhat.com, robh@kernel.org, krzysztof.kozlowski@linaro.org, quic_c_gdjako@quicinc.com, konrad.dybcio@linaro.org, iommu@lists.linux.dev, linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v13 4/6] iommu/arm-smmu: add ACTLR data and support for SM8550 Message-ID: <20240703130209.GA5750@willie-the-truck> References: <20240628140435.1652374-1-quic_bibekkum@quicinc.com> <20240628140435.1652374-5-quic_bibekkum@quicinc.com> <6da77880-2ba4-4b02-8b3e-cb0fbd0a9daf@quicinc.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <6da77880-2ba4-4b02-8b3e-cb0fbd0a9daf@quicinc.com> User-Agent: Mutt/1.10.1 (2018-07-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240703_060221_809807_98CEDAE9 X-CRM114-Status: GOOD ( 23.96 ) 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 On Wed, Jul 03, 2024 at 05:45:23PM +0530, Bibek Kumar Patro wrote: > > > On 7/2/2024 12:04 AM, Dmitry Baryshkov wrote: > > On Fri, Jun 28, 2024 at 07:34:33PM GMT, Bibek Kumar Patro wrote: > > > Add ACTLR data table for SM8550 along with support for > > > same including SM8550 specific implementation operations. > > > > > > Signed-off-by: Bibek Kumar Patro > > > --- > > > drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c | 89 ++++++++++++++++++++++ > > > 1 file changed, 89 insertions(+) > > > > > > diff --git a/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c b/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c > > > index 77c9abffe07d..b4521471ffe9 100644 > > > --- a/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c > > > +++ b/drivers/iommu/arm/arm-smmu/arm-smmu-qcom.c > > > @@ -23,6 +23,85 @@ > > > > > > #define CPRE (1 << 1) > > > #define CMTLB (1 << 0) > > > +#define PREFETCH_SHIFT 8 > > > +#define PREFETCH_DEFAULT 0 > > > +#define PREFETCH_SHALLOW (1 << PREFETCH_SHIFT) > > > +#define PREFETCH_MODERATE (2 << PREFETCH_SHIFT) > > > +#define PREFETCH_DEEP (3 << PREFETCH_SHIFT) > > > + > > > +static const struct actlr_config sm8550_apps_actlr_cfg[] = { > > > + { 0x18a0, 0x0000, PREFETCH_SHALLOW | CPRE | CMTLB }, > > > + { 0x18e0, 0x0000, PREFETCH_SHALLOW | CPRE | CMTLB }, > > > + { 0x0800, 0x0020, PREFETCH_DEFAULT | CMTLB }, > > > + { 0x1800, 0x00c0, PREFETCH_DEFAULT | CMTLB }, > > > + { 0x1820, 0x0000, PREFETCH_DEFAULT | CMTLB }, > > > + { 0x1860, 0x0000, PREFETCH_DEFAULT | CMTLB }, > > > + { 0x0c01, 0x0020, PREFETCH_DEEP | CPRE | CMTLB }, > > > > - Please keep the list sorted > > Sure Dmitry, will sort this list in reverse-christmas-tree order > in next iteration. Thanks for this input. > > > - Please comment, which devices use these settings. > > As discussed in earlier versions of this patch, these table entries > are kind of just blind values for SMMU device, where SMMU do not have > idea on which SID belong to which client. During probe time when the > clients' Stream-ID has corresponding ACTLR entry then the driver would > set value in register. I'm still firmly of the opinion that this stuff needs a higher-level description in the device-tree and should not be hard-coded in the driver like this. It's not just a list of opaque values; it describes SoC-specific topological information that should not be this rigid. Will