Yidi Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47293 )
Change subject: mediatek/mt8192: memlayout: Add DRAM DMA region
......................................................................
Patch Set 10:
Patch Set 10:
The SPM DMA can be abandoned after it's fully loaded right? I wonder if we can leverage the MCU region for it, and always load SPM before MCU.
Yes, I did try the MCU region and SPM DMA hardware also works fine with MCU region.
But anyway, given we're loading more and more blobs, I agree having a post-DRAM DMA maybe the better idea.
Thank you.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/47293
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: If6e803da23126419a96ffc0337d35edd0e181871
Gerrit-Change-Number: 47293
Gerrit-PatchSet: 10
Gerrit-Owner: Yidi Lin
yidi.lin@mediatek.com
Gerrit-Reviewer: Hung-Te Lin
hungte@chromium.org
Gerrit-Reviewer: Julius Werner
jwerner@chromium.org
Gerrit-Reviewer: Martin Roth
martinroth@google.com
Gerrit-Reviewer: Patrick Georgi
pgeorgi@google.com
Gerrit-Reviewer: Roger Lu
roger.lu@mediatek.com
Gerrit-Reviewer: Yu-Ping Wu
yupingso@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Nicolas Boichat
drinkcat@chromium.org
Gerrit-CC: Nicolas Boichat
drinkcat@google.com
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Thu, 19 Nov 2020 08:29:20 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment