From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751677AbbIKHbA (ORCPT ); Fri, 11 Sep 2015 03:31:00 -0400 Received: from mail-qk0-f177.google.com ([209.85.220.177]:35543 "EHLO mail-qk0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751115AbbIKHa7 (ORCPT ); Fri, 11 Sep 2015 03:30:59 -0400 From: =?UTF-8?q?Rapha=C3=ABl=20Beamonte?= To: Greg Kroah-Hartman Cc: Dan Carpenter , =?UTF-8?q?Rapha=C3=ABl=20Beamonte?= , Paul Gortmaker , Cristina Opriceana , Andy Shevchenko , "Ragnar B. Johannsson" , Joe Perches , Marti Bolivar , Greg Donald , Ksenija Stanojevic , Arnd Bergmann , Luis de Bethencourt , Aya Mahfouz , Haneen Mohammed , Somya Anand , Rene Kolarik , Vaishali Thakkar , Melike Yurtoglu , =?UTF-8?q?Hatice=20ERT=C3=9CRK?= , devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: [PATCHv2 00/16] staging: rtl8192u: code clean up Date: Fri, 11 Sep 2015 03:29:08 -0400 Message-Id: X-Mailer: git-send-email 2.5.1 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, Following comments from Dan Carpenter on my previous patch to limit the lines lengths for rtl8192u/r8192U_core.c [1], please find attached a set of patches splitting those operations. I also took care of keeping the code the more readable possible, some of those patchs even aim to clean the original content. The last patch gets rid of the last WARNING of checkpatch about the 80 lines, as well as the TO_DO_LIST macro that was used in that module to comment out unused or unusable code. Thanks, - R. [1]: https://lkml.org/lkml/2015/8/19/173 Raphaël Beamonte (16): staging: rtl8192u: r8192U_core: fix comments lines over 80 characters staging: rtl8192u: r8192U_core: add line breaks to keep lines under 80 characters staging: rtl8192u: r8192U_core: add temporary variables to keep lines under 80 characters staging: rtl8192u: r8192U_core: reverse conditions to get lines under 80 characters staging: rtl8192u: r8192U_core: rtl8192_adapter_start: reorganize function staging: rtl8192u: r8192U_core: rtl8192_read_eeprom_info: reorganize function staging: rtl8192u: r8192U_core: rtl8192_process_phyinfo: rename variable pprevious_stats to prev_stats staging: rtl8192u: r8192U_core: rtl8192_process_phyinfo: rename variable slide_beacon_adc_pwdb_index to sb_index staging: rtl8192u: r8192U_core: rtl8192_process_phyinfo: rename variable slide_beacon_adc_pwdb_statistics to sb_stats staging: rtl8192u: r8192U_core: rtl8192_process_phyinfo: remove unneeded variable staging: rtl8192u: r8192U_core: rtl8192_process_phyinfo: rename variable rfpath to rfp staging: rtl8192u: r8192U_core: rtl8192_process_phyinfo: reorganize function staging: rtl8192u: r8192U_core: rtl8192_tx: replace some inline conditions staging: rtl8192u: r8192U_core: rtl8192_ioctl: reorganize function staging: rtl8192u: r8192U_core: replace else { if() {} } by else if () {} staging: rtl8192u: remove all code framed by symbol TO_DO_LIST drivers/staging/rtl8192u/ieee80211/ieee80211.h | 4 +- drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c | 23 - .../staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 4 - .../staging/rtl8192u/ieee80211/rtl819x_TSProc.c | 5 +- drivers/staging/rtl8192u/r8192U_core.c | 1548 ++++++++++++-------- drivers/staging/rtl8192u/r819xU_phy.c | 57 - 6 files changed, 940 insertions(+), 701 deletions(-) -- 2.5.1