Commit 3926e0c1 authored by Manikanta Pubbisetty's avatar Manikanta Pubbisetty Committed by Kalle Valo
Browse files

ath11k: Fix warnings reported by checkpatch 



Fix warnings reported by checkpatch tool. Below are the errors fixed,

drivers/net/wireless/ath/ath11k/hal_rx.c:760: 'recevied' may be misspelled - perhaps 'received?
drivers/net/wireless/ath/ath11k/qmi.c:2232: Prefer strscpy over strlcpy
drivers/net/wireless/ath/ath11k/qmi.c:2238: Prefer strscpy over strlcpy
drivers/net/wireless/ath/ath11k/qmi.c:2662: Prefer strscpy over strlcpy
drivers/net/wireless/ath/ath11k/mac.c:7836: 'atleast' may be misspelled - perhaps 'at least'?
drivers/net/wireless/ath/ath11k/wmi.c:6566: 'succeded' may be misspelled - perhaps 'succeeded'?

Tested-on: WCN6750 hw1.0 AHB WLAN.MSL.1.0.1-00887-QCAMSLSWPLZ-1

Signed-off-by: default avatarManikanta Pubbisetty <quic_mpubbise@quicinc.com>
Signed-off-by: default avatarKalle Valo <quic_kvalo@quicinc.com>
Link: https://lore.kernel.org/r/20220602131350.29486-1-quic_mpubbise@quicinc.com
parent 4c1fc4f6

drivers/net/wireless/ath/ath11k/hal_rx.c

+1 −1
Original line number Diff line number Diff line
@@ -757,7 +757,7 @@ void ath11k_hal_reo_qdesc_setup(void *vaddr, int tid, u32 ba_window_size, 


	/* TODO: HW queue descriptors are currently allocated for max BA

	 * window size for all QOS TIDs so that same descriptor can be used

	 * later when ADDBA request is recevied. This should be changed to

	 * later when ADDBA request is received. This should be changed to

	 * allocate HW queue descriptors based on BA window size being

	 * negotiated (0 for non BA cases), and reallocate when BA window

	 * size changes and also send WMI message to FW to change the REO

drivers/net/wireless/ath/ath11k/qmi.c

+3 −3
Original line number Diff line number Diff line
@@ -2229,13 +2229,13 @@ static int ath11k_qmi_request_target_cap(struct ath11k_base *ab) 


	if (resp.fw_version_info_valid) {

		ab->qmi.target.fw_version = resp.fw_version_info.fw_version;

		strlcpy(ab->qmi.target.fw_build_timestamp,

		strscpy(ab->qmi.target.fw_build_timestamp,

			resp.fw_version_info.fw_build_timestamp,

			sizeof(ab->qmi.target.fw_build_timestamp));

	}



	if (resp.fw_build_id_valid)

		strlcpy(ab->qmi.target.fw_build_id, resp.fw_build_id,

		strscpy(ab->qmi.target.fw_build_id, resp.fw_build_id,

			sizeof(ab->qmi.target.fw_build_id));



	if (resp.eeprom_read_timeout_valid) {
@@ -2659,7 +2659,7 @@ static int ath11k_qmi_wlanfw_wlan_cfg_send(struct ath11k_base *ab) 
	memset(&resp, 0, sizeof(resp));



	req->host_version_valid = 1;

	strlcpy(req->host_version, ATH11K_HOST_VERSION_STRING,

	strscpy(req->host_version, ATH11K_HOST_VERSION_STRING,

		sizeof(req->host_version));



	req->tgt_cfg_valid = 1;

drivers/net/wireless/ath/ath11k/wmi.c

+1 −1
Original line number Diff line number Diff line
@@ -6563,7 +6563,7 @@ static int ath11k_reg_chan_list_event(struct ath11k_base *ab, struct sk_buff *sk 


fallback:

	/* Fallback to older reg (by sending previous country setting

	 * again if fw has succeded and we failed to process here.

	 * again if fw has succeeded and we failed to process here.

	 * The Regdomain should be uniform across driver and fw. Since the

	 * FW has processed the command and sent a success status, we expect

	 * this function to succeed as well. If it doesn't, CTRY needs to be

drivers/net/wireless/ath/ath11k/mac.c

+1 −1

File changed.

Contains only whitespace changes.