Hi Ong, Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on drm/drm-next] [also build test WARNING on v4.17-rc7 next-20180531] [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/Hean-Loong-Ong/ARM-drm-ivip-Intel-FPGA-Video-and-Image-Processing-Suite/20180601-132429 base: git://people.freedesktop.org/~airlied/linux.git drm-next config: sparc64-allyesconfig (attached as .config) compiler: sparc64-linux-gnu-gcc (Debian 7.2.0-11) 7.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=sparc64 All warnings (new ones prefixed by >>): In file included from include/linux/cdev.h:8:0, from include/drm/drmP.h:36, from drivers/gpu//drm/ivip/intel_vip_core.c:24: drivers/gpu//drm/ivip/intel_vip_core.c: In function 'intelvipfb_enable': >> drivers/gpu//drm/ivip/intel_vip_core.c:58:33: warning: format '%x' expects >> argument of type 'unsigned int', but argument 3 has type 'dma_addr_t {aka >> long long unsigned int}' [-Wformat=] dev_info(pipe->plane.dev->dev, "Address 0x%x\n", addr); ^ include/linux/device.h:1382:51: note: in definition of macro 'dev_info' #define dev_info(dev, fmt, arg...) _dev_info(dev, fmt, ##arg) ^~~ vim +58 drivers/gpu//drm/ivip/intel_vip_core.c > 24 #include <drm/drmP.h> 25 #include <drm/drm_atomic.h> 26 #include <drm/drm_atomic_helper.h> 27 #include <drm/drm_crtc_helper.h> 28 #include <drm/drm_fb_helper.h> 29 #include <drm/drm_fb_cma_helper.h> 30 #include <drm/drm_gem_cma_helper.h> 31 #include <drm/drm_plane_helper.h> 32 #include <drm/drm_simple_kms_helper.h> 33 #include <drm/drm_gem_framebuffer_helper.h> 34 35 #include <linux/init.h> 36 #include <linux/kernel.h> 37 #include <linux/module.h> 38 39 #include "intel_vip_drv.h" 40 41 static void intelvipfb_enable(struct drm_simple_display_pipe *pipe, 42 struct drm_crtc_state *crtc_state) 43 { 44 /* 45 * The frameinfo variable has to correspond to the size of the VIP Suite 46 * Frame Reader register 7 which will determine the maximum size used 47 * in this frameinfo 48 */ 49 50 u32 frameinfo; 51 struct intelvipfb_priv *priv = pipe->plane.dev->dev_private; 52 void __iomem *base = priv->base; 53 struct drm_plane_state *state = pipe->plane.state; 54 dma_addr_t addr; 55 56 addr = drm_fb_cma_get_gem_addr(state->fb, state, 0); 57 > 58 dev_info(pipe->plane.dev->dev, "Address 0x%x\n", addr); 59 60 frameinfo = 61 readl(base + INTELVIPFB_FRAME_READER) & 0x00ffffff; 62 writel(frameinfo, base + INTELVIPFB_FRAME_INFO); 63 writel(addr, base + INTELVIPFB_FRAME_START); 64 /* Finally set the control register to 1 to start streaming */ 65 writel(1, base + INTELVIPFB_CONTROL); 66 } 67 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation
.config.gz
Description: application/gzip
_______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
