Re: [PATCH v2] media: venus: add debugfs support

From: kbuild test robot
Date: Sat Dec 29 2018 - 13:23:28 EST


Hi Malathi,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on linuxtv-media/master]
[also build test ERROR on next-20181224]
[cannot apply to v4.20]
[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/Malathi-Gottam/media-venus-add-debugfs-support/20181228-172634
base: git://linuxtv.org/media_tree.git master
config: microblaze-allyesconfig (attached as .config)
compiler: microblaze-linux-gcc (GCC) 8.1.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=8.1.0 make.cross ARCH=microblaze

All errors (new ones prefixed by >>):

In file included from include/linux/kernel.h:14,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
drivers/media/platform/qcom/venus/helpers.c: In function 'venus_helper_check_codec':
>> drivers/media/platform/qcom/venus/helpers.c:78:40: error: 'pixfmt' undeclared (first use in this function); did you mean 'pr_fmt'?
dprintk(WARN, "Unknown format:%x\n", pixfmt);
^~~~~~
include/linux/printk.h:315:34: note: in definition of macro 'pr_info'
printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
^~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:78:3: note: in expansion of macro 'dprintk'
dprintk(WARN, "Unknown format:%x\n", pixfmt);
^~~~~~~
drivers/media/platform/qcom/venus/helpers.c:78:40: note: each undeclared identifier is reported only once for each function it appears in
dprintk(WARN, "Unknown format:%x\n", pixfmt);
^~~~~~
include/linux/printk.h:315:34: note: in definition of macro 'pr_info'
printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
^~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:78:3: note: in expansion of macro 'dprintk'
dprintk(WARN, "Unknown format:%x\n", pixfmt);
^~~~~~~
In file included from include/linux/printk.h:7,
from include/linux/kernel.h:14,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
drivers/media/platform/qcom/venus/helpers.c: In function 'venus_helper_queue_dpb_bufs':
include/linux/kern_levels.h:5:18: warning: format '%d' expects argument of type 'int', but argument 3 has type 'const char *' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^~~~~~
include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH'
#define KERN_INFO KERN_SOH "6" /* informational */
^~~~~~~~
include/linux/printk.h:315:9: note: in expansion of macro 'KERN_INFO'
printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
^~~~~~~~~
drivers/media/platform/qcom/venus/core.h:55:4: note: in expansion of macro 'pr_info'
pr_info("venus:" fmt, \
^~~~~~~
drivers/media/platform/qcom/venus/helpers.c:107:4: note: in expansion of macro 'dprintk'
dprintk(ERR, "%s: Failed to queue dpb buf to hfi: %d\n",
^~~~~~~
drivers/media/platform/qcom/venus/helpers.c:107:55: note: format string is defined here
dprintk(ERR, "%s: Failed to queue dpb buf to hfi: %d\n",
~^
%s
In file included from include/linux/printk.h:7,
from include/linux/kernel.h:14,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
include/linux/kern_levels.h:5:18: warning: too many arguments for format [-Wformat-extra-args]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^~~~~~
include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH'
#define KERN_INFO KERN_SOH "6" /* informational */
^~~~~~~~
include/linux/printk.h:315:9: note: in expansion of macro 'KERN_INFO'
printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
^~~~~~~~~
drivers/media/platform/qcom/venus/core.h:55:4: note: in expansion of macro 'pr_info'
pr_info("venus:" fmt, \
^~~~~~~
drivers/media/platform/qcom/venus/helpers.c:107:4: note: in expansion of macro 'dprintk'
dprintk(ERR, "%s: Failed to queue dpb buf to hfi: %d\n",
^~~~~~~
drivers/media/platform/qcom/venus/helpers.c: In function 'venus_helper_alloc_dpb_bufs':
include/linux/kern_levels.h:5:18: warning: format '%d' expects argument of type 'int', but argument 2 has type 'char *' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^~~~~~
include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH'
#define KERN_INFO KERN_SOH "6" /* informational */
^~~~~~~~
include/linux/printk.h:315:9: note: in expansion of macro 'KERN_INFO'
printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
^~~~~~~~~
drivers/media/platform/qcom/venus/core.h:55:4: note: in expansion of macro 'pr_info'
pr_info("venus:" fmt, \
^~~~~~~
drivers/media/platform/qcom/venus/helpers.c:164:2: note: in expansion of macro 'dprintk'
dprintk(DBG, "buf count min %d", count);
^~~~~~~
drivers/media/platform/qcom/venus/helpers.c:164:31: note: format string is defined here
dprintk(DBG, "buf count min %d", count);
~^
%s
In file included from include/linux/printk.h:7,
from include/linux/kernel.h:14,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
include/linux/kern_levels.h:5:18: warning: too many arguments for format [-Wformat-extra-args]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^~~~~~
include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH'
#define KERN_INFO KERN_SOH "6" /* informational */
^~~~~~~~
include/linux/printk.h:315:9: note: in expansion of macro 'KERN_INFO'
printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
^~~~~~~~~
drivers/media/platform/qcom/venus/core.h:55:4: note: in expansion of macro 'pr_info'
pr_info("venus:" fmt, \
^~~~~~~
drivers/media/platform/qcom/venus/helpers.c:164:2: note: in expansion of macro 'dprintk'
dprintk(DBG, "buf count min %d", count);
^~~~~~~
include/linux/kern_levels.h:5:18: warning: too many arguments for format [-Wformat-extra-args]
--
printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
^~~~~~~~~
drivers/media/platform/qcom/venus/core.h:55:4: note: in expansion of macro 'pr_info'
pr_info("venus:" fmt, \
^~~~~~~
drivers/media/platform/qcom/venus/helpers.c:756:3: note: in expansion of macro 'dprintk'
dprintk(ERR,
^~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%d' expects argument of type 'int', but argument 2 has type 'char *' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^~~~~~
include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH'
#define KERN_INFO KERN_SOH "6" /* informational */
^~~~~~~~
include/linux/printk.h:315:9: note: in expansion of macro 'KERN_INFO'
printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
^~~~~~~~~
drivers/media/platform/qcom/venus/core.h:55:4: note: in expansion of macro 'pr_info'
pr_info("venus:" fmt, \
^~~~~~~
drivers/media/platform/qcom/venus/helpers.c:766:3: note: in expansion of macro 'dprintk'
dprintk(ERR,
^~~~~~~
drivers/media/platform/qcom/venus/helpers.c:767:40: note: format string is defined here
"Failed to set actual buffer count %d for buffer type %d\n",
~^
%s
In file included from include/linux/printk.h:7,
from include/linux/kernel.h:14,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
include/linux/kern_levels.h:5:18: warning: too many arguments for format [-Wformat-extra-args]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^~~~~~
include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH'
#define KERN_INFO KERN_SOH "6" /* informational */
^~~~~~~~
include/linux/printk.h:315:9: note: in expansion of macro 'KERN_INFO'
printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
^~~~~~~~~
drivers/media/platform/qcom/venus/core.h:55:4: note: in expansion of macro 'pr_info'
pr_info("venus:" fmt, \
^~~~~~~
drivers/media/platform/qcom/venus/helpers.c:766:3: note: in expansion of macro 'dprintk'
dprintk(ERR,
^~~~~~~
include/linux/kern_levels.h:5:18: warning: format '%d' expects argument of type 'int', but argument 2 has type 'char *' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^~~~~~
include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH'
#define KERN_INFO KERN_SOH "6" /* informational */
^~~~~~~~
include/linux/printk.h:315:9: note: in expansion of macro 'KERN_INFO'
printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
^~~~~~~~~
drivers/media/platform/qcom/venus/core.h:55:4: note: in expansion of macro 'pr_info'
pr_info("venus:" fmt, \
^~~~~~~
drivers/media/platform/qcom/venus/helpers.c:771:2: note: in expansion of macro 'dprintk'
dprintk(DBG, "output buf: num = %d, input buf = %d\n",
^~~~~~~
drivers/media/platform/qcom/venus/helpers.c:771:35: note: format string is defined here
dprintk(DBG, "output buf: num = %d, input buf = %d\n",
~^
%s
In file included from include/linux/printk.h:7,
from include/linux/kernel.h:14,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
include/linux/kern_levels.h:5:18: warning: too many arguments for format [-Wformat-extra-args]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^~~~~~
include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH'
#define KERN_INFO KERN_SOH "6" /* informational */
^~~~~~~~
include/linux/printk.h:315:9: note: in expansion of macro 'KERN_INFO'
printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
^~~~~~~~~
drivers/media/platform/qcom/venus/core.h:55:4: note: in expansion of macro 'pr_info'
pr_info("venus:" fmt, \
^~~~~~~
drivers/media/platform/qcom/venus/helpers.c:771:2: note: in expansion of macro 'dprintk'
dprintk(DBG, "output buf: num = %d, input buf = %d\n",
^~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
include/linux/export.h:79:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
extern typeof(sym) sym; \
^~~~~~
include/linux/export.h:120:25: note: in expansion of macro '___EXPORT_SYMBOL'
#define __EXPORT_SYMBOL ___EXPORT_SYMBOL
^~~~~~~~~~~~~~~~
include/linux/export.h:127:2: note: in expansion of macro '__EXPORT_SYMBOL'
__EXPORT_SYMBOL(sym, "_gpl")
^~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:782:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_set_num_bufs);
^~~~~~~~~~~~~~~~~
>> drivers/media/platform/qcom/venus/helpers.c:795:19: error: non-static declaration of 'venus_helper_set_raw_format' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_set_raw_format);
^~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:795:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_set_raw_format);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:784:5: note: previous definition of 'venus_helper_set_raw_format' was here
int venus_helper_set_raw_format(struct venus_inst *inst, u32 hfi_format,
^~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/printk.h:7,
from include/linux/kernel.h:14,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
drivers/media/platform/qcom/venus/helpers.c: In function 'venus_helper_set_color_format':
include/linux/kern_levels.h:5:18: warning: format '%x' expects argument of type 'unsigned int', but argument 2 has type 'char *' [-Wformat=]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^~~~~~
include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH'
#define KERN_INFO KERN_SOH "6" /* informational */
^~~~~~~~
include/linux/printk.h:315:9: note: in expansion of macro 'KERN_INFO'
printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
^~~~~~~~~
drivers/media/platform/qcom/venus/core.h:55:4: note: in expansion of macro 'pr_info'
pr_info("venus:" fmt, \
^~~~~~~
drivers/media/platform/qcom/venus/helpers.c:810:3: note: in expansion of macro 'dprintk'
dprintk(ERR, "Using unsupported colorformat %#x\n",
^~~~~~~
drivers/media/platform/qcom/venus/helpers.c:810:49: note: format string is defined here
dprintk(ERR, "Using unsupported colorformat %#x\n",
~~^
%#s
In file included from include/linux/printk.h:7,
from include/linux/kernel.h:14,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
include/linux/kern_levels.h:5:18: warning: too many arguments for format [-Wformat-extra-args]
#define KERN_SOH "\001" /* ASCII Start Of Header */
^~~~~~
include/linux/kern_levels.h:14:19: note: in expansion of macro 'KERN_SOH'
#define KERN_INFO KERN_SOH "6" /* informational */
^~~~~~~~
include/linux/printk.h:315:9: note: in expansion of macro 'KERN_INFO'
printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)
^~~~~~~~~
drivers/media/platform/qcom/venus/core.h:55:4: note: in expansion of macro 'pr_info'
pr_info("venus:" fmt, \
^~~~~~~
drivers/media/platform/qcom/venus/helpers.c:810:3: note: in expansion of macro 'dprintk'
dprintk(ERR, "Using unsupported colorformat %#x\n",
^~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
drivers/media/platform/qcom/venus/helpers.c: In function 'venus_helper_set_num_bufs':
>> drivers/media/platform/qcom/venus/helpers.c:817:19: error: non-static declaration of 'venus_helper_set_color_format' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_set_color_format);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:817:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_set_color_format);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:797:5: note: previous definition of 'venus_helper_set_color_format' was here
int venus_helper_set_color_format(struct venus_inst *inst, u32 pixfmt)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
drivers/media/platform/qcom/venus/helpers.c:838:19: error: non-static declaration of 'venus_helper_set_multistream' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_set_multistream);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:838:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_set_multistream);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:819:5: note: previous definition of 'venus_helper_set_multistream' was here
int venus_helper_set_multistream(struct venus_inst *inst, bool out_en,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
drivers/media/platform/qcom/venus/helpers.c:860:19: error: non-static declaration of 'venus_helper_set_dyn_bufmode' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_set_dyn_bufmode);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:860:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_set_dyn_bufmode);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:840:5: note: previous definition of 'venus_helper_set_dyn_bufmode' was here
int venus_helper_set_dyn_bufmode(struct venus_inst *inst)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
>> drivers/media/platform/qcom/venus/helpers.c:872:19: error: non-static declaration of 'venus_helper_set_bufsize' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_set_bufsize);
^~~~~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:872:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_set_bufsize);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:862:5: note: previous definition of 'venus_helper_set_bufsize' was here
int venus_helper_set_bufsize(struct venus_inst *inst, u32 bufsize, u32 buftype)
^~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
>> drivers/media/platform/qcom/venus/helpers.c:887:19: error: non-static declaration of 'venus_helper_get_opb_size' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_get_opb_size);
^~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:887:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_get_opb_size);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:874:14: note: previous definition of 'venus_helper_get_opb_size' was here
unsigned int venus_helper_get_opb_size(struct venus_inst *inst)
^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:889:13: error: invalid storage class for function 'delayed_process_buf_func'
static void delayed_process_buf_func(struct work_struct *work)
^~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
drivers/media/platform/qcom/venus/helpers.c:928:19: error: non-static declaration of 'venus_helper_release_buf_ref' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_release_buf_ref);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:928:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_release_buf_ref);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:916:6: note: previous definition of 'venus_helper_release_buf_ref' was here
void venus_helper_release_buf_ref(struct venus_inst *inst, unsigned int idx)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
drivers/media/platform/qcom/venus/helpers.c:936:19: error: non-static declaration of 'venus_helper_acquire_buf_ref' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_acquire_buf_ref);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:936:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_acquire_buf_ref);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:930:6: note: previous definition of 'venus_helper_acquire_buf_ref' was here
void venus_helper_acquire_buf_ref(struct vb2_v4l2_buffer *vbuf)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:938:12: error: invalid storage class for function 'is_buf_refed'
static int is_buf_refed(struct venus_inst *inst, struct vb2_v4l2_buffer *vbuf)
^~~~~~~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
>> drivers/media/platform/qcom/venus/helpers.c:961:19: error: non-static declaration of 'venus_helper_find_buf' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_find_buf);
^~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:961:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_find_buf);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:952:1: note: previous definition of 'venus_helper_find_buf' was here
venus_helper_find_buf(struct venus_inst *inst, unsigned int type, u32 idx)
^~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
drivers/media/platform/qcom/venus/helpers.c:982:19: error: non-static declaration of 'venus_helper_vb2_buf_init' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_vb2_buf_init);
^~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:982:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_vb2_buf_init);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:963:5: note: previous definition of 'venus_helper_vb2_buf_init' was here
int venus_helper_vb2_buf_init(struct vb2_buffer *vb)
^~~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
>> drivers/media/platform/qcom/venus/helpers.c:998:19: error: non-static declaration of 'venus_helper_vb2_buf_prepare' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_vb2_buf_prepare);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:998:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_vb2_buf_prepare);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:984:5: note: previous definition of 'venus_helper_vb2_buf_prepare' was here
int venus_helper_vb2_buf_prepare(struct vb2_buffer *vb)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
drivers/media/platform/qcom/venus/helpers.c:1025:19: error: non-static declaration of 'venus_helper_vb2_buf_queue' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_vb2_buf_queue);
^~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:1025:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_vb2_buf_queue);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:1000:6: note: previous definition of 'venus_helper_vb2_buf_queue' was here
void venus_helper_vb2_buf_queue(struct vb2_buffer *vb)
^~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
>> drivers/media/platform/qcom/venus/helpers.c:1037:19: error: non-static declaration of 'venus_helper_buffers_done' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_buffers_done);
^~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:1037:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_buffers_done);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:1027:6: note: previous definition of 'venus_helper_buffers_done' was here
void venus_helper_buffers_done(struct venus_inst *inst,
^~~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
drivers/media/platform/qcom/venus/helpers.c:1075:19: error: non-static declaration of 'venus_helper_vb2_stop_streaming' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_vb2_stop_streaming);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:1075:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_vb2_stop_streaming);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:1039:6: note: previous definition of 'venus_helper_vb2_stop_streaming' was here
void venus_helper_vb2_stop_streaming(struct vb2_queue *q)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
>> drivers/media/platform/qcom/venus/helpers.c:1116:19: error: non-static declaration of 'venus_helper_vb2_start_streaming' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_vb2_start_streaming);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:1116:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_vb2_start_streaming);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:1077:5: note: previous definition of 'venus_helper_vb2_start_streaming' was here
int venus_helper_vb2_start_streaming(struct venus_inst *inst)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
>> drivers/media/platform/qcom/venus/helpers.c:1141:19: error: non-static declaration of 'venus_helper_m2m_device_run' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_m2m_device_run);
^~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:1141:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_m2m_device_run);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:1118:6: note: previous definition of 'venus_helper_m2m_device_run' was here
void venus_helper_m2m_device_run(void *priv)
^~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
>> drivers/media/platform/qcom/venus/helpers.c:1149:19: error: non-static declaration of 'venus_helper_m2m_job_abort' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_m2m_job_abort);
^~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:1149:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_m2m_job_abort);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:1143:6: note: previous definition of 'venus_helper_m2m_job_abort' was here
void venus_helper_m2m_job_abort(void *priv)
^~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
drivers/media/platform/qcom/venus/helpers.c:1159:19: error: non-static declaration of 'venus_helper_init_instance' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_init_instance);
^~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:1159:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_init_instance);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:1151:6: note: previous definition of 'venus_helper_init_instance' was here
void venus_helper_init_instance(struct venus_inst *inst)
^~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:1161:13: error: invalid storage class for function 'find_fmt_from_caps'
static bool find_fmt_from_caps(struct venus_caps *caps, u32 buftype, u32 fmt)
^~~~~~~~~~~~~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
>> drivers/media/platform/qcom/venus/helpers.c:1220:19: error: non-static declaration of 'venus_helper_get_out_fmts' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_get_out_fmts);
^~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:1220:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_get_out_fmts);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:1174:5: note: previous definition of 'venus_helper_get_out_fmts' was here
int venus_helper_get_out_fmts(struct venus_inst *inst, u32 v4l2_fmt,
^~~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
>> drivers/media/platform/qcom/venus/helpers.c:1269:19: error: non-static declaration of 'venus_helper_power_enable' follows static declaration
EXPORT_SYMBOL_GPL(venus_helper_power_enable);
^~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/export.h:79:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
drivers/media/platform/qcom/venus/helpers.c:1269:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_power_enable);
^~~~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:1222:5: note: previous definition of 'venus_helper_power_enable' was here
int venus_helper_power_enable(struct venus_core *core, u32 session_type,
^~~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/linkage.h:7,
from include/linux/kernel.h:7,
from include/linux/clk.h:16,
from drivers/media/platform/qcom/venus/helpers.c:15:
>> include/linux/export.h:67:22: error: expected declaration or statement at end of input
static const struct kernel_symbol __ksymtab_##sym \
^~~~~~~~~~~~~
include/linux/export.h:84:2: note: in expansion of macro '__KSYMTAB_ENTRY'
__KSYMTAB_ENTRY(sym, sec)
^~~~~~~~~~~~~~~
include/linux/export.h:120:25: note: in expansion of macro '___EXPORT_SYMBOL'
#define __EXPORT_SYMBOL ___EXPORT_SYMBOL
^~~~~~~~~~~~~~~~
include/linux/export.h:127:2: note: in expansion of macro '__EXPORT_SYMBOL'
__EXPORT_SYMBOL(sym, "_gpl")
^~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:1269:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_power_enable);
^~~~~~~~~~~~~~~~~
include/linux/export.h:67:22: warning: control reaches end of non-void function [-Wreturn-type]
static const struct kernel_symbol __ksymtab_##sym \
^~~~~~~~~~~~~
include/linux/export.h:84:2: note: in expansion of macro '__KSYMTAB_ENTRY'
__KSYMTAB_ENTRY(sym, sec)
^~~~~~~~~~~~~~~
include/linux/export.h:120:25: note: in expansion of macro '___EXPORT_SYMBOL'
#define __EXPORT_SYMBOL ___EXPORT_SYMBOL
^~~~~~~~~~~~~~~~
include/linux/export.h:127:2: note: in expansion of macro '__EXPORT_SYMBOL'
__EXPORT_SYMBOL(sym, "_gpl")
^~~~~~~~~~~~~~~
drivers/media/platform/qcom/venus/helpers.c:1269:1: note: in expansion of macro 'EXPORT_SYMBOL_GPL'
EXPORT_SYMBOL_GPL(venus_helper_power_enable);
^~~~~~~~~~~~~~~~~
..

