diff --git a/dp/wifi3.0/dp_htt.c b/dp/wifi3.0/dp_htt.c index b648686f9e..b16fa957b9 100644 --- a/dp/wifi3.0/dp_htt.c +++ b/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); diff --git a/dp/wifi3.0/dp_main.c b/dp/wifi3.0/dp_main.c index 8e9a79906f..a7595d8c11 100644 --- a/dp/wifi3.0/dp_main.c +++ b/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)); diff --git a/dp/wifi3.0/dp_rings_main.c b/dp/wifi3.0/dp_rings_main.c index e0223ef504..87b266d4a4 100644 --- a/dp/wifi3.0/dp_rings_main.c +++ b/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); diff --git a/dp/wifi3.0/dp_rx_defrag.c b/dp/wifi3.0/dp_rx_defrag.c index dc9818e99d..24df4e9696 100644 --- a/dp/wifi3.0/dp_rx_defrag.c +++ b/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); diff --git a/dp/wifi3.0/dp_stats.c b/dp/wifi3.0/dp_stats.c index 9ebbdcc1da..cad6b215c5 100644 --- a/dp/wifi3.0/dp_stats.c +++ b/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", diff --git a/hal/wifi3.0/hal_rx_flow.c b/hal/wifi3.0/hal_rx_flow.c index 7a119190a4..be69ef0b55 100644 --- a/hal/wifi3.0/hal_rx_flow.c +++ b/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; } diff --git a/ipa/core/src/wlan_ipa_core.c b/ipa/core/src/wlan_ipa_core.c index 07fe47ba09..e8f11d5ff0 100644 --- a/ipa/core/src/wlan_ipa_core.c +++ b/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", diff --git a/wlan_cfg/wlan_cfg.c b/wlan_cfg/wlan_cfg.c index 94b94926fe..e817feb588 100644 --- a/wlan_cfg/wlan_cfg.c +++ b/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);