Skip to content
Snippets Groups Projects
Commit dbe832e2 authored by Suhrid Subramaniam's avatar Suhrid Subramaniam Committed by Oleksii Kutuzov
Browse files

GENIO: arm64: dts: mediatek: genio-700-evk: Correct the reserved memory size for the DSP

The reserved memory size for ADSP specified in the DTS does not match
the memory protection range set in TF-A, leading the kernel to believe
it can still use the physical addresses that are actually protected.
Consequently, when hardware IPs attempt to access these protected
addresses,it results in a condition where R/W cannot be performed.

(am from https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/5400820

)

This is to align the setting of Chromebook. Genio platforms didn't set
the memory protection for the multimedia hardware, so this issue won't
happen on Genio platforms. However, we still apply this fix to prevent
the issue in case someone wants to add the protection in the future.

After applying this change, the memory region for audio will be:

17 MiB reserved for Audio (AFE, ADSP)
+-----------------------+ 0x61100000
|      ADSP DMA 1MiB    |
+-----------------------+ 0x61000000
|      AFE MEM  1MiB    |
+-----------------------+ 0x60f00000
|      ADSP MEM 15MiB   |
+-----------------------+ 0x60000000

Change-Id: I14e53ff7417e4af832f5fd1080bb1675607a60ce
Signed-off-by: default avatarSuhrid Subramaniam <suhrid.subramaniam@mediatek.com>
parent 5b6f045a
No related branches found
No related tags found
1 merge request!14[UPSTREAM] Add Media-related patches from Mediatek's Release 24.0
......@@ -57,6 +57,18 @@ reserved-memory {
#size-cells = <2>;
ranges;
adsp_dma_mem_reserved: adsp-dma-mem-region {
compatible = "shared-dma-pool";
reg = <0 0x61000000 0 0x100000>;
no-map;
};
adsp_mem_reserved: adsp-mem-region {
compatible = "shared-dma-pool";
no-map;
reg = <0 0x60000000 0 0xf00000>;
};
apu_reserve_memory: apu-reserve-memory{
compatible = "shared-dma-pool";
size = <0 0x1400000>; //20 MB
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment