Staging: ieee80211: Place constant on right side of the test.

Sandhya-Bankar · gregkh · commit 56c4a79958eb · 2016-03-11T22:09:09.000-08:00
Place constant on right side of the test.

Signed-off-by: Sandhya Bankar &lt;bankarsandhya512@gmail.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
diff --git a/drivers/staging/rtl8192u/ieee80211/dot11d.c b/drivers/staging/rtl8192u/ieee80211/dot11d.c
@@ -103,7 +103,7 @@ u8 DOT11D_GetMaxTxPwrInDbm(struct ieee80211_device *dev, u8 Channel)
 	PRT_DOT11D_INFO pDot11dInfo = GET_DOT11D_INFO(dev);
 	u8 MaxTxPwrInDbm = 255;
 
-	if (MAX_CHANNEL_NUMBER < Channel) {
+	if (Channel > MAX_CHANNEL_NUMBER) {
 		netdev_err(dev->dev, "DOT11D_GetMaxTxPwrInDbm(): Invalid Channel\n");
 		return MaxTxPwrInDbm;
 	}
@@ -139,7 +139,7 @@ int IsLegalChannel(struct ieee80211_device *dev, u8 channel)
 {
 	PRT_DOT11D_INFO pDot11dInfo = GET_DOT11D_INFO(dev);
 
-	if (MAX_CHANNEL_NUMBER < channel) {
+	if (channel > MAX_CHANNEL_NUMBER) {
 		netdev_err(dev->dev, "IsLegalChannel(): Invalid Channel\n");
 		return 0;
 	}
@@ -162,7 +162,7 @@ int ToLegalChannel(struct ieee80211_device *dev, u8 channel)
 		}
 	}
 
-	if (MAX_CHANNEL_NUMBER < channel) {
+	if (channel > MAX_CHANNEL_NUMBER) {
 		netdev_err(dev->dev, "IsLegalChannel(): Invalid Channel\n");
 		return default_chn;
 	}

Original file line number	Diff line number	Diff line change
`@@ -103,7 +103,7 @@ u8 DOT11D_GetMaxTxPwrInDbm(struct ieee80211_device *dev, u8 Channel)`
`103`	`103`	`PRT_DOT11D_INFO pDot11dInfo = GET_DOT11D_INFO(dev);`
`104`	`104`	`u8 MaxTxPwrInDbm = 255;`
`105`	`105`
`106`		`- if (MAX_CHANNEL_NUMBER < Channel) {`
	`106`	`+ if (Channel > MAX_CHANNEL_NUMBER) {`
`107`	`107`	`netdev_err(dev->dev, "DOT11D_GetMaxTxPwrInDbm(): Invalid Channel\n");`
`108`	`108`	`return MaxTxPwrInDbm;`
`109`	`109`	`}`
`@@ -139,7 +139,7 @@ int IsLegalChannel(struct ieee80211_device *dev, u8 channel)`
`139`	`139`	`{`
`140`	`140`	`PRT_DOT11D_INFO pDot11dInfo = GET_DOT11D_INFO(dev);`
`141`	`141`
`142`		`- if (MAX_CHANNEL_NUMBER < channel) {`
	`142`	`+ if (channel > MAX_CHANNEL_NUMBER) {`
`143`	`143`	`netdev_err(dev->dev, "IsLegalChannel(): Invalid Channel\n");`
`144`	`144`	`return 0;`
`145`	`145`	`}`
`@@ -162,7 +162,7 @@ int ToLegalChannel(struct ieee80211_device *dev, u8 channel)`
`162`	`162`	`}`
`163`	`163`	`}`
`164`	`164`
`165`		`- if (MAX_CHANNEL_NUMBER < channel) {`
	`165`	`+ if (channel > MAX_CHANNEL_NUMBER) {`
`166`	`166`	`netdev_err(dev->dev, "IsLegalChannel(): Invalid Channel\n");`
`167`	`167`	`return default_chn;`
`168`	`168`	`}`