Hi Larry, On Mon, Jan 5, 2015 at 12:45 PM, Larry Finger <Larry.Finger@xxxxxxxxxxxx> wrote: > On 01/03/2015 06:03 AM, Rickard Strandqvist wrote: >> >> 2015-01-03 7:05 GMT+01:00 Larry Finger <Larry.Finger@xxxxxxxxxxxx>: >> >>> 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 <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. I suspect he means how he makes the subject lines. I'm guessing something like: echo $PATH | sed 's/^.*\/drivers\///;s/\//: /;s/\.c$/: Remove some unused functions/;' Thanks, -- Julian Calaby Email: julian.calaby@xxxxxxxxx Profile: http://www.google.com/profiles/julian.calaby/ -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html