mbox series

[v2,0/2] wifi: ath12k: Pass correct channel frequency parameters

Message ID 20250304095315.3050325-1-quic_surapk@quicinc.com
Headers show
Series wifi: ath12k: Pass correct channel frequency parameters | expand

Message

Suraj P Kizhakkethil March 4, 2025, 9:53 a.m. UTC
For higher bandwidths, the channel frequency parameters passed to
the firmware are incorrect. Pass correct values of channel
frequency parameters.

v2:
 - Replaced arg->band_center_freq1 to local variable center_freq1,
   which is initialized to arg->band_center_freq1
 - Rebased on ToT

Sriram R (1):
  wifi: ath12k: Pass correct values of center freq1 and center freq2 for
    320 MHz

Suraj P Kizhakkethil (1):
  wifi: ath12k: Pass correct values of center freq1 and center freq2 for
    160 MHz

 drivers/net/wireless/ath/ath12k/wmi.c | 24 +++++++++++++++++++++---
 1 file changed, 21 insertions(+), 3 deletions(-)


base-commit: 3148fc3cf193dbbd2e14eee59468a510a38bf604

Comments

Jeff Johnson March 4, 2025, 5:12 p.m. UTC | #1
On 3/4/2025 1:53 AM, Suraj P Kizhakkethil wrote:
> For higher bandwidths, the channel frequency parameters passed to
> the firmware are incorrect. Pass correct values of channel
> frequency parameters.
> 
> v2:
>  - Replaced arg->band_center_freq1 to local variable center_freq1,
>    which is initialized to arg->band_center_freq1
>  - Rebased on ToT
> 
> Sriram R (1):
>   wifi: ath12k: Pass correct values of center freq1 and center freq2 for
>     320 MHz
> 
> Suraj P Kizhakkethil (1):
>   wifi: ath12k: Pass correct values of center freq1 and center freq2 for
>     160 MHz
> 
>  drivers/net/wireless/ath/ath12k/wmi.c | 24 +++++++++++++++++++++---
>  1 file changed, 21 insertions(+), 3 deletions(-)
> 
> 
> base-commit: 3148fc3cf193dbbd2e14eee59468a510a38bf604

Some reminders:
1) per new guidance you should have added the destination tree name (ath-next)
as a patch tag, i.e.: [PATCH ath-next v2 0/2]
2) since these were trivial changes you should have propagated the
Reviewed-by: tags that were given for the v1 patches.

No need to resend for this, but please keep in mind for future reference.

/jeff
Suraj P Kizhakkethil March 5, 2025, 3:34 a.m. UTC | #2
On 04-Mar-25 10:42 PM, Jeff Johnson wrote:
> On 3/4/2025 1:53 AM, Suraj P Kizhakkethil wrote:
>> For higher bandwidths, the channel frequency parameters passed to
>> the firmware are incorrect. Pass correct values of channel
>> frequency parameters.
>>
>> v2:
>>   - Replaced arg->band_center_freq1 to local variable center_freq1,
>>     which is initialized to arg->band_center_freq1
>>   - Rebased on ToT
>>
>> Sriram R (1):
>>    wifi: ath12k: Pass correct values of center freq1 and center freq2 for
>>      320 MHz
>>
>> Suraj P Kizhakkethil (1):
>>    wifi: ath12k: Pass correct values of center freq1 and center freq2 for
>>      160 MHz
>>
>>   drivers/net/wireless/ath/ath12k/wmi.c | 24 +++++++++++++++++++++---
>>   1 file changed, 21 insertions(+), 3 deletions(-)
>>
>>
>> base-commit: 3148fc3cf193dbbd2e14eee59468a510a38bf604
> Some reminders:
> 1) per new guidance you should have added the destination tree name (ath-next)
> as a patch tag, i.e.: [PATCH ath-next v2 0/2]
> 2) since these were trivial changes you should have propagated the
> Reviewed-by: tags that were given for the v1 patches.
>
> No need to resend for this, but please keep in mind for future reference.
>
> /jeff

Sure Jeff,

I'll ensure to follow these guidance in the future

Thanks