[coreboot-gerrit] New patch to review for coreboot: arch/arm64: add DMA_COHERENT region macros to memlayout

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Tue Dec 1 19:57:16 CET 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/12584

-gerrit

commit 6abde733d29ea137f8557c5a22647bbf71e7060f
Author: Jimmy Huang <jimmy.huang at mediatek.com>
Date:   Tue Jun 16 12:18:50 2015 +0800

    arch/arm64: add DMA_COHERENT region macros to memlayout
    
    BRANCH=none
    BUG=none
    TEST=build pass
    
    Change-Id: Ia997ce97ad42234ab020af7bd007d57d7191ee86
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 604ac738e33fdfbaf093989ea13162c8506b9360
    Original-Change-Id: I636a1a38d0f5af97926d4446f3edb91a359cce4c
    Original-Signed-off-by: Jimmy Huang <jimmy.huang at mediatek.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/292551
    Original-Commit-Ready: Yidi Lin <yidi.lin at mediatek.com>
    Original-Tested-by: Yidi Lin <yidi.lin at mediatek.com>
    Original-Reviewed-by: Julius Werner <jwerner at chromium.org>
---
 src/arch/arm64/include/arch/memlayout.h | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/arch/arm64/include/arch/memlayout.h b/src/arch/arm64/include/arch/memlayout.h
index 0bd0835..ac09934 100644
--- a/src/arch/arm64/include/arch/memlayout.h
+++ b/src/arch/arm64/include/arch/memlayout.h
@@ -22,6 +22,11 @@
 	REGION(ttb, addr, size, 4K) \
 	_ = ASSERT(size % 4K == 0, "TTB size must be divisible by 4K!");
 
+#define DMA_COHERENT(addr, size) \
+	REGION(dma_coherent, addr, size, 4K) \
+	_ = ASSERT(size % 4K == 0, \
+		"DMA buffer should be multiple of smallest page size (4K)!");
+
 /* ARM64 stacks need 16-byte alignment. */
 #define STACK(addr, size) \
 	REGION(stack, addr, size, 16) \



More information about the coreboot-gerrit mailing list