lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <b58b5b2e-bf9f-480c-810b-2cef29aab82c@quicinc.com>
Date: Fri, 11 Oct 2024 09:51:49 -0700
From: Jeff Johnson <quic_jjohnson@...cinc.com>
To: Paweł Owoc <frut3k7@...il.com>
CC: Kalle Valo <kvalo@...nel.org>, Jeff Johnson <jjohnson@...nel.org>,
        <linux-wireless@...r.kernel.org>, <ath10k@...ts.infradead.org>,
        <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2] wifi: ath10k: add channel 177 for 5 GHz band

On 10/8/2024 11:49 PM, Paweł Owoc wrote:> Add support for channel 177 (5885
MHz ) for the 5 GHz band.
> 
> Tested-on: qca988x hw2.0 firmware ver 10.2.4-1.0-00047

Can you elaborate on what was tested in your commit text? And more
importantly, what is the impact on existing devices, especially given that
existing devices would not have calibration data for this channel in the board
files? Does the QCA988x board file even have calibration data for this channel?

> 
> Signed-off-by: Paweł Owoc <frut3k7@...il.com>
> ---

what is the diff from v1?
for future reference when you submit a new version of a patch you should
include a patch changelog after the ---, see:
https://www.kernel.org/doc/html/latest/process/submitting-patches.html#the-canonical-patch-format

>  drivers/net/wireless/ath/ath10k/core.h | 4 ++--
>  drivers/net/wireless/ath/ath10k/mac.c  | 1 +
>  2 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/wireless/ath/ath10k/core.h b/drivers/net/wireless/ath/ath10k/core.h
> index 446dca74f06a..3dff8c028526 100644
> --- a/drivers/net/wireless/ath/ath10k/core.h
> +++ b/drivers/net/wireless/ath/ath10k/core.h
> @@ -39,8 +39,8 @@
>  #define WMI_READY_TIMEOUT (5 * HZ)
>  #define ATH10K_FLUSH_TIMEOUT_HZ (5 * HZ)
>  #define ATH10K_CONNECTION_LOSS_HZ (3 * HZ)
> -#define ATH10K_NUM_CHANS 41
> -#define ATH10K_MAX_5G_CHAN 173
> +#define ATH10K_NUM_CHANS 42
> +#define ATH10K_MAX_5G_CHAN 177
>  
>  /* Antenna noise floor */
>  #define ATH10K_DEFAULT_NOISE_FLOOR -95
> diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c
> index 646e1737d4c4..cee6a4d287b5 100644
> --- a/drivers/net/wireless/ath/ath10k/mac.c
> +++ b/drivers/net/wireless/ath/ath10k/mac.c
> @@ -9543,6 +9543,7 @@ static const struct ieee80211_channel ath10k_5ghz_channels[] = {
>  	CHAN5G(165, 5825, 0),
>  	CHAN5G(169, 5845, 0),
>  	CHAN5G(173, 5865, 0),
> +	CHAN5G(177, 5885, 0),
>  	/* If you add more, you may need to change ATH10K_MAX_5G_CHAN */
>  	/* And you will definitely need to change ATH10K_NUM_CHANS in core.h */
>  };


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