Attention is currently required from: Hung-Te Lin, Paul Menzel, Yu-Ping Wu. Rex-BC Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63605 )
Change subject: soc/mediatek/mt8195: Reserve 17MB DRAM for audio and DSP ......................................................................
Patch Set 2:
(4 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/63605/comment/78115b80_45bb68be PS1, Line 7: Reserve DRAM memory for usage of audio and DSP
Reserve DRAM for audio and DSP […]
Done
https://review.coreboot.org/c/coreboot/+/63605/comment/5cff0822_2ff66814 PS1, Line 7: DRAM memory
Also mention the size?
Done
https://review.coreboot.org/c/coreboot/+/63605/comment/c7289a14_2a5498d2 PS1, Line 9: Audio and DSP use the 0x60000000 of 17M to do some operations in : kernel drivers. Therefore, we reserve this DRAM address to prevent : this address is used in bootloader stage.
In the future, please use 72 characters per line.
done
File src/soc/mediatek/mt8195/include/soc/memlayout.ld:
https://review.coreboot.org/c/coreboot/+/63605/comment/13320adf_be3d08b5 PS1, Line 72: REGION(audio_reserved, 0x60000000, 17M, 4)
This only creates a symbol. […]
Done