Nicolas Boichat has uploaded this change for review.

View Change

WIP: mediatek/mt8192: spm: Align spm_firmware_binary to 16-byte

16-byte alignement is critical, else the DMA transfer will corrupt
data at 4096-byte boundaries.

BUG=b:172177570
TEST=Asurada boots up to shell, suspend works

Change-Id: I016062d04dd07950405d38f42e5e0ee01371c174
Signed-off-by: Nicolas Boichat <drinkcat@chromium.org>
---
M src/soc/mediatek/mt8192/spm.c
1 file changed, 5 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/82/47282/1
diff --git a/src/soc/mediatek/mt8192/spm.c b/src/soc/mediatek/mt8192/spm.c
index a6f2109..4847377 100644
--- a/src/soc/mediatek/mt8192/spm.c
+++ b/src/soc/mediatek/mt8192/spm.c
@@ -9,7 +9,11 @@
#include <soc/spm.h>
#include <timer.h>

-static const u32 spm_firmware_binary[] = {
+/*
+ * 16-byte alignement is critical, else the DMA transfer will corrupt data at
+ * 4096-byte boundaries.
+ */
+static const u32 spm_firmware_binary[] __aligned(16) = {
0x3200001e, 0x6a00001e, 0x6e00001e, 0x6800001e, 0x6200001e, 0x6000001e,
0x5e00001e, 0x5c00001e, 0x6000001e, 0x5800001e, 0x5600001e, 0x5400001e,
0x5200001e, 0x5000001e, 0x4e00001e, 0x4c00001e, 0xd402001e, 0xee01001e,

To view, visit change 47282. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I016062d04dd07950405d38f42e5e0ee01371c174
Gerrit-Change-Number: 47282
Gerrit-PatchSet: 1
Gerrit-Owner: Nicolas Boichat <drinkcat@chromium.org>
Gerrit-MessageType: newchange