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