[PATCH] ath11k: add raw mode and software crypto support

[Kalle Valo]

Venkateswara Naralasetty <vnaralas at codeaurora.org> writes:

> From: Manikanta Pubbisetty <mpubbise at codeaurora.org>
>
> Adding raw mode tx/rx support; also, adding support
> for software crypto which depends on raw mode.
>
> To enable raw mode tx/rx:
> insmod ath11k.ko frame_mode=0
>
> To enable software crypto:
> insmod ath11k.ko cryptmode=1
>
> These modes could be helpful in debugging crypto related issues.
>
> Tested-on: IPQ8074 WLAN.HK.2.1.0.1-01228-QCAHKSWPL_SILICONZ-1
>
> Signed-off-by: Manikanta Pubbisetty <mpubbise at codeaurora.org>
> Signed-off-by: Venkateswara Naralasetty <vnaralas at codeaurora.org>

Adds new warnings:

drivers/net/wireless/ath/ath11k/core.c:23:14: warning: symbol 'cryptmode' was not declared. Should it be static?
drivers/net/wireless/ath/ath11k/mac.c:46:14: warning: symbol 'ath11k_frame_mode' was not declared. Should it be static?

> --- a/drivers/net/wireless/ath/ath11k/core.c
> +++ b/drivers/net/wireless/ath/ath11k/core.c
> @@ -13,10 +13,16 @@
>  #include "debug.h"
>  #include "hif.h"
>  
> +extern unsigned int ath11k_frame_mode;

Please move this to core.h

>  unsigned int ath11k_debug_mask;
>  module_param_named(debug_mask, ath11k_debug_mask, uint, 0644);
>  MODULE_PARM_DESC(debug_mask, "Debugging mask");
>  
> +unsigned int cryptmode;

static unsigned int ath11k_crypto_mode;

> +module_param_named(cryptmode, cryptmode, uint, 0644);

module_param_named(crypto_mode, ath11k_crypto_mode, uint, 0644);

[...]

> --- a/drivers/net/wireless/ath/ath11k/mac.c
> +++ b/drivers/net/wireless/ath/ath11k/mac.c
> @@ -34,7 +34,7 @@
>  }
>  
>  /* frame mode values are mapped as per enum ath11k_hw_txrx_mode */
> -static unsigned int ath11k_frame_mode = ATH11K_HW_TXRX_NATIVE_WIFI;
> +unsigned int ath11k_frame_mode = ATH11K_HW_TXRX_NATIVE_WIFI;
>  module_param_named(frame_mode, ath11k_frame_mode, uint, 0644);
>  MODULE_PARM_DESC(frame_mode,
>  		 "Datapath frame mode (0: raw, 1: native wifi (default), 2: ethernet)"

Better to move this to core.c so that all module parameters are in one
place.

-- 
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches