Change in coreboot[master]: WIP: mediatek/mt8192: spm: Align spm_firmware_binary to 16-byte
Nicolas Boichat has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/47282 ) Change subject: WIP: mediatek/mt8192: spm: Align spm_firmware_binary to 16-byte ...................................................................... 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 https://review.coreboot.org/c/coreboot/+/47282 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/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
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47282 ) Change subject: WIP: mediatek/mt8192: spm: Align spm_firmware_binary to 16-byte ...................................................................... Patch Set 1: (1 comment) https://review.coreboot.org/c/coreboot/+/47282/1/src/soc/mediatek/mt8192/spm... File src/soc/mediatek/mt8192/spm.c: https://review.coreboot.org/c/coreboot/+/47282/1/src/soc/mediatek/mt8192/spm... PS1, Line 13: * 16-byte alignement is critical, else the DMA transfer will corrupt data at 'alignement' may be misspelled - perhaps 'alignment'? -- To view, visit https://review.coreboot.org/c/coreboot/+/47282 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/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-CC: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-Comment-Date: Fri, 06 Nov 2020 06:18:26 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
Nicolas Boichat has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47282 ) Change subject: WIP: mediatek/mt8192: spm: Align spm_firmware_binary to 16-byte ...................................................................... Patch Set 1: Critical to ensure data is copied properly (SPM _may_ appear to work if the data corruption happens in some locations) -- To view, visit https://review.coreboot.org/c/coreboot/+/47282 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/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-Reviewer: Roger Lu <roger.lu@mediatek.com> Gerrit-Reviewer: Sj Huang <sj.huang@mediatek.com> Gerrit-Reviewer: Yidi Lin <yidi.lin@mediatek.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Hung-Te Lin <hungte@chromium.org> Gerrit-CC: Nicolas Boichat <drinkcat@google.com> Gerrit-CC: Yu-Ping Wu <yupingso@google.com> Gerrit-Comment-Date: Fri, 06 Nov 2020 06:22:17 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Nicolas Boichat has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/47282 ) Change subject: WIP: mediatek/mt8192: spm: Align spm_firmware_binary to 16-byte ...................................................................... Abandoned Also fixed by MTK. -- To view, visit https://review.coreboot.org/c/coreboot/+/47282 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/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-Reviewer: Roger Lu <roger.lu@mediatek.com> Gerrit-Reviewer: Sj Huang <sj.huang@mediatek.com> Gerrit-Reviewer: Yidi Lin <yidi.lin@mediatek.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Hung-Te Lin <hungte@chromium.org> Gerrit-CC: Nicolas Boichat <drinkcat@google.com> Gerrit-CC: Yu-Ping Wu <yupingso@google.com> Gerrit-MessageType: abandon
participants (2)
-
build bot (Jenkins) (Code Review) -
Nicolas Boichat (Code Review)