* [PATCH] wifi: ath11k: remove duplicate definitions in wmi.h
@ 2024-03-14 2:16 Kang Yang
2024-03-14 15:05 ` Jeff Johnson
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Kang Yang @ 2024-03-14 2:16 UTC (permalink / raw)
To: ath11k; +Cc: linux-wireless, quic_kangyang
There are some duplicate definitions in wmi.h, remove them.
No functional changes, compile tested only.
Signed-off-by: Kang Yang <quic_kangyang@quicinc.com>
---
drivers/net/wireless/ath/ath11k/wmi.h | 6 ------
1 file changed, 6 deletions(-)
diff --git a/drivers/net/wireless/ath/ath11k/wmi.h b/drivers/net/wireless/ath/ath11k/wmi.h
index bb419e3abb00..9652f4eb2f32 100644
--- a/drivers/net/wireless/ath/ath11k/wmi.h
+++ b/drivers/net/wireless/ath/ath11k/wmi.h
@@ -60,10 +60,6 @@ struct wmi_tlv {
#define WLAN_SCAN_MAX_HINT_BSSID 10
#define MAX_RNR_BSS 5
-#define WLAN_SCAN_MAX_HINT_S_SSID 10
-#define WLAN_SCAN_MAX_HINT_BSSID 10
-#define MAX_RNR_BSS 5
-
#define WLAN_SCAN_PARAMS_MAX_SSID 16
#define WLAN_SCAN_PARAMS_MAX_BSSID 4
#define WLAN_SCAN_PARAMS_MAX_IE_LEN 256
@@ -5740,8 +5736,6 @@ struct wmi_debug_log_config_cmd_fixed_param {
u32 value;
} __packed;
-#define WMI_MAX_MEM_REQS 32
-
#define MAX_RADIOS 3
#define WMI_SERVICE_READY_TIMEOUT_HZ (5 * HZ)
base-commit: ffd4e60352172cc167ff74e8bfd8a331a5830013
--
2.34.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] wifi: ath11k: remove duplicate definitions in wmi.h
2024-03-14 2:16 [PATCH] wifi: ath11k: remove duplicate definitions in wmi.h Kang Yang
@ 2024-03-14 15:05 ` Jeff Johnson
2024-03-20 13:54 ` Kalle Valo
2024-03-21 11:41 ` Kalle Valo
2 siblings, 0 replies; 4+ messages in thread
From: Jeff Johnson @ 2024-03-14 15:05 UTC (permalink / raw)
To: Kang Yang, ath11k; +Cc: linux-wireless
On 3/13/2024 7:16 PM, Kang Yang wrote:
> There are some duplicate definitions in wmi.h, remove them.
>
> No functional changes, compile tested only.
>
> Signed-off-by: Kang Yang <quic_kangyang@quicinc.com>
> ---
> drivers/net/wireless/ath/ath11k/wmi.h | 6 ------
> 1 file changed, 6 deletions(-)
>
> diff --git a/drivers/net/wireless/ath/ath11k/wmi.h b/drivers/net/wireless/ath/ath11k/wmi.h
> index bb419e3abb00..9652f4eb2f32 100644
> --- a/drivers/net/wireless/ath/ath11k/wmi.h
> +++ b/drivers/net/wireless/ath/ath11k/wmi.h
> @@ -60,10 +60,6 @@ struct wmi_tlv {
> #define WLAN_SCAN_MAX_HINT_BSSID 10
> #define MAX_RNR_BSS 5
>
> -#define WLAN_SCAN_MAX_HINT_S_SSID 10
> -#define WLAN_SCAN_MAX_HINT_BSSID 10
> -#define MAX_RNR_BSS 5
> -
> #define WLAN_SCAN_PARAMS_MAX_SSID 16
> #define WLAN_SCAN_PARAMS_MAX_BSSID 4
> #define WLAN_SCAN_PARAMS_MAX_IE_LEN 256
This patch doesn't apply (when using b4 shazam) when trying to verify with the
'pending' branch due to a pending change to WLAN_SCAN_PARAMS_MAX_IE_LEN.
So I verified against ath-next instead.
I'll let Kalle comment on if he can handle the conflict himself or if he wants
a rebased patch.
Acked-by: Jeff Johnson <quic_jjohnson@quicinc.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] wifi: ath11k: remove duplicate definitions in wmi.h
2024-03-14 2:16 [PATCH] wifi: ath11k: remove duplicate definitions in wmi.h Kang Yang
2024-03-14 15:05 ` Jeff Johnson
@ 2024-03-20 13:54 ` Kalle Valo
2024-03-21 11:41 ` Kalle Valo
2 siblings, 0 replies; 4+ messages in thread
From: Kalle Valo @ 2024-03-20 13:54 UTC (permalink / raw)
To: Kang Yang; +Cc: ath11k, linux-wireless, quic_kangyang
Kang Yang <quic_kangyang@quicinc.com> wrote:
> There are some duplicate definitions in wmi.h, remove them.
>
> No functional changes, compile tested only.
>
> Signed-off-by: Kang Yang <quic_kangyang@quicinc.com>
> Acked-by: Jeff Johnson <quic_jjohnson@quicinc.com>
> Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
There was a conflict but git-am handled that automatically:
https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=pending&id=5955a10e0a46d46216420b8d2b6c0a1c895892af
--
https://patchwork.kernel.org/project/linux-wireless/patch/20240314021654.304451-1-quic_kangyang@quicinc.com/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] wifi: ath11k: remove duplicate definitions in wmi.h
2024-03-14 2:16 [PATCH] wifi: ath11k: remove duplicate definitions in wmi.h Kang Yang
2024-03-14 15:05 ` Jeff Johnson
2024-03-20 13:54 ` Kalle Valo
@ 2024-03-21 11:41 ` Kalle Valo
2 siblings, 0 replies; 4+ messages in thread
From: Kalle Valo @ 2024-03-21 11:41 UTC (permalink / raw)
To: Kang Yang; +Cc: ath11k, linux-wireless, quic_kangyang
Kang Yang <quic_kangyang@quicinc.com> wrote:
> There are some duplicate definitions in wmi.h, remove them.
>
> No functional changes, compile tested only.
>
> Signed-off-by: Kang Yang <quic_kangyang@quicinc.com>
> Acked-by: Jeff Johnson <quic_jjohnson@quicinc.com>
> Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Patch applied to ath-next branch of ath.git, thanks.
3b8e475b27d7 wifi: ath11k: remove duplicate definitions in wmi.h
--
https://patchwork.kernel.org/project/linux-wireless/patch/20240314021654.304451-1-quic_kangyang@quicinc.com/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2024-03-21 11:41 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-03-14 2:16 [PATCH] wifi: ath11k: remove duplicate definitions in wmi.h Kang Yang
2024-03-14 15:05 ` Jeff Johnson
2024-03-20 13:54 ` Kalle Valo
2024-03-21 11:41 ` Kalle Valo
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.