[PATCH v2] hostapd: remove invalid condition check on vht_oper_chwidth

peter.oh at bowerswilkins.com peter.oh at bowerswilkins.com
Tue Jun 13 16:21:11 PDT 2017


From: Peter Oh <peter.oh at 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 at bowerswilkins.com>
---
 src/common/hw_features_common.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

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)
-- 
2.7.4




More information about the Hostap mailing list