Hi Chenyang, Thank you for the patch! Yet something to improve: [auto build test ERROR on drm-misc/drm-misc-next] [also build test ERROR on linus/master v5.19-rc3 next-20220624] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/intel-lab-lkp/linux/commits/Chenyang-Li/drm-loongson-Add-DRM-Driver-for-Loongson-7A1000-bridge-chip/20220625-171037 base: git://anongit.freedesktop.org/drm/drm-misc drm-misc-next config: powerpc-allmodconfig compiler: powerpc-linux-gcc (GCC) 11.3.0 reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # https://github.com/intel-lab-lkp/linux/commit/438d0791edb6352903bf09dfe214453526081075 git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Chenyang-Li/drm-loongson-Add-DRM-Driver-for-Loongson-7A1000-bridge-chip/20220625-171037 git checkout 438d0791edb6352903bf09dfe214453526081075 # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=powerpc SHELL=/bin/bash drivers/gpu/drm/loongson/ If you fix the issue, kindly add following tag where applicable Reported-by: kernel test robot <lkp@xxxxxxxxx> All error/warnings (new ones prefixed by >>): drivers/gpu/drm/loongson/loongson_crtc.c: In function 'loongson_crtc_mode_set_nofb': >> drivers/gpu/drm/loongson/loongson_crtc.c:128:42: error: invalid use of undefined type 'struct drm_framebuffer' 128 | format = crtc->primary->state->fb->format; | ^~ -- drivers/gpu/drm/loongson/loongson_device.c: In function 'loongson_gpu_offset': >> drivers/gpu/drm/loongson/loongson_device.c:14:44: error: invalid use of undefined type 'struct drm_framebuffer' 14 | gbo = drm_gem_vram_of_gem(state->fb->obj[0]); | ^~ -- In file included from include/linux/device.h:15, from include/linux/pci.h:37, from drivers/gpu/drm/loongson/loongson_drv.c:14: drivers/gpu/drm/loongson/loongson_drv.c: In function 'loongson_device_init': >> include/drm/drm_print.h:425:39: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 3 has type 'resource_size_t' {aka 'unsigned int'} [-Wformat=] 425 | dev_##level##type((drm)->dev, "[drm] " fmt, ##__VA_ARGS__) | ^~~~~~~~ include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap' 110 | _p_func(dev, fmt, ##__VA_ARGS__); \ | ^~~ include/linux/dev_printk.h:150:58: note: in expansion of macro 'dev_fmt' 150 | dev_printk_index_wrap(_dev_info, KERN_INFO, dev, dev_fmt(fmt), ##__VA_ARGS__) | ^~~~~~~ include/drm/drm_print.h:425:9: note: in expansion of macro 'dev_info' 425 | dev_##level##type((drm)->dev, "[drm] " fmt, ##__VA_ARGS__) | ^~~~ include/drm/drm_print.h:429:9: note: in expansion of macro '__drm_printk' 429 | __drm_printk((drm), info,, fmt, ##__VA_ARGS__) | ^~~~~~~~~~~~ drivers/gpu/drm/loongson/loongson_drv.c:91:9: note: in expansion of macro 'drm_info' 91 | drm_info(dev, "DC mmio base 0x%llx size 0x%llx io 0x%llx\n", | ^~~~~~~~ include/drm/drm_print.h:425:39: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'resource_size_t' {aka 'unsigned int'} [-Wformat=] 425 | dev_##level##type((drm)->dev, "[drm] " fmt, ##__VA_ARGS__) | ^~~~~~~~ include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap' 110 | _p_func(dev, fmt, ##__VA_ARGS__); \ | ^~~ include/linux/dev_printk.h:150:58: note: in expansion of macro 'dev_fmt' 150 | dev_printk_index_wrap(_dev_info, KERN_INFO, dev, dev_fmt(fmt), ##__VA_ARGS__) | ^~~~~~~ include/drm/drm_print.h:425:9: note: in expansion of macro 'dev_info' 425 | dev_##level##type((drm)->dev, "[drm] " fmt, ##__VA_ARGS__) | ^~~~ include/drm/drm_print.h:429:9: note: in expansion of macro '__drm_printk' 429 | __drm_printk((drm), info,, fmt, ##__VA_ARGS__) | ^~~~~~~~~~~~ drivers/gpu/drm/loongson/loongson_drv.c:91:9: note: in expansion of macro 'drm_info' 91 | drm_info(dev, "DC mmio base 0x%llx size 0x%llx io 0x%llx\n", | ^~~~~~~~ drivers/gpu/drm/loongson/loongson_drv.c: At top level: drivers/gpu/drm/loongson/loongson_drv.c:99:5: warning: no previous prototype for 'loongson_modeset_init' [-Wmissing-prototypes] 99 | int loongson_modeset_init(struct loongson_device *ldev) | ^~~~~~~~~~~~~~~~~~~~~ -- drivers/gpu/drm/loongson/loongson_plane.c: In function 'loongson_plane_atomic_update': >> drivers/gpu/drm/loongson/loongson_plane.c:27:27: error: invalid use of undefined type 'struct drm_framebuffer' 27 | pitch = lstate->fb->pitches[0]; | ^~ drivers/gpu/drm/loongson/loongson_plane.c:33:27: error: invalid use of undefined type 'struct drm_framebuffer' 33 | depth = lstate->fb->format->cpp[0] << 3; | ^~ vim +128 drivers/gpu/drm/loongson/loongson_crtc.c 102 103 static void loongson_crtc_mode_set_nofb(struct drm_crtc *crtc) 104 { 105 struct drm_device *dev = crtc->dev; 106 struct loongson_device *ldev = to_loongson_device(dev); 107 struct loongson_crtc *lcrtc = to_loongson_crtc(crtc); 108 struct drm_display_mode *mode = &crtc->state->adjusted_mode; 109 const struct drm_format_info *format; 110 struct pix_pll pll_cfg; 111 u32 hr, hss, hse, hfl; 112 u32 vr, vss, vse, vfl; 113 u32 pix_freq; 114 u32 reg_offset; 115 116 hr = mode->hdisplay; 117 hss = mode->hsync_start; 118 hse = mode->hsync_end; 119 hfl = mode->htotal; 120 121 vr = mode->vdisplay; 122 vss = mode->vsync_start; 123 vse = mode->vsync_end; 124 vfl = mode->vtotal; 125 126 pix_freq = mode->clock; 127 reg_offset = lcrtc->reg_offset; > 128 format = crtc->primary->state->fb->format; 129 130 ls7a_mm_wreg(ldev, FB_DITCFG_REG + reg_offset, 0); 131 ls7a_mm_wreg(ldev, FB_DITTAB_LO_REG + reg_offset, 0); 132 ls7a_mm_wreg(ldev, FB_DITTAB_HI_REG + reg_offset, 0); 133 ls7a_mm_wreg(ldev, FB_PANCFG_REG + reg_offset, FB_PANCFG_DEF); 134 ls7a_mm_wreg(ldev, FB_PANTIM_REG + reg_offset, 0); 135 136 ls7a_mm_wreg(ldev, FB_HDISPLAY_REG + reg_offset, (hfl << 16) | hr); 137 ls7a_mm_wreg(ldev, FB_HSYNC_REG + reg_offset, 138 FB_HSYNC_PULSE | (hse << 16) | hss); 139 140 ls7a_mm_wreg(ldev, FB_VDISPLAY_REG + reg_offset, (vfl << 16) | vr); 141 ls7a_mm_wreg(ldev, FB_VSYNC_REG + reg_offset, 142 FB_VSYNC_PULSE | (vse << 16) | vss); 143 144 switch (format->format) { 145 case DRM_FORMAT_RGB565: 146 lcrtc->cfg_reg |= 0x3; 147 break; 148 case DRM_FORMAT_RGB888: 149 case DRM_FORMAT_XRGB8888: 150 case DRM_FORMAT_ARGB8888: 151 default: 152 lcrtc->cfg_reg |= 0x4; 153 break; 154 } 155 156 ls7a_mm_wreg(ldev, FB_CFG_REG + reg_offset, lcrtc->cfg_reg); 157 158 cal_freq(pix_freq, &pll_cfg); 159 config_pll(ldev, LS7A_PIX_PLL + reg_offset, &pll_cfg); 160 } 161 -- 0-DAY CI Kernel Test Service https://01.org/lkp
