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

On Wed, 2017-12-20 at 15:45 +0800, yan.wang@linux.intel.com wrote:

From: Yan Wang yan.wang@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@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;

Applied.