mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Mateusz Kulikowski <mateusz.kulikowski@gmail.com>
To: gregkh@linuxfoundation.org
Cc: Mateusz Kulikowski <mateusz.kulikowski@gmail.com>,
	devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org
Subject: [PATCH v2 00/21] staging: rtl8192e: Fix more checkpatch.pl warnings
Date: Mon, 13 Apr 2015 23:47:23 +0200	[thread overview]
Message-ID: <1428961664-4241-1-git-send-email-mateusz.kulikowski@gmail.com> (raw)

New cleanup patchset for rtl8192e. It fixes (mostly) checkpatch.pl warnings.
When applied, checkpatch.pl warning count drops to 34 (from 1111).
It should apply cleanly to staging-next/testing (c610f7f7) branches.

Series was smoke tested on rtl8192e card vs staging-next:
- Module load/unload
- Interface up/down

Most of changes are related to checkpatch.pl with the exception of
- bugfix - staging: rtl8192e: Fix DeviceID in rtl8192_pci_findadapter()
  This is clearly (in my opinion) typo - I checked two rtl8192e cards from 
  different vendors and both had DeviceID == 0x8192.
- Removal of unimplemented/unused iwpriv handlers - they just give false sense 
  of hope to iwpriv user

Additional changes in v2:
- Fix (most) LONG_LINE warnings
- Replace memcpy() with custom macro - later __aligned(2) will be added to 
  structures (where possible) - I prefer not to do this change before I have 
  working hardware. Unfortunantely these cards are grumpy and refuse to enable 
  radio in PC mPCIe slot (even after soldering W_DISABLE override and using two 
  different antenna sets)
- Fix (most) remaining PREFER_PR_LEVEL warnings

Notes from v1:
This series of patches fixes another set of checkpatch.pl warnings.

Most of the patches are trivial, with the exception of #8, #7 and #5;
Driver logic should not be affected. 
Some of the patches cause LONG_LINE warnings, but fix has to wait until
I do more driver refactorings (It's hard to keep line length when variable 
names have 30 characters).

Mateusz Kulikowski (21):
  staging: rtl8192e: Fix SPACE_BEFORE_TAB warnings
  staging: rtl8192e: Copy comments from r819XE_phyreg.h to r8192E_phyreg.h
  staging: rtl8192e: remove r819xE_phyreg.h
  staging: rtl8192e: Fix SPACING errors
  staging: rtl8192e: Remove bb tx gains from r8192_priv
  staging: rtl8192e: Fix LINE_SPACING warning
  staging: rtl8192e: Fix DEEP_INDENTATION warnings in rtllib_parse_info_param()
  staging: rtl8192e: Fix DEEP_INDENTATION warnings in rtl_dm.c
  staging: rtl8192e: Fix DeviceID in rtl8192_pci_findadapter()
  staging: rtl8192e: replace memcpy() -> ether_addr_copy_unaligned()
  staging: rtl8192e: Fix DEEP_INDENTATION warning in rtllib_parse_info_param()
  staging: rtl8192e: replace memcmp() with ether_addr_equal_unaligned()
  staging: rtl8192e: Fix PREFER_PR_LEVEL warnings
  staging: rtl8192e: Fix LONG_LINE warnings
  staging: rtl8192e: Fix LONG_LINE warnings - modify variables
  staging: rtl8192e: Fix LONG_LING in rtllib_parse_info_param()
  staging: rtl8192e: Fix LONG_LINE warnings (conditions)
  staging: rtl8192e: fix LONG_LINE warnings - replace ?: expressions
  staging: rtl8192e: Fix LONG_LINE in rtllib_parse_mife_generic()
  staging: rtl8192e: Fix LONG_LINE in simplify rtl8192_process_phyinfo()
  staging: rtl8192e: Remove unimplemented iwpriv handlers

 drivers/staging/rtl8192e/dot11d.h                  |    4 +-
 drivers/staging/rtl8192e/rtl8192e/r8192E_dev.c     |  144 +-
 .../staging/rtl8192e/rtl8192e/r8192E_firmware.c    |   42 +-
 drivers/staging/rtl8192e/rtl8192e/r8192E_phy.c     |   51 +-
 drivers/staging/rtl8192e/rtl8192e/r8192E_phyreg.h  | 1496 ++++++++++---------
 drivers/staging/rtl8192e/rtl8192e/r819xE_phyreg.h  |  908 ------------
 drivers/staging/rtl8192e/rtl8192e/rtl_cam.c        |   53 +-
 drivers/staging/rtl8192e/rtl8192e/rtl_core.c       |   10 +-
 drivers/staging/rtl8192e/rtl8192e/rtl_core.h       |   21 +-
 drivers/staging/rtl8192e/rtl8192e/rtl_dm.c         | 1546 ++++++++------------
 drivers/staging/rtl8192e/rtl8192e/rtl_dm.h         |    9 +
 drivers/staging/rtl8192e/rtl8192e/rtl_pci.c        |    2 +-
 drivers/staging/rtl8192e/rtl8192e/rtl_wx.c         |   89 +-
 drivers/staging/rtl8192e/rtl819x_BAProc.c          |   12 +-
 drivers/staging/rtl8192e/rtl819x_HTProc.c          |    3 +-
 drivers/staging/rtl8192e/rtl819x_TSProc.c          |   21 +-
 drivers/staging/rtl8192e/rtllib.h                  |    8 +-
 drivers/staging/rtl8192e/rtllib_crypt.c            |   17 +-
 drivers/staging/rtl8192e/rtllib_crypt_tkip.c       |   77 +-
 drivers/staging/rtl8192e/rtllib_debug.h            |    3 +-
 drivers/staging/rtl8192e/rtllib_rx.c               |  797 +++++-----
 drivers/staging/rtl8192e/rtllib_softmac.c          |  360 ++---
 drivers/staging/rtl8192e/rtllib_softmac_wx.c       |    5 +-
 drivers/staging/rtl8192e/rtllib_tx.c               |   87 +-
 drivers/staging/rtl8192e/rtllib_wx.c               |    4 +-
 25 files changed, 2289 insertions(+), 3480 deletions(-)
 delete mode 100644 drivers/staging/rtl8192e/rtl8192e/r819xE_phyreg.h

