Checkpatch emits WARNING: Comparisons should place the constant on the
right side of the test.

Move constant to right hand side of test, modify operator to ensure
logic is maintained.

Signed-off-by: Tobin C. Harding <m...@tobin.cc>
---
 drivers/staging/ks7010/ks_wlan_net.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/ks7010/ks_wlan_net.c 
b/drivers/staging/ks7010/ks_wlan_net.c
index 59eabb9..cee3971 100644
--- a/drivers/staging/ks7010/ks_wlan_net.c
+++ b/drivers/staging/ks7010/ks_wlan_net.c
@@ -2164,7 +2164,7 @@ static int ks_wlan_set_pmksa(struct net_device *dev,
                                }
                        }
                        if (ptr == &priv->pmklist.head) {       /* not find 
address. */
-                               if (PMK_LIST_MAX > priv->pmklist.size) {        
/* new cache data */
+                               if (priv->pmklist.size <= PMK_LIST_MAX) {       
/* new cache data */
                                        for (i = 0; i < PMK_LIST_MAX; i++) {
                                                pmk = &priv->pmklist.pmk[i];
                                                if (!memcmp
-- 
2.7.4

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to