[v2] hostapd: remove invalid condition check on vht_oper_chwidth

Submitted by peter.oh@bowerswilkins.com on June 13, 2017, 11:21 p.m.

Details

Message ID 9294a852badcba94279018b5b4ebaea7ac7122a6.1497396033.git.peter.oh@bowerswilkins.com
State New
Headers show

Commit Message

peter.oh@bowerswilkins.com June 13, 2017, 11:21 p.m.
From: Peter Oh <peter.oh@bowerswilkins.com>

vht_oper_chwidth value under the switch - case condition is
already fixed to VHT_CHANWIDTH_80MHZ, so no meaning to inspect
whether vht_oper_chwidth is VHT_CHANWIDTH_80MHZ or not.
Also there is no way that it has VHT_CHANWIDTH_80P80MHZ value
at that point. Hence remove the useless condition check.

Signed-off-by: Peter Oh <peter.oh@bowerswilkins.com>
---
 src/common/hw_features_common.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

Patch hide | download patch | download mbox

diff --git a/src/common/hw_features_common.c b/src/common/hw_features_common.c
index 7a96f96..1459e97 100644
--- a/src/common/hw_features_common.c
+++ b/src/common/hw_features_common.c
@@ -388,11 +388,7 @@  int hostapd_set_freq_params(struct hostapd_freq_params *data,
 		/* fall through */
 	case VHT_CHANWIDTH_80MHZ:
 		data->bandwidth = 80;
-		if ((vht_oper_chwidth == VHT_CHANWIDTH_80MHZ &&
-		     center_segment1) ||
-		    (vht_oper_chwidth == VHT_CHANWIDTH_80P80MHZ &&
-		     !center_segment1) ||
-		    !sec_channel_offset)
+		if (center_segment1 || !sec_channel_offset)
 			return -1;
 		if (!center_segment0) {
 			if (channel <= 48)