Checkpatch.pl complains that these lines are over 80 characters. Use the "psecuritypriv" pointer for consistency, remove unnecessary parantheses and fix the alignment. This patch just cleans up a condition, it doesn't affect runtime. Signed-off-by: Anirudh Rayabharam <anirudh.rayabharam@xxxxxxxxx> --- v2: Made the commit message clearer, removed unnecessary parantheses and fixed the alignment as suggested by Dan Carpenter <dan.carpenter@xxxxxxxxxx> drivers/staging/rtl8723bs/core/rtw_ap.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8723bs/core/rtw_ap.c b/drivers/staging/rtl8723bs/core/rtw_ap.c index 18fabf5ff44b..8062b7f36de2 100644 --- a/drivers/staging/rtl8723bs/core/rtw_ap.c +++ b/drivers/staging/rtl8723bs/core/rtw_ap.c @@ -2336,8 +2336,8 @@ void rtw_ap_restore_network(struct adapter *padapter) Update_RA_Entry(padapter, psta); /* pairwise key */ /* per sta pairwise key and settings */ - if ((padapter->securitypriv.dot11PrivacyAlgrthm == _TKIP_) || - (padapter->securitypriv.dot11PrivacyAlgrthm == _AES_)) { + if (psecuritypriv->dot11PrivacyAlgrthm == _TKIP_ || + psecuritypriv->dot11PrivacyAlgrthm == _AES_) { rtw_setstakey_cmd(padapter, psta, true, false); } } -- 2.17.1 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel