All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ath10k: Do not limit RTS threshold value to 2347
@ 2014-11-24 11:51 Vivek Natarajan
  0 siblings, 0 replies; 6+ messages in thread
From: Vivek Natarajan @ 2014-11-24 11:51 UTC (permalink / raw)
  To: kvalo; +Cc: linux-wireless, ath10

Increase the rts threshold from the legacy value of 2347 to support higher
threshold limit.

Signed-off-by: Vivek Natarajan <nataraja@qti.qualcomm.com>
---
 drivers/net/wireless/ath/ath10k/mac.c |    6 +-----
 drivers/net/wireless/ath/ath10k/wmi.h |    1 -
 2 files changed, 1 insertion(+), 6 deletions(-)

diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c
index 8b60749..61f7e22 100644
--- a/drivers/net/wireless/ath/ath10k/mac.c
+++ b/drivers/net/wireless/ath/ath10k/mac.c
@@ -391,15 +391,11 @@ static int ath10k_mac_set_kickout(struct ath10k_vif *arvif)
 	return 0;
 }
 
-static int  ath10k_mac_set_rts(struct ath10k_vif *arvif, u32 value)
+static int ath10k_mac_set_rts(struct ath10k_vif *arvif, u32 value)
 {
 	struct ath10k *ar = arvif->ar;
 	u32 vdev_param;
 
-	if (value != 0xFFFFFFFF)
-		value = min_t(u32, arvif->ar->hw->wiphy->rts_threshold,
-			      ATH10K_RTS_MAX);
-
 	vdev_param = ar->wmi.vdev_param->rts_threshold;
 	return ath10k_wmi_vdev_set_param(ar, arvif->vdev_id, vdev_param, value);
 }
diff --git a/drivers/net/wireless/ath/ath10k/wmi.h b/drivers/net/wireless/ath/ath10k/wmi.h
index a38d788..8a4e779 100644
--- a/drivers/net/wireless/ath/ath10k/wmi.h
+++ b/drivers/net/wireless/ath/ath10k/wmi.h
@@ -4547,7 +4547,6 @@ struct wmi_dbglog_cfg_cmd {
 	__le32 config_valid;
 } __packed;
 
-#define ATH10K_RTS_MAX		2347
 #define ATH10K_FRAGMT_THRESHOLD_MIN	540
 #define ATH10K_FRAGMT_THRESHOLD_MAX	2346
 
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH] ath10k: Do not limit RTS threshold value to 2347
@ 2014-11-24 11:57 Vivek Natarajan
  2014-11-24 18:35 ` Ben Greear
  2014-12-01  7:04 ` Kalle Valo
  0 siblings, 2 replies; 6+ messages in thread
From: Vivek Natarajan @ 2014-11-24 11:57 UTC (permalink / raw)
  To: kvalo; +Cc: ath10k

Increase the rts threshold from the legacy value of 2347 to support higher
threshold limit.

Signed-off-by: Vivek Natarajan <nataraja@qti.qualcomm.com>
---
 drivers/net/wireless/ath/ath10k/mac.c |    6 +-----
 drivers/net/wireless/ath/ath10k/wmi.h |    1 -
 2 files changed, 1 insertion(+), 6 deletions(-)

diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c
index 8b60749..61f7e22 100644
--- a/drivers/net/wireless/ath/ath10k/mac.c
+++ b/drivers/net/wireless/ath/ath10k/mac.c
@@ -391,15 +391,11 @@ static int ath10k_mac_set_kickout(struct ath10k_vif *arvif)
 	return 0;
 }
 
-static int  ath10k_mac_set_rts(struct ath10k_vif *arvif, u32 value)
+static int ath10k_mac_set_rts(struct ath10k_vif *arvif, u32 value)
 {
 	struct ath10k *ar = arvif->ar;
 	u32 vdev_param;
 
-	if (value != 0xFFFFFFFF)
-		value = min_t(u32, arvif->ar->hw->wiphy->rts_threshold,
-			      ATH10K_RTS_MAX);
-
 	vdev_param = ar->wmi.vdev_param->rts_threshold;
 	return ath10k_wmi_vdev_set_param(ar, arvif->vdev_id, vdev_param, value);
 }
diff --git a/drivers/net/wireless/ath/ath10k/wmi.h b/drivers/net/wireless/ath/ath10k/wmi.h
index a38d788..8a4e779 100644
--- a/drivers/net/wireless/ath/ath10k/wmi.h
+++ b/drivers/net/wireless/ath/ath10k/wmi.h
@@ -4547,7 +4547,6 @@ struct wmi_dbglog_cfg_cmd {
 	__le32 config_valid;
 } __packed;
 
-#define ATH10K_RTS_MAX		2347
 #define ATH10K_FRAGMT_THRESHOLD_MIN	540
 #define ATH10K_FRAGMT_THRESHOLD_MAX	2346
 
