From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Osipenko Subject: Re: [PATCH v2 1/2] staging: Introduce NVIDIA Tegra20 video decoder driver Date: Fri, 6 Oct 2017 13:46:42 +0300 Message-ID: References: <201710061105.oOGaoiDu%fengguang.wu@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <201710061105.oOGaoiDu%fengguang.wu@intel.com> Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: driverdev-devel-bounces@linuxdriverproject.org Sender: "devel" To: kbuild test robot Cc: devel@driverdev.osuosl.org, devicetree@vger.kernel.org, Stephen Warren , Greg Kroah-Hartman , linux-kernel@vger.kernel.org, Rob Herring , Jonathan Hunter , Thierry Reding , kbuild-all@01.org, linux-tegra@vger.kernel.org, Mauro Carvalho Chehab , Dan Carpenter , linux-media@vger.kernel.org List-Id: devicetree@vger.kernel.org On 06.10.2017 06:57, kbuild test robot wrote: > 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: ia64-allmodconfig (attached as .config) > compiler: ia64-linux-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=ia64 > > 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 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:51:11: warning: format '%X' expects argument of type 'unsigned int', but argument 5 has type 'phys_addr_t {aka long 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:435:2: note: in expansion of macro 'VDE_WR' > VDE_WR(bitstream_data_paddr, vde->regs + SXE(0x6C)); > ^~~~~~ > 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, " > ^ > drivers/staging/tegra-vde/vde.c: In function 'tegra_vde_ioctl_decode_h264': > drivers/staging/tegra-vde/vde.c:855:16: warning: format '%X' expects argument of type 'unsigned int', but argument 3 has type 'phys_addr_t {aka long long unsigned int}' [-Wformat=] > dev_err(dev, "Decoding failed, " > ^~~~~~~~~~~~~~~~~~~ > > vim +51 drivers/staging/tegra-vde/vde.c > > > 11 #include > 12 #include > 13 #include > 14 #include > 15 #include > 16 #include > 17 #include > 18 #include > 19 #include > 20 #include > 21 #include > 22 #include > 23 #include > 24 > 25 #include > 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 > I'll silence these warnings in v3.