From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from hemlock.osuosl.org (smtp2.osuosl.org [140.211.166.133]) by ash.osuosl.org (Postfix) with ESMTP id 827681C0E79 for ; Mon, 13 Mar 2017 22:54:21 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by hemlock.osuosl.org (Postfix) with ESMTP id 7BD8289BFC for ; Mon, 13 Mar 2017 22:54:21 +0000 (UTC) Received: from hemlock.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id Easi8NiAHj2r for ; Mon, 13 Mar 2017 22:54:20 +0000 (UTC) Received: from out1-smtp.messagingengine.com (out1-smtp.messagingengine.com [66.111.4.25]) by hemlock.osuosl.org (Postfix) with ESMTPS id 8541489BCC for ; Mon, 13 Mar 2017 22:54:20 +0000 (UTC) From: "Tobin C. Harding" Subject: [PATCH 00/12] staging: ks7010: fix checkpatch ks_wlan_net.c Date: Tue, 14 Mar 2017 09:53:58 +1100 Message-Id: <1489445650-11398-1-git-send-email-me@tobin.cc> List-Id: Linux Driver Project Developer List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: driverdev-devel-bounces@linuxdriverproject.org Sender: "devel" To: Greg Kroah-Hartman Cc: driverdev-devel@linuxdriverproject.org, Wolfram Sang Checkpatch emits various checks, warnings, and errors when parsing ks_wlan_net.c. Patch 01 fixes spacing issues. Patch 02 converts block comments to use networking style block comments. Patch 03 converts function comments to use kernel doc format. Patch 04 moves logical operators to the end of the previous line. Patch 05 removes commented out code (dead code). Patch 06 removes multiple assignment on single line. Patch 07 moves comparison constant to the right hand side. Patch 08 removes unnecessary else statements. Patch 09 removes unnecessary cast from kmalloc() call. Patch 10 replaces single byte memset with assignment. Patch 11 reduces the level of indentation multiple times. This patch is the meat of the patch set. I was not able to easily visually parse the diff, admittedly I do not have a maintainers skill at this but if there is an easier way to split this patch up I'm happy to re-work it. First I tried reducing one level of indentation per patch but then found there was no easy way to uniquely describe each patch. Patch breaks out of case block in multiple places - is this ok? The result of this patch is less indentation, enhanced readability, and most importantly opens up the way to refactoring that fixes multiple types and instances of checkpatch warnings. Multiple break statements are intermingled, breaking out of loops and case blocks. For this reason a comment string was added to each 'break' in similar fashion to that used on #endif preprocessor directives. Patch 12 does whitespace refactor taking advantage of changes made in the previous patch. Code has not been tested. Series has been run through checkpatch --strict, Smatch, and Sparse and does not introduce any new errors. Each patch in series has been applied and built on x86_64 and PowerPC. Tobin C. Harding (12): staging: ks7010: fix checkpatch SPACING staging: ks7010: fix checkpatch BLOCK_COMMENT_STYLE staging: ks7010: convert comments to kernel doc format staging: ks7010: fix logical line continuation staging: ks7010: remove dead code staging: ks7010: remove multiple assignment staging: ks7010: move comparison to right hand side staging: ks7010: remove unnecessary else statement staging: ks7010: remove unnecessary cast staging: ks7010: fix checkpatch memset warning staging: ks7010: reduce level of indentation staging: ks7010: refactor, whitespace only drivers/staging/ks7010/ks_wlan_net.c | 1040 +++++++++++++++++++++------------- 1 file changed, 640 insertions(+), 400 deletions(-) -- 2.7.4 _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel