[Sound-open-firmware] [PATCH] Change type of size in trace_work() callback from uint32_t to int32_t.

yan.wang at linux.intel.com yan.wang at linux.intel.com
Wed Dec 20 08:45:27 CET 2017


From: Yan Wang <yan.wang at linux.intel.com>

This variable is set by the return value of dma_copy_to_host_nowait().
Unsigned type will mislead error checking.

Signed-off-by: Yan Wang <yan.wang at linux.intel.com>
---
 src/lib/dma-trace.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/lib/dma-trace.c b/src/lib/dma-trace.c
index ea63ab4..1fbd050 100644
--- a/src/lib/dma-trace.c
+++ b/src/lib/dma-trace.c
@@ -48,7 +48,7 @@ static uint64_t trace_work(void *data, uint64_t delay)
 	struct dma_sg_config *config = &d->config;
 	unsigned long flags;
 	uint32_t avail = buffer->avail;
-	uint32_t size;
+	int32_t size;
 	uint32_t hsize;
 	uint32_t lsize;
 
-- 
2.7.4



More information about the Sound-open-firmware mailing list