diff options
| author | Vinod Koul <vinod.koul@intel.com> | 2014-06-02 08:02:59 +0400 |
|---|---|---|
| committer | Vinod Koul <vinod.koul@intel.com> | 2014-06-03 10:05:24 +0400 |
| commit | 9d9f71a804314e7d50e2fa9e6e61bc77e2d6ae1c (patch) | |
| tree | 43a9be74ba0ef8676b0752da96e5c46ff5068b09 | |
| parent | a68765430772fac92ad4f0dd9e33d31b4276546e (diff) | |
| download | linux-9d9f71a804314e7d50e2fa9e6e61bc77e2d6ae1c.tar.xz | |
dmaengine: sh: fix print specifier warnings
As documented in Documentation/printk-formats.txt we should use %zu/%zx
specifiers for size_t type variables for the code to compile on different
architectures. This is uncovered as COMPILE_TEST has been enabled recently for
this driver
drivers/dma/sh/shdma-base.c: In function 'shdma_prep_dma_cyclic':
>> drivers/dma/sh/shdma-base.c:683:4: warning: format '%d' expects argument of
>> type 'int', but argument 4 has type 'size_t' [-Wformat=]
__func__, buf_len, period_len, slave_id);
>> drivers/dma/sh/shdma-base.c:683:4: warning: format '%d' expects argument of
>> type 'int', but argument 5 has type 'size_t' [-Wformat=]
Reported-by: kbuild test robot <fengguang.wu@intel.com>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
| -rw-r--r-- | drivers/dma/sh/shdma-base.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/dma/sh/shdma-base.c b/drivers/dma/sh/shdma-base.c index 66b4a35f7bd1..591b9d8cfd7f 100644 --- a/drivers/dma/sh/shdma-base.c +++ b/drivers/dma/sh/shdma-base.c @@ -679,7 +679,7 @@ static struct dma_async_tx_descriptor *shdma_prep_dma_cyclic( /* Someone calling slave DMA on a generic channel? */ if (slave_id < 0 || (buf_len < period_len)) { dev_warn(schan->dev, - "%s: bad parameter: buf_len=%d, period_len=%d, id=%d\n", + "%s: bad parameter: buf_len=%zu, period_len=%zu, id=%d\n", __func__, buf_len, period_len, slave_id); return NULL; } |