From: kernel test robot <lkp@intel.com>
To: michael.nemanov@ti.com
Cc: oe-kbuild-all@lists.linux.dev
Subject: Re: [RFC PATCH 17/17] Add Kconfig, Makefile and integrate into wireless/ti folder
Date: Tue, 14 May 2024 12:33:59 +0800 [thread overview]
Message-ID: <202405141253.rZTcUmiw-lkp@intel.com> (raw)
In-Reply-To: <20240512183247.2190242-18-michael.nemanov@ti.com>
Hi,
[This is a private test report for your RFC patch.]
kernel test robot noticed the following build warnings:
[auto build test WARNING on 83127ecada257e27f4740dbca9644dd0e838bc36]
url: https://github.com/intel-lab-lkp/linux/commits/michael-nemanov-ti-com/Add-cc33xx-h-cc33xx_i-h/20240513-023728
base: 83127ecada257e27f4740dbca9644dd0e838bc36
patch link: https://lore.kernel.org/r/20240512183247.2190242-18-michael.nemanov%40ti.com
patch subject: [RFC PATCH 17/17] Add Kconfig, Makefile and integrate into wireless/ti folder
config: alpha-allyesconfig (https://download.01.org/0day-ci/archive/20240514/202405141253.rZTcUmiw-lkp@intel.com/config)
compiler: alpha-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240514/202405141253.rZTcUmiw-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202405141253.rZTcUmiw-lkp@intel.com/
All warnings (new ones prefixed by >>):
In file included from arch/alpha/include/asm/bug.h:23,
from include/linux/bug.h:5,
from include/linux/thread_info.h:13,
from include/asm-generic/current.h:6,
from ./arch/alpha/include/generated/asm/current.h:1,
from include/linux/sched.h:12,
from include/linux/ratelimit.h:6,
from include/linux/dev_printk.h:16,
from include/linux/device.h:15,
from include/linux/platform_device.h:13,
from drivers/net/wireless/ti/cc33xx/main.c:6:
drivers/net/wireless/ti/cc33xx/main.c: In function 'parse_control_message':
>> drivers/net/wireless/ti/cc33xx/main.c:857:33: warning: format '%x' expects argument of type 'unsigned int', but argument 5 has type 'long unsigned int' [-Wformat=]
857 | WARN(1, "Error processing device message @ offset %x",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
858 | (size_t)(buffer - start_of_payload));
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| |
| long unsigned int
include/asm-generic/bug.h:99:62: note: in definition of macro '__WARN_printf'
99 | warn_slowpath_fmt(__FILE__, __LINE__, taint, arg); \
| ^~~
drivers/net/wireless/ti/cc33xx/main.c:857:25: note: in expansion of macro 'WARN'
857 | WARN(1, "Error processing device message @ offset %x",
| ^~~~
drivers/net/wireless/ti/cc33xx/main.c:857:76: note: format string is defined here
857 | WARN(1, "Error processing device message @ offset %x",
| ~^
| |
| unsigned int
| %lx
drivers/net/wireless/ti/cc33xx/main.c: At top level:
>> drivers/net/wireless/ti/cc33xx/main.c:199:36: warning: 'cc33xx_clk_table' defined but not used [-Wunused-const-variable=]
199 | static const struct cc33xx_clk_cfg cc33xx_clk_table[NUM_CLOCK_CONFIGS] = {
| ^~~~~~~~~~~~~~~~
>> drivers/net/wireless/ti/cc33xx/main.c:187:36: warning: 'cc33xx_clk_table_coex' defined but not used [-Wunused-const-variable=]
187 | static const struct cc33xx_clk_cfg cc33xx_clk_table_coex[NUM_CLOCK_CONFIGS] = {
| ^~~~~~~~~~~~~~~~~~~~~
In file included from drivers/net/wireless/ti/cc33xx/main.c:13:
>> drivers/net/wireless/ti/cc33xx/acx.h:825:18: warning: 'cc33xx_idx_to_rate_100kbps' defined but not used [-Wunused-const-variable=]
825 | static const u16 cc33xx_idx_to_rate_100kbps[] = {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
--
In file included from include/asm-generic/bug.h:22,
from arch/alpha/include/asm/bug.h:23,
from include/linux/bug.h:5,
from include/net/mac80211.h:16,
from drivers/net/wireless/ti/cc33xx/cc33xx_i.h:8,
from drivers/net/wireless/ti/cc33xx/cc33xx.h:8,
from drivers/net/wireless/ti/cc33xx/cmd.h:8,
from drivers/net/wireless/ti/cc33xx/acx.h:8,
from drivers/net/wireless/ti/cc33xx/cmd.c:5:
drivers/net/wireless/ti/cc33xx/cmd.c: In function '__cc33xx_cmd_send':
>> drivers/net/wireless/ti/cc33xx/debug.h:8:21: warning: format '%d' expects argument of type 'int', but argument 4 has type 'size_t' {aka 'long unsigned int'} [-Wformat=]
8 | #define DRIVER_NAME "cc33xx"
| ^~~~~~~~
include/linux/printk.h:347:21: note: in definition of macro 'pr_fmt'
347 | #define pr_fmt(fmt) fmt
| ^~~
include/linux/dynamic_debug.h:248:9: note: in expansion of macro '__dynamic_func_call_cls'
248 | __dynamic_func_call_cls(__UNIQUE_ID(ddebug), cls, fmt, func, ##__VA_ARGS__)
| ^~~~~~~~~~~~~~~~~~~~~~~
include/linux/dynamic_debug.h:250:9: note: in expansion of macro '_dynamic_func_call_cls'
250 | _dynamic_func_call_cls(_DPRINTK_CLASS_DFLT, fmt, func, ##__VA_ARGS__)
| ^~~~~~~~~~~~~~~~~~~~~~
include/linux/dynamic_debug.h:269:9: note: in expansion of macro '_dynamic_func_call'
269 | _dynamic_func_call(fmt, __dynamic_pr_debug, \
| ^~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debug.h:60:25: note: in expansion of macro 'dynamic_pr_debug'
60 | dynamic_pr_debug(DRIVER_PREFIX fmt "\n", ##arg); \
| ^~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debug.h:9:23: note: in expansion of macro 'DRIVER_NAME'
9 | #define DRIVER_PREFIX DRIVER_NAME ": "
| ^~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debug.h:60:42: note: in expansion of macro 'DRIVER_PREFIX'
60 | dynamic_pr_debug(DRIVER_PREFIX fmt "\n", ##arg); \
| ^~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/cmd.c:112:17: note: in expansion of macro 'cc33xx_debug'
112 | cc33xx_debug(DEBUG_CMD,
| ^~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/cmd.c: In function 'cc33xx_cmd_send_failsafe':
>> drivers/net/wireless/ti/cc33xx/debug.h:8:21: warning: format '%d' expects argument of type 'int', but argument 4 has type 'size_t' {aka 'long unsigned int'} [-Wformat=]
8 | #define DRIVER_NAME "cc33xx"
| ^~~~~~~~
include/linux/printk.h:347:21: note: in definition of macro 'pr_fmt'
347 | #define pr_fmt(fmt) fmt
| ^~~
include/linux/dynamic_debug.h:248:9: note: in expansion of macro '__dynamic_func_call_cls'
248 | __dynamic_func_call_cls(__UNIQUE_ID(ddebug), cls, fmt, func, ##__VA_ARGS__)
| ^~~~~~~~~~~~~~~~~~~~~~~
include/linux/dynamic_debug.h:250:9: note: in expansion of macro '_dynamic_func_call_cls'
250 | _dynamic_func_call_cls(_DPRINTK_CLASS_DFLT, fmt, func, ##__VA_ARGS__)
| ^~~~~~~~~~~~~~~~~~~~~~
include/linux/dynamic_debug.h:269:9: note: in expansion of macro '_dynamic_func_call'
269 | _dynamic_func_call(fmt, __dynamic_pr_debug, \
| ^~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debug.h:60:25: note: in expansion of macro 'dynamic_pr_debug'
60 | dynamic_pr_debug(DRIVER_PREFIX fmt "\n", ##arg); \
| ^~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debug.h:9:23: note: in expansion of macro 'DRIVER_NAME'
9 | #define DRIVER_PREFIX DRIVER_NAME ": "
| ^~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debug.h:60:42: note: in expansion of macro 'DRIVER_PREFIX'
60 | dynamic_pr_debug(DRIVER_PREFIX fmt "\n", ##arg); \
| ^~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/cmd.c:146:9: note: in expansion of macro 'cc33xx_debug'
146 | cc33xx_debug(DEBUG_TESTMODE, "CMD# %d, len=%d", id, len);
| ^~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/acx.h: At top level:
>> drivers/net/wireless/ti/cc33xx/acx.h:825:18: warning: 'cc33xx_idx_to_rate_100kbps' defined but not used [-Wunused-const-variable=]
825 | static const u16 cc33xx_idx_to_rate_100kbps[] = {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
--
In file included from drivers/net/wireless/ti/cc33xx/event.c:5:
>> drivers/net/wireless/ti/cc33xx/acx.h:825:18: warning: 'cc33xx_idx_to_rate_100kbps' defined but not used [-Wunused-const-variable=]
825 | static const u16 cc33xx_idx_to_rate_100kbps[] = {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
--
drivers/net/wireless/ti/cc33xx/tx.c: In function 'cc33xx_tx_fill_hdr':
>> drivers/net/wireless/ti/cc33xx/tx.c:326:13: warning: variable 'hosttime' set but not used [-Wunused-but-set-variable]
326 | s64 hosttime;
| ^~~~~~~~
In file included from drivers/net/wireless/ti/cc33xx/tx.c:5:
drivers/net/wireless/ti/cc33xx/acx.h: At top level:
>> drivers/net/wireless/ti/cc33xx/acx.h:825:18: warning: 'cc33xx_idx_to_rate_100kbps' defined but not used [-Wunused-const-variable=]
825 | static const u16 cc33xx_idx_to_rate_100kbps[] = {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
--
>> drivers/net/wireless/ti/cc33xx/cmd.c:896: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
* send test command to firmware
drivers/net/wireless/ti/cc33xx/cmd.c:924: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
* read acx from firmware
drivers/net/wireless/ti/cc33xx/cmd.c:952: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
* read debug acx from firmware
drivers/net/wireless/ti/cc33xx/cmd.c:980: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
* write acx value to firmware
drivers/net/wireless/ti/cc33xx/cmd.c:1029: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
* write acx value to firmware
--
drivers/net/wireless/ti/cc33xx/rx.c: In function 'cc33xx_rx_get_packet_data':
>> drivers/net/wireless/ti/cc33xx/rx.c:165:12: warning: variable 'is_data' set but not used [-Wunused-but-set-variable]
165 | u8 is_data = 0;
| ^~~~~~~
drivers/net/wireless/ti/cc33xx/rx.c: In function 'cc33xx_rx_filter_enable':
>> drivers/net/wireless/ti/cc33xx/rx.c:391:1: warning: control reaches end of non-void function [-Wreturn-type]
391 | }
| ^
drivers/net/wireless/ti/cc33xx/rx.c: In function 'cc33xx_rx_filter_clear_all':
drivers/net/wireless/ti/cc33xx/rx.c:393:52: warning: control reaches end of non-void function [-Wreturn-type]
393 | int cc33xx_rx_filter_clear_all(struct cc33xx *cc) {}
| ^
In file included from drivers/net/wireless/ti/cc33xx/rx.c:5:
drivers/net/wireless/ti/cc33xx/acx.h: At top level:
>> drivers/net/wireless/ti/cc33xx/acx.h:825:18: warning: 'cc33xx_idx_to_rate_100kbps' defined but not used [-Wunused-const-variable=]
825 | static const u16 cc33xx_idx_to_rate_100kbps[] = {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
--
drivers/net/wireless/ti/cc33xx/init.c: In function 'cc33xx_init_vif_specific':
>> drivers/net/wireless/ti/cc33xx/init.c:109:29: warning: variable 'conf_tid' set but not used [-Wunused-but-set-variable]
109 | struct conf_tx_tid *conf_tid;
| ^~~~~~~~
drivers/net/wireless/ti/cc33xx/init.c: In function 'cc33xx_hw_init':
>> drivers/net/wireless/ti/cc33xx/init.c:196:13: warning: variable 'ret' set but not used [-Wunused-but-set-variable]
196 | int ret = 0;
| ^~~
In file included from include/asm-generic/bug.h:22,
from arch/alpha/include/asm/bug.h:23,
from include/linux/bug.h:5,
from include/linux/thread_info.h:13,
from include/asm-generic/preempt.h:5,
from ./arch/alpha/include/generated/asm/preempt.h:1,
from include/linux/preempt.h:79,
from include/linux/spinlock.h:56,
from include/linux/mmzone.h:8,
from include/linux/gfp.h:7,
from include/linux/firmware.h:8,
from drivers/net/wireless/ti/cc33xx/init.c:5:
drivers/net/wireless/ti/cc33xx/init.c: In function 'cc33xx_download_ini_params_and_wait':
>> drivers/net/wireless/ti/cc33xx/debug.h:8:21: warning: format '%d' expects argument of type 'int', but argument 3 has type 'long unsigned int' [-Wformat=]
8 | #define DRIVER_NAME "cc33xx"
| ^~~~~~~~
include/linux/printk.h:347:21: note: in definition of macro 'pr_fmt'
347 | #define pr_fmt(fmt) fmt
| ^~~
include/linux/dynamic_debug.h:248:9: note: in expansion of macro '__dynamic_func_call_cls'
248 | __dynamic_func_call_cls(__UNIQUE_ID(ddebug), cls, fmt, func, ##__VA_ARGS__)
| ^~~~~~~~~~~~~~~~~~~~~~~
include/linux/dynamic_debug.h:250:9: note: in expansion of macro '_dynamic_func_call_cls'
250 | _dynamic_func_call_cls(_DPRINTK_CLASS_DFLT, fmt, func, ##__VA_ARGS__)
| ^~~~~~~~~~~~~~~~~~~~~~
include/linux/dynamic_debug.h:269:9: note: in expansion of macro '_dynamic_func_call'
269 | _dynamic_func_call(fmt, __dynamic_pr_debug, \
| ^~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debug.h:60:25: note: in expansion of macro 'dynamic_pr_debug'
60 | dynamic_pr_debug(DRIVER_PREFIX fmt "\n", ##arg); \
| ^~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debug.h:9:23: note: in expansion of macro 'DRIVER_NAME'
9 | #define DRIVER_PREFIX DRIVER_NAME ": "
| ^~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debug.h:60:42: note: in expansion of macro 'DRIVER_PREFIX'
60 | dynamic_pr_debug(DRIVER_PREFIX fmt "\n", ##arg); \
| ^~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/init.c:214:9: note: in expansion of macro 'cc33xx_debug'
214 | cc33xx_debug(DEBUG_ACX,
| ^~~~~~~~~~~~
In file included from drivers/net/wireless/ti/cc33xx/init.c:6:
drivers/net/wireless/ti/cc33xx/acx.h: At top level:
>> drivers/net/wireless/ti/cc33xx/acx.h:825:18: warning: 'cc33xx_idx_to_rate_100kbps' defined but not used [-Wunused-const-variable=]
825 | static const u16 cc33xx_idx_to_rate_100kbps[] = {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
--
In file included from drivers/net/wireless/ti/cc33xx/debugfs.c:5:
drivers/net/wireless/ti/cc33xx/debugfs.c: In function 'tx_tx_retry_per_rate_read':
>> drivers/net/wireless/ti/cc33xx/debugfs.h:71:13: warning: variable 'res' set but not used [-Wunused-but-set-variable]
71 | int res, i; \
| ^~~
drivers/net/wireless/ti/cc33xx/debugfs.c:15:9: note: in expansion of macro 'DEBUGFS_FWSTATS_FILE_ARRAY'
15 | DEBUGFS_FWSTATS_FILE_ARRAY(a, b, c, cc33xx_acx_statistics)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.c:52:1: note: in expansion of macro 'CC33XX_DEBUGFS_FWSTATS_FILE_ARRAY'
52 | CC33XX_DEBUGFS_FWSTATS_FILE_ARRAY(tx, tx_retry_per_rate,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.c: In function 'pwr_cont_miss_bcns_spread_read':
>> drivers/net/wireless/ti/cc33xx/debugfs.h:71:13: warning: variable 'res' set but not used [-Wunused-but-set-variable]
71 | int res, i; \
| ^~~
drivers/net/wireless/ti/cc33xx/debugfs.c:15:9: note: in expansion of macro 'DEBUGFS_FWSTATS_FILE_ARRAY'
15 | DEBUGFS_FWSTATS_FILE_ARRAY(a, b, c, cc33xx_acx_statistics)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.c:103:1: note: in expansion of macro 'CC33XX_DEBUGFS_FWSTATS_FILE_ARRAY'
103 | CC33XX_DEBUGFS_FWSTATS_FILE_ARRAY(pwr, cont_miss_bcns_spread,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.c: In function 'rx_rate_rx_frames_per_rates_read':
>> drivers/net/wireless/ti/cc33xx/debugfs.h:71:13: warning: variable 'res' set but not used [-Wunused-but-set-variable]
71 | int res, i; \
| ^~~
drivers/net/wireless/ti/cc33xx/debugfs.c:15:9: note: in expansion of macro 'DEBUGFS_FWSTATS_FILE_ARRAY'
15 | DEBUGFS_FWSTATS_FILE_ARRAY(a, b, c, cc33xx_acx_statistics)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.c:124:1: note: in expansion of macro 'CC33XX_DEBUGFS_FWSTATS_FILE_ARRAY'
124 | CC33XX_DEBUGFS_FWSTATS_FILE_ARRAY(rx_rate, rx_frames_per_rates, 50);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.c: In function 'aggr_size_tx_agg_rate_read':
>> drivers/net/wireless/ti/cc33xx/debugfs.h:71:13: warning: variable 'res' set but not used [-Wunused-but-set-variable]
71 | int res, i; \
| ^~~
drivers/net/wireless/ti/cc33xx/debugfs.c:15:9: note: in expansion of macro 'DEBUGFS_FWSTATS_FILE_ARRAY'
15 | DEBUGFS_FWSTATS_FILE_ARRAY(a, b, c, cc33xx_acx_statistics)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.c:126:1: note: in expansion of macro 'CC33XX_DEBUGFS_FWSTATS_FILE_ARRAY'
126 | CC33XX_DEBUGFS_FWSTATS_FILE_ARRAY(aggr_size, tx_agg_rate,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.c: In function 'aggr_size_tx_agg_len_read':
>> drivers/net/wireless/ti/cc33xx/debugfs.h:71:13: warning: variable 'res' set but not used [-Wunused-but-set-variable]
71 | int res, i; \
| ^~~
drivers/net/wireless/ti/cc33xx/debugfs.c:15:9: note: in expansion of macro 'DEBUGFS_FWSTATS_FILE_ARRAY'
15 | DEBUGFS_FWSTATS_FILE_ARRAY(a, b, c, cc33xx_acx_statistics)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.c:128:1: note: in expansion of macro 'CC33XX_DEBUGFS_FWSTATS_FILE_ARRAY'
128 | CC33XX_DEBUGFS_FWSTATS_FILE_ARRAY(aggr_size, tx_agg_len,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.c: In function 'aggr_size_rx_size_read':
>> drivers/net/wireless/ti/cc33xx/debugfs.h:71:13: warning: variable 'res' set but not used [-Wunused-but-set-variable]
71 | int res, i; \
| ^~~
drivers/net/wireless/ti/cc33xx/debugfs.c:15:9: note: in expansion of macro 'DEBUGFS_FWSTATS_FILE_ARRAY'
15 | DEBUGFS_FWSTATS_FILE_ARRAY(a, b, c, cc33xx_acx_statistics)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.c:130:1: note: in expansion of macro 'CC33XX_DEBUGFS_FWSTATS_FILE_ARRAY'
130 | CC33XX_DEBUGFS_FWSTATS_FILE_ARRAY(aggr_size, rx_size,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.c: In function 'pipeline_pipeline_fifo_full_read':
>> drivers/net/wireless/ti/cc33xx/debugfs.h:71:13: warning: variable 'res' set but not used [-Wunused-but-set-variable]
71 | int res, i; \
| ^~~
drivers/net/wireless/ti/cc33xx/debugfs.c:15:9: note: in expansion of macro 'DEBUGFS_FWSTATS_FILE_ARRAY'
15 | DEBUGFS_FWSTATS_FILE_ARRAY(a, b, c, cc33xx_acx_statistics)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.c:146:1: note: in expansion of macro 'CC33XX_DEBUGFS_FWSTATS_FILE_ARRAY'
146 | CC33XX_DEBUGFS_FWSTATS_FILE_ARRAY(pipeline, pipeline_fifo_full,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.c: In function 'diversity_num_of_packets_per_ant_read':
>> drivers/net/wireless/ti/cc33xx/debugfs.h:71:13: warning: variable 'res' set but not used [-Wunused-but-set-variable]
71 | int res, i; \
| ^~~
drivers/net/wireless/ti/cc33xx/debugfs.c:15:9: note: in expansion of macro 'DEBUGFS_FWSTATS_FILE_ARRAY'
15 | DEBUGFS_FWSTATS_FILE_ARRAY(a, b, c, cc33xx_acx_statistics)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.c:149:1: note: in expansion of macro 'CC33XX_DEBUGFS_FWSTATS_FILE_ARRAY'
149 | CC33XX_DEBUGFS_FWSTATS_FILE_ARRAY(diversity, num_of_packets_per_ant,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.c: In function 'calib_fail_count_read':
>> drivers/net/wireless/ti/cc33xx/debugfs.h:71:13: warning: variable 'res' set but not used [-Wunused-but-set-variable]
71 | int res, i; \
| ^~~
drivers/net/wireless/ti/cc33xx/debugfs.c:15:9: note: in expansion of macro 'DEBUGFS_FWSTATS_FILE_ARRAY'
15 | DEBUGFS_FWSTATS_FILE_ARRAY(a, b, c, cc33xx_acx_statistics)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.c:160:1: note: in expansion of macro 'CC33XX_DEBUGFS_FWSTATS_FILE_ARRAY'
160 | CC33XX_DEBUGFS_FWSTATS_FILE_ARRAY(calib, fail_count,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.c: In function 'cc33xx_debugfs_add_files_helper':
drivers/net/wireless/ti/cc33xx/debugfs.h:37:37: error: 'cc' undeclared (first use in this function)
37 | cc, &name## _ops)
| ^~
drivers/net/wireless/ti/cc33xx/debugfs.c:399:9: note: in expansion of macro 'DEBUGFS_ADD'
399 | DEBUGFS_ADD(radar_debug_mode, moddir);
| ^~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.h:37:37: note: each undeclared identifier is reported only once for each function it appears in
37 | cc, &name## _ops)
| ^~
drivers/net/wireless/ti/cc33xx/debugfs.c:399:9: note: in expansion of macro 'DEBUGFS_ADD'
399 | DEBUGFS_ADD(radar_debug_mode, moddir);
| ^~~~~~~~~~~
drivers/net/wireless/ti/cc33xx/debugfs.c: In function 'ble_enable_write':
>> drivers/net/wireless/ti/cc33xx/debugfs.c:1182:13: warning: variable 'ret' set but not used [-Wunused-but-set-variable]
1182 | int ret;
| ^~~
In file included from drivers/net/wireless/ti/cc33xx/debugfs.c:6:
drivers/net/wireless/ti/cc33xx/acx.h: At top level:
>> drivers/net/wireless/ti/cc33xx/acx.h:825:18: warning: 'cc33xx_idx_to_rate_100kbps' defined but not used [-Wunused-const-variable=]
825 | static const u16 cc33xx_idx_to_rate_100kbps[] = {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
..
vim +857 drivers/net/wireless/ti/cc33xx/main.c
53500b0b54c6a11 Michael Nemanov 2024-05-12 802
53500b0b54c6a11 Michael Nemanov 2024-05-12 803 static int parse_control_message(struct cc33xx *cc,
53500b0b54c6a11 Michael Nemanov 2024-05-12 804 const u8 *buffer, size_t buffer_length)
53500b0b54c6a11 Michael Nemanov 2024-05-12 805 {
53500b0b54c6a11 Michael Nemanov 2024-05-12 806 u8 *const end_of_payload = (u8 *const)buffer + buffer_length;
53500b0b54c6a11 Michael Nemanov 2024-05-12 807 u8 *const start_of_payload = (u8 *const)buffer;
53500b0b54c6a11 Michael Nemanov 2024-05-12 808 struct control_info_descriptor *control_info_descriptor;
53500b0b54c6a11 Michael Nemanov 2024-05-12 809 const u8 *event_data, *cmd_result_data;
53500b0b54c6a11 Michael Nemanov 2024-05-12 810 int ctrl_info_type, ctrl_info_length;
53500b0b54c6a11 Michael Nemanov 2024-05-12 811
53500b0b54c6a11 Michael Nemanov 2024-05-12 812 while (buffer < end_of_payload) {
53500b0b54c6a11 Michael Nemanov 2024-05-12 813 control_info_descriptor =
53500b0b54c6a11 Michael Nemanov 2024-05-12 814 (struct control_info_descriptor *)buffer;
53500b0b54c6a11 Michael Nemanov 2024-05-12 815
53500b0b54c6a11 Michael Nemanov 2024-05-12 816 ctrl_info_type = get_type(control_info_descriptor);
53500b0b54c6a11 Michael Nemanov 2024-05-12 817 ctrl_info_length = get_length(control_info_descriptor);
53500b0b54c6a11 Michael Nemanov 2024-05-12 818
53500b0b54c6a11 Michael Nemanov 2024-05-12 819 cc33xx_debug(DEBUG_CMD, "Processing message type %d, len %d",
53500b0b54c6a11 Michael Nemanov 2024-05-12 820 ctrl_info_type, ctrl_info_length);
53500b0b54c6a11 Michael Nemanov 2024-05-12 821
53500b0b54c6a11 Michael Nemanov 2024-05-12 822 switch (ctrl_info_type) {
53500b0b54c6a11 Michael Nemanov 2024-05-12 823 case CTRL_MSG_EVENT:
53500b0b54c6a11 Michael Nemanov 2024-05-12 824 event_data = buffer + sizeof(*control_info_descriptor);
53500b0b54c6a11 Michael Nemanov 2024-05-12 825
53500b0b54c6a11 Michael Nemanov 2024-05-12 826 deffer_event(cc, event_data, ctrl_info_length);
53500b0b54c6a11 Michael Nemanov 2024-05-12 827 break;
53500b0b54c6a11 Michael Nemanov 2024-05-12 828
53500b0b54c6a11 Michael Nemanov 2024-05-12 829 case CTRL_MSG_COMMND_COMPLETE:
53500b0b54c6a11 Michael Nemanov 2024-05-12 830 cmd_result_data = buffer;
53500b0b54c6a11 Michael Nemanov 2024-05-12 831 cmd_result_data += sizeof(*control_info_descriptor);
53500b0b54c6a11 Michael Nemanov 2024-05-12 832
53500b0b54c6a11 Michael Nemanov 2024-05-12 833 if (ctrl_info_length > sizeof(cc->command_result)) {
53500b0b54c6a11 Michael Nemanov 2024-05-12 834 print_hex_dump(KERN_DEBUG, "message dump:",
53500b0b54c6a11 Michael Nemanov 2024-05-12 835 DUMP_PREFIX_OFFSET, 16, 1,
53500b0b54c6a11 Michael Nemanov 2024-05-12 836 cmd_result_data,
53500b0b54c6a11 Michael Nemanov 2024-05-12 837 ctrl_info_length, false);
53500b0b54c6a11 Michael Nemanov 2024-05-12 838
53500b0b54c6a11 Michael Nemanov 2024-05-12 839 WARN(1, "Error device response exceeds result buffer size");
53500b0b54c6a11 Michael Nemanov 2024-05-12 840
53500b0b54c6a11 Michael Nemanov 2024-05-12 841 goto message_parse_error;
53500b0b54c6a11 Michael Nemanov 2024-05-12 842 }
53500b0b54c6a11 Michael Nemanov 2024-05-12 843
53500b0b54c6a11 Michael Nemanov 2024-05-12 844 memcpy(cc->command_result,
53500b0b54c6a11 Michael Nemanov 2024-05-12 845 cmd_result_data, ctrl_info_length);
53500b0b54c6a11 Michael Nemanov 2024-05-12 846
53500b0b54c6a11 Michael Nemanov 2024-05-12 847 cc->result_length = ctrl_info_length;
53500b0b54c6a11 Michael Nemanov 2024-05-12 848
53500b0b54c6a11 Michael Nemanov 2024-05-12 849 complete(&cc->command_complete);
53500b0b54c6a11 Michael Nemanov 2024-05-12 850 break;
53500b0b54c6a11 Michael Nemanov 2024-05-12 851
53500b0b54c6a11 Michael Nemanov 2024-05-12 852 default:
53500b0b54c6a11 Michael Nemanov 2024-05-12 853 print_hex_dump(KERN_DEBUG, "message dump:",
53500b0b54c6a11 Michael Nemanov 2024-05-12 854 DUMP_PREFIX_OFFSET, 16, 1,
53500b0b54c6a11 Michael Nemanov 2024-05-12 855 start_of_payload, buffer_length, false);
53500b0b54c6a11 Michael Nemanov 2024-05-12 856
53500b0b54c6a11 Michael Nemanov 2024-05-12 @857 WARN(1, "Error processing device message @ offset %x",
53500b0b54c6a11 Michael Nemanov 2024-05-12 858 (size_t)(buffer - start_of_payload));
53500b0b54c6a11 Michael Nemanov 2024-05-12 859
53500b0b54c6a11 Michael Nemanov 2024-05-12 860 goto message_parse_error;
53500b0b54c6a11 Michael Nemanov 2024-05-12 861 }
53500b0b54c6a11 Michael Nemanov 2024-05-12 862
53500b0b54c6a11 Michael Nemanov 2024-05-12 863 buffer += sizeof(*control_info_descriptor);
53500b0b54c6a11 Michael Nemanov 2024-05-12 864 buffer += ctrl_info_length;
53500b0b54c6a11 Michael Nemanov 2024-05-12 865 }
53500b0b54c6a11 Michael Nemanov 2024-05-12 866
53500b0b54c6a11 Michael Nemanov 2024-05-12 867 return 0;
53500b0b54c6a11 Michael Nemanov 2024-05-12 868
53500b0b54c6a11 Michael Nemanov 2024-05-12 869 message_parse_error:
53500b0b54c6a11 Michael Nemanov 2024-05-12 870 return -EIO;
53500b0b54c6a11 Michael Nemanov 2024-05-12 871 }
53500b0b54c6a11 Michael Nemanov 2024-05-12 872
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2024-05-14 4:34 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-05-12 18:32 [RFC PATCH 00/17] wifi: cc33xx: Add driver for new TI CC33xx wireless device family michael.nemanov
2024-05-12 18:32 ` [RFC PATCH 01/17] Add cc33xx.h, cc33xx_i.h michael.nemanov
2024-05-12 18:32 ` [RFC PATCH 02/17] Add debug.h michael.nemanov
2024-05-12 18:32 ` [RFC PATCH 03/17] Add sdio.c, io.c, io.h michael.nemanov
2024-05-12 18:32 ` [RFC PATCH 04/17] Add cmd.c, cmd.h michael.nemanov
2024-05-12 18:32 ` [RFC PATCH 05/17] Add acx.c, acx.h michael.nemanov
2024-05-12 18:32 ` [RFC PATCH 06/17] Add event.c, event.h michael.nemanov
2024-05-12 18:32 ` [RFC PATCH 07/17] Add boot.c, boot.h michael.nemanov
2024-05-12 18:32 ` [RFC PATCH 08/17] Add main.c michael.nemanov
2024-05-12 18:32 ` [RFC PATCH 09/17] Add rx.c, rx.h michael.nemanov
2024-05-12 18:32 ` [RFC PATCH 10/17] Add tx.c, tx.h michael.nemanov
2024-05-12 18:32 ` [RFC PATCH 11/17] Add init.c, init.h michael.nemanov
2024-05-12 18:32 ` [RFC PATCH 12/17] Add debugfs.c, debugfs.h michael.nemanov
2024-05-12 18:32 ` [RFC PATCH 13/17] Add scan.c, scan.h michael.nemanov
2024-05-12 18:32 ` [RFC PATCH 14/17] Add conf.h michael.nemanov
2024-05-12 18:32 ` [RFC PATCH 15/17] Add ps.c, ps.h michael.nemanov
2024-05-12 18:32 ` [RFC PATCH 16/17] Add testmode.c, testmode.h michael.nemanov
2024-05-12 18:32 ` [RFC PATCH 17/17] Add Kconfig, Makefile and integrate into wireless/ti folder michael.nemanov
2024-05-13 6:51 ` kernel test robot
2024-05-13 7:46 ` kernel test robot
2024-05-14 4:33 ` kernel test robot [this message]
2024-05-13 6:42 ` [RFC PATCH 00/17] wifi: cc33xx: Add driver for new TI CC33xx wireless device family Kalle Valo
2024-05-13 11:27 ` Nemanov, Michael
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=202405141253.rZTcUmiw-lkp@intel.com \
--to=lkp@intel.com \
--cc=michael.nemanov@ti.com \
--cc=oe-kbuild-all@lists.linux.dev \
/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.