Hi Nas, kernel test robot noticed the following build warnings: [auto build test WARNING on linuxtv-media-pending/master] [also build test WARNING on arm64/for-next/core robh/for-next linus/master v6.14-rc2 next-20250210] [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#_base_tree_information] url: https://github.com/intel-lab-lkp/linux/commits/Nas-Chung/media-platform-chips-media-wave5-Rename-Kconfig-parameter/20250210-171144 base: https://git.linuxtv.org/media-ci/media-pending.git master patch link: https://lore.kernel.org/r/20250210090725.4580-9-nas.chung%40chipsnmedia.com patch subject: [PATCH 8/8] media: chips-media: wave6: Improve debugging capabilities config: loongarch-allyesconfig (https://download.01.org/0day-ci/archive/20250212/202502120017.COEwtixF-lkp@xxxxxxxxx/config) compiler: loongarch64-linux-gcc (GCC) 14.2.0 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250212/202502120017.COEwtixF-lkp@xxxxxxxxx/reproduce) If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot <lkp@xxxxxxxxx> | Closes: https://lore.kernel.org/oe-kbuild-all/202502120017.COEwtixF-lkp@xxxxxxxxx/ All warnings (new ones prefixed by >>): drivers/media/platform/chips-media/wave6/wave6-vpu-dec.c: In function 'wave6_vpu_dec_handle_dst_buffer': >> drivers/media/platform/chips-media/wave6/wave6-vpu-dec.c:261:13: warning: variable 'buf_size' set but not used [-Wunused-but-set-variable] 261 | u32 buf_size; | ^~~~~~~~ -- drivers/media/platform/chips-media/wave6/wave6-hw.c: In function 'wave6_vpu_dec_flush': >> drivers/media/platform/chips-media/wave6/wave6-hw.c:1022:37: warning: variable 'fb' set but not used [-Wunused-but-set-variable] 1022 | struct frame_buffer fb; | ^~ vim +/buf_size +261 drivers/media/platform/chips-media/wave6/wave6-vpu-dec.c de204321b54004 Nas Chung 2025-02-10 254 de204321b54004 Nas Chung 2025-02-10 255 static void wave6_vpu_dec_handle_dst_buffer(struct vpu_instance *inst) de204321b54004 Nas Chung 2025-02-10 256 { de204321b54004 Nas Chung 2025-02-10 257 struct vb2_v4l2_buffer *dst_buf; de204321b54004 Nas Chung 2025-02-10 258 struct v4l2_m2m_buffer *v4l2_m2m_buf; de204321b54004 Nas Chung 2025-02-10 259 struct vpu_buffer *vpu_buf; de204321b54004 Nas Chung 2025-02-10 260 dma_addr_t buf_addr_y, buf_addr_cb, buf_addr_cr; de204321b54004 Nas Chung 2025-02-10 @261 u32 buf_size; de204321b54004 Nas Chung 2025-02-10 262 u32 fb_stride = inst->dst_fmt.plane_fmt[0].bytesperline; de204321b54004 Nas Chung 2025-02-10 263 u32 luma_size = fb_stride * inst->dst_fmt.height; de204321b54004 Nas Chung 2025-02-10 264 u32 chroma_size = (fb_stride / 2) * (inst->dst_fmt.height / 2); de204321b54004 Nas Chung 2025-02-10 265 struct frame_buffer disp_buffer = {0}; de204321b54004 Nas Chung 2025-02-10 266 struct dec_initial_info initial_info = {0}; de204321b54004 Nas Chung 2025-02-10 267 int consumed_num = wave6_vpu_get_consumed_fb_num(inst); de204321b54004 Nas Chung 2025-02-10 268 int ret; de204321b54004 Nas Chung 2025-02-10 269 de204321b54004 Nas Chung 2025-02-10 270 wave6_vpu_dec_give_command(inst, DEC_GET_SEQ_INFO, &initial_info); de204321b54004 Nas Chung 2025-02-10 271 de204321b54004 Nas Chung 2025-02-10 272 v4l2_m2m_for_each_dst_buf(inst->v4l2_fh.m2m_ctx, v4l2_m2m_buf) { de204321b54004 Nas Chung 2025-02-10 273 dst_buf = &v4l2_m2m_buf->vb; de204321b54004 Nas Chung 2025-02-10 274 vpu_buf = wave6_to_vpu_buf(dst_buf); de204321b54004 Nas Chung 2025-02-10 275 de204321b54004 Nas Chung 2025-02-10 276 if (vpu_buf->consumed) de204321b54004 Nas Chung 2025-02-10 277 continue; de204321b54004 Nas Chung 2025-02-10 278 de204321b54004 Nas Chung 2025-02-10 279 if (consumed_num >= WAVE6_MAX_FBS) de204321b54004 Nas Chung 2025-02-10 280 break; de204321b54004 Nas Chung 2025-02-10 281 de204321b54004 Nas Chung 2025-02-10 282 if (inst->dst_fmt.num_planes == 1) { de204321b54004 Nas Chung 2025-02-10 283 buf_size = vb2_plane_size(&dst_buf->vb2_buf, 0); de204321b54004 Nas Chung 2025-02-10 284 buf_addr_y = wave6_get_dma_addr(dst_buf, 0); de204321b54004 Nas Chung 2025-02-10 285 buf_addr_cb = buf_addr_y + luma_size; de204321b54004 Nas Chung 2025-02-10 286 buf_addr_cr = buf_addr_cb + chroma_size; de204321b54004 Nas Chung 2025-02-10 287 } else if (inst->dst_fmt.num_planes == 2) { de204321b54004 Nas Chung 2025-02-10 288 buf_size = vb2_plane_size(&dst_buf->vb2_buf, 0) + de204321b54004 Nas Chung 2025-02-10 289 vb2_plane_size(&dst_buf->vb2_buf, 1); de204321b54004 Nas Chung 2025-02-10 290 buf_addr_y = wave6_get_dma_addr(dst_buf, 0); de204321b54004 Nas Chung 2025-02-10 291 buf_addr_cb = wave6_get_dma_addr(dst_buf, 1); de204321b54004 Nas Chung 2025-02-10 292 buf_addr_cr = buf_addr_cb + chroma_size; de204321b54004 Nas Chung 2025-02-10 293 } else if (inst->dst_fmt.num_planes == 3) { de204321b54004 Nas Chung 2025-02-10 294 buf_size = vb2_plane_size(&dst_buf->vb2_buf, 0) + de204321b54004 Nas Chung 2025-02-10 295 vb2_plane_size(&dst_buf->vb2_buf, 1) + de204321b54004 Nas Chung 2025-02-10 296 vb2_plane_size(&dst_buf->vb2_buf, 2); de204321b54004 Nas Chung 2025-02-10 297 buf_addr_y = wave6_get_dma_addr(dst_buf, 0); de204321b54004 Nas Chung 2025-02-10 298 buf_addr_cb = wave6_get_dma_addr(dst_buf, 1); de204321b54004 Nas Chung 2025-02-10 299 buf_addr_cr = wave6_get_dma_addr(dst_buf, 2); de204321b54004 Nas Chung 2025-02-10 300 } de204321b54004 Nas Chung 2025-02-10 301 disp_buffer.buf_y = buf_addr_y; de204321b54004 Nas Chung 2025-02-10 302 disp_buffer.buf_cb = buf_addr_cb; de204321b54004 Nas Chung 2025-02-10 303 disp_buffer.buf_cr = buf_addr_cr; de204321b54004 Nas Chung 2025-02-10 304 disp_buffer.width = inst->src_fmt.width; de204321b54004 Nas Chung 2025-02-10 305 disp_buffer.height = inst->src_fmt.height; de204321b54004 Nas Chung 2025-02-10 306 disp_buffer.stride = fb_stride; de204321b54004 Nas Chung 2025-02-10 307 disp_buffer.map_type = LINEAR_FRAME_MAP; de204321b54004 Nas Chung 2025-02-10 308 disp_buffer.luma_bitdepth = initial_info.luma_bitdepth; de204321b54004 Nas Chung 2025-02-10 309 disp_buffer.chroma_bitdepth = initial_info.chroma_bitdepth; de204321b54004 Nas Chung 2025-02-10 310 disp_buffer.chroma_format_idc = initial_info.chroma_format_idc; de204321b54004 Nas Chung 2025-02-10 311 de204321b54004 Nas Chung 2025-02-10 312 ret = wave6_vpu_dec_register_display_buffer_ex(inst, disp_buffer); de204321b54004 Nas Chung 2025-02-10 313 if (ret) { de204321b54004 Nas Chung 2025-02-10 314 dev_err(inst->dev->dev, "fail register display buffer %d", ret); de204321b54004 Nas Chung 2025-02-10 315 break; de204321b54004 Nas Chung 2025-02-10 316 } de204321b54004 Nas Chung 2025-02-10 317 de204321b54004 Nas Chung 2025-02-10 318 vpu_buf->consumed = true; de204321b54004 Nas Chung 2025-02-10 319 consumed_num++; de204321b54004 Nas Chung 2025-02-10 320 } de204321b54004 Nas Chung 2025-02-10 321 } de204321b54004 Nas Chung 2025-02-10 322 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki
