summaryrefslogtreecommitdiff
path: root/payloads
diff options
context:
space:
mode:
authorYu-Ping Wu <yupingso@chromium.org>2022-08-23 16:40:03 +0800
committerTim Wawrzynczak <twawrzynczak@chromium.org>2022-08-24 21:27:46 +0000
commit30d8e724e7b045a81815f07b5a568d4aa37461a9 (patch)
tree80e797d6252cc4f731b2b2a9cdfb7dce93fd7220 /payloads
parent403d22076ce2fe3d6a0c53e65f4cc6c528f73027 (diff)
downloadcoreboot-30d8e724e7b045a81815f07b5a568d4aa37461a9.tar.gz
libpayload: Add const for dma_coherent argument
Add the const modifier for the ptr argument of dma_coherent to avoid unnecessary type casting in payloads. BUG=none TEST=emerge-corsola libpayload BRANCH=none Change-Id: Ic4bb1d8318c7e83fd3ab3054aa0333cb27afe588 Signed-off-by: Yu-Ping Wu <yupingso@chromium.org> Reviewed-on: https://review.coreboot.org/c/coreboot/+/66961 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Hung-Te Lin <hungte@chromium.org> Reviewed-by: Hsin-Te Yuan <yuanhsinte@google.com>
Diffstat (limited to 'payloads')
-rw-r--r--payloads/libpayload/include/stdlib.h2
-rw-r--r--payloads/libpayload/libc/malloc.c2
2 files changed, 2 insertions, 2 deletions
diff --git a/payloads/libpayload/include/stdlib.h b/payloads/libpayload/include/stdlib.h
index 18f1dca8a4..ac386c1fa2 100644
--- a/payloads/libpayload/include/stdlib.h
+++ b/payloads/libpayload/include/stdlib.h
@@ -136,7 +136,7 @@ void print_malloc_map(void);
void init_dma_memory(void *start, u32 size);
int dma_initialized(void);
-int dma_coherent(void *ptr);
+int dma_coherent(const void *ptr);
static inline void *xmalloc_work(size_t size, const char *file,
const char *func, int line)
diff --git a/payloads/libpayload/libc/malloc.c b/payloads/libpayload/libc/malloc.c
index 20f9ef38e1..f94f33e5df 100644
--- a/payloads/libpayload/libc/malloc.c
+++ b/payloads/libpayload/libc/malloc.c
@@ -118,7 +118,7 @@ int dma_initialized()
}
/* For boards that don't initialize DMA we assume all locations are coherent */
-int dma_coherent(void *ptr)
+int dma_coherent(const void *ptr)
{
return !dma_initialized() || (dma->start <= ptr && dma->end > ptr);
}