Received-SPF: pass (google.com: domain of linux-kernel+bounces-157358-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Date: Wed, 24 Apr 2024 09:55:30 -0700
From: Bjorn Andersson <quic_bjorande@quicinc.com>
To: Luca Weiss <luca@z3ntu.xyz>
CC: <~postmarketos/upstreaming@lists.sr.ht>, <phone-devel@vger.kernel.org>,
        Bjorn Andersson <andersson@kernel.org>,
        Mathieu Poirier
	<mathieu.poirier@linaro.org>,
        Konrad Dybcio <konrad.dybcio@linaro.org>,
        Rob
 Herring <robh@kernel.org>,
        Krzysztof Kozlowski <krzk+dt@kernel.org>,
        Conor
 Dooley <conor+dt@kernel.org>, <linux-arm-msm@vger.kernel.org>,
        <linux-remoteproc@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        <devicetree@vger.kernel.org>
Subject: Re: [PATCH 0/7] Use mboxes instead of syscon for APCS (arm32 & arm64)
Message-ID: <Zik5gmTixGpk13qj@hu-bjorande-lv.qualcomm.com>
References: <20240424-apcs-mboxes-v1-0-6556c47cb501@z3ntu.xyz>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20240424-apcs-mboxes-v1-0-6556c47cb501@z3ntu.xyz>

On Wed, Apr 24, 2024 at 06:23:53PM +0200, Luca Weiss wrote:
> The first patch is for removing a bogus error warning I've noticed while
> developing this on msm8226 - there the patches are also coming later for
> this SoC since apcs is getting hooked up to cpufreq there also.
> 
> Apart from usages from the qcom,smsm driver (patches coming!) all other
> usages of the apcs mailbox now go via the mailbox driver - where one is
> used, so some arm32 boards will continue using "qcom,ipc*" properties in
> the short or long term.
> 

Very nice, thank you for cleaning this up.

Regards,
Bjorn

> Only compile-tested apart from msm8953 (tested on sdm632-fairphone-fp3)
> and msm8974 (tested on msm8974pro-fairphone-fp2), but I don't expect any
> complications with this.
> 
> Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
> ---
> Luca Weiss (7):
>       rpmsg: qcom_smd: Don't print error during probe deferral
>       ARM: dts: qcom: msm8974: Use mboxes properties for APCS
>       arm64: dts: qcom: msm8916: Use mboxes properties for APCS
>       arm64: dts: qcom: msm8939: Use mboxes properties for APCS
>       arm64: dts: qcom: msm8953: Use mboxes properties for APCS
>       arm64: dts: qcom: msm8976: Use mboxes properties for APCS
>       arm64: dts: qcom: msm8994: Use mboxes properties for APCS
> 
>  arch/arm/boot/dts/qcom/qcom-msm8974.dtsi | 14 +++++++-------
>  arch/arm64/boot/dts/qcom/msm8916.dtsi    | 10 +++++-----
>  arch/arm64/boot/dts/qcom/msm8939.dtsi    |  4 ++--
>  arch/arm64/boot/dts/qcom/msm8953.dtsi    | 10 +++++-----
>  arch/arm64/boot/dts/qcom/msm8976.dtsi    |  8 ++++----
>  arch/arm64/boot/dts/qcom/msm8994.dtsi    |  6 +++---
>  drivers/rpmsg/qcom_smd.c                 |  3 ++-
>  7 files changed, 28 insertions(+), 27 deletions(-)
> ---
> base-commit: 43173e6dbaa227f3107310d4df4a3bacd5e0df33
> change-id: 20240423-apcs-mboxes-12ee6c01a5b3
> 
> Best regards,
> -- 
> Luca Weiss <luca@z3ntu.xyz>
>