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
