Return-path: Received: from aibo.runbox.com ([91.220.196.211]:44495 "EHLO aibo.runbox.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751166AbbCOAPT (ORCPT ); Sat, 14 Mar 2015 20:15:19 -0400 From: "M. Vefa Bicakci" To: linux-kernel@vger.kernel.org, linux-wireless@vger.kernel.org Cc: joe@perches.com, Larry.Finger@lwfinger.net, gregkh@linuxfoundation.org, Jes.Sorensen@redhat.com, m.v.b@runbox.com Subject: [PATCH v2 00/16] checkpatch clean-up of rtl8723au's rtw_security.c Date: Sat, 14 Mar 2015 20:10:25 -0400 Message-Id: <1426378241-32469-1-git-send-email-m.v.b@runbox.com> (sfid-20150315_012008_176615_86042CFC) Sender: linux-wireless-owner@vger.kernel.org List-ID: These commits address a number of checkpatch.pl warnings and errors in rtl8723au's rtw_security.c. Prior to this set of commits, checkpatch.pl reported the following: total: 77 errors, 138 warnings, 1621 lines checked After applying this set of commits, checkpatch.pl reports the following: total: 0 errors, 34 warnings, 1613 lines checked where the only remaining warnings relate to lines longer than 80 characters and quoted strings split across multiple lines. There are also a number of commits which improve the code in other ways. For example, the commit titled "Reorganize a few functions to remove indentation" reworks two functions to make them more readable, whereas the commit titled "Rework two byte array comparisons" reworks two CRC verifications for the same reason. All of the individual commits have been verified to compile successfully individually via the use of the interactive rebase feature of git. v2: Please see the individual commit messages for the changes since v1. In summary, the following commits have been updated: - Reformat whitespace to increase readability - Fix "before/around/after" whitespace issues - Rework two byte array comparisons M. Vefa Bicakci (16): staging: rtl8723au: Reformat whitespace to increase readability staging: rtl8723au: Fix "before/around/after" whitespace issues staging: rtl8723au: else should follow close brace staging: rtl8723au: Fix the indentation of two lines staging: rtl8723au: Reorganize a few functions to remove indentation staging: rtl8723au: Do not initialize a static to 0 staging: rtl8723au: else is not generally useful after a return staging: rtl8723au: Remove unneeded curly braces staging: rtl8723au: trailing statements should be on next line staging: rtl8723au: that open brace should be on the previous line staging: rtl8723au: No spaces at the start of a line staging: rtl8723au: Adjust whitespace in and around comments staging: rtl8723au: suspect code indent for conditional statements staging: rtl8723au: Rework two byte array comparisons staging: rtl8723au: Correct a typo in a trace log line staging: rtl8723au: Remove unneeded comments drivers/staging/rtl8723au/core/rtw_security.c | 772 +++++++++++++------------- 1 file changed, 382 insertions(+), 390 deletions(-) -- 2.1.4