Re: [PATCH] net: wireless: rtlwifi: btcoexist: halbtc8821a2ant: Remove some unused functions

[Larry Finger]

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.

Larry

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/