-- 
1.7.9.5


_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] ath10k: Do not limit RTS threshold value to 2347
  2014-11-24 11:57 [PATCH] ath10k: Do not limit RTS threshold value to 2347 Vivek Natarajan
@ 2014-11-24 18:35 ` Ben Greear
  2014-11-26  7:16   ` Kalle Valo
  2014-11-26  7:29   ` Natarajan, Vivekanandan
  2014-12-01  7:04 ` Kalle Valo
  1 sibling, 2 replies; 6+ messages in thread
From: Ben Greear @ 2014-11-24 18:35 UTC (permalink / raw)
  To: Vivek Natarajan; +Cc: kvalo, ath10k

On 11/24/2014 03:57 AM, Vivek Natarajan wrote:
> Increase the rts threshold from the legacy value of 2347 to support higher
> threshold limit.

Does any firmware actually support RTS?  It didn't work last I tested
it on 10.1.

Thanks,
Ben

-- 
Ben Greear <greearb@candelatech.com>
Candela Technologies Inc  http://www.candelatech.com


_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] ath10k: Do not limit RTS threshold value to 2347
  2014-11-24 18:35 ` Ben Greear
@ 2014-11-26  7:16   ` Kalle Valo
  2014-11-26  7:29   ` Natarajan, Vivekanandan
  1 sibling, 0 replies; 6+ messages in thread
From: Kalle Valo @ 2014-11-26  7:16 UTC (permalink / raw)
  To: Ben Greear; +Cc: Vivek Natarajan, ath10k

Ben Greear <greearb@candelatech.com> writes:

> On 11/24/2014 03:57 AM, Vivek Natarajan wrote:
>> Increase the rts threshold from the legacy value of 2347 to support higher
>> threshold limit.
>
> Does any firmware actually support RTS?  It didn't work last I tested
> it on 10.1.

Vivek?

-- 
Kalle Valo

_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k

^ permalink raw reply	[flat|nested] 6+ messages in thread

* RE: [PATCH] ath10k: Do not limit RTS threshold value to 2347
  2014-11-24 18:35 ` Ben Greear
  2014-11-26  7:16   ` Kalle Valo
@ 2014-11-26  7:29   ` Natarajan, Vivekanandan
  1 sibling, 0 replies; 6+ messages in thread
From: Natarajan, Vivekanandan @ 2014-11-26  7:29 UTC (permalink / raw)
  To: Ben Greear; +Cc: Valo, Kalle, ath10k@lists.infradead.org

It works with the 10.2 firmware.

Moreover, the issue reported here was rts_threshold displayed in debugfs was different than the one that we had configured to the firmware(2347) .

Thanks
________________________________________
From: Ben Greear <greearb@candelatech.com>
Sent: Tuesday, November 25, 2014 12:05 AM
To: Natarajan, Vivekanandan
Cc: Valo, Kalle; ath10k@lists.infradead.org
Subject: Re: [PATCH] ath10k: Do not limit RTS threshold value to 2347

On 11/24/2014 03:57 AM, Vivek Natarajan wrote:
> Increase the rts threshold from the legacy value of 2347 to support higher
> threshold limit.

Does any firmware actually support RTS?  It didn't work last I tested
it on 10.1.

Thanks,
Ben

--
Ben Greear <greearb@candelatech.com>
Candela Technologies Inc  http://www.candelatech.com


_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] ath10k: Do not limit RTS threshold value to 2347
  2014-11-24 11:57 [PATCH] ath10k: Do not limit RTS threshold value to 2347 Vivek Natarajan
  2014-11-24 18:35 ` Ben Greear
@ 2014-12-01  7:04 ` Kalle Valo
  1 sibling, 0 replies; 6+ messages in thread
From: Kalle Valo @ 2014-12-01  7:04 UTC (permalink / raw)
  To: Vivek Natarajan; +Cc: ath10k

Vivek Natarajan <nataraja@qti.qualcomm.com> writes:

> Increase the rts threshold from the legacy value of 2347 to support higher
> threshold limit.
>
> Signed-off-by: Vivek Natarajan <nataraja@qti.qualcomm.com>

Thanks, applied.

In the future, please also CC linux-wireless as documented here:

http://wireless.kernel.org/en/users/Drivers/ath10k/sources#Submitting_patches

-- 
Kalle Valo

_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2014-12-01  7:05 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-11-24 11:57 [PATCH] ath10k: Do not limit RTS threshold value to 2347 Vivek Natarajan
2014-11-24 18:35 ` Ben Greear
2014-11-26  7:16   ` Kalle Valo
2014-11-26  7:29   ` Natarajan, Vivekanandan
2014-12-01  7:04 ` Kalle Valo
  -- strict thread matches above, loose matches on Subject: below --
2014-11-24 11:51 Vivek Natarajan

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.