On 09/03/17 21:17, Mauro Carvalho Chehab wrote:
> drivers/media/platform/coda/coda-common.c: In function ‘coda_alloc_aux_buf’:
> ./include/linux/kern_levels.h:4:18: warning: format ‘%u’ expects argument of type ‘unsigned int’, but argument 4 has type ‘size_t {aka long unsigned int}’ [-Wformat=]
> #define KERN_SOH "\001" /* ASCII Start Of Header */
> ^
> ./include/media/v4l2-common.h:69:9: note: in definition of macro ‘v4l2_printk’
> printk(level "%s: " fmt, (dev)->name , ## arg)
> ^~~~~
> ./include/linux/kern_levels.h:10:18: note: in expansion of macro ‘KERN_SOH’
> #define KERN_ERR KERN_SOH "3" /* error conditions */
> ^~~~~~~~
> ./include/media/v4l2-common.h:72:14: note: in expansion of macro ‘KERN_ERR’
> v4l2_printk(KERN_ERR, dev, fmt , ## arg)
> ^~~~~~~~
> drivers/media/platform/coda/coda-common.c:1341:3: note: in expansion of macro ‘v4l2_err’
> v4l2_err(&dev->v4l2_dev,
> ^~~~~~~~
Yes, this too is in my pull request for coda :-)
Regards,
Hans
>
> Signed-off-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxxx>
> ---
> drivers/media/platform/coda/coda-common.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/media/platform/coda/coda-common.c b/drivers/media/platform/coda/coda-common.c
> index cb76c96759b9..dc51ae2050cc 100644
> --- a/drivers/media/platform/coda/coda-common.c
> +++ b/drivers/media/platform/coda/coda-common.c
> @@ -1339,7 +1339,7 @@ int coda_alloc_aux_buf(struct coda_dev *dev, struct coda_aux_buf *buf,
> GFP_KERNEL);
> if (!buf->vaddr) {
> v4l2_err(&dev->v4l2_dev,
> - "Failed to allocate %s buffer of size %u\n",
> + "Failed to allocate %s buffer of size %zd\n",
> name, size);
> return -ENOMEM;
> }
>
