samsung-sm8650/android_kernel_samsung_sm8650-modules
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

qcacmn: Fix compilation with WMI_INTERFACE_EVENT_LOGGING disabled

Browse Source 
Compilation error occurs due to undefined struct when
WMI_INTERFACE_EVENT_LOGGING disabled.

Use void pointer as argument to wmi_mgmt_cmd_record() and cast as header
struct inside logging function to avoid casting as undeclared type.

Change-Id: I7d0922ee2009b235473febdbcbc3317e432a1386
CRs-Fixed: 1043597
This commit is contained in:
Sandeep Puligilla
2016-07-19 13:20:57 -07:00
committed by Vishwajith Upendra
parent 3ba23ea33f
commit 828a45f7f1
2 changed files with 8 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
wmi_unified.c
View File

@@ -892,20 +892,21 @@ static QDF_STATUS wmi_debugfs_init(wmi_unified_t wmi_handle)
* *
* @wmi_handle: wmi handle * @wmi_handle: wmi handle
* @cmd: mgmt command * @cmd: mgmt command
* @type: 802.11 frame type * @header: pointer to 802.11 header
* @subtype: 802.11 fram subtype
* @vdev_id: vdev id * @vdev_id: vdev id
* @chanfreq: channel frequency * @chanfreq: channel frequency
* *
* Return: none * Return: none
*/ */
void wmi_mgmt_cmd_record(wmi_unified_t wmi_handle, WMI_CMD_ID cmd, void wmi_mgmt_cmd_record(wmi_unified_t wmi_handle, WMI_CMD_ID cmd,
uint32_t type, uint32_t subtype, void *header, uint32_t vdev_id, uint32_t chanfreq)
uint32_t vdev_id, uint32_t chanfreq)
{ {
qdf_spin_lock_bh(&wmi_handle->log_info.wmi_record_lock); qdf_spin_lock_bh(&wmi_handle->log_info.wmi_record_lock);
WMI_MGMT_COMMAND_RECORD(cmd, type, subtype, vdev_id, chanfreq); WMI_MGMT_COMMAND_RECORD(cmd,
((struct wmi_command_header *)header)->type,
((struct wmi_command_header *)header)->sub_type,
vdev_id, chanfreq);
qdf_spin_unlock_bh(&wmi_handle->log_info.wmi_record_lock); qdf_spin_unlock_bh(&wmi_handle->log_info.wmi_record_lock);
} }
@@ -920,8 +921,7 @@ void wmi_mgmt_cmd_record(wmi_unified_t wmi_handle, WMI_CMD_ID cmd,
*/ */
static void wmi_debugfs_remove(wmi_unified_t wmi_handle) { } static void wmi_debugfs_remove(wmi_unified_t wmi_handle) { }
void wmi_mgmt_cmd_record(wmi_unified_t wmi_handle, WMI_CMD_ID cmd, void wmi_mgmt_cmd_record(wmi_unified_t wmi_handle, WMI_CMD_ID cmd,
uint32_t type, uint32_t subtype, void *header, uint32_t vdev_id, uint32_t chanfreq) { }
uint32_t vdev_id, uint32_t chanfreq) { }
#endif /*WMI_INTERFACE_EVENT_LOGGING */ #endif /*WMI_INTERFACE_EVENT_LOGGING */
int wmi_get_host_credits(wmi_unified_t wmi_handle); int wmi_get_host_credits(wmi_unified_t wmi_handle);

4
wmi_unified_tlv.c
View File

@@ -1641,9 +1641,7 @@ QDF_STATUS send_mgmt_cmd_tlv(wmi_unified_t wmi_handle,
cmd->buf_len = bufp_len; cmd->buf_len = bufp_len;
wmi_mgmt_cmd_record(wmi_handle, WMI_MGMT_TX_SEND_CMDID, wmi_mgmt_cmd_record(wmi_handle, WMI_MGMT_TX_SEND_CMDID,
((struct wmi_command_header *)bufp)->type, bufp, cmd->vdev_id, cmd->chanfreq);
((struct wmi_command_header *)bufp)->sub_type,
cmd->vdev_id, cmd->chanfreq);
if (wmi_unified_cmd_send(wmi_handle, buf, cmd_len, if (wmi_unified_cmd_send(wmi_handle, buf, cmd_len,
WMI_MGMT_TX_SEND_CMDID)) { WMI_MGMT_TX_SEND_CMDID)) {