linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] cma_heap: fix implicit function declaration
@ 2020-12-17 12:30 siyanteng01
  2020-12-17 19:40 ` John Stultz
  0 siblings, 1 reply; 2+ messages in thread
From: siyanteng01 @ 2020-12-17 12:30 UTC (permalink / raw)
  To: Sumit Semwal
  Cc: Benjamin Gaignard, Liam Mark, Laura Abbott, Brian Starkey,
	John Stultz, Christian König, linux-media, dri-devel,
	linaro-mm-sig, linux-kernel, siyanteng

From: siyanteng <siyanteng01@gmail.com>

When building cma_heap the following error shows up:

drivers/dma-buf/heaps/cma_heap.c:195:10: error: implicit declaration of function 'vmap'; did you mean 'kmap'? [-Werror=implicit-function-declaration]
195 |  vaddr = vmap(buffer->pages, buffer->pagecount, VM_MAP, PAGE_KERNEL);
    |          ^~~~
    |          kmap

Use this include: linux-next/include/linux/vmalloc.h

Signed-off-by: siyanteng <siyanteng01@gmail.com>
---
 drivers/dma-buf/heaps/cma_heap.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/dma-buf/heaps/cma_heap.c b/drivers/dma-buf/heaps/cma_heap.c
index 5e7c3436310c..3c4e34301172 100644
--- a/drivers/dma-buf/heaps/cma_heap.c
+++ b/drivers/dma-buf/heaps/cma_heap.c
@@ -20,6 +20,7 @@
 #include <linux/module.h>
 #include <linux/scatterlist.h>
 #include <linux/slab.h>
+#include <linux/vmalloc.h>
 
 
 struct cma_heap {
-- 
2.27.0


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] cma_heap: fix implicit function declaration
  2020-12-17 12:30 [PATCH] cma_heap: fix implicit function declaration siyanteng01
@ 2020-12-17 19:40 ` John Stultz
  0 siblings, 0 replies; 2+ messages in thread
From: John Stultz @ 2020-12-17 19:40 UTC (permalink / raw)
  To: siyanteng01
  Cc: Sumit Semwal, Benjamin Gaignard, Liam Mark, Laura Abbott,
	Brian Starkey, Christian König, linux-media, dri-devel,
	moderated list:DMA BUFFER SHARING FRAMEWORK, lkml

On Thu, Dec 17, 2020 at 4:31 AM <siyanteng01@gmail.com> wrote:
>
> From: siyanteng <siyanteng01@gmail.com>
>
> When building cma_heap the following error shows up:
>
> drivers/dma-buf/heaps/cma_heap.c:195:10: error: implicit declaration of function 'vmap'; did you mean 'kmap'? [-Werror=implicit-function-declaration]
> 195 |  vaddr = vmap(buffer->pages, buffer->pagecount, VM_MAP, PAGE_KERNEL);
>     |          ^~~~
>     |          kmap
>
> Use this include: linux-next/include/linux/vmalloc.h
>
> Signed-off-by: siyanteng <siyanteng01@gmail.com>

Thanks for submitting this! My apologies for the trouble!

We already have a similar patch queued here:
  https://cgit.freedesktop.org/drm/drm-misc/commit/?h=drm-misc-next-fixes&id=8075c3005e4b1efa12dbbf6e84bc412a713de92c
so hopefully that will land upstream soon.

thanks again!
-john

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-12-17 19:41 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-17 12:30 [PATCH] cma_heap: fix implicit function declaration siyanteng01
2020-12-17 19:40 ` John Stultz

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).