forked from luck/tmp_suning_uos_patched
mwifiex: use sizeof(array) to print_hex_dump_bytes
DBG_CMD_NUM is the number of commands, not the actual bytes of data for printing. Also remove the duplicated DBG_CMD_NUM definition. Reported-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Signed-off-by: Bing Zhao <bzhao@marvell.com> Signed-off-by: John W. Linville <linville@tuxdriver.com>
This commit is contained in:
parent
7a66205a21
commit
f575f65897
@ -918,20 +918,23 @@ mwifiex_cmd_timeout_func(unsigned long function_context)
|
||||
dev_err(adapter->dev, "last_cmd_index = %d\n",
|
||||
adapter->dbg.last_cmd_index);
|
||||
print_hex_dump_bytes("last_cmd_id: ", DUMP_PREFIX_OFFSET,
|
||||
adapter->dbg.last_cmd_id, DBG_CMD_NUM);
|
||||
adapter->dbg.last_cmd_id,
|
||||
sizeof(adapter->dbg.last_cmd_id));
|
||||
print_hex_dump_bytes("last_cmd_act: ", DUMP_PREFIX_OFFSET,
|
||||
adapter->dbg.last_cmd_act, DBG_CMD_NUM);
|
||||
adapter->dbg.last_cmd_act,
|
||||
sizeof(adapter->dbg.last_cmd_act));
|
||||
|
||||
dev_err(adapter->dev, "last_cmd_resp_index = %d\n",
|
||||
adapter->dbg.last_cmd_resp_index);
|
||||
print_hex_dump_bytes("last_cmd_resp_id: ", DUMP_PREFIX_OFFSET,
|
||||
adapter->dbg.last_cmd_resp_id,
|
||||
DBG_CMD_NUM);
|
||||
sizeof(adapter->dbg.last_cmd_resp_id));
|
||||
|
||||
dev_err(adapter->dev, "last_event_index = %d\n",
|
||||
adapter->dbg.last_event_index);
|
||||
print_hex_dump_bytes("last_event: ", DUMP_PREFIX_OFFSET,
|
||||
adapter->dbg.last_event, DBG_CMD_NUM);
|
||||
adapter->dbg.last_event,
|
||||
sizeof(adapter->dbg.last_event));
|
||||
|
||||
dev_err(adapter->dev, "data_sent=%d cmd_sent=%d\n",
|
||||
adapter->data_sent, adapter->cmd_sent);
|
||||
|
@ -115,8 +115,6 @@ enum {
|
||||
#define MWIFIEX_TYPE_DATA 0
|
||||
#define MWIFIEX_TYPE_EVENT 3
|
||||
|
||||
#define DBG_CMD_NUM 5
|
||||
|
||||
#define MAX_BITMAP_RATES_SIZE 10
|
||||
|
||||
#define MAX_CHANNEL_BAND_BG 14
|
||||
|
Loading…
Reference in New Issue
Block a user