Re: [PATCH v2 1/2] staging: Introduce NVIDIA Tegra20 video decoder driver

[kbuild test robot]

Hi Dmitry,

[auto build test WARNING on staging/staging-testing]
[also build test WARNING on v4.14-rc3 next-20170929]
[cannot apply to tegra/for-next]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Dmitry-Osipenko/staging-Introduce-NVIDIA-Tegra20-video-decoder-driver/20171006-101015
config: blackfin-allyesconfig (attached as .config)
compiler: bfin-uclinux-gcc (GCC) 6.2.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=blackfin 

All warnings (new ones prefixed by >>):

   In file included from include/linux/kernel.h:13:0,
                    from include/linux/clk.h:16,
                    from drivers/staging/tegra-vde/vde.c:11:
   drivers/staging/tegra-vde/vde.c: In function 'tegra_vde_setup_hw_context':
>> drivers/staging/tegra-vde/vde.c:51:11: warning: format '%X' expects argument of type 'unsigned int', but argument 5 has type 'long unsigned int' [-Wformat=]
     pr_debug("%s: %d: 0x%08X => " #__addr ")\n", \
              ^
   include/linux/printk.h:285:21: note: in definition of macro 'pr_fmt'
    #define pr_fmt(fmt) fmt
                        ^~~
   include/linux/printk.h:333:2: note: in expansion of macro 'dynamic_pr_debug'
     dynamic_pr_debug(fmt, ##__VA_ARGS__)
     ^~~~~~~~~~~~~~~~
>> drivers/staging/tegra-vde/vde.c:51:2: note: in expansion of macro 'pr_debug'
     pr_debug("%s: %d: 0x%08X => " #__addr ")\n", \
     ^~~~~~~~
>> drivers/staging/tegra-vde/vde.c:362:2: note: in expansion of macro 'VDE_WR'
     VDE_WR(bitstream_data_paddr + bitstream_data_size,
     ^~~~~~
   drivers/staging/tegra-vde/vde.c: In function 'tegra_vde_attach_dmabuf':
>> drivers/staging/tegra-vde/vde.c:531:40: warning: format '%d' expects argument of type 'int', but argument 3 has type 'size_t {aka long unsigned int}' [-Wformat=]
      dev_err(dev, "Too small dmabuf size %d @0x%lX, "
                                           ^

vim +51 drivers/staging/tegra-vde/vde.c

  > 11	#include <linux/clk.h>
    12	#include <linux/delay.h>
    13	#include <linux/dma-buf.h>
    14	#include <linux/interrupt.h>
    15	#include <linux/io.h>
    16	#include <linux/iopoll.h>
    17	#include <linux/miscdevice.h>
    18	#include <linux/module.h>
    19	#include <linux/platform_device.h>
    20	#include <linux/pm_runtime.h>
    21	#include <linux/reset.h>
    22	#include <linux/slab.h>
    23	#include <linux/uaccess.h>
    24	
    25	#include <soc/tegra/pmc.h>
    26	
    27	#include "uapi.h"
    28	
    29	#define SXE(offt)		(0x0000 + (offt)) /* Syntax Engine */
    30	#define BSEV(offt)		(0x1000 + (offt)) /* Video Bitstream Engine */
    31	#define MBE(offt)		(0x2000 + (offt)) /* Macroblock Engine */
    32	#define PPE(offt)		(0x2200 + (offt)) /* Post-processing Engine */
    33	#define MCE(offt)		(0x2400 + (offt)) /* Motion Compensation Eng. */
    34	#define TFE(offt)		(0x2600 + (offt)) /* Transform Engine */
    35	#define VDMA(offt)		(0x2A00 + (offt)) /* Video DMA */
    36	#define FRAMEID(offt)		(0x3800 + (offt))
    37	
    38	#define ICMDQUE_WR		0x00
    39	#define CMDQUE_CONTROL		0x08
    40	#define INTR_STATUS		0x18
    41	#define BSE_INT_ENB		0x40
    42	#define BSE_CONFIG		0x44
    43	
    44	#define BSE_ICMDQUE_EMPTY	BIT(3)
    45	#define BSE_DMA_BUSY		BIT(23)
    46	
    47	#define TEGRA_VDE_TIMEOUT	msecs_to_jiffies(1000)
    48	
    49	#define VDE_WR(__data, __addr)				\
    50	do {							\
  > 51		pr_debug("%s: %d: 0x%08X => " #__addr ")\n",	\
    52			  __func__, __LINE__, (__data));	\
    53		writel_relaxed(__data, __addr);			\
    54	} while (0)
    55	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip