mac80211: Fix incorrect condition when checking rx timestamp

AviStern · jmberg-intel · commit 204a7dbcb27b · 2017-06-13T10:24:32.000+02:00
If the driver reports the rx timestamp at PLCP start, mac80211 can only handle legacy encoding, but the code checks that the encoding is not legacy. Fix this. Fixes: da6a435 ("mac80211: separate encoding/bandwidth from flags") Signed-off-by: Avraham Stern <avraham.stern@intel.com> Signed-off-by: Luca Coelho <luciano.coelho@intel.com> Signed-off-by: Johannes Berg <johannes.berg@intel.com>
diff --git a/net/mac80211/ieee80211_i.h b/net/mac80211/ieee80211_i.h
@@ -1531,7 +1531,7 @@ ieee80211_have_rx_timestamp(struct ieee80211_rx_status *status)
 		return true;
 	/* can't handle non-legacy preamble yet */
 	if (status->flag & RX_FLAG_MACTIME_PLCP_START &&
-	    status->encoding != RX_ENC_LEGACY)
+	    status->encoding == RX_ENC_LEGACY)
 		return true;
 	return false;
 }

Original file line number	Diff line number	Diff line change
`@@ -1531,7 +1531,7 @@ ieee80211_have_rx_timestamp(struct ieee80211_rx_status *status)`
`1531`	`1531`	`return true;`
`1532`	`1532`	`/* can't handle non-legacy preamble yet */`
`1533`	`1533`	`if (status->flag & RX_FLAG_MACTIME_PLCP_START &&`
`1534`		`- status->encoding != RX_ENC_LEGACY)`
	`1534`	`+ status->encoding == RX_ENC_LEGACY)`
`1535`	`1535`	`return true;`
`1536`	`1536`	`return false;`
`1537`	`1537`	`}`