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 688BAC7EE29 for ; Fri, 9 Jun 2023 15:23:16 +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:In-Reply-To:Subject:From:References:Cc: To:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=RcH2R/PpLbmGBr2WWMbepMAbPpu17WAV09sRRhIuBqI=; b=fr20u4MCPEqtu/ OpT0fgEeY5ksOgBPpm7pf0/ZMcIM4THeoGa4yCNtlG/wJG5xfwWNOqQ9BXW3FyXIMlZvvbK36VQrM jecRvuqQFL2OU1HICrpP7JLui7joqPRcLZCgAgqVL5N81TiXcRNt6BoL88xRw642ZiD04zQ2LoecB hKg1f1xZLNUqz2+hvUZ3McTbO1vMp/xW8u7ikbBCVnc+rAgf7xpsG2EvmZs5iO2H21SP5mYdheTiC 4GuPCQQHFrWKlxrLMxRhoyHFwFBAjZeEGuARBi/D7UgDl7ak5CbXBzpigpJT6PYxWdiypX2qfaMzF lqfYS699rvm/9o8LulKg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1q7dwy-00DPvw-2Q; Fri, 09 Jun 2023 15:22:52 +0000 Received: from mail-lf1-x12b.google.com ([2a00:1450:4864:20::12b]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1q7dwv-00DPui-2Z for linux-arm-kernel@lists.infradead.org; Fri, 09 Jun 2023 15:22:51 +0000 Received: by mail-lf1-x12b.google.com with SMTP id 2adb3069b0e04-4f4b2bc1565so2432919e87.2 for ; Fri, 09 Jun 2023 08:22:47 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1686324166; x=1688916166; h=content-transfer-encoding:in-reply-to:subject:from:references:cc:to :content-language:user-agent:mime-version:date:message-id:from:to:cc :subject:date:message-id:reply-to; bh=Z86xdDsI0kGtRKtFM/tLOoPg7PoS51U7g8ShD0LwtSQ=; b=g+0YEjiq32Yj0Wa0AXNiU8bJ+JSClgdE/hZ428+TspCD8Mk36atPrCzhCMjgBfNU7i FO4dld8TogXyFJ/ENAsy/Q/TGQwSbJVpX0TXOYdLg3SgcWLXrB/yHs+kVzPQJxEUQfpf y7D4hWcN2yCgn9ctd/hhtpGGl4jJkYEqy0M4EiVZDgObiinjgqFHn93eqaStKnZNk2uU 1PzEy+TrscO+6qFUIJ6jVLe/G1kOkDEHaHndYGswtHTYiM/5jd0vFIGOgLx5CLFzJxrB tC0JjqQEVvJpiUflPUMzFOaRkbEJ3wyiWZnLaSER6nXEjOsSIMMpamCKshCvOW8rXt8j k/mQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1686324166; x=1688916166; h=content-transfer-encoding:in-reply-to:subject:from:references:cc:to :content-language:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=Z86xdDsI0kGtRKtFM/tLOoPg7PoS51U7g8ShD0LwtSQ=; b=XVxzCqLet4GrXBYRfux3mpaeJ8pU08qB4dtt+UD7l/26Lqt5nbxZyUIZ5ElBB3kedD WJPktAHF0sLtaCjiLlFplJH55NoVtMjIjFiYkW+ovydH4QH5FW/r+OMJU7e59Hy1BaAp IdIlPK/whmV8fBFHMicjOVWC1slbstOvVcvHDZXWv/dOpUbIf7NKLj6Me1kFwuEXmsQb bndk2fiaCBohx+4sVTHSYgiY+M8NG5Tnzmgu+aZXLKPXLWAh/iT4YwBGzsKaiA/iulDx 4MFOQzWD8FTw+V6/dZl/cNH2MLBuXxqhKEQcEjvuRbpJeH3sbl1o+OFfF0IvOgPz/PR1 UGTg== X-Gm-Message-State: AC+VfDzexE65RxsykfnhD2WItZ1AWykesL8GKueVEv4zPd9ifUYyUHlm ppH3ue8zaBsa9dxeOZXqlFlPow== X-Google-Smtp-Source: ACHHUZ644vSJDB1y/RM+1wyEjsx5kTGuUMSwyzXP5Ku8VIMsRpk/Ae2ppr2P3Yn9zp8yhGbfpP+dvA== X-Received: by 2002:ac2:498e:0:b0:4ec:8615:303e with SMTP id f14-20020ac2498e000000b004ec8615303emr1384935lfl.33.1686324166361; Fri, 09 Jun 2023 08:22:46 -0700 (PDT) Received: from [192.168.1.101] (abyj190.neoplus.adsl.tpnet.pl. [83.9.29.190]) by smtp.gmail.com with ESMTPSA id y27-20020ac2447b000000b004f4c3feb9fbsm574947lfl.61.2023.06.09.08.22.45 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 09 Jun 2023 08:22:46 -0700 (PDT) Message-ID: Date: Fri, 9 Jun 2023 17:22:44 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.11.2 Content-Language: en-US To: Robin Murphy , Parikshit Pareek Cc: Will Deacon , Joerg Roedel , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Andy Gross , Bjorn Andersson , Manivannan Sadhasivam , Dmitry Baryshkov , Marijn Suijten , Adam Skladowski , linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, iommu@lists.linux.dev, devicetree@vger.kernel.org, "linux-kernel @ vger . kernel . org Prasanna Kumar" , Shazad Hussain References: <20230609054141.18938-1-quic_ppareek@quicinc.com> <79206b05-674b-1f6c-6eb1-ed45e6bd5637@linaro.org> <20230609125631.GA29252@hu-ppareek-blr.qualcomm.com> <2881f374-70e2-0057-f43e-7be12d32ae22@arm.com> <54da68f0-9f9d-cea4-13fb-5e10b8171746@arm.com> From: Konrad Dybcio Subject: Re: [PATCH 0/3] arm64: dts: qcom: sa8775p: Add interconnect to SMMU In-Reply-To: <54da68f0-9f9d-cea4-13fb-5e10b8171746@arm.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230609_082249_836356_415C3991 X-CRM114-Status: GOOD ( 20.16 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 9.06.2023 17:07, Robin Murphy wrote: > On 2023-06-09 15:52, Konrad Dybcio wrote: >> >> >> On 9.06.2023 16:45, Robin Murphy wrote: >>> On 2023-06-09 13:56, Parikshit Pareek wrote: >>>> On Fri, Jun 09, 2023 at 10:52:26AM +0200, Konrad Dybcio wrote: >>>>> >>>>> >>>>> On 9.06.2023 07:41, Parikshit Pareek wrote: >>>>>> Some qcom SoCs have SMMUs, which need the interconnect bandwidth to be >>>>>> This series introduce the due support for associated interconnect, and >>>>>> setting of the due interconnect-bandwidth. Setting due interconnect >>>>>> bandwidth is needed to avoid the issues like [1], caused by not having >>>>>> due clock votes(indirectly dependent upon interconnect bandwidth). >>>>> >>>>> [1] ??? >>>> >>>> My bad. Intended to mention following: >>>> https://lore.kernel.org/linux-arm-msm/20230418165224.vmok75fwcjqdxspe@echanude/ >>> >>> This sounds super-dodgy - do you really have to rely on configuration of the interconnect path from the SMMU's pagetable walker to RAM to keep a completely different interconnect path clocked for the CPU to access SMMU registers? You can't just request the programming interface clock directly like on other SoCs? >> On Qualcomm platforms, particularly so with the more recent ones, some >> clocks are managed by various remote cores. Half of what the interconnect >> infra does on these SoCs is telling one such core to change the internally >> managed clock's rate based on the requested bw. > > That much I get, it just seems like an arse-backwards design decision if it's really necessary to pretend the SMMU needs to access memory in order for the CPU to be able to access the SMMU. The respective SMMU interfaces are functionally independent of each other - even if it is the case in the integration that both interfaces and/or the internal TCU clock do happen to be driven synchronously from the same parent clock - and in any sane interconnect the CPU->SMMU and SMMU->RAM routes would be completely different and not intersect at all. Well, it's not the first time we stumble into a.. peculiar.. design decision on these SoCs.. That said, we can't do much about it now.. On older SoCs, some interconnect paths were strongly associated with specific TBUs which were responsible for specific SID ranges.. In this specific case, it looks like SIDs 0x000-0x3ff should correspond to PCIE0 and 0x400-0x7ff to PCIE1. But the line isn't drawn very clearly this time around, so maybe there's some internal spaghetti. Konrad > > Thanks, > Robin. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel