[PATCH 00/10] Grab IPA IMEM slice through DT, part 2

Konrad Dybcio posted 10 patches 6 months, 3 weeks ago
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(-)
[PATCH 00/10] Grab IPA IMEM slice through DT, part 2
Posted by Konrad Dybcio 6 months, 3 weeks ago
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@oss.qualcomm.com>
---
Konrad Dybcio (10):
      dt-bindings: sram: qcom,imem: Add a number of missing compatibles
      arm64: dts: qcom: sdm845: Expand IMEM region
      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@oss.qualcomm.com>
prerequisite-patch-id: f6aa0c354d434ec11dd88b93528c05cb3a45bb07
prerequisite-patch-id: 89f72ef0c3d3f29753b0cb9e290a4036ba380cc1
prerequisite-patch-id: 330c94ac7c2b42dd86c5f763b133f0d2598fda40

Best regards,
-- 
Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Re: [PATCH 00/10] Grab IPA IMEM slice through DT, part 2
Posted by Dmitry Baryshkov 6 months, 3 weeks ago
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@oss.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@oss.qualcomm.com>
> prerequisite-patch-id: f6aa0c354d434ec11dd88b93528c05cb3a45bb07
> prerequisite-patch-id: 89f72ef0c3d3f29753b0cb9e290a4036ba380cc1
> prerequisite-patch-id: 330c94ac7c2b42dd86c5f763b133f0d2598fda40
> 
> Best regards,
> -- 
> Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
> 

-- 
With best wishes
Dmitry
Re: [PATCH 00/10] Grab IPA IMEM slice through DT, part 2
Posted by Bjorn Andersson 5 months, 4 weeks ago
On Fri, 23 May 2025 01:18:15 +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.
> 
> [...]

Applied, thanks!

[01/10] dt-bindings: sram: qcom,imem: Add a number of missing compatibles
        commit: 9cea10a4f5a39fde32bf7b8addfa5f9175174e0e
[02/10] arm64: dts: qcom: sdm845: Expand IMEM region
        commit: 81a4a7de3d4031e77b5796479ef21aefb0862807
[03/10] arm64: dts: qcom: sc7180: Expand IMEM region
        commit: 965e28cad4739b11f1bc58c0a9935e025938bb1f
[04/10] arm64: dts: qcom: sc7180: Explicitly describe the IPA IMEM slice
        (no commit info)
[05/10] arm64: dts: qcom: sc7280: Explicitly describe the IPA IMEM slice
        (no commit info)
[06/10] arm64: dts: qcom: sdm845: Explicitly describe the IPA IMEM slice
        (no commit info)
[07/10] arm64: dts: qcom: sm6350: Explicitly describe the IPA IMEM slice
        (no commit info)
[08/10] arm64: dts: qcom: sm8350: Explicitly describe the IPA IMEM slice
        (no commit info)
[09/10] arm64: dts: qcom: sm8550: Explicitly describe the IPA IMEM slice
        (no commit info)
[10/10] arm64: dts: qcom: sm8650: Explicitly describe the IPA IMEM slice
        (no commit info)

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>