Thanks Aditya, I agree with your suggestion. I will make these changes and submit an updated patchet v11. Thank you for your valuable input.ret = ath12k_wmi_send_peer_assoc_cmd(ar, peer_arg); if (ret) {ath12k_warn(ar->ab, "failed to run peer assoc for %pM vdev %i: %d\n", @@ -3873,6 +4012,13 @@ static void ath12k_mac_bss_info_changed(struct ath12k *ar,ether_addr_copy(arvif->bssid, info->bssid); if (changed & BSS_CHANGED_BEACON_ENABLED) { + if (info->enable_beacon) { + ret = ath12k_mac_set_he_txbf_conf(arvif); + if (ret) + ath12k_warn(ar->ab, + "failed to set HE TXBF config for vdev: %d\n", + arvif->vdev_idSo when you bring up AP in non-HE mode i.e HT/VHT and all, this print will appear which will be misleading.