On Tue, May 14, 2019 at 01:11:21AM +0530, Puranjay Mohan wrote: > @@ -2856,7 +2856,7 @@ static int ieee80211_wpa_set_encryption(struct ieee80211_device *ieee, > goto done; > } > > - if (*crypt == NULL || (*crypt)->ops != ops) { > + if (!(*crypt) || (*crypt)->ops != ops) { ^ ^ Please don't add these parentheses. > struct ieee80211_crypt_data *new_crypt; > > ieee80211_crypt_delayed_deinit(ieee, crypt); regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel