Re: [PATCH v3 3/4] arm64: dts: qcom: sdm630: describe adsp_mem region properly

From: Nickolay Goppen

Date: Mon Apr 27 2026 - 08:29:08 EST



27.04.2026 15:08, Konrad Dybcio пишет:
On 4/27/26 2:00 PM, Nickolay Goppen wrote:
27.04.2026 14:49, Konrad Dybcio пишет:
On 4/22/26 5:39 PM, Nickolay Goppen wrote:
Downstream [1] this region is marked as shared, reusable and dynamic so
describe it that way.

[1]: https://github.com/xiaomi-sdm660/android_kernel_xiaomi_sdm660/blob/11-EAS/arch/arm/boot/dts/qcom/sdm660.dtsi#L448

Signed-off-by: Nickolay Goppen <setotau@xxxxxxxxxxxxxx>
---
  arch/arm64/boot/dts/qcom/sdm630.dtsi | 9 ++++++---
  1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
index 4b47efdb57b2..36b419dea153 100644
--- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
@@ -494,9 +494,12 @@ venus_region: venus@9f800000 {
              no-map;
          };
  -        adsp_mem: adsp-region@f6000000 {
-            reg = <0x0 0xf6000000 0x0 0x800000>;
-            no-map;
+        adsp_mem: adsp-region {
+            compatible = "shared-dma-pool";
+            alloc-ranges = <0x0 0x00000000 0x0 0xffffffff>;
Please make this = <0x0 0x80000000 0x0 0x80000000>; instead, since the RAM
Maybe you meant <0x0 0x80000000 0x0 0x8fffffff>?
No, <0x0 0x80000000 0x0 0x80000000> means "start at 2 GiB, until the end of
the 32-bit address range"
Thanks, I've misunderstood.
Konrad

--
Best regards,
Nickolay