From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH v8 08/15] media: amphion: add vpu v4l2 m2m support
Date: Wed, 08 Sep 2021 04:08:20 +0800 [thread overview]
Message-ID: <202109080336.c8r1bAhj-lkp@intel.com> (raw)
In-Reply-To: <3a9b7f137756571db1c7d217c21446a0b32133a0.1631002447.git.ming.qian@nxp.com>
[-- Attachment #1: Type: text/plain, Size: 9034 bytes --]
Hi Ming,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on 9c3a0f285248899dfa81585bc5d5bc9ebdb8fead]
url: https://github.com/0day-ci/linux/commits/Ming-Qian/amphion-video-decoder-encoder-driver/20210907-175342
base: 9c3a0f285248899dfa81585bc5d5bc9ebdb8fead
config: arm-defconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 11.2.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/0day-ci/linux/commit/fc6ae31812624f861c29b263d4215f9d059ddcb6
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Ming-Qian/amphion-video-decoder-encoder-driver/20210907-175342
git checkout fc6ae31812624f861c29b263d4215f9d059ddcb6
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=arm
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/media/platform/amphion/vpu_msgs.c: In function 'vpu_handle_msg':
>> drivers/media/platform/amphion/vpu_msgs.c:314:1: warning: the frame size of 1040 bytes is larger than 1024 bytes [-Wframe-larger-than=]
314 | }
| ^
drivers/media/platform/amphion/vpu_msgs.c: In function 'vpu_inst_run_work':
drivers/media/platform/amphion/vpu_msgs.c:265:1: warning: the frame size of 1040 bytes is larger than 1024 bytes [-Wframe-larger-than=]
265 | }
| ^
--
drivers/media/platform/amphion/vpu_dbg.c: In function 'vpu_dbg_instance':
>> drivers/media/platform/amphion/vpu_dbg.c:169:83: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 6 has type 'dma_addr_t' {aka 'unsigned int'} [-Wformat=]
169 | num = scnprintf(str, sizeof(str), "stream_buffer = %d / %d, [0x%llx, 0x%llx]\n",
| ~~~^
| |
| long long unsigned int
| %x
......
172 | inst->stream_buffer.phys,
| ~~~~~~~~~~~~~~~~~~~~~~~~
| |
| dma_addr_t {aka unsigned int}
drivers/media/platform/amphion/vpu_dbg.c:169:91: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 7 has type 'dma_addr_t' {aka 'unsigned int'} [-Wformat=]
169 | num = scnprintf(str, sizeof(str), "stream_buffer = %d / %d, [0x%llx, 0x%llx]\n",
| ~~~^
| |
| long long unsigned int
| %x
......
173 | inst->stream_buffer.phys + inst->stream_buffer.length);
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| |
| dma_addr_t {aka unsigned int}
drivers/media/platform/amphion/vpu_dbg.c: In function 'vpu_dbg_core':
drivers/media/platform/amphion/vpu_dbg.c:224:67: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'dma_addr_t' {aka 'unsigned int'} [-Wformat=]
224 | num = scnprintf(str, sizeof(str), "boot_region = <0x%08llx, 0x%x>\n",
| ~~~~~^
| |
| long long unsigned int
| %08x
225 | core->fw.phys, core->fw.length);
| ~~~~~~~~~~~~~
| |
| dma_addr_t {aka unsigned int}
drivers/media/platform/amphion/vpu_dbg.c:228:67: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'dma_addr_t' {aka 'unsigned int'} [-Wformat=]
228 | num = scnprintf(str, sizeof(str), "rpc_region = <0x%08llx, 0x%x> used = 0x%x\n",
| ~~~~~^
| |
| long long unsigned int
| %08x
229 | core->rpc.phys, core->rpc.length, core->rpc.bytesused);
| ~~~~~~~~~~~~~~
| |
| dma_addr_t {aka unsigned int}
drivers/media/platform/amphion/vpu_dbg.c:232:67: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'dma_addr_t' {aka 'unsigned int'} [-Wformat=]
232 | num = scnprintf(str, sizeof(str), "fwlog_region = <0x%08llx, 0x%x>\n",
| ~~~~~^
| |
| long long unsigned int
| %08x
233 | core->log.phys, core->log.length);
| ~~~~~~~~~~~~~~
| |
| dma_addr_t {aka unsigned int}
vim +314 drivers/media/platform/amphion/vpu_msgs.c
f1cf0764ca594c8 Ming Qian 2021-09-07 284
f1cf0764ca594c8 Ming Qian 2021-09-07 285 static int vpu_handle_msg(struct vpu_core *core)
f1cf0764ca594c8 Ming Qian 2021-09-07 286 {
f1cf0764ca594c8 Ming Qian 2021-09-07 287 struct vpu_rpc_event pkt;
f1cf0764ca594c8 Ming Qian 2021-09-07 288 struct vpu_inst *inst;
f1cf0764ca594c8 Ming Qian 2021-09-07 289 int ret;
f1cf0764ca594c8 Ming Qian 2021-09-07 290
f1cf0764ca594c8 Ming Qian 2021-09-07 291 memset(&pkt, 0, sizeof(pkt));
f1cf0764ca594c8 Ming Qian 2021-09-07 292 while (!vpu_iface_receive_msg(core, &pkt)) {
f1cf0764ca594c8 Ming Qian 2021-09-07 293 vpu_dbg(LVL_DEBUG, "event index = %d, id = %d, num = %d\n",
f1cf0764ca594c8 Ming Qian 2021-09-07 294 pkt.hdr.index, pkt.hdr.id, pkt.hdr.num);
f1cf0764ca594c8 Ming Qian 2021-09-07 295
f1cf0764ca594c8 Ming Qian 2021-09-07 296 ret = vpu_iface_convert_msg_id(core, pkt.hdr.id);
f1cf0764ca594c8 Ming Qian 2021-09-07 297 if (ret < 0)
f1cf0764ca594c8 Ming Qian 2021-09-07 298 continue;
f1cf0764ca594c8 Ming Qian 2021-09-07 299
f1cf0764ca594c8 Ming Qian 2021-09-07 300 inst = vpu_core_find_instance(core, pkt.hdr.index);
f1cf0764ca594c8 Ming Qian 2021-09-07 301 if (inst) {
f1cf0764ca594c8 Ming Qian 2021-09-07 302 vpu_response_cmd(inst, ret, 0);
f1cf0764ca594c8 Ming Qian 2021-09-07 303 mutex_lock(&core->cmd_lock);
f1cf0764ca594c8 Ming Qian 2021-09-07 304 vpu_inst_record_flow(inst, ret);
f1cf0764ca594c8 Ming Qian 2021-09-07 305 mutex_unlock(&core->cmd_lock);
f1cf0764ca594c8 Ming Qian 2021-09-07 306
f1cf0764ca594c8 Ming Qian 2021-09-07 307 vpu_inst_handle_msg(inst, &pkt);
f1cf0764ca594c8 Ming Qian 2021-09-07 308 vpu_inst_put(inst);
f1cf0764ca594c8 Ming Qian 2021-09-07 309 }
f1cf0764ca594c8 Ming Qian 2021-09-07 310 memset(&pkt, 0, sizeof(pkt));
f1cf0764ca594c8 Ming Qian 2021-09-07 311 }
f1cf0764ca594c8 Ming Qian 2021-09-07 312
f1cf0764ca594c8 Ming Qian 2021-09-07 313 return 0;
f1cf0764ca594c8 Ming Qian 2021-09-07 @314 }
f1cf0764ca594c8 Ming Qian 2021-09-07 315
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 54075 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: Ming Qian <ming.qian@nxp.com>,
mchehab@kernel.org, shawnguo@kernel.org, robh+dt@kernel.org,
s.hauer@pengutronix.de
Cc: kbuild-all@lists.01.org, hverkuil-cisco@xs4all.nl,
kernel@pengutronix.de, festevam@gmail.com, linux-imx@nxp.com,
aisheng.dong@nxp.com, linux-media@vger.kernel.org
Subject: Re: [PATCH v8 08/15] media: amphion: add vpu v4l2 m2m support
Date: Wed, 8 Sep 2021 04:08:20 +0800 [thread overview]
Message-ID: <202109080336.c8r1bAhj-lkp@intel.com> (raw)
In-Reply-To: <3a9b7f137756571db1c7d217c21446a0b32133a0.1631002447.git.ming.qian@nxp.com>
[-- Attachment #1: Type: text/plain, Size: 8904 bytes --]
Hi Ming,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on 9c3a0f285248899dfa81585bc5d5bc9ebdb8fead]
url: https://github.com/0day-ci/linux/commits/Ming-Qian/amphion-video-decoder-encoder-driver/20210907-175342
base: 9c3a0f285248899dfa81585bc5d5bc9ebdb8fead
config: arm-defconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 11.2.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/0day-ci/linux/commit/fc6ae31812624f861c29b263d4215f9d059ddcb6
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Ming-Qian/amphion-video-decoder-encoder-driver/20210907-175342
git checkout fc6ae31812624f861c29b263d4215f9d059ddcb6
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=arm
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/media/platform/amphion/vpu_msgs.c: In function 'vpu_handle_msg':
>> drivers/media/platform/amphion/vpu_msgs.c:314:1: warning: the frame size of 1040 bytes is larger than 1024 bytes [-Wframe-larger-than=]
314 | }
| ^
drivers/media/platform/amphion/vpu_msgs.c: In function 'vpu_inst_run_work':
drivers/media/platform/amphion/vpu_msgs.c:265:1: warning: the frame size of 1040 bytes is larger than 1024 bytes [-Wframe-larger-than=]
265 | }
| ^
--
drivers/media/platform/amphion/vpu_dbg.c: In function 'vpu_dbg_instance':
>> drivers/media/platform/amphion/vpu_dbg.c:169:83: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 6 has type 'dma_addr_t' {aka 'unsigned int'} [-Wformat=]
169 | num = scnprintf(str, sizeof(str), "stream_buffer = %d / %d, [0x%llx, 0x%llx]\n",
| ~~~^
| |
| long long unsigned int
| %x
......
172 | inst->stream_buffer.phys,
| ~~~~~~~~~~~~~~~~~~~~~~~~
| |
| dma_addr_t {aka unsigned int}
drivers/media/platform/amphion/vpu_dbg.c:169:91: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 7 has type 'dma_addr_t' {aka 'unsigned int'} [-Wformat=]
169 | num = scnprintf(str, sizeof(str), "stream_buffer = %d / %d, [0x%llx, 0x%llx]\n",
| ~~~^
| |
| long long unsigned int
| %x
......
173 | inst->stream_buffer.phys + inst->stream_buffer.length);
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| |
| dma_addr_t {aka unsigned int}
drivers/media/platform/amphion/vpu_dbg.c: In function 'vpu_dbg_core':
drivers/media/platform/amphion/vpu_dbg.c:224:67: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'dma_addr_t' {aka 'unsigned int'} [-Wformat=]
224 | num = scnprintf(str, sizeof(str), "boot_region = <0x%08llx, 0x%x>\n",
| ~~~~~^
| |
| long long unsigned int
| %08x
225 | core->fw.phys, core->fw.length);
| ~~~~~~~~~~~~~
| |
| dma_addr_t {aka unsigned int}
drivers/media/platform/amphion/vpu_dbg.c:228:67: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'dma_addr_t' {aka 'unsigned int'} [-Wformat=]
228 | num = scnprintf(str, sizeof(str), "rpc_region = <0x%08llx, 0x%x> used = 0x%x\n",
| ~~~~~^
| |
| long long unsigned int
| %08x
229 | core->rpc.phys, core->rpc.length, core->rpc.bytesused);
| ~~~~~~~~~~~~~~
| |
| dma_addr_t {aka unsigned int}
drivers/media/platform/amphion/vpu_dbg.c:232:67: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'dma_addr_t' {aka 'unsigned int'} [-Wformat=]
232 | num = scnprintf(str, sizeof(str), "fwlog_region = <0x%08llx, 0x%x>\n",
| ~~~~~^
| |
| long long unsigned int
| %08x
233 | core->log.phys, core->log.length);
| ~~~~~~~~~~~~~~
| |
| dma_addr_t {aka unsigned int}
vim +314 drivers/media/platform/amphion/vpu_msgs.c
f1cf0764ca594c8 Ming Qian 2021-09-07 284
f1cf0764ca594c8 Ming Qian 2021-09-07 285 static int vpu_handle_msg(struct vpu_core *core)
f1cf0764ca594c8 Ming Qian 2021-09-07 286 {
f1cf0764ca594c8 Ming Qian 2021-09-07 287 struct vpu_rpc_event pkt;
f1cf0764ca594c8 Ming Qian 2021-09-07 288 struct vpu_inst *inst;
f1cf0764ca594c8 Ming Qian 2021-09-07 289 int ret;
f1cf0764ca594c8 Ming Qian 2021-09-07 290
f1cf0764ca594c8 Ming Qian 2021-09-07 291 memset(&pkt, 0, sizeof(pkt));
f1cf0764ca594c8 Ming Qian 2021-09-07 292 while (!vpu_iface_receive_msg(core, &pkt)) {
f1cf0764ca594c8 Ming Qian 2021-09-07 293 vpu_dbg(LVL_DEBUG, "event index = %d, id = %d, num = %d\n",
f1cf0764ca594c8 Ming Qian 2021-09-07 294 pkt.hdr.index, pkt.hdr.id, pkt.hdr.num);
f1cf0764ca594c8 Ming Qian 2021-09-07 295
f1cf0764ca594c8 Ming Qian 2021-09-07 296 ret = vpu_iface_convert_msg_id(core, pkt.hdr.id);
f1cf0764ca594c8 Ming Qian 2021-09-07 297 if (ret < 0)
f1cf0764ca594c8 Ming Qian 2021-09-07 298 continue;
f1cf0764ca594c8 Ming Qian 2021-09-07 299
f1cf0764ca594c8 Ming Qian 2021-09-07 300 inst = vpu_core_find_instance(core, pkt.hdr.index);
f1cf0764ca594c8 Ming Qian 2021-09-07 301 if (inst) {
f1cf0764ca594c8 Ming Qian 2021-09-07 302 vpu_response_cmd(inst, ret, 0);
f1cf0764ca594c8 Ming Qian 2021-09-07 303 mutex_lock(&core->cmd_lock);
f1cf0764ca594c8 Ming Qian 2021-09-07 304 vpu_inst_record_flow(inst, ret);
f1cf0764ca594c8 Ming Qian 2021-09-07 305 mutex_unlock(&core->cmd_lock);
f1cf0764ca594c8 Ming Qian 2021-09-07 306
f1cf0764ca594c8 Ming Qian 2021-09-07 307 vpu_inst_handle_msg(inst, &pkt);
f1cf0764ca594c8 Ming Qian 2021-09-07 308 vpu_inst_put(inst);
f1cf0764ca594c8 Ming Qian 2021-09-07 309 }
f1cf0764ca594c8 Ming Qian 2021-09-07 310 memset(&pkt, 0, sizeof(pkt));
f1cf0764ca594c8 Ming Qian 2021-09-07 311 }
f1cf0764ca594c8 Ming Qian 2021-09-07 312
f1cf0764ca594c8 Ming Qian 2021-09-07 313 return 0;
f1cf0764ca594c8 Ming Qian 2021-09-07 @314 }
f1cf0764ca594c8 Ming Qian 2021-09-07 315
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 54075 bytes --]
next prev parent reply other threads:[~2021-09-07 20:08 UTC|newest]
Thread overview: 62+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-07 9:49 [PATCH v8 00/15] amphion video decoder/encoder driver Ming Qian
2021-09-07 9:49 ` Ming Qian
2021-09-07 9:49 ` [PATCH v8 01/15] dt-bindings: media: amphion: add amphion video codec bindings Ming Qian
2021-09-07 9:49 ` Ming Qian
2021-09-07 9:49 ` [PATCH v8 02/15] media:Add nt8 and nt10 video format Ming Qian
2021-09-07 9:49 ` Ming Qian
2021-09-07 9:49 ` [PATCH v8 03/15] media:Add v4l2 buf flag codec data Ming Qian
2021-09-07 9:49 ` Ming Qian
2021-09-08 13:14 ` Nicolas Dufresne
2021-09-08 13:14 ` Nicolas Dufresne
2021-09-09 2:20 ` [EXT] " Ming Qian
2021-09-09 2:20 ` Ming Qian
2021-09-09 19:40 ` Nicolas Dufresne
2021-09-09 19:40 ` Nicolas Dufresne
2021-09-10 1:33 ` Ming Qian
2021-09-10 1:33 ` Ming Qian
2021-09-07 9:49 ` [PATCH v8 04/15] media:Add v4l2 event codec_error and skip Ming Qian
2021-09-07 9:49 ` Ming Qian
2021-09-08 13:32 ` Nicolas Dufresne
2021-09-08 13:32 ` Nicolas Dufresne
2021-09-09 3:13 ` [EXT] " Ming Qian
2021-09-09 3:13 ` Ming Qian
2021-09-09 19:54 ` Nicolas Dufresne
2021-09-09 19:54 ` Nicolas Dufresne
2021-09-10 1:50 ` Ming Qian
2021-09-10 1:50 ` Ming Qian
2022-01-13 7:18 ` Ming Qian
2022-01-13 7:18 ` Ming Qian
2022-01-21 22:25 ` Nicolas Dufresne
2022-01-21 22:25 ` Nicolas Dufresne
2022-01-25 1:54 ` Ming Qian
2022-01-25 1:54 ` Ming Qian
2022-01-25 20:23 ` Nicolas Dufresne
2022-01-25 20:23 ` Nicolas Dufresne
2021-09-07 9:49 ` [PATCH v8 05/15] media: amphion: add amphion vpu device driver Ming Qian
2021-09-07 9:49 ` Ming Qian
2021-09-07 9:49 ` [PATCH v8 06/15] media: amphion: add vpu core driver Ming Qian
2021-09-07 9:49 ` Ming Qian
2021-09-07 9:49 ` [PATCH v8 07/15] media: amphion: implement vpu core communication based on mailbox Ming Qian
2021-09-07 9:49 ` Ming Qian
2021-09-07 19:04 ` kernel test robot
2021-09-07 19:04 ` kernel test robot
2021-09-07 9:49 ` [PATCH v8 08/15] media: amphion: add vpu v4l2 m2m support Ming Qian
2021-09-07 9:49 ` Ming Qian
2021-09-07 20:08 ` kernel test robot [this message]
2021-09-07 20:08 ` kernel test robot
2021-09-07 9:49 ` [PATCH v8 09/15] media: amphion: add v4l2 m2m vpu encoder stateful driver Ming Qian
2021-09-07 9:49 ` Ming Qian
2021-09-07 9:49 ` [PATCH v8 10/15] media: amphion: add v4l2 m2m vpu decoder " Ming Qian
2021-09-07 9:49 ` Ming Qian
2021-09-07 9:49 ` [PATCH v8 11/15] media: amphion: implement windsor encoder rpc interface Ming Qian
2021-09-07 9:49 ` Ming Qian
2021-09-07 9:49 ` [PATCH v8 12/15] media: amphion: implement malone decoder " Ming Qian
2021-09-07 9:49 ` Ming Qian
2021-09-07 9:49 ` [PATCH v8 13/15] ARM64: dts: freescale: imx8q: add imx vpu codec entries Ming Qian
2021-09-07 9:49 ` Ming Qian
2021-09-07 9:49 ` [PATCH v8 14/15] firmware: imx: scu-pd: imx8q: add vpu mu resources Ming Qian
2021-09-07 9:49 ` Ming Qian
2021-09-07 9:49 ` [PATCH v8 15/15] MAINTAINERS: add AMPHION VPU CODEC V4L2 driver entry Ming Qian
2021-09-07 9:49 ` Ming Qian
2021-09-07 14:35 ` [PATCH v8 00/15] amphion video decoder/encoder driver Nicolas Dufresne
2021-09-07 14:35 ` Nicolas Dufresne
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202109080336.c8r1bAhj-lkp@intel.com \
--to=lkp@intel.com \
--cc=kbuild-all@lists.01.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.