From: kbuild test robot <lkp@intel.com>
To: Michael Tretter <m.tretter@pengutronix.de>
Cc: kbuild-all@01.org, linux-media@vger.kernel.org,
devicetree@vger.kernel.org, kernel@pengutronix.de,
robh+dt@kernel.org, mchehab@kernel.org, tfiga@chromium.org,
dshah@xilinx.com
Subject: Re: [PATCH v3 2/3] [media] allegro: add Allegro DVT video IP core driver
Date: Mon, 25 Feb 2019 04:01:05 +0800 [thread overview]
Message-ID: <201902250310.h831vF9s%fengguang.wu@intel.com> (raw)
In-Reply-To: <20190213175124.3695-3-m.tretter@pengutronix.de>
[-- Attachment #1: Type: text/plain, Size: 18137 bytes --]
Hi Michael,
I love your patch! Perhaps something to improve:
[auto build test WARNING on linuxtv-media/master]
[also build test WARNING on v5.0-rc4 next-20190222]
[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/Michael-Tretter/Add-ZynqMP-VCU-Allegro-DVT-H-264-encoder-driver/20190214-090312
base: git://linuxtv.org/media_tree.git master
config: nds32-allmodconfig (attached as .config)
compiler: nds32le-linux-gcc (GCC) 6.4.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
GCC_VERSION=6.4.0 make.cross ARCH=nds32
All warnings (new ones prefixed by >>):
In file included from include/media/v4l2-subdev.h:24:0,
from include/media/v4l2-device.h:25,
from drivers/staging/media/allegro-dvt/allegro-core.c:22:
drivers/staging/media/allegro-dvt/allegro-core.c: In function 'allegro_mbox_write':
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
>> drivers/staging/media/allegro-dvt/allegro-core.c:749:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
>> drivers/staging/media/allegro-dvt/allegro-core.c:749:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:756:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%lx' expects argument of type 'long unsigned int', but argument 4 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:769:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c: In function 'allegro_mbox_read':
include/linux/kern_levels.h:5:18: warning: format '%lx' expects argument of type 'long unsigned int', but argument 4 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:798:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'unsigned int' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:810:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:810:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:816:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'ssize_t {aka int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:816:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c: In function 'allegro_receive_message':
include/linux/kern_levels.h:5:18: warning: format '%ld' expects argument of type 'long int', but argument 3 has type 'ssize_t {aka int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:1542:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'unsigned int' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:1542:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c: In function 'allegro_copy_firmware':
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:15:20: note: in expansion of macro 'KERN_SOH'
#define KERN_DEBUG KERN_SOH "7" /* debug-level messages */
^~~~~~~~
>> include/media/v4l2-common.h:85:16: note: in expansion of macro 'KERN_DEBUG'
v4l2_printk(KERN_DEBUG, dev, fmt , ## arg); \
^~~~~~~~~~
>> drivers/staging/media/allegro-dvt/allegro-core.c:1618:2: note: in expansion of macro 'v4l2_dbg'
v4l2_dbg(1, debug, &dev->v4l2_dev,
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c: In function 'allegro_copy_fw_codec':
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:1640:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%ld' expects argument of type 'long int', but argument 3 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:15:20: note: in expansion of macro 'KERN_SOH'
#define KERN_DEBUG KERN_SOH "7" /* debug-level messages */
^~~~~~~~
>> include/media/v4l2-common.h:85:16: note: in expansion of macro 'KERN_DEBUG'
v4l2_printk(KERN_DEBUG, dev, fmt , ## arg); \
^~~~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:1645:2: note: in expansion of macro 'v4l2_dbg'
v4l2_dbg(1, debug, &dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'dma_addr_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:15:20: note: in expansion of macro 'KERN_SOH'
#define KERN_DEBUG KERN_SOH "7" /* debug-level messages */
^~~~~~~~
>> include/media/v4l2-common.h:85:16: note: in expansion of macro 'KERN_DEBUG'
v4l2_printk(KERN_DEBUG, dev, fmt , ## arg); \
^~~~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:1645:2: note: in expansion of macro 'v4l2_dbg'
v4l2_dbg(1, debug, &dev->v4l2_dev,
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c: In function 'allegro_buf_queue':
include/linux/kern_levels.h:5:18: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'dma_addr_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:15:20: note: in expansion of macro 'KERN_SOH'
#define KERN_DEBUG KERN_SOH "7" /* debug-level messages */
^~~~~~~~
>> include/media/v4l2-common.h:85:16: note: in expansion of macro 'KERN_DEBUG'
v4l2_printk(KERN_DEBUG, dev, fmt , ## arg); \
^~~~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:1949:3: note: in expansion of macro 'v4l2_dbg'
v4l2_dbg(1, debug, &dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%ld' expects argument of type 'long int', but argument 5 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:15:20: note: in expansion of macro 'KERN_SOH'
#define KERN_DEBUG KERN_SOH "7" /* debug-level messages */
^~~~~~~~
>> include/media/v4l2-common.h:85:16: note: in expansion of macro 'KERN_DEBUG'
v4l2_printk(KERN_DEBUG, dev, fmt , ## arg); \
^~~~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:1949:3: note: in expansion of macro 'v4l2_dbg'
v4l2_dbg(1, debug, &dev->v4l2_dev,
^~~~~~~~
vim +/v4l2_err +749 drivers/staging/media/allegro-dvt/allegro-core.c
736
737 static int allegro_mbox_write(struct allegro_dev *dev,
738 struct allegro_mbox *mbox, void *src, size_t size)
739 {
740 struct mcu_msg_header *header = src;
741 unsigned int tail;
742 size_t size_no_wrap;
743 int err = 0;
744
745 if (!src)
746 return -EINVAL;
747
748 if (size > mbox->size) {
> 749 v4l2_err(&dev->v4l2_dev,
750 "message (%lu bytes) to large for mailbox (%lu bytes)\n",
751 size, mbox->size);
752 return -EINVAL;
753 }
754
755 if (header->length != size - sizeof(*header)) {
756 v4l2_err(&dev->v4l2_dev,
757 "invalid message length: %u bytes (expected %lu bytes)\n",
758 header->length, size - sizeof(*header));
759 return -EINVAL;
760 }
761
762 v4l2_dbg(2, debug, &dev->v4l2_dev,
763 "write command message: type %s, body length %d\n",
764 msg_type_name(header->type), header->length);
765
766 mutex_lock(&mbox->lock);
767 regmap_read(dev->sram, mbox->tail, &tail);
768 if (tail > mbox->size) {
769 v4l2_err(&dev->v4l2_dev,
770 "invalid tail (0x%x): must be smaller than mailbox size (0x%lx)\n",
771 tail, mbox->size);
772 err = -EIO;
773 goto out;
774 }
775 size_no_wrap = min(size, mbox->size - (size_t)tail);
776 regmap_bulk_write(dev->sram, mbox->data + tail, src, size_no_wrap / 4);
777 regmap_bulk_write(dev->sram, mbox->data,
778 src + size_no_wrap, (size - size_no_wrap) / 4);
779 regmap_write(dev->sram, mbox->tail, (tail + size) % mbox->size);
780
781 out:
782 mutex_unlock(&mbox->lock);
783
784 return err;
785 }
786
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 49776 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kbuild test robot <lkp@intel.com>
To: Michael Tretter <m.tretter@pengutronix.de>
Cc: kbuild-all@01.org, linux-media@vger.kernel.org,
devicetree@vger.kernel.org, kernel@pengutronix.de,
robh+dt@kernel.org, mchehab@kernel.org, tfiga@chromium.org,
dshah@xilinx.com, Michael Tretter <m.tretter@pengutronix.de>
Subject: Re: [PATCH v3 2/3] [media] allegro: add Allegro DVT video IP core driver
Date: Mon, 25 Feb 2019 04:01:05 +0800 [thread overview]
Message-ID: <201902250310.h831vF9s%fengguang.wu@intel.com> (raw)
In-Reply-To: <20190213175124.3695-3-m.tretter@pengutronix.de>
[-- Attachment #1: Type: text/plain, Size: 18137 bytes --]
Hi Michael,
I love your patch! Perhaps something to improve:
[auto build test WARNING on linuxtv-media/master]
[also build test WARNING on v5.0-rc4 next-20190222]
[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/Michael-Tretter/Add-ZynqMP-VCU-Allegro-DVT-H-264-encoder-driver/20190214-090312
base: git://linuxtv.org/media_tree.git master
config: nds32-allmodconfig (attached as .config)
compiler: nds32le-linux-gcc (GCC) 6.4.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
GCC_VERSION=6.4.0 make.cross ARCH=nds32
All warnings (new ones prefixed by >>):
In file included from include/media/v4l2-subdev.h:24:0,
from include/media/v4l2-device.h:25,
from drivers/staging/media/allegro-dvt/allegro-core.c:22:
drivers/staging/media/allegro-dvt/allegro-core.c: In function 'allegro_mbox_write':
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
>> drivers/staging/media/allegro-dvt/allegro-core.c:749:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
>> drivers/staging/media/allegro-dvt/allegro-core.c:749:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:756:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%lx' expects argument of type 'long unsigned int', but argument 4 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:769:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c: In function 'allegro_mbox_read':
include/linux/kern_levels.h:5:18: warning: format '%lx' expects argument of type 'long unsigned int', but argument 4 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:798:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'unsigned int' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:810:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:810:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:816:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'ssize_t {aka int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:816:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c: In function 'allegro_receive_message':
include/linux/kern_levels.h:5:18: warning: format '%ld' expects argument of type 'long int', but argument 3 has type 'ssize_t {aka int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:1542:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'unsigned int' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:1542:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c: In function 'allegro_copy_firmware':
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:15:20: note: in expansion of macro 'KERN_SOH'
#define KERN_DEBUG KERN_SOH "7" /* debug-level messages */
^~~~~~~~
>> include/media/v4l2-common.h:85:16: note: in expansion of macro 'KERN_DEBUG'
v4l2_printk(KERN_DEBUG, dev, fmt , ## arg); \
^~~~~~~~~~
>> drivers/staging/media/allegro-dvt/allegro-core.c:1618:2: note: in expansion of macro 'v4l2_dbg'
v4l2_dbg(1, debug, &dev->v4l2_dev,
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c: In function 'allegro_copy_fw_codec':
include/linux/kern_levels.h:5:18: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH'
#define KERN_ERR KERN_SOH "3" /* error conditions */
^~~~~~~~
>> include/media/v4l2-common.h:72:14: note: in expansion of macro 'KERN_ERR'
v4l2_printk(KERN_ERR, dev, fmt , ## arg)
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:1640:3: note: in expansion of macro 'v4l2_err'
v4l2_err(&dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%ld' expects argument of type 'long int', but argument 3 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:15:20: note: in expansion of macro 'KERN_SOH'
#define KERN_DEBUG KERN_SOH "7" /* debug-level messages */
^~~~~~~~
>> include/media/v4l2-common.h:85:16: note: in expansion of macro 'KERN_DEBUG'
v4l2_printk(KERN_DEBUG, dev, fmt , ## arg); \
^~~~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:1645:2: note: in expansion of macro 'v4l2_dbg'
v4l2_dbg(1, debug, &dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'dma_addr_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:15:20: note: in expansion of macro 'KERN_SOH'
#define KERN_DEBUG KERN_SOH "7" /* debug-level messages */
^~~~~~~~
>> include/media/v4l2-common.h:85:16: note: in expansion of macro 'KERN_DEBUG'
v4l2_printk(KERN_DEBUG, dev, fmt , ## arg); \
^~~~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:1645:2: note: in expansion of macro 'v4l2_dbg'
v4l2_dbg(1, debug, &dev->v4l2_dev,
^~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c: In function 'allegro_buf_queue':
include/linux/kern_levels.h:5:18: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'dma_addr_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:15:20: note: in expansion of macro 'KERN_SOH'
#define KERN_DEBUG KERN_SOH "7" /* debug-level messages */
^~~~~~~~
>> include/media/v4l2-common.h:85:16: note: in expansion of macro 'KERN_DEBUG'
v4l2_printk(KERN_DEBUG, dev, fmt , ## arg); \
^~~~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:1949:3: note: in expansion of macro 'v4l2_dbg'
v4l2_dbg(1, debug, &dev->v4l2_dev,
^~~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%ld' expects argument of type 'long int', but argument 5 has type 'size_t {aka unsigned int}' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^
include/media/v4l2-common.h:69:9: note: in definition of macro 'v4l2_printk'
printk(level "%s: " fmt, (dev)->name , ## arg)
^~~~~
include/linux/kern_levels.h:15:20: note: in expansion of macro 'KERN_SOH'
#define KERN_DEBUG KERN_SOH "7" /* debug-level messages */
^~~~~~~~
>> include/media/v4l2-common.h:85:16: note: in expansion of macro 'KERN_DEBUG'
v4l2_printk(KERN_DEBUG, dev, fmt , ## arg); \
^~~~~~~~~~
drivers/staging/media/allegro-dvt/allegro-core.c:1949:3: note: in expansion of macro 'v4l2_dbg'
v4l2_dbg(1, debug, &dev->v4l2_dev,
^~~~~~~~
vim +/v4l2_err +749 drivers/staging/media/allegro-dvt/allegro-core.c
736
737 static int allegro_mbox_write(struct allegro_dev *dev,
738 struct allegro_mbox *mbox, void *src, size_t size)
739 {
740 struct mcu_msg_header *header = src;
741 unsigned int tail;
742 size_t size_no_wrap;
743 int err = 0;
744
745 if (!src)
746 return -EINVAL;
747
748 if (size > mbox->size) {
> 749 v4l2_err(&dev->v4l2_dev,
750 "message (%lu bytes) to large for mailbox (%lu bytes)\n",
751 size, mbox->size);
752 return -EINVAL;
753 }
754
755 if (header->length != size - sizeof(*header)) {
756 v4l2_err(&dev->v4l2_dev,
757 "invalid message length: %u bytes (expected %lu bytes)\n",
758 header->length, size - sizeof(*header));
759 return -EINVAL;
760 }
761
762 v4l2_dbg(2, debug, &dev->v4l2_dev,
763 "write command message: type %s, body length %d\n",
764 msg_type_name(header->type), header->length);
765
766 mutex_lock(&mbox->lock);
767 regmap_read(dev->sram, mbox->tail, &tail);
768 if (tail > mbox->size) {
769 v4l2_err(&dev->v4l2_dev,
770 "invalid tail (0x%x): must be smaller than mailbox size (0x%lx)\n",
771 tail, mbox->size);
772 err = -EIO;
773 goto out;
774 }
775 size_no_wrap = min(size, mbox->size - (size_t)tail);
776 regmap_bulk_write(dev->sram, mbox->data + tail, src, size_no_wrap / 4);
777 regmap_bulk_write(dev->sram, mbox->data,
778 src + size_no_wrap, (size - size_no_wrap) / 4);
779 regmap_write(dev->sram, mbox->tail, (tail + size) % mbox->size);
780
781 out:
782 mutex_unlock(&mbox->lock);
783
784 return err;
785 }
786
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 49776 bytes --]
next prev parent reply other threads:[~2019-02-24 20:01 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-02-13 17:51 [PATCH v3 0/3] Add ZynqMP VCU/Allegro DVT H.264 encoder driver Michael Tretter
2019-02-13 17:51 ` [PATCH v3 1/3] media: dt-bindings: media: document allegro-dvt bindings Michael Tretter
2019-02-18 18:12 ` Rob Herring
2019-02-18 18:12 ` Rob Herring
2019-02-13 17:51 ` [PATCH v3 2/3] [media] allegro: add Allegro DVT video IP core driver Michael Tretter
2019-02-14 11:15 ` kbuild test robot
2019-02-14 11:15 ` kbuild test robot
2019-02-24 20:01 ` kbuild test robot [this message]
2019-02-24 20:01 ` kbuild test robot
2019-02-25 11:10 ` Hans Verkuil
2019-02-13 17:51 ` [PATCH v3 3/3] [media] allegro: add SPS/PPS nal unit writer Michael Tretter
2019-02-25 11:13 ` Hans Verkuil
2019-02-13 17:54 ` [PATCH v3 0/3] Add ZynqMP VCU/Allegro DVT H.264 encoder driver Michael Tretter
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=201902250310.h831vF9s%fengguang.wu@intel.com \
--to=lkp@intel.com \
--cc=devicetree@vger.kernel.org \
--cc=dshah@xilinx.com \
--cc=kbuild-all@01.org \
--cc=kernel@pengutronix.de \
--cc=linux-media@vger.kernel.org \
--cc=m.tretter@pengutronix.de \
--cc=mchehab@kernel.org \
--cc=robh+dt@kernel.org \
--cc=tfiga@chromium.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.