linux-kernel - Re: [PATCH 00/10] Grab IPA IMEM slice through DT, part 2

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <j23rf3rjfv6rh73ce22ka6g5e4tujeeg25fmobvetpxo4kth7p@rvi3krl5hvwh>
Date: Fri, 23 May 2025 08:51:45 +0300
From: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
To: Konrad Dybcio <konradybcio@...nel.org>
Cc: Bjorn Andersson <andersson@...nel.org>, Rob Herring <robh@...nel.org>,
        Krzysztof Kozlowski <krzk+dt@...nel.org>,
        Conor Dooley <conor+dt@...nel.org>,
        cros-qcom-dts-watchers@...omium.org,
        Sai Prakash Ranjan <quic_saipraka@...cinc.com>,
        Marijn Suijten <marijn.suijten@...ainline.org>,
        linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org,
        Konrad Dybcio <konrad.dybcio@....qualcomm.com>
Subject: Re: [PATCH 00/10] Grab IPA IMEM slice through DT, part 2

On Fri, May 23, 2025 at 01:18:15AM +0200, Konrad Dybcio wrote:
> The IPA IMEM slice addresses/sizes are hardcoded in the driver. That's
> mucho no bueno, especially since the same versions of IPA are used
> across a number of vastly different platforms, which invalidates that
> approach completely.
> 
> This series wires up the IMEM slices in DT on almost all platforms
> (need some more time for SDX55/65) and fills in the necessary bindings
> holes.
> 
> Tested on SC7280 only, FWIW.
> 
> Patches 1-3 are good to go instantly, the rest must wait for the
> bindings changes that were submitted in the series marked as a
> dependency.
> 
> Signed-off-by: Konrad Dybcio <konrad.dybcio@....qualcomm.com>
> ---
> Konrad Dybcio (10):
>       dt-bindings: sram: qcom,imem: Add a number of missing compatibles
>       arm64: dts: qcom: sdm845: Expand IMEM region

You've expanded IMEM region for SDM845, but you didn't add IPA slice. Is
that expected?

>       arm64: dts: qcom: sc7180: Expand IMEM region
>       arm64: dts: qcom: sc7180: Explicitly describe the IPA IMEM slice
>       arm64: dts: qcom: sc7280: Explicitly describe the IPA IMEM slice
>       arm64: dts: qcom: sdm845: Explicitly describe the IPA IMEM slice
>       arm64: dts: qcom: sm6350: Explicitly describe the IPA IMEM slice
>       arm64: dts: qcom: sm8350: Explicitly describe the IPA IMEM slice
>       arm64: dts: qcom: sm8550: Explicitly describe the IPA IMEM slice
>       arm64: dts: qcom: sm8650: Explicitly describe the IPA IMEM slice
> 
>  Documentation/devicetree/bindings/sram/qcom,imem.yaml | 13 +++++++++++++
>  arch/arm64/boot/dts/qcom/sc7180.dtsi                  | 16 +++++++++++-----
>  arch/arm64/boot/dts/qcom/sc7280.dtsi                  |  6 ++++++
>  arch/arm64/boot/dts/qcom/sdm845.dtsi                  | 16 +++++++++++-----
>  arch/arm64/boot/dts/qcom/sm6350.dtsi                  | 16 ++++++++++++++++
>  arch/arm64/boot/dts/qcom/sm8350.dtsi                  | 16 ++++++++++++++++
>  arch/arm64/boot/dts/qcom/sm8550.dtsi                  | 16 ++++++++++++++++
>  arch/arm64/boot/dts/qcom/sm8650.dtsi                  | 16 ++++++++++++++++
>  8 files changed, 105 insertions(+), 10 deletions(-)
> ---
> base-commit: 6add743d2854d744c3037235b87c1c9d164fd132
> change-id: 20250523-topic-ipa_mem_dts-58f5d73a1f21
> prerequisite-message-id: <20250523-topic-ipa_imem-v1-0-b5d536291c7f@....qualcomm.com>
> prerequisite-patch-id: f6aa0c354d434ec11dd88b93528c05cb3a45bb07
> prerequisite-patch-id: 89f72ef0c3d3f29753b0cb9e290a4036ba380cc1
> prerequisite-patch-id: 330c94ac7c2b42dd86c5f763b133f0d2598fda40
> 
> Best regards,
> -- 
> Konrad Dybcio <konrad.dybcio@....qualcomm.com>
> 

-- 
With best wishes
Dmitry