diff --git a/wmi/src/wmi_unified_nan_tlv.c b/wmi/src/wmi_unified_nan_tlv.c index b7f2b07db899..9c6f9bc3d1b6 100644 --- a/wmi/src/wmi_unified_nan_tlv.c +++ b/wmi/src/wmi_unified_nan_tlv.c @@ -795,13 +795,15 @@ static QDF_STATUS extract_ndp_ind_tlv(wmi_unified_t wmi_handle, WMI_MAC_ADDR_TO_CHAR_ARRAY(&fixed_params->peer_discovery_mac_addr, rsp->peer_discovery_mac_addr.bytes); - WMI_LOGD("WMI_NDP_INDICATION_EVENTID(0x%X) received. vdev %d service_instance %d, ndp_instance %d, role %d, policy %d csid: %d, scid_len: %d, peer_addr: %pM, peer_disc_addr: %pM ndp_cfg - %d bytes ndp_app_info - %d bytes", + WMI_LOGD("WMI_NDP_INDICATION_EVENTID(0x%X) received. vdev %d service_instance %d, ndp_instance %d, role %d, policy %d csid: %d, scid_len: %d, peer_addr: "QDF_MAC_ADDR_FMT", peer_disc_addr: "QDF_MAC_ADDR_FMT" ndp_cfg - %d bytes ndp_app_info - %d bytes", WMI_NDP_INDICATION_EVENTID, fixed_params->vdev_id, fixed_params->service_instance_id, fixed_params->ndp_instance_id, fixed_params->self_ndp_role, fixed_params->accept_policy, fixed_params->nan_csid, - fixed_params->nan_scid_len, rsp->peer_mac_addr.bytes, - rsp->peer_discovery_mac_addr.bytes, fixed_params->ndp_cfg_len, + fixed_params->nan_scid_len, + QDF_MAC_ADDR_REF(rsp->peer_mac_addr.bytes), + QDF_MAC_ADDR_REF(rsp->peer_discovery_mac_addr.bytes), + fixed_params->ndp_cfg_len, fixed_params->ndp_app_info_len); rsp->ncs_sk_type = fixed_params->nan_csid; @@ -1007,9 +1009,10 @@ static QDF_STATUS extract_ndp_responder_rsp_tlv(wmi_unified_t wmi_handle, rsp->create_peer = fixed_params->create_peer; WMI_MAC_ADDR_TO_CHAR_ARRAY(&fixed_params->peer_ndi_mac_addr, rsp->peer_mac_addr.bytes); - WMI_LOGD("WMI_NDP_RESPONDER_RSP_EVENTID(0x%X) received. vdev_id: %d, peer_mac_addr: %pM,transaction_id: %d, status_code %d, reason_code: %d, create_peer: %d", + WMI_LOGD("WMI_NDP_RESPONDER_RSP_EVENTID(0x%X) received. vdev_id: %d, peer_mac_addr: "QDF_MAC_ADDR_FMT",transaction_id: %d, status_code %d, reason_code: %d, create_peer: %d", WMI_NDP_RESPONDER_RSP_EVENTID, fixed_params->vdev_id, - rsp->peer_mac_addr.bytes, rsp->transaction_id, + QDF_MAC_ADDR_REF(rsp->peer_mac_addr.bytes), + rsp->transaction_id, rsp->status, rsp->reason, rsp->create_peer); return QDF_STATUS_SUCCESS; diff --git a/wmi/src/wmi_unified_ocb_ut.c b/wmi/src/wmi_unified_ocb_ut.c index 5c0c9df84a1d..b3d086897c8f 100644 --- a/wmi/src/wmi_unified_ocb_ut.c +++ b/wmi/src/wmi_unified_ocb_ut.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. + * Copyright (c) 2018-2020 The Linux Foundation. All rights reserved. * * Permission to use, copy, modify, and/or distribute this software for * any purpose with or without fee is hereby granted, provided that the @@ -282,11 +282,11 @@ static QDF_STATUS fake_ocb_set_config_cmd_tlv(wmi_unified_t wmi_handle, for (i = 0; i < config->channel_count; i++) { WMI_LOGI("%s: channel info for channel %d" - " chan_freq=%d, bandwidth=%d, " QDF_MAC_ADDRESS_STR + " chan_freq=%d, bandwidth=%d, " QDF_MAC_ADDR_FMT " max_pwr=%d, min_pwr=%d, reg_pwr=%d, antenna_max=%d, " "flags=%d", __func__, i, config->channels[i].chan_freq, config->channels[i].bandwidth, - QDF_MAC_ADDR_ARRAY( + QDF_MAC_ADDR_REF( config->channels[i].mac_address.bytes), config->channels[i].max_pwr, config->channels[i].min_pwr, diff --git a/wmi/src/wmi_unified_p2p_tlv.c b/wmi/src/wmi_unified_p2p_tlv.c index 4ea6c630b0f3..387f93c3d6a8 100644 --- a/wmi/src/wmi_unified_p2p_tlv.c +++ b/wmi/src/wmi_unified_p2p_tlv.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2013-2018 The Linux Foundation. All rights reserved. + * Copyright (c) 2013-2018, 2020 The Linux Foundation. All rights reserved. * * Permission to use, copy, modify, and/or distribute this software for * any purpose with or without fee is hereby granted, provided that the @@ -250,8 +250,9 @@ send_set_mac_addr_rx_filter_cmd_tlv(wmi_unified_t wmi_handle, cmd->enable = 1; else cmd->enable = 0; - WMI_LOGD("set random mac rx vdev %d freq %d set %d %pM", - param->vdev_id, param->freq, param->set, param->mac); + WMI_LOGD("set random mac rx vdev %d freq %d set %d "QDF_MAC_ADDR_FMT, + param->vdev_id, param->freq, param->set, + QDF_MAC_ADDR_REF(param->mac)); ret = wmi_unified_cmd_send(wmi_handle, buf, len, WMI_VDEV_ADD_MAC_ADDR_TO_RX_FILTER_CMDID); if (ret) { diff --git a/wmi/src/wmi_unified_pmo_tlv.c b/wmi/src/wmi_unified_pmo_tlv.c index 4945e5e9845b..1e05e3d0f5a5 100644 --- a/wmi/src/wmi_unified_pmo_tlv.c +++ b/wmi/src/wmi_unified_pmo_tlv.c @@ -596,8 +596,9 @@ static QDF_STATUS send_add_clear_mcbc_filter_cmd_tlv(wmi_unified_t wmi_handle, cmd->vdev_id = vdev_id; WMI_CHAR_ARRAY_TO_MAC_ADDR(multicast_addr.bytes, &cmd->mcastbdcastaddr); - WMI_LOGD("Action:%d; vdev_id:%d; clearList:%d; MCBC MAC Addr: %pM", - cmd->action, vdev_id, clearList, multicast_addr.bytes); + WMI_LOGD("Action:%d; vdev_id:%d; clearList:%d; MCBC MAC Addr: "QDF_MAC_ADDR_FMT, + cmd->action, vdev_id, clearList, + QDF_MAC_ADDR_REF(multicast_addr.bytes)); wmi_mtrace(WMI_SET_MCASTBCAST_FILTER_CMDID, cmd->vdev_id, 0); err = wmi_unified_cmd_send(wmi_handle, buf, @@ -1846,14 +1847,15 @@ static QDF_STATUS send_set_app_type2_params_in_fw_cmd_tlv(wmi_unified_t wmi_hand cmd->tcp_tx_timeout_val = appType2Params->tcp_tx_timeout_val; cmd->tcp_rx_timeout_val = appType2Params->tcp_rx_timeout_val; - WMI_LOGD("%s: vdev_id %d gateway_mac %pM " + WMI_LOGD("%s: vdev_id %d gateway_mac "QDF_MAC_ADDR_FMT" " "rc4_key %.16s rc4_key_len %u " "ip_id %x ip_device_ip %x ip_server_ip %x " "tcp_src_port %u tcp_dst_port %u tcp_seq %u " "tcp_ack_seq %u keepalive_init %u keepalive_min %u " "keepalive_max %u keepalive_inc %u " "tcp_tx_timeout_val %u tcp_rx_timeout_val %u", - __func__, cmd->vdev_id, appType2Params->gateway_mac.bytes, + __func__, cmd->vdev_id, + QDF_MAC_ADDR_REF(appType2Params->gateway_mac.bytes), cmd->rc4_key, cmd->rc4_key_len, cmd->ip_id, cmd->ip_device_ip, cmd->ip_server_ip, cmd->tcp_src_port, cmd->tcp_dst_port, cmd->tcp_seq, @@ -1911,10 +1913,11 @@ static QDF_STATUS send_app_type1_params_in_fw_cmd_tlv(wmi_unified_t wmi_handle, qdf_mem_copy(cmd->passwd, app_type1_params->password, 16); cmd->passwd_len = app_type1_params->pass_length; - WMI_LOGD("%s: vdev_id %d wakee_mac_addr %pM " + WMI_LOGD("%s: vdev_id %d wakee_mac_addr "QDF_MAC_ADDR_FMT" " "identification_id %.8s id_length %u " "password %.16s pass_length %u", - __func__, cmd->vdev_id, app_type1_params->wakee_mac_addr.bytes, + __func__, cmd->vdev_id, + QDF_MAC_ADDR_REF(app_type1_params->wakee_mac_addr.bytes), cmd->ident, cmd->ident_len, cmd->passwd, cmd->passwd_len); wmi_mtrace(WMI_EXTWOW_SET_APP_TYPE1_PARAMS_CMDID, cmd->vdev_id, 0); diff --git a/wmi/src/wmi_unified_sta_tlv.c b/wmi/src/wmi_unified_sta_tlv.c index 224893dd6a6e..7e2ffdc5e85d 100644 --- a/wmi/src/wmi_unified_sta_tlv.c +++ b/wmi/src/wmi_unified_sta_tlv.c @@ -763,10 +763,11 @@ send_update_tdls_peer_state_cmd_tlv(wmi_unified_t wmi_handle, cmd->peer_state = peer_state->peer_state; - WMI_LOGD("%s: vdev_id: %d, peermac: %pM, " + WMI_LOGD("%s: vdev_id: %d, peermac: "QDF_MAC_ADDR_FMT", " "peer_macaddr.mac_addr31to0: 0x%x, " "peer_macaddr.mac_addr47to32: 0x%x, peer_state: %d", - __func__, cmd->vdev_id, peer_state->peer_macaddr, + __func__, cmd->vdev_id, + QDF_MAC_ADDR_REF(peer_state->peer_macaddr), cmd->peer_macaddr.mac_addr31to0, cmd->peer_macaddr.mac_addr47to32, cmd->peer_state); @@ -971,8 +972,9 @@ static QDF_STATUS extract_vdev_tdls_ev_param_tlv(wmi_unified_t wmi_handle, return QDF_STATUS_E_INVAL; }; - WMI_LOGD("%s: tdls event, peer: %pM, type: 0x%x, reason: %d, vdev: %d", - __func__, param->peermac.bytes, param->message_type, + WMI_LOGD("%s: tdls event, peer: "QDF_MAC_ADDR_FMT", type: 0x%x, reason: %d, vdev: %d", + __func__, QDF_MAC_ADDR_REF(param->peermac.bytes), + param->message_type, param->peer_reason, param->vdev_id); return QDF_STATUS_SUCCESS; diff --git a/wmi/src/wmi_unified_tlv.c b/wmi/src/wmi_unified_tlv.c index 150458e7c3e7..25bf760c55d1 100644 --- a/wmi/src/wmi_unified_tlv.c +++ b/wmi/src/wmi_unified_tlv.c @@ -762,10 +762,9 @@ static QDF_STATUS send_vdev_create_cmd_tlv(wmi_unified_t wmi_handle, cmd->num_cfg_txrx_streams = num_bands; copy_vdev_create_pdev_id(wmi_handle, cmd, param); WMI_CHAR_ARRAY_TO_MAC_ADDR(macaddr, &cmd->vdev_macaddr); - WMI_LOGD("%s: ID = %d[pdev:%d] VAP Addr = %02x:%02x:%02x:%02x:%02x:%02x", + WMI_LOGD("%s: ID = %d[pdev:%d] VAP Addr = "QDF_MAC_ADDR_FMT, __func__, param->vdev_id, cmd->pdev_id, - macaddr[0], macaddr[1], macaddr[2], - macaddr[3], macaddr[4], macaddr[5]); + QDF_MAC_ADDR_REF(macaddr)); buf_ptr = (uint8_t *)cmd + sizeof(*cmd); WMITLV_SET_HDR(buf_ptr, WMITLV_TAG_ARRAY_STRUC, (num_bands * sizeof(wmi_vdev_txrx_streams))); @@ -1129,9 +1128,9 @@ static QDF_STATUS send_peer_flush_tids_cmd_tlv(wmi_unified_t wmi, WMI_CHAR_ARRAY_TO_MAC_ADDR(peer_addr, &cmd->peer_macaddr); cmd->peer_tid_bitmap = param->peer_tid_bitmap; cmd->vdev_id = param->vdev_id; - WMI_LOGD("%s: peer_addr %pM vdev_id %d and peer bitmap %d", __func__, - peer_addr, param->vdev_id, - param->peer_tid_bitmap); + WMI_LOGD("%s: peer_addr "QDF_MAC_ADDR_FMT" vdev_id %d and peer bitmap %d", + __func__, QDF_MAC_ADDR_REF(peer_addr), param->vdev_id, + param->peer_tid_bitmap); wmi_mtrace(WMI_PEER_FLUSH_TIDS_CMDID, cmd->vdev_id, 0); if (wmi_unified_cmd_send(wmi, buf, len, WMI_PEER_FLUSH_TIDS_CMDID)) { WMI_LOGP("%s: Failed to send flush tid command", __func__); @@ -1169,7 +1168,8 @@ static QDF_STATUS send_peer_delete_cmd_tlv(wmi_unified_t wmi, WMI_CHAR_ARRAY_TO_MAC_ADDR(peer_addr, &cmd->peer_macaddr); cmd->vdev_id = vdev_id; - WMI_LOGD("%s: peer_addr %pM vdev_id %d", __func__, peer_addr, vdev_id); + WMI_LOGD("%s: peer_addr "QDF_MAC_ADDR_FMT" vdev_id %d", + __func__, QDF_MAC_ADDR_REF(peer_addr), vdev_id); wmi_mtrace(WMI_PEER_DELETE_CMDID, cmd->vdev_id, 0); if (wmi_unified_cmd_send(wmi, buf, len, WMI_PEER_DELETE_CMDID)) { WMI_LOGP("%s: Failed to send peer delete command", __func__); @@ -1279,8 +1279,9 @@ static QDF_STATUS send_peer_param_cmd_tlv(wmi_unified_t wmi, cmd->param_id = param_id; cmd->param_value = param->param_value; - WMI_LOGD("%s: vdev_id %d peer_mac: %pM param_id: %u param_value: %x", - __func__, cmd->vdev_id, peer_addr, param->param_id, + WMI_LOGD("%s: vdev_id %d peer_mac: "QDF_MAC_ADDR_FMT" param_id: %u param_value: %x", + __func__, cmd->vdev_id, + QDF_MAC_ADDR_REF(peer_addr), param->param_id, cmd->param_value); wmi_mtrace(WMI_PEER_SET_PARAM_CMDID, cmd->vdev_id, 0); @@ -1313,8 +1314,9 @@ static QDF_STATUS send_vdev_up_cmd_tlv(wmi_unified_t wmi, int32_t len = sizeof(*cmd); WMI_LOGD("%s: VDEV_UP", __func__); - WMI_LOGD("%s: vdev_id %d aid %d bssid %pM", __func__, - params->vdev_id, params->assoc_id, bssid); + WMI_LOGD("%s: vdev_id %d aid %d bssid "QDF_MAC_ADDR_FMT, + __func__, + params->vdev_id, params->assoc_id, QDF_MAC_ADDR_REF(bssid)); buf = wmi_buf_alloc(wmi, len); if (!buf) return QDF_STATUS_E_NOMEM; @@ -1374,8 +1376,9 @@ static QDF_STATUS send_peer_create_cmd_tlv(wmi_unified_t wmi, wmi_buf_free(buf); return QDF_STATUS_E_FAILURE; } - WMI_LOGD("%s: peer_addr %pM vdev_id %d", __func__, param->peer_addr, - param->vdev_id); + WMI_LOGD("%s: peer_addr "QDF_MAC_ADDR_FMT" vdev_id %d", + __func__, QDF_MAC_ADDR_REF(param->peer_addr), + param->vdev_id); return 0; } @@ -1423,8 +1426,10 @@ QDF_STATUS send_peer_rx_reorder_queue_setup_cmd_tlv(wmi_unified_t wmi, wmi_buf_free(buf); return QDF_STATUS_E_FAILURE; } - WMI_LOGD("%s: peer_macaddr %pM vdev_id %d, tid %d", __func__, - param->peer_macaddr, param->vdev_id, param->tid); + WMI_LOGD("%s: peer_macaddr "QDF_MAC_ADDR_FMT" vdev_id %d, tid %d", + __func__, + QDF_MAC_ADDR_REF(param->peer_macaddr), + param->vdev_id, param->tid); return QDF_STATUS_SUCCESS; } @@ -1467,8 +1472,10 @@ QDF_STATUS send_peer_rx_reorder_queue_remove_cmd_tlv(wmi_unified_t wmi, wmi_buf_free(buf); return QDF_STATUS_E_FAILURE; } - WMI_LOGD("%s: peer_macaddr %pM vdev_id %d, tid_map %d", __func__, - param->peer_macaddr, param->vdev_id, param->peer_tid_bitmap); + WMI_LOGD("%s: peer_macaddr "QDF_MAC_ADDR_FMT" vdev_id %d, tid_map %d", + __func__, + QDF_MAC_ADDR_REF(param->peer_macaddr), + param->vdev_id, param->peer_tid_bitmap); return QDF_STATUS_SUCCESS; } @@ -2665,10 +2672,8 @@ static QDF_STATUS send_peer_assoc_cmd_tlv(wmi_unified_t wmi_handle, param->peer_he_tx_mcs_set[WMI_HOST_HE_TXRX_MCS_NSS_IDX_160]); WMI_LOGD("param->peer_he_rx_mcs_set[160MHz]=%x", param->peer_he_rx_mcs_set[WMI_HOST_HE_TXRX_MCS_NSS_IDX_160]); - WMI_LOGD("peer_mac=%02x:%02x:%02x:%02x:%02x:%02x", - param->peer_mac[0], param->peer_mac[1], - param->peer_mac[2], param->peer_mac[3], - param->peer_mac[4], param->peer_mac[5]); + WMI_LOGD("peer_mac="QDF_MAC_ADDR_FMT, + QDF_MAC_ADDR_REF(param->peer_mac)); } WMI_LOGD("%s: vdev_id %d associd %d peer_flags %x rate_caps %x " @@ -5081,7 +5086,8 @@ static QDF_STATUS send_process_ll_stats_clear_cmd_tlv(wmi_unified_t wmi_handle, WMI_LOGD("StopReq: %d", cmd->stop_stats_collection_req); WMI_LOGD("Vdev Id: %d", cmd->vdev_id); WMI_LOGD("Clear Stat Mask: %d", cmd->stats_clear_req_mask); - WMI_LOGD("Peer MAC Addr: %pM", clear_req->peer_macaddr.bytes); + WMI_LOGD("Peer MAC Addr: "QDF_MAC_ADDR_FMT, + QDF_MAC_ADDR_REF(clear_req->peer_macaddr.bytes)); wmi_mtrace(WMI_CLEAR_LINK_STATS_CMDID, cmd->vdev_id, 0); ret = wmi_unified_cmd_send(wmi_handle, buf, len, @@ -5185,9 +5191,9 @@ static QDF_STATUS send_process_ll_stats_get_cmd_tlv(wmi_unified_t wmi_handle, WMI_CHAR_ARRAY_TO_MAC_ADDR(get_req->peer_macaddr.bytes, &cmd->peer_macaddr); - WMI_LOGD("LINK_LAYER_STATS - Get Request Params Request ID: %u Stats Type: %0x Vdev ID: %d Peer MAC Addr: %pM", + WMI_LOGD("LINK_LAYER_STATS - Get Request Params Request ID: %u Stats Type: %0x Vdev ID: %d Peer MAC Addr: "QDF_MAC_ADDR_FMT, cmd->request_id, cmd->stats_type, cmd->vdev_id, - get_req->peer_macaddr.bytes); + QDF_MAC_ADDR_REF(get_req->peer_macaddr.bytes)); wmi_mtrace(WMI_REQUEST_LINK_STATS_CMDID, cmd->vdev_id, 0); ret = wmi_unified_cmd_send_pm_chk(wmi_handle, buf, len,