Return-path: Received: from esa1.microchip.iphmx.com ([68.232.147.91]:37871 "EHLO esa1.microchip.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753000AbeDZHSV (ORCPT ); Thu, 26 Apr 2018 03:18:21 -0400 Date: Thu, 26 Apr 2018 11:46:15 +0530 From: Ajay Singh To: , CC: , , , , , Subject: Re: [PATCH v2 00/21] staging: wilc1000: remove unused code and reorder functions Message-ID: <20180426114615.563850f3@ajaysk-VirtualBox> (sfid-20180426_091825_432909_5D90F834) In-Reply-To: <1524676706-13179-1-git-send-email-ajay.kathat@microchip.com> References: <1524676706-13179-1-git-send-email-ajay.kathat@microchip.com> MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Sender: linux-wireless-owner@vger.kernel.org List-ID: Hi Greg, This series is actually V1 and not V2. Sorry, it's mistake from my side. If you want me to resend this series, please let me know. On Wed, 25 Apr 2018 22:48:05 +0530 Ajay Singh wrote: > This patch series contains fixes to remove the unused code. It also > has patches to reorganize the function and simplify the function > logic. > > Ajay Singh (21): > staging: wilc1000: replace crc7_byte() with inline macro CRC7_BYTE > staging: wilc1000: align the #define in wilc_spi file > staging: wilc1000: remove unnecessary assingment from 'if' > conditions staging: wilc1000: remove 'cmd' variable in wilc_spi > functions staging: wicl1000: removed the unsed variables in > wilc_parse_network_info() > staging: wilc1000: remove inner {} in wilc_parse_network_info() > staging: wilc1000: simplified if conditions in spi_data_write() > staging: wilc1000: remove unused variable scan_while_connected > staging: wilc1000: remove unsed typedef wilc_debug_func > staging: wilc1000: remove used #define HIF_SDIO_GPIO_IRQ > staging: wilc1000: remove unused #define related to MAC status > staging: wilc1000: rename mac status macros and moved related > #define together > staging: wilc1000: rename WILC_WFI_stats to avoid uppercase > staging: wilc1000: rename num_reg_frame macro to have uppercase in > macro name > staging: wilc1000: rename wlan init and deinit function prefixed > with wil1000 > staging: wilc1000: remove unused elements in 'wilc_priv' struct > staging: wilc1000: remove unused enum 'stats_flags' > staging: wilc1000: rename P2P_LISTEN_STATE variable to use > lowercase staging: wilc1000: remove inner block '{}' in > handle_remain_on_chan() staging: wilc1000: change function to > static in linux_wlan staging: wilc1000: reorder functions to avoid > forward declaration in linux_wlan > > drivers/staging/wilc1000/coreconfigurator.c | 84 > +++++++-------- drivers/staging/wilc1000/host_interface.c > | 64 ++++-------- > drivers/staging/wilc1000/linux_wlan.c | 122 > ++++++++++------------ > drivers/staging/wilc1000/wilc_sdio.c | 12 +-- > drivers/staging/wilc1000/wilc_spi.c | 70 > ++++++------- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | > 2 +- drivers/staging/wilc1000/wilc_wfi_netdevice.h | 23 +--- > drivers/staging/wilc1000/wilc_wlan.h | 7 -- > drivers/staging/wilc1000/wilc_wlan_if.h | 11 +- 9 files > changed, 158 insertions(+), 237 deletions(-) > Regards, Ajay