linux-kernel - Re: [PATCH 2/2] ARM: dts: qcom: msm8974-*: re-add remoteproc supplies

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Wed, 06 Jul 2022 00:06:05 +0200
From:   Luca Weiss <luca@...tu.xyz>
To:     Bjorn Andersson <bjorn.andersson@...aro.org>
Cc:     linux-arm-msm@...r.kernel.org,
        ~postmarketos/upstreaming@...ts.sr.ht, phone-devel@...r.kernel.org,
        Andy Gross <agross@...nel.org>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Konrad Dybcio <konrad.dybcio@...ainline.org>,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] ARM: dts: qcom: msm8974-*: re-add remoteproc supplies

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@...tu.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