Return-path: Received: from mail-ob0-f169.google.com ([209.85.214.169]:39001 "EHLO mail-ob0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752808AbbAEBpq (ORCPT ); Sun, 4 Jan 2015 20:45:46 -0500 Message-ID: <54A9ECC7.6040900@lwfinger.net> (sfid-20150105_024608_054031_9F7BA537) Date: Sun, 04 Jan 2015 19:45:43 -0600 From: Larry Finger MIME-Version: 1.0 To: Rickard Strandqvist CC: Chaoming Li , Kalle Valo , Greg Kroah-Hartman , Fengguang Wu , "linux-wireless@vger.kernel.org" , Network Development , Linux Kernel Mailing List Subject: Re: [PATCH] net: wireless: rtlwifi: btcoexist: halbtc8821a2ant: Remove some unused functions References: <1420230361-1862-1-git-send-email-rickard_strandqvist@spectrumdigital.se> <54A786A4.2050202@lwfinger.net> In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Sender: linux-wireless-owner@vger.kernel.org List-ID: On 01/03/2015 06:03 AM, Rickard Strandqvist wrote: > 2015-01-03 7:05 GMT+01:00 Larry Finger : > >> On 01/02/2015 02:26 PM, Rickard Strandqvist wrote: >> >>> Removes some functions that are not used anywhere: >>> ex_halbtc8821a2ant_periodical() ex_halbtc8821a2ant_halt_notify() >>> ex_halbtc8821a2ant_bt_info_notify() >>> ex_halbtc8821a2ant_special_packet_notify() >>> ex_halbtc8821a2ant_connect_notify() ex_halbtc8821a2ant_scan_notify() >>> ex_halbtc8821a2ant_lps_notify() ex_halbtc8821a2ant_ips_notify() >>> ex_halbtc8821a2ant_display_coex_info() ex_halbtc8821a2ant_init_coex_dm() >>> ex_halbtc8821a2ant_init_hwconfig() >>> >>> This was partially found by using a static code analysis program called >>> cppcheck. >>> >>> Signed-off-by: Rickard Strandqvist >> spectrumdigital.se> >>> >> >> I know that you have been told that including "net: wireless:" in the >> subject line is discouraged. Please do not do this again. The staging >> directory is different as GregKH uses "staging:" in the subject to route >> patches, but wireless does not. >> >> As to the patch, NACK for the simple reason that I am currently working on >> a number of changes to btcoexist. Some of these routines may end up being >> removed, but others will not. Having your patch remove them, and one of >> mine adding them back just constitutes a lot of churning of the source. In >> addition, it greatly increases the probability of the source trees becoming >> unsynchronized and getting merge conflicts. >> >> Larry >> >> > > Hi Larry > > I do not recognize that there has been no diskution on the subject of "net: > wireless:" > I use some sed call for this, so it's easy to fix. You want the "net: > wireless:" part completely erased then? > I check in Documentation/ but find any clear info for this. > > Sorry for the patch, is there any way for me to see that this is something > that is being worked on? These patches are the result of private communications between the Realtek engineers and me. The only publication is in the "rock" branch of the git repo at http://github.com/lwfinger/rtlwifi_new.git. I do not understand the "use of some 'sed' call". If you edit the source and add it to the git repo with the add and commit operations, then format-patch gives you exactly what you need, and send-email does that operation. Larry