From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752052AbbDMVr5 (ORCPT ); Mon, 13 Apr 2015 17:47:57 -0400 Received: from mail-lb0-f179.google.com ([209.85.217.179]:33861 "EHLO mail-lb0-f179.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751514AbbDMVr4 (ORCPT ); Mon, 13 Apr 2015 17:47:56 -0400 From: Mateusz Kulikowski To: gregkh@linuxfoundation.org Cc: Mateusz Kulikowski , 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 Message-Id: <1428961664-4241-1-git-send-email-mateusz.kulikowski@gmail.com> X-Mailer: git-send-email 1.8.4.1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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