From: Johan Hovold <johan@kernel.org>
To: Abel Vesa <abel.vesa@linaro.org>
Cc: Konrad Dybcio <konrad.dybcio@linaro.org>,
Bjorn Andersson <andersson@kernel.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Rajendra Nayak <quic_rjendra@quicinc.com>,
Sibi Sankar <quic_sibis@quicinc.com>,
linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] arm64: dts: qcom: sm8450: Add Broadcast_AND region in LLCC block
Date: Wed, 9 Oct 2024 18:34:24 +0200 [thread overview]
Message-ID: <ZwawkF17sxYRk_vC@hovoldconsulting.com> (raw)
In-Reply-To: <ZpkOgpGCKp/w13la@linaro.org>
On Thu, Jul 18, 2024 at 03:45:54PM +0300, Abel Vesa wrote:
> On 24-07-18 13:28:49, Johan Hovold wrote:
> > On Thu, Jul 18, 2024 at 01:20:37PM +0200, Konrad Dybcio wrote:
> > > On 18.07.2024 12:20 PM, Abel Vesa wrote:
> > > > Add missing Broadcast_AND region to the LLCC block for x1e80100,
> > > > as the LLCC version on this platform is 4.1 and it provides the region.
> > > >
> > > > This also fixes the following error caused by the missing region:
> > > >
> > > > [ 3.797768] qcom-llcc 25000000.system-cache-controller: error -EINVAL: invalid resource (null)
> >
> > Please say something about this error only showing up in linux-next,
> > which has 055afc34fd21 ("soc: qcom: llcc: Add regmap for Broadcast_AND region"
> > region")).
>
> Hm, now that I think of it, maybe I should drop the fixes tag
> altogether, as it fits more into the "new support" category strictly
> because of that commit you mentioned.
Looks like you forgot to respin this one so that the error is now logged
with mainline since 6.11 and commit 055afc34fd21 ("soc: qcom: llcc: Add
regmap for Broadcast_AND region"):
qcom-llcc 25000000.system-cache-controller: error -EINVAL: invalid resource (null)
Can you please respin as soon as possible with an updated commit message
and a CC stable tag to suppress the error?
> > > > Fixes: af16b00578a7 ("arm64: dts: qcom: Add base X1E80100 dtsi and the QCP dts")
> > > > Signed-off-by: Abel Vesa <abel.vesa@linaro.org>
> > > > ---
> > >
> > > Please fix the commit title
> >
> > And that.
>
> Yep. Will do.
Don't forget to update Subject.
Johan
next prev parent reply other threads:[~2024-10-09 16:34 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-18 10:20 [PATCH] arm64: dts: qcom: sm8450: Add Broadcast_AND region in LLCC block Abel Vesa
2024-07-18 11:20 ` Konrad Dybcio
2024-07-18 11:28 ` Johan Hovold
2024-07-18 12:45 ` Abel Vesa
2024-10-09 16:34 ` Johan Hovold [this message]
2024-10-14 7:42 ` Abel Vesa
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ZwawkF17sxYRk_vC@hovoldconsulting.com \
--to=johan@kernel.org \
--cc=abel.vesa@linaro.org \
--cc=andersson@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=konrad.dybcio@linaro.org \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=quic_rjendra@quicinc.com \
--cc=quic_sibis@quicinc.com \
--cc=robh@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.