From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f174.google.com (mail-pf1-f174.google.com [209.85.210.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 04A141A9FB8 for ; Mon, 25 Aug 2025 22:19:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.174 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756160397; cv=none; b=gyAI8BVWosfxc5UpWLynWkKBjfMYdG8F6iLAqd/2PRPIHnZXB2oaO8eFLKpJ1goiEE/gWi8u1lnXzehiidOfVLhqHJI219PfCi3iF7JT3RlIh6fIjDCemoGSC6MgFsJYnzlFx3aJuB+FBFMAoaTaO38iEFiukQ77ExEVNiVr6fY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756160397; c=relaxed/simple; bh=pF5m4XVh5+kykGCQizUldzCrFgr+3qmcWGuNjVxySro=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=i3owrUUcELeRFvA+qzrhvyOiGnuULqy0b7sKpvRj27IglCWfmJtbfu1Dve8zGnJgkXPs+PBDI/1nScGvxmCQv1JTX5njee2glW/GlamEpu7pvvOLrQvOaTDlrV6g8hrniT8TCGZWR2f2LM0tE+QW248B2izJ2WPi++aAXo62ZSc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=chromium.org; spf=pass smtp.mailfrom=chromium.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b=YNFU7y2y; arc=none smtp.client-ip=209.85.210.174 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=chromium.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=chromium.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b="YNFU7y2y" Received: by mail-pf1-f174.google.com with SMTP id d2e1a72fcca58-7704799d798so1612295b3a.3 for ; Mon, 25 Aug 2025 15:19:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; t=1756160395; x=1756765195; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=/JpVTU3PWKRezYGNXaHQneZI3VniMUHuCsVrcgfQ/mw=; b=YNFU7y2y7zStDluE1E8zyp4nokpQ4jS0ZipW6DwTqqqti+NthkCLsP5bAeuIjxv7l8 sg+j5/kr85OmM+rEU8bAqj+xnHh4ZzAvLQ0TsfRbyT3c9l96NVR9TFVUbHkFlJsuFlxO FFtaHLshuTGDYuVAKnghyS91ibQ1QptcnAucg= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1756160395; x=1756765195; h=in-reply-to: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=/JpVTU3PWKRezYGNXaHQneZI3VniMUHuCsVrcgfQ/mw=; b=lgDz+/zE/XHkll0f5QCk+9y+jFxNxfhZLKqc+I41nA8OZz/5WOPlG/Cqam5bIc5zrw wpWqqZV1vuhx6BGV1ZKttlzBA6UfOoVp0AAbj4+IfefqtfcCnkT74JaWz3It/Q1UdfIP R+FWTTBPyAsDBi7JIfJhpkKD2MEGg8/X4rWqhO7nKyH0gyCbyEBagesm8ZjNAgOsIfqe taSJxciWq38BgTT8elG5hehfVZDCKx3+veoT+JIlah7nswmj8O1YUG8xA60ZUfdJoPJc tqufEm78cH2xpQRQw79LR+gi7rfxE3YVvKCXfupqot6YEtApp1ih84UiuRZ2HI1pV0p8 Ozuw== X-Forwarded-Encrypted: i=1; AJvYcCXnQxXzNugccWovePAIWJCFGjzCDRheOJGM46P+FGkBRHpuNYJIvU/6G5KWt0D8sZMAo2jXO3ffq3w/sj48@vger.kernel.org X-Gm-Message-State: AOJu0YxdrAsfH3kMAKUP/rElpfR00GZPPf0NYAfFT3SreFu9xtFfoZfx JJFyfDbBcIzMbUpU4eR5CpE2tlyTpGTTJbPuFa0z1aS+9c7z5mJwZihEX2/qUDMtLg== X-Gm-Gg: ASbGncsNSv3SVOwOKRSR4Ilx9wJaWDmzFCnWxw0fIniOORQnCFQCUvTZWJhfP9fk0p4 U/7O/q803Ew4PpoUs21EZ8YGufg7ZaaFEZHRMLEZV4YIWALO70ErzcihIpOm4ctIRZ7V/BKETTf ycJ5dIC4BcWeThgyopRVyLTLd6sw4uEKUveI64s06fIe05C4Y8QfgnXx5HJ4dY6kJ8Xsgy6uiyJ YUyTLNcE3qmnSRL4tqopuXdwzDWHX4bmhunHpfeDHf17bkU8AEtCLMPsxNzE1XmRe4Xp/1blV8e mYMtiizdq93YODIEW78DZAVj3rccYmXYGHtKMvB5rbesxKGmRqsNV5STb4iC37gOjsJT+0N3JGG iyIHQe+dllPNRA04nX/lK20n4Y0muEnOy6QA7xY1Fhkdmvhp/bWnMYJg9uQ== X-Google-Smtp-Source: AGHT+IHGJg8be7xU6VaEaBVE+3EWYZ4SopGxSXLSailrKgYeCMuf54WK7If+tDGPHqm1jXQLMQoqYA== X-Received: by 2002:a05:6a00:14c7:b0:770:5987:5b3a with SMTP id d2e1a72fcca58-77059879144mr6483981b3a.16.1756160395127; Mon, 25 Aug 2025 15:19:55 -0700 (PDT) Received: from localhost ([2a00:79e0:2e14:7:230e:95:218f:e216]) by smtp.gmail.com with UTF8SMTPSA id d2e1a72fcca58-770401b01b2sm8327239b3a.64.2025.08.25.15.19.53 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 25 Aug 2025 15:19:54 -0700 (PDT) Date: Mon, 25 Aug 2025 15:19:52 -0700 From: Brian Norris To: Dmitry Baryshkov Cc: Bjorn Andersson , Konrad Dybcio , Georgi Djakov , Odelu Kukatla , Douglas Anderson , Krzysztof Kozlowski , Konrad Dybcio , linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, Conor Dooley , Rob Herring , cros-qcom-dts-watchers@chromium.org Subject: Re: [RFC PATCH] arm64: dts: qcom: sc7280: Drop aggre{1,2}_noc QOS clocks on Herobrine Message-ID: References: <20250823123718.RFC.1.Idebf1d8bd8ff507462fef9dc1ff47e84c01e9b60@changeid> <3i7pdzata6gxsc3svb3eygubfdfmnetlekxdd25bb4ljkdrlh7@bvufiwmdusqz> 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=us-ascii Content-Disposition: inline In-Reply-To: <3i7pdzata6gxsc3svb3eygubfdfmnetlekxdd25bb4ljkdrlh7@bvufiwmdusqz> Hi Dmitry, Thanks for the response. On Mon, Aug 25, 2025 at 12:22:03PM +0300, Dmitry Baryshkov wrote: > On Sat, Aug 23, 2025 at 12:37:18PM -0700, Brian Norris wrote: > > Specifically, they fail in qcom_icc_set_qos() when trying to write the > > QoS settings for qhm_qup1. Several of the previous nodes (qhm_qspi, > > qhm_qup0, ...) seem to configure without crashing. > > > > I don't really know what's unique about Herobrine systems vs other > > sc7280 systems that presumably work fine. I'd guess there's some > > conflict with something configured by the boot firmware. > > I think it well might be that Herobrine's TZ doesn't export QoS regions > to Linux. That may be. I don't really know what to look for to verify that without a lot of research, aside from crashes like this though. I'm fine with assuming this though. > > I'm submitting as an RFC just to get thoughts from people who hopefully > > know better than me what might be going wrong here. > > > > Fixes: fbd908bb8bc0 ("interconnect: qcom: sc7280: enable QoS configuration") > > Fixes: 2b5004956aff ("arm64: dts: qcom: sc7280: Add clocks for QOS configuration") > > Signed-off-by: Brian Norris > > --- > > > > arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi | 10 ++++++++++ > > 1 file changed, 10 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi > > index 2ba4ea60cb14..59203ce58c61 100644 > > --- a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi > > +++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi > > @@ -394,6 +394,16 @@ &vreg_l2c_1p8 { > > > > /* ADDITIONS TO NODES DEFINED IN PARENT DEVICE TREE FILES */ > > > > +/* QoS seems to have conflicts with boot firmware on these devices. */ > > +&aggre1_noc { > > + /delete-property/ clocks; > > While it might be enough for Linux to make it skip the QoS, I think a > more correct fix would be to remove the 'reg' instead / in addition. That should work too. However, besides simply violating Documentation/devicetree/bindings/interconnect/qcom,sc7280-rpmh.yaml (which is easy enough to tweak), that also violates basic simple-bus rules: arch/arm64/boot/dts/qcom/sc7280-herobrine-crd.dtb: soc@0 (simple-bus): interconnect@16e0000: 'anyOf' conditional failed, one must be fixed: 'reg' is a required property 'ranges' is a required property from schema $id: http://devicetree.org/schemas/simple-bus.yaml# [...] > On the other hand, having those boards used by only a few people it > might be easier to just pick up the simple fix rather than implementing > a 'proper' one. > > Nevertheless, this would require changing the schema too, see > Documentation/devicetree/bindings/interconnect/qcom,sc7280-rpmh.yaml Sure, I've tested out a change just to tweak the clock requirements. I'll plan on submitting that if no solution arises for the 'reg' schema. Brian