Re: [PATCH] usb: xhci: add missing dma-mapping.h includes

From: Sarah Sharp
Date: Tue Jul 30 2013 - 16:41:35 EST


Thanks for the patch James! I'll send it off to Greg in a couple days.

Sarah Sharp

On Fri, Jul 26, 2013 at 01:34:43PM +0100, James Hogan wrote:
> A randconfig build hit the following build errors because xhci.c and
> xhci-mem.c use dma mapping functions but don't include
> <linux/dma-mapping.h>. Add the missing includes to fix the build errors.
>
> drivers/usb/host/xhci.c In function 'xhci_gen_setup':
> drivers/usb/host/xhci.c +4872 : error: implicit declaration of function 'dma_set_mask'
> drivers/usb/host/xhci.c +4872 : error: implicit declaration of function 'DMA_BIT_MASK'
>
> drivers/usb/host/xhci-mem.c In function 'xhci_free_stream_ctx':
> drivers/usb/host/xhci-mem.c +435 : error: implicit declaration of function 'dma_free_coherent'
> drivers/usb/host/xhci-mem.c In function 'xhci_alloc_stream_ctx':
> drivers/usb/host/xhci-mem.c +463 : error: implicit declaration of function 'dma_alloc_coherent'
>
> Signed-off-by: James Hogan <james.hogan@xxxxxxxxxx>
> Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> Cc: linux-usb@xxxxxxxxxxxxxxx
> Cc: Sarah Sharp <sarah.a.sharp@xxxxxxxxxxxxxxx>
> ---
> drivers/usb/host/xhci-mem.c | 1 +
> drivers/usb/host/xhci.c | 1 +
> 2 files changed, 2 insertions(+)
>
> diff --git a/drivers/usb/host/xhci-mem.c b/drivers/usb/host/xhci-mem.c
> index fbf75e5..3e27d92 100644
> --- a/drivers/usb/host/xhci-mem.c
> +++ b/drivers/usb/host/xhci-mem.c
> @@ -24,6 +24,7 @@
> #include <linux/pci.h>
> #include <linux/slab.h>
> #include <linux/dmapool.h>
> +#include <linux/dma-mapping.h>
>
> #include "xhci.h"
>
> diff --git a/drivers/usb/host/xhci.c b/drivers/usb/host/xhci.c
> index d8f640b..247020c 100644
> --- a/drivers/usb/host/xhci.c
> +++ b/drivers/usb/host/xhci.c
> @@ -27,6 +27,7 @@
> #include <linux/moduleparam.h>
> #include <linux/slab.h>
> #include <linux/dmi.h>
> +#include <linux/dma-mapping.h>
>
> #include "xhci.h"
>
> --
> 1.8.1.2
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/