-- 
1.8.4.1


             reply	other threads:[~2015-04-13 21:47 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-13 21:47 Mateusz Kulikowski [this message]
2015-04-13 21:47 ` [PATCH v2 01/21] staging: rtl8192e: Fix SPACE_BEFORE_TAB warnings Mateusz Kulikowski
2015-04-13 21:47 ` [PATCH v2 02/21] staging: rtl8192e: Copy comments from r819XE_phyreg.h to r8192E_phyreg.h Mateusz Kulikowski
2015-04-13 21:47 ` [PATCH v2 03/21] staging: rtl8192e: remove r819xE_phyreg.h Mateusz Kulikowski
2015-04-13 21:47 ` [PATCH v2 04/21] staging: rtl8192e: Fix SPACING errors Mateusz Kulikowski
2015-04-13 21:47 ` [PATCH v2 05/21] staging: rtl8192e: Remove bb tx gains from r8192_priv Mateusz Kulikowski
2015-04-13 21:47 ` [PATCH v2 06/21] staging: rtl8192e: Fix LINE_SPACING warning Mateusz Kulikowski
2015-04-13 21:47 ` [PATCH v2 07/21] staging: rtl8192e: Fix DEEP_INDENTATION warnings in rtllib_parse_info_param() Mateusz Kulikowski
2015-04-13 21:47 ` [PATCH v2 08/21] staging: rtl8192e: Fix DEEP_INDENTATION warnings in rtl_dm.c Mateusz Kulikowski
2015-04-13 21:47 ` [PATCH v2 09/21] staging: rtl8192e: Fix DeviceID in rtl8192_pci_findadapter() Mateusz Kulikowski
2015-04-13 21:47 ` [PATCH v2 10/21] staging: rtl8192e: replace memcpy() -> ether_addr_copy_unaligned() Mateusz Kulikowski
2015-04-14  8:00   ` Dan Carpenter
2015-04-15 21:08     ` Mateusz Kulikowski
2015-04-15 21:36       ` Joe Perches
2015-05-03 19:10   ` Greg KH
2015-05-03 19:17     ` Mateusz Kulikowski
2015-04-13 21:47 ` [PATCH v2 11/21] staging: rtl8192e: Fix DEEP_INDENTATION warning in rtllib_parse_info_param() Mateusz Kulikowski
2015-04-13 21:47 ` [PATCH v2 12/21] staging: rtl8192e: replace memcmp() with ether_addr_equal_unaligned() Mateusz Kulikowski
2015-04-13 21:47 ` [PATCH v2 13/21] staging: rtl8192e: Fix PREFER_PR_LEVEL warnings Mateusz Kulikowski
2015-04-14  8:07   ` Dan Carpenter
2015-04-15 21:27     ` Mateusz Kulikowski
2015-04-14  8:09   ` Dan Carpenter
2015-04-13 21:47 ` [PATCH v2 14/21] staging: rtl8192e: Fix LONG_LINE warnings Mateusz Kulikowski
2015-04-13 21:47 ` [PATCH v2 15/21] staging: rtl8192e: Fix LONG_LINE warnings - modify variables Mateusz Kulikowski
2015-04-14  8:40   ` Dan Carpenter
2015-04-15 21:56     ` Mateusz Kulikowski
2015-04-13 21:47 ` [PATCH v2 16/21] staging: rtl8192e: Fix LONG_LING in rtllib_parse_info_param() Mateusz Kulikowski
2015-04-13 21:47 ` [PATCH v2 17/21] staging: rtl8192e: Fix LONG_LINE warnings (conditions) Mateusz Kulikowski
2015-04-13 21:47 ` [PATCH v2 18/21] staging: rtl8192e: fix LONG_LINE warnings - replace ?: expressions Mateusz Kulikowski
2015-04-13 21:47 ` [PATCH v2 19/21] staging: rtl8192e: Fix LONG_LINE in rtllib_parse_mife_generic() Mateusz Kulikowski
2015-04-13 21:47 ` [PATCH v2 20/21] staging: rtl8192e: Fix LONG_LINE in simplify rtl8192_process_phyinfo() Mateusz Kulikowski
2015-04-13 21:47 ` [PATCH v2 21/21] staging: rtl8192e: Remove unimplemented iwpriv handlers Mateusz Kulikowski
2015-04-13 22:12 ` [PATCH v2 00/21] staging: rtl8192e: Fix more checkpatch.pl warnings Dan Carpenter
2015-04-13 22:15   ` Mateusz Kulikowski
2015-04-13 22:33     ` Dan Carpenter
2015-04-13 22:39       ` Mateusz Kulikowski

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=1428961664-4241-1-git-send-email-mateusz.kulikowski@gmail.com \
    --to=mateusz.kulikowski@gmail.com \
    --cc=devel@driverdev.osuosl.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-kernel@vger.kernel.org \
    /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

all inboxes | Powered by JetHome®