[Sound-open-firmware] [PATCH] Change type of size in trace_work() callback from uint32_t to int32_t.
Liam Girdwood
liam.r.girdwood at linux.intel.com
Wed Dec 20 10:21:15 CET 2017
On Wed, 2017-12-20 at 15:45 +0800, yan.wang at linux.intel.com wrote:
> 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;
>
Applied.
More information about the Sound-open-firmware
mailing list