Re: [PATCH v5] vcodec: mediatek: Add g/s_selection support for V4L2 Encoder

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Tiffany,

[auto build test WARNING on linuxtv-media/master]
[also build test WARNING on v4.8-rc1 next-20160812]
[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/Tiffany-Lin/vcodec-mediatek-Add-g-s_selection-support-for-V4L2-Encoder/20160812-221617
base:   git://linuxtv.org/media_tree.git master
config: mips-allyesconfig (attached as .config)
compiler: mips-linux-gnu-gcc (Debian 5.4.0-6) 5.4.0 20160609
reproduce:
        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=mips 

All warnings (new ones prefixed by >>):

   drivers/media/platform/mtk-vcodec/mtk_vcodec_enc.c: In function 'vidioc_venc_s_selection':
>> drivers/media/platform/mtk-vcodec/mtk_vcodec_enc.c:673:6: warning: unused variable 'err' [-Wunused-variable]
     int err;
         ^

vim +/err +673 drivers/media/platform/mtk-vcodec/mtk_vcodec_enc.c

   657			s->r.left = 0;
   658			s->r.width = q_data->visible_width;
   659			s->r.height = q_data->visible_height;
   660			break;
   661		default:
   662			return -EINVAL;
   663		}
   664	
   665		return 0;
   666	}
   667	
   668	static int vidioc_venc_s_selection(struct file *file, void *priv,
   669					     struct v4l2_selection *s)
   670	{
   671		struct mtk_vcodec_ctx *ctx = fh_to_ctx(priv);
   672		struct mtk_q_data *q_data;
 > 673		int err;
   674	
   675		if (s->type != V4L2_BUF_TYPE_VIDEO_OUTPUT)
   676			return -EINVAL;
   677	
   678		q_data = mtk_venc_get_q_data(ctx, s->type);
   679		if (!q_data)
   680			return -EINVAL;
   681	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: Binary data


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux