Merge pull request #51 from mmaka1/dma-caps

dma: separated dma queries for host connections and hda/gp caps

Author: lgirdwood

src/audio/dai.c

@@ -197,6 +197,9 @@ static struct comp_dev *dai_new(struct sof_ipc_comp *comp)
 	}
 
 	/* request GP LP DMA with shared access privilege */
+	/* TODO: hda: retrieve req'ed caps from the dai,
+	 * dmas are not cross-compatible.
+	 */
 	dir = DMA_DIR_MEM_TO_DEV | DMA_DIR_DEV_TO_MEM;
 	caps = DMA_CAP_GP_LP | DMA_CAP_GP_HP;
 	dma_dev = DMA_DEV_SSP | DMA_DEV_DMIC;

src/audio/host.c

@@ -513,7 +513,7 @@ static struct comp_dev *host_new(struct sof_ipc_comp *comp)
 		dir =  DMA_DIR_LMEM_TO_HMEM;
 
 	caps = 0;
-	dma_dev = DMA_DEV_HDA;
+	dma_dev = DMA_DEV_HOST;
 	hd->dma = dma_get(dir, caps, dma_dev, DMA_ACCESS_SHARED);
 	if (hd->dma == NULL) {
 		trace_host_error("eDM");

src/include/sof/bit.h

@@ -0,0 +1,40 @@
+/*
+ * Copyright (c) 2018, Intel Corporation
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are met:
+ *   * Redistributions of source code must retain the above copyright
+ *     notice, this list of conditions and the following disclaimer.
+ *   * Redistributions in binary form must reproduce the above copyright
+ *     notice, this list of conditions and the following disclaimer in the
+ *     documentation and/or other materials provided with the distribution.
+ *   * Neither the name of the Intel Corporation nor the
+ *     names of its contributors may be used to endorse or promote products
+ *     derived from this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE
+ * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ * POSSIBILITY OF SUCH DAMAGE.
+ *
+ * Author: Liam Girdwood <liam.r.girdwood@linux.intel.com>
+ */
+
+#ifndef __INCLUDE_BIT__
+#define __INCLUDE_BIT__
+
+#define BIT(b)			(1 << (b))
+#define MASK(b_hi, b_lo)	((1 << ((b_hi) - (b_lo) + 1)) - 1)
+#define SET_BIT(b, x)		(((x) & 1) << (b))
+#define SET_BITS(b_hi, b_lo, x)	\
+	(((x) & ((1 << ((b_hi) - (b_lo) + 1)) - 1)) << (b_lo))
+
+#endif

src/include/sof/dma.h

@@ -44,6 +44,7 @@
 #include <sof/lock.h>
 #include <sof/sof.h>
 #include <sof/wait.h>
+#include <sof/bit.h>
 #include <arch/atomic.h>
 
 /** \addtogroup sof_dma_drivers DMA Drivers
@@ -52,29 +53,32 @@
  */
 
 /* DMA direction bitmasks used to define DMA copy direction */
-#define DMA_DIR_MEM_TO_MEM	(1 << 0) /* local memory copy */
-#define DMA_DIR_HMEM_TO_LMEM	(1 << 1) /* host memory to local mem copy */
-#define DMA_DIR_LMEM_TO_HMEM	(1 << 2) /* local mem to host mem copy */
-#define DMA_DIR_MEM_TO_DEV	(1 << 3) /* local mem to dev copy */
-#define DMA_DIR_DEV_TO_MEM	(1 << 4) /* dev to local mem copy */
-#define DMA_DIR_DEV_TO_DEV	(1 << 5) /* dev to dev copy */
+#define DMA_DIR_MEM_TO_MEM	BIT(0) /**< local memory copy */
+#define DMA_DIR_HMEM_TO_LMEM	BIT(1) /**< host memory to local mem copy */
+#define DMA_DIR_LMEM_TO_HMEM	BIT(2) /**< local mem to host mem copy */
+#define DMA_DIR_MEM_TO_DEV	BIT(3) /**< local mem to dev copy */
+#define DMA_DIR_DEV_TO_MEM	BIT(4) /**< dev to local mem copy */
+#define DMA_DIR_DEV_TO_DEV	BIT(5) /**< dev to dev copy */
 
 /* DMA capabilities bitmasks used to define the type of DMA */
-#define DMA_CAP_GP_LP		(1 << 0)
-#define DMA_CAP_GP_HP		(1 << 1)
+#define DMA_CAP_HDA		BIT(0) /**< HDA DMA */
+#define DMA_CAP_GP_LP		BIT(1) /**< GP LP DMA */
+#define DMA_CAP_GP_HP		BIT(2) /**< GP HP DMA */
 
 /* DMA dev type bitmasks used to define the type of DMA */
-#define DMA_DEV_HDA		(1 << 0)
-#define DMA_DEV_SSP		(1 << 1)
-#define DMA_DEV_DMIC		(1 << 2)
+
+#define DMA_DEV_HOST		BIT(0) /**< connectable to host */
+#define DMA_DEV_HDA		BIT(1) /**< connectable to HD/A link */
+#define DMA_DEV_SSP		BIT(2) /**< connectable to SSP fifo */
+#define DMA_DEV_DMIC		BIT(3) /**< connectable to DMIC fifo */
 
 /* DMA access privilege flag */
 #define DMA_ACCESS_EXCLUSIVE	1
 #define DMA_ACCESS_SHARED	0
 
 /* DMA IRQ types */
-#define DMA_IRQ_TYPE_BLOCK	(1 << 0)
-#define DMA_IRQ_TYPE_LLIST	(1 << 1)
+#define DMA_IRQ_TYPE_BLOCK	BIT(0)
+#define DMA_IRQ_TYPE_LLIST	BIT(1)
 
 
 /* We will use this macro in cb handler to inform dma that

src/include/sof/io.h

@@ -32,13 +32,7 @@
 #define __INCLUDE_IO__
 
 #include <stdint.h>
-
-/* Macros for register bits access */
-#define BIT(b)			(1 << (b))
-#define MASK(b_hi, b_lo)	((1 << ((b_hi) - (b_lo) + 1)) - 1)
-#define SET_BIT(b, x)		(((x) & 1) << (b))
-#define SET_BITS(b_hi, b_lo, x)	\
-	(((x) & ((1 << ((b_hi) - (b_lo) + 1)) - 1)) << (b_lo))
+#include <sof/bit.h>
 
 static inline uint32_t io_reg_read(uint32_t reg)
 {

src/ipc/apl-ipc.c

@@ -199,10 +199,10 @@ int platform_ipc_init(struct ipc *ipc)
 	if (iipc->page_table)
 		bzero(iipc->page_table, HOST_PAGE_SIZE);
 
-	/* request GP DMA with shared access privilege */
+	/* request HDA DMA with shared access privilege */
 	caps = 0;
 	dir = DMA_DIR_HMEM_TO_LMEM;
-	dev = DMA_DEV_HDA;
+	dev = DMA_DEV_HOST;
 	iipc->dmac = dma_get(dir, caps, dev, DMA_ACCESS_SHARED);
 
 	/* PM */

src/ipc/byt-ipc.c

@@ -228,7 +228,7 @@ int platform_ipc_init(struct ipc *ipc)
 	/* request HDA DMA with shared access privilege */
 	caps = 0;
 	dir = DMA_DIR_HMEM_TO_LMEM;
-	dev = DMA_DEV_HDA;
+	dev = DMA_DEV_HOST;
 	iipc->dmac = dma_get(dir, caps, dev, DMA_ACCESS_SHARED);
 
 	/* PM */

src/ipc/cnl-ipc.c

@@ -199,10 +199,10 @@ int platform_ipc_init(struct ipc *ipc)
 	if (iipc->page_table)
 		bzero(iipc->page_table, HOST_PAGE_SIZE);
 
-	/* request GP DMA with shared access privilege */
+	/* request HDA DMA with shared access privilege */
 	caps = 0;
 	dir = DMA_DIR_HMEM_TO_LMEM;
-	dev = DMA_DEV_HDA;
+	dev = DMA_DEV_HOST;
 	iipc->dmac = dma_get(dir, caps, dev, DMA_ACCESS_SHARED);
 
 	/* PM */

src/ipc/dma-copy.c

@@ -379,7 +379,7 @@ int dma_copy_new(struct dma_copy *dc)
 
 	/* request HDA DMA in the dir LMEM->HMEM with shared access */
 	dir = DMA_DIR_LMEM_TO_HMEM;
-	dev = DMA_DEV_HDA;
+	dev = DMA_DEV_HOST;
 	cap = 0;
 	dc->dmac = dma_get(dir, cap, dev, DMA_ACCESS_SHARED);
 	if (dc->dmac == NULL) {

src/ipc/hsw-ipc.c

@@ -224,7 +224,7 @@ int platform_ipc_init(struct ipc *ipc)
 	/* request GP DMA with shared access privilege */
 	caps = 0;
 	dir = DMA_DIR_HMEM_TO_LMEM;
-	dev = DMA_DEV_HDA;
+	dev = DMA_DEV_HOST;
 	iipc->dmac = dma_get(dir, caps, dev, DMA_ACCESS_SHARED);
 
 	/* PM */