From: Luca Weiss <luca@z3ntu.xyz>
To: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: linux-arm-msm@vger.kernel.org,
~postmarketos/upstreaming@lists.sr.ht,
phone-devel@vger.kernel.org, Andy Gross <agross@kernel.org>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Konrad Dybcio <konrad.dybcio@somainline.org>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2/2] ARM: dts: qcom: msm8974-*: re-add remoteproc supplies
Date: Wed, 06 Jul 2022 00:06:05 +0200 [thread overview]
Message-ID: <12009239.O9o76ZdvQC@g550jk> (raw)
In-Reply-To: <Yr5mua6i6WxoAlzB@builder.lan>
Hi Bjorn,
On Freitag, 1. Juli 2022 05:15:05 CEST Bjorn Andersson wrote:
> On Mon 06 Jun 11:04 CDT 2022, Luca Weiss wrote:
> > As part of a recent cleanup commit, the remoteproc supplies for adsp and
> > modem were removed from msm8974.dtsi and now need to be set in the
> > device dts. Do so.
> >
> > Fixes: f300826d27be ("ARM: dts: qcom-msm8974: Sort and clean up nodes")
> > Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
> > ---
> > Bjorn, could you please pick this up for -fixes so it lands in an
> > upcoming 5.19-rc?
>
> As reported by lkp, this doesn't build. Am I missing something?
I briefly mentioned in IRC, maybe you missed it.
Castor is actually still missing pm8841 regulators upstream.
I'll send this old (rebased) commit from you to add them, is that ok with you?
https://github.com/msm8974-mainline/linux/commit/
d01e20807a00e78570abf3dd91bea555dff06fa9
Otherwise we can just keep castor out of this patch for now and add it later,
I don't really mind either way.
Regards
Luca
>
> Regards,
> Bjorn
next prev parent reply other threads:[~2022-07-05 22:06 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-06-06 16:04 [PATCH 1/2] ARM: dts: qcom: msm8974: re-add missing pinctrl Luca Weiss
2022-06-06 16:04 ` [PATCH 2/2] ARM: dts: qcom: msm8974-*: re-add remoteproc supplies Luca Weiss
2022-06-09 9:32 ` kernel test robot
2022-07-01 3:15 ` Bjorn Andersson
2022-07-05 22:06 ` Luca Weiss [this message]
2022-06-06 18:38 ` [PATCH 1/2] ARM: dts: qcom: msm8974: re-add missing pinctrl Konrad Dybcio
2022-07-03 3:56 ` Bjorn Andersson
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=12009239.O9o76ZdvQC@g550jk \
--to=luca@z3ntu.xyz \
--cc=agross@kernel.org \
--cc=bjorn.andersson@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=konrad.dybcio@somainline.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=phone-devel@vger.kernel.org \
--cc=robh+dt@kernel.org \
--cc=~postmarketos/upstreaming@lists.sr.ht \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).