qcacmn: Remove trailing comma/period from the DP logs

Remove trailing comma/period from the DP logs.

Change-Id: Ifdaf508157274c44f04e775c1883c4eaa2924c44
CRs-Fixed: 3492731

dp/wifi3.0/dp_htt.c

@@ -3643,7 +3643,7 @@ static void dp_ipa_rx_cce_super_rule_setup_done_handler(struct htt_soc *soc,
 		dp_info("opt_dp:: Wrong Super rule setup response");
 	};
 
-	dp_info("opt_dp:: cce super rule resp type: %d, is_rules_enough: %d,",
+	dp_info("opt_dp:: cce super rule resp type: %d, is_rules_enough: %d",
 		resp_type, is_rules_enough);
 	dp_info("num_rules_avail: %d, rslt0: %d, rslt1: %d",
 		num_rules_avail, filter0_result, filter1_result);

dp/wifi3.0/dp_main.c

@@ -4843,7 +4843,7 @@ QDF_STATUS dp_peer_mlo_setup(
 			 * during mld peer creation.
 			 */
 			prev_vdev = mld_peer->vdev;
-			dp_info("Primary link is not the first link. vdev: %pK,"
+			dp_info("Primary link is not the first link. vdev: %pK "
 				"vdev_id %d vdev_ref_cnt %d",
 				mld_peer->vdev, vdev_id,
 				qdf_atomic_read(&mld_peer->vdev->ref_cnt));

dp/wifi3.0/dp_rings_main.c

@@ -4236,7 +4236,7 @@ qdf_export_symbol(dp_soc_set_txrx_ring_map);
 static void dp_soc_cfg_dump(struct dp_soc *soc, uint32_t target_type)
 {
 	dp_init_info("DP soc Dump for Target = %d", target_type);
-	dp_init_info("ast_override_support = %d, da_war_enabled = %d,",
+	dp_init_info("ast_override_support = %d da_war_enabled = %d",
 		     soc->ast_override_support, soc->da_war_enabled);
 
 	wlan_cfg_dp_soc_ctx_dump(soc->wlan_cfg_ctx);

dp/wifi3.0/dp_rx_defrag.c

@@ -1849,8 +1849,8 @@ dp_rx_defrag_store_fragment(struct dp_soc *soc,
 	uint32_t msdu_len;
 
 	if (qdf_nbuf_len(frag) > 0) {
-		dp_info("Dropping unexpected packet with skb_len: %d,"
-			"data len: %d, cookie: %d",
+		dp_info("Dropping unexpected packet with skb_len: %d "
+			"data len: %d cookie: %d",
 			(uint32_t)qdf_nbuf_len(frag), frag->data_len,
 			rx_desc->cookie);
 		DP_STATS_INC(soc, rx.rx_frag_err_len_error, 1);

dp/wifi3.0/dp_stats.c

@@ -7048,7 +7048,7 @@ void dp_print_peer_stats(struct dp_peer *peer,
 			       peer_stats->rx.rcvd_reo[i].bytes);
 	}
 	for (i = 0; i < CDP_MAX_LMACS; i++)
-		DP_PRINT_STATS("Packets Received on lmac[%d] = %d ( %llu ),",
+		DP_PRINT_STATS("Packets Received on lmac[%d] = %d ( %llu )",
 			       i, peer_stats->rx.rx_lmac[i].num,
 			       peer_stats->rx.rx_lmac[i].bytes);
 
@@ -7275,7 +7275,7 @@ void dp_txrx_path_stats(struct dp_soc *soc)
 		DP_PRINT_STATS("Invalid TX desc from completion ring: %u",
 			       soc->stats.tx.invalid_tx_comp_desc);
 		DP_PRINT_STATS("Dropped in host:");
-		DP_PRINT_STATS("Total packets dropped: %u,",
+		DP_PRINT_STATS("Total packets dropped: %u",
 			       pdev->stats.tx_i.dropped.dropped_pkt.num);
 		DP_PRINT_STATS("Descriptor not available: %u",
 			       pdev->stats.tx_i.dropped.desc_na.num);
@@ -7329,25 +7329,25 @@ void dp_txrx_path_stats(struct dp_soc *soc)
 
 		DP_PRINT_STATS("Rx path statistics");
 
-		DP_PRINT_STATS("delivered %u msdus ( %llu bytes),",
+		DP_PRINT_STATS("delivered %u msdus ( %llu bytes)",
 			       pdev->stats.rx.to_stack.num,
 			       pdev->stats.rx.to_stack.bytes);
 		for (i = 0; i < CDP_MAX_RX_RINGS; i++) {
 			if (!pdev->stats.rx.rcvd_reo[i].num)
 				continue;
 			DP_PRINT_STATS(
-				       "received on reo[%d] %u msdus( %llu bytes),",
+				       "received on reo[%d] %u msdus( %llu bytes)",
 				       i, pdev->stats.rx.rcvd_reo[i].num,
 				       pdev->stats.rx.rcvd_reo[i].bytes);
 		}
 		for (i = 0; i < CDP_MAX_LMACS; i++)
-			DP_PRINT_STATS("received on lmac[%d] %u msdus (%llu bytes),",
+			DP_PRINT_STATS("received on lmac[%d] %u msdus (%llu bytes)",
 				       i, pdev->stats.rx.rx_lmac[i].num,
 				       pdev->stats.rx.rx_lmac[i].bytes);
-		DP_PRINT_STATS("intra-bss packets %u msdus ( %llu bytes),",
+		DP_PRINT_STATS("intra-bss packets %u msdus ( %llu bytes)",
 			       pdev->stats.rx.intra_bss.pkts.num,
 			       pdev->stats.rx.intra_bss.pkts.bytes);
-		DP_PRINT_STATS("intra-bss fails %u msdus ( %llu bytes),",
+		DP_PRINT_STATS("intra-bss fails %u msdus ( %llu bytes)",
 			       pdev->stats.rx.intra_bss.fail.num,
 			       pdev->stats.rx.intra_bss.fail.bytes);
 		DP_PRINT_STATS("intra-bss no mdns fwds %u msdus",
@@ -7355,7 +7355,7 @@ void dp_txrx_path_stats(struct dp_soc *soc)
 		DP_PRINT_STATS("intra-bss EAPOL drops: %u",
 			       soc->stats.rx.err.intrabss_eapol_drop);
 
-		DP_PRINT_STATS("raw packets %u msdus ( %llu bytes),",
+		DP_PRINT_STATS("raw packets %u msdus ( %llu bytes)",
 			       pdev->stats.rx.raw.num,
 			       pdev->stats.rx.raw.bytes);
 		DP_PRINT_STATS("mic errors %u",

hal/wifi3.0/hal_rx_flow.c

@@ -363,7 +363,7 @@ hal_rx_fst_attach(hal_soc_handle_t hal_soc_hdl,
 
 	if (!fst) {
 		QDF_TRACE(QDF_MODULE_ID_TXRX, QDF_TRACE_LEVEL_ERROR,
-			  FL("hal fst allocation failed,"));
+			  FL("hal fst allocation failed"));
 		return NULL;
 	}
 

ipa/core/src/wlan_ipa_core.c

@@ -5180,7 +5180,7 @@ int wlan_ipa_wdi_opt_dpath_flt_add_cb(
 	dp_flt_param->num_filters = num_flts;
 	qdf_event_reset(&ipa_obj->ipa_flt_evnt);
 
-	ipa_info("opt_dp: op %d, pdev_id %d. num_flts %d,",
+	ipa_info("opt_dp: op %d, pdev_id %d. num_flts %d",
 		 dp_flt_param->op, dp_flt_param->pdev_id, num_flts);
 	for (i = 0; i < num_flts; i++)
 		ipa_info("version %d, valid %d, src addr_ %08lx, evnt reqd %d",
@@ -5255,7 +5255,7 @@ int wlan_ipa_wdi_opt_dpath_flt_rem_cb(
 	dp_flt_params->num_filters = num_flts;
 	qdf_event_reset(&ipa_obj->ipa_flt_evnt);
 
-	ipa_info("opt_dp: op %d, pdev_id %d. num_flts %d,",
+	ipa_info("opt_dp: op %d, pdev_id %d. num_flts %d",
 		 dp_flt_params->op, dp_flt_params->pdev_id, num_flts);
 	for (i = 0; i < num_flts; i++)
 		ipa_info("version %d, valid %d, src addr_ %08lx, evnt_reqd %d",

wlan_cfg/wlan_cfg.c

@@ -5370,13 +5370,13 @@ qdf_export_symbol(wlan_cfg_is_delay_mon_replenish);
 #ifdef WLAN_SOFTUMAC_SUPPORT
 void wlan_cfg_dp_soc_ctx_dump(struct wlan_cfg_dp_soc_ctxt *cfg)
 {
-	dp_info("DP CFG SoC ctx: delay_mon_replenish = %d,",
+	dp_info("DP CFG SoC ctx: delay_mon_replenish = %d",
 		cfg->delay_mon_replenish);
 }
 #else
 void wlan_cfg_dp_soc_ctx_dump(struct wlan_cfg_dp_soc_ctxt *cfg)
 {
-	dp_info("DP CFG SoC ctx: delay_mon_replenish = %d,",
+	dp_info("DP CFG SoC ctx: delay_mon_replenish = %d",
 		cfg->delay_mon_replenish);
 	dp_info("reo_dst_ring_size = %d, delayed_replenish_entries = %d",
 		cfg->reo_dst_ring_size, cfg->delayed_replenish_entries);