netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Kirtika Ruchandani <kirtika.ruchandani@gmail.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: linux-wireless@vger.kernel.org, ruchandani.tina@gmail.com,
	netdev@vger.kernel.org, Julian.Calaby@gmail.com
Subject: [PATCH v2 00/10] nl80211: Fix checkpatch.pl issues
Date: Sun, 29 May 2016 19:49:57 -0700	[thread overview]
Message-ID: <cover.1464576022.git.kirtika.ruchandani@gmail.com> (raw)

This patch-set attempts to fix checkpatch.pl issues found in nl80211.c.
Each patch describes the specific warnings/checks/errors it addresses.
This patch-set ignores 'line over 80 chars' warnings, except for 2 cases.
Please check for issues with the 'fixes' for 'alignment should match
open parenthesis'. Some of them do not display correctly with git diff.
This patch-set removes 3 errors, 66 warnings and 46 checks.

Kirtika Ruchandani (10):
  nl80211: Fix checkpatch warnings
  nl80211: Fix checkpatch warnings about blank lines
  nl80211: Prefer ether_addr_copy
  nl80211: Fix checkpatch.pl warning about braces
  nl80211: Fix checkpatch.pl NULL comparison warning
  nl80211: Various checkpatch.pl spacing fixes
  nl80211: Avoid multiple assignments on same line
  nl80211: Fix spelling
  nl80211: Fix checkpatch.pl warning
  nl80211: Prefer kcalloc over kzalloc

Changes in v2:
- Break down patch into smaller independent patches as suggested by Julian 
  Calaby.This breaks down patch 3/3 in set v1 into patches 3-10 of v2.

 net/wireless/nl80211.c | 301 ++++++++++++++++++++++---------------------------
 1 file changed, 136 insertions(+), 165 deletions(-)

-- 
2.8.0.rc3.226.g39d4020

             reply	other threads:[~2016-05-30  2:50 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-30  2:49 Kirtika Ruchandani [this message]
2016-05-30  2:52 ` [PATCH v2 05/10] nl80211: Fix checkpatch.pl NULL comparison warning Kirtika Ruchandani
2016-05-30  5:22   ` Julian Calaby
2016-05-30  2:53 ` [PATCH v2 06/10] nl80211: Various checkpatch.pl spacing fixes Kirtika Ruchandani
2016-05-30  5:25   ` Julian Calaby
2016-05-30  6:04     ` Kirtika Ruchandani
2016-05-30  6:06       ` Julian Calaby
2016-05-30  6:07   ` Julian Calaby
2016-05-30  2:53 ` [PATCH v2 07/10] nl80211: Avoid multiple assignments on same line Kirtika Ruchandani
     [not found]   ` <637c25b980379d1a98d7c57cc247d49ef295cbc5.1464576022.git.kirtika.ruchandani-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-05-30  5:30     ` Julian Calaby
     [not found] ` <cover.1464576022.git.kirtika.ruchandani-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-05-30  2:50   ` [PATCH v2 01/10] nl80211: Fix checkpatch warnings Kirtika Ruchandani
2016-05-30  2:51   ` [PATCH v2 02/10] nl80211: Fix checkpatch warnings about blank lines Kirtika Ruchandani
2016-05-30  2:52   ` [PATCH v2 03/10] nl80211: Prefer ether_addr_copy Kirtika Ruchandani
     [not found]     ` <49c03996365360077fe7317d108072c30354b555.1464576022.git.kirtika.ruchandani-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-05-30  5:21       ` Julian Calaby
2016-05-30  6:50         ` Kirtika Ruchandani
2016-05-31 10:34           ` Johannes Berg
2016-05-30  2:52   ` [PATCH v2 04/10] nl80211: Fix checkpatch.pl warning about braces Kirtika Ruchandani
     [not found]     ` <aef84fd2cf9f2e8fc81b366c83cb38402513f5b1.1464576022.git.kirtika.ruchandani-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-05-30  5:22       ` Julian Calaby
2016-05-30  2:54   ` [PATCH v2 08/10] nl80211: Fix spelling Kirtika Ruchandani
2016-05-30  5:31     ` Julian Calaby
2016-05-30  2:54   ` [PATCH v2 09/10] nl80211: Fix checkpatch.pl warning Kirtika Ruchandani
     [not found]     ` <2b42b54b8128bb953569021ddadce77499859a79.1464576022.git.kirtika.ruchandani-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-05-30  5:31       ` Julian Calaby
2016-05-30  2:54 ` [PATCH v2 10/10] nl80211: Prefer kcalloc over kzalloc Kirtika Ruchandani
     [not found]   ` <ca13e9bde679512a872e5136c177676d0238dc25.1464576022.git.kirtika.ruchandani-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-05-30  5:32     ` Julian Calaby

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1464576022.git.kirtika.ruchandani@gmail.com \
    --to=kirtika.ruchandani@gmail.com \
    --cc=Julian.Calaby@gmail.com \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=ruchandani.tina@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).