Browse Source

qcacld-3.0: Add context to the logs

Logs without meaningful message or values serves no purpose in debugging.
Hence, add more context to the logs.

Change-Id: Ie913f7460356a1550b5252fd72ed5b454c67b3cf
CRs-Fixed: 3492709
Srinivas Girigowda 1 year ago
parent
commit
3acf9cdec0

+ 4 - 2
components/dp/core/src/wlan_dp_main.c

@@ -849,7 +849,8 @@ dp_peer_obj_create_notification(struct wlan_objmgr_peer *peer, void *arg)
 						       sta_info,
 						       QDF_STATUS_SUCCESS);
 	if (QDF_IS_STATUS_ERROR(status)) {
-		dp_err("DP peer attach failed");
+		dp_err("DP peer ("QDF_MAC_ADDR_FMT") attach failed",
+			QDF_MAC_ADDR_REF(peer->macaddr));
 		qdf_mem_free(sta_info);
 		return status;
 	}
@@ -881,7 +882,8 @@ dp_peer_obj_destroy_notification(struct wlan_objmgr_peer *peer, void *arg)
 	status = wlan_objmgr_peer_component_obj_detach(peer, WLAN_COMP_DP,
 						       sta_info);
 	if (QDF_IS_STATUS_ERROR(status))
-		dp_err("DP peer detach failed");
+		dp_err("DP peer ("QDF_MAC_ADDR_FMT") detach failed",
+			QDF_MAC_ADDR_REF(peer->macaddr));
 
 	qdf_mem_free(sta_info);
 

+ 1 - 1
components/target_if/dp/src/target_if_dp_comp.c

@@ -291,7 +291,7 @@ target_if_dp_lro_config_cmd(struct wlan_objmgr_psoc *psoc,
 
 	wmi_handle = get_wmi_unified_hdl_from_psoc(psoc);
 	if (!dp_lro_cmd || !wmi_handle) {
-		dp_err("Invalid input!");
+		dp_err("wmi_handle or dp_lro_cmd is null");
 		return QDF_STATUS_E_FAILURE;
 	}