[lkml]   [2022]   [Jul]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 1/5] media: mediatek: vcodec: Add encoder driver support for 34-bit iova
Hi Irui,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on media-tree/master]
[also build test WARNING on robh/for-next linus/master v5.19-rc6 next-20220715]
[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]

base: git:// master
config: powerpc-allmodconfig (
compiler: powerpc-linux-gcc (GCC) 12.1.0
reproduce (this is a W=1 build):
wget -O ~/bin/make.cross
chmod +x ~/bin/make.cross
git remote add linux-review
git fetch --no-tags linux-review Irui-Wang/support-mt8188-h264-encoder/20220716-235804
git checkout cfa5713276c701af0193ab5371d6678515cea42f
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=powerpc SHELL=/bin/bash drivers/media/platform/mediatek/vcodec/

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <>

All warnings (new ones prefixed by >>):

>> drivers/media/platform/mediatek/vcodec/venc/venc_h264_if.c:194: warning: expecting prototype for struct venc_h264_vsi_64. Prototype was for struct venc_h264_vsi_34 instead

vim +194 drivers/media/platform/mediatek/vcodec/venc/venc_h264_if.c

185 /**
186 * struct venc_h264_vsi_64 - Structure for VPU driver control and info share
187 * Used for 64 bit iova sharing
188 * @config: h264 encoder configuration
189 * @work_bufs: working buffer information in VPU side
190 */
191 struct venc_h264_vsi_34 {
192 struct venc_h264_vpu_config_ext config;
193 struct venc_h264_vpu_buf_34 work_bufs[VENC_H264_VPU_WORK_BUF_MAX];
> 194 };

0-DAY CI Kernel Test Service

 \ /
  Last update: 2022-07-16 23:30    [W:0.119 / U:0.384 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site