vim +78 drivers/media/platform/qcom/venus/helpers.c

> 15 #include <linux/clk.h>
16 #include <linux/iopoll.h>
17 #include <linux/list.h>
18 #include <linux/mutex.h>
19 #include <linux/pm_runtime.h>
20 #include <linux/slab.h>
21 #include <media/videobuf2-dma-sg.h>
22 #include <media/v4l2-mem2mem.h>
23 #include <asm/div64.h>
24
25 #include "core.h"
26 #include "helpers.h"
27 #include "hfi_helper.h"
28 #include "hfi_venus_io.h"
29
30 struct intbuf {
31 struct list_head list;
32 u32 type;
33 size_t size;
34 void *va;
35 dma_addr_t da;
36 unsigned long attrs;
37 };
38
39 bool venus_helper_check_codec(struct venus_inst *inst, u32 v4l2_pixfmt)
40 {
41 struct venus_core *core = inst->core;
42 u32 session_type = inst->session_type;
43 u32 codec;
44
45 switch (v4l2_pixfmt) {
46 case V4L2_PIX_FMT_H264:
47 codec = HFI_VIDEO_CODEC_H264;
48 break;
49 case V4L2_PIX_FMT_H263:
50 codec = HFI_VIDEO_CODEC_H263;
51 break;
52 case V4L2_PIX_FMT_MPEG1:
53 codec = HFI_VIDEO_CODEC_MPEG1;
54 break;
55 case V4L2_PIX_FMT_MPEG2:
56 codec = HFI_VIDEO_CODEC_MPEG2;
57 break;
58 case V4L2_PIX_FMT_MPEG4:
59 codec = HFI_VIDEO_CODEC_MPEG4;
60 break;
61 case V4L2_PIX_FMT_VC1_ANNEX_G:
62 case V4L2_PIX_FMT_VC1_ANNEX_L:
63 codec = HFI_VIDEO_CODEC_VC1;
64 break;
65 case V4L2_PIX_FMT_VP8:
66 codec = HFI_VIDEO_CODEC_VP8;
67 break;
68 case V4L2_PIX_FMT_VP9:
69 codec = HFI_VIDEO_CODEC_VP9;
70 break;
71 case V4L2_PIX_FMT_XVID:
72 codec = HFI_VIDEO_CODEC_DIVX;
73 break;
74 case V4L2_PIX_FMT_HEVC:
75 codec = HFI_VIDEO_CODEC_HEVC;
76 break;
77 default:
> 78 dprintk(WARN, "Unknown format:%x\n", pixfmt);
79 return false;
80 }
81
82 if (session_type == VIDC_SESSION_TYPE_ENC && core->enc_codecs & codec)
83 return true;
84
85 if (session_type == VIDC_SESSION_TYPE_DEC && core->dec_codecs & codec)
86 return true;
87
88 return false;
89 }
90 EXPORT_SYMBOL_GPL(venus_helper_check_codec);
91

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

Attachment: .config.gz
Description: application/gzip