Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-06 Thread Rickard Strandqvist
2015-01-05 7:34 GMT+01:00 Rafał Miłecki : > On 3 January 2015 at 13:28, Rickard Strandqvist > wrote: >> 2015-01-02 22:34 GMT+01:00 Rafał Miłecki : >>> >>> 1) I gave you Ack for the changes >>> 2) You could drop "net: wireless: " or better use something Sedat proposed >> >> Nice, yes I miss the

Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-06 Thread Rickard Strandqvist
2015-01-05 7:34 GMT+01:00 Rafał Miłecki zaj...@gmail.com: On 3 January 2015 at 13:28, Rickard Strandqvist rickard_strandqv...@spectrumdigital.se wrote: 2015-01-02 22:34 GMT+01:00 Rafał Miłecki zaj...@gmail.com: 1) I gave you Ack for the changes 2) You could drop net: wireless: or better use

Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-04 Thread Rafał Miłecki
On 3 January 2015 at 13:28, Rickard Strandqvist wrote: > 2015-01-02 22:34 GMT+01:00 Rafał Miłecki : >> >> 1) I gave you Ack for the changes >> 2) You could drop "net: wireless: " or better use something Sedat proposed > > Nice, yes I miss the Ack :) > > I just got one more complaining about my

Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-04 Thread Rafał Miłecki
On 3 January 2015 at 13:28, Rickard Strandqvist rickard_strandqv...@spectrumdigital.se wrote: 2015-01-02 22:34 GMT+01:00 Rafał Miłecki zaj...@gmail.com: 1) I gave you Ack for the changes 2) You could drop net: wireless: or better use something Sedat proposed Nice, yes I miss the Ack :) I

Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-03 Thread Rickard Strandqvist
2015-01-03 15:51 GMT+01:00 Sedat Dilek : > On Sat, Jan 3, 2015 at 3:45 PM, Rickard Strandqvist > wrote: > [ ... ] >>> > I just got one more complaining about my subject-line in "net: wireless: " >>> > I use some sed call for this, so it's easy to fix. I will now remove that >>> > part

Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-03 Thread Sedat Dilek
On Sat, Jan 3, 2015 at 3:45 PM, Rickard Strandqvist wrote: [ ... ] >> > I just got one more complaining about my subject-line in "net: wireless: " >> > I use some sed call for this, so it's easy to fix. I will now remove that >> > part hereinafter. >> > I check in Documentation/ but did not find

Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-03 Thread Rickard Strandqvist
2015-01-03 14:19 GMT+01:00 Sedat Dilek : > > On Sat, Jan 3, 2015 at 1:28 PM, Rickard Strandqvist > wrote: > > 2015-01-02 22:34 GMT+01:00 Rafał Miłecki : > >> > >> On 2 January 2015 at 18:46, Rickard Strandqvist > >> wrote: > >> > 2015-01-02 13:14 GMT+01:00 Sedat Dilek : > >> >> > >> >> On Fri,

Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-03 Thread Sedat Dilek
On Sat, Jan 3, 2015 at 1:28 PM, Rickard Strandqvist wrote: > 2015-01-02 22:34 GMT+01:00 Rafał Miłecki : >> >> On 2 January 2015 at 18:46, Rickard Strandqvist >> wrote: >> > 2015-01-02 13:14 GMT+01:00 Sedat Dilek : >> >> >> >> On Fri, Jan 2, 2015 at 1:06 PM, Rafał Miłecki wrote: >> >> > On 2

Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-03 Thread Rickard Strandqvist
2015-01-03 14:19 GMT+01:00 Sedat Dilek sedat.di...@gmail.com: On Sat, Jan 3, 2015 at 1:28 PM, Rickard Strandqvist rickard_strandqv...@spectrumdigital.se wrote: 2015-01-02 22:34 GMT+01:00 Rafał Miłecki zaj...@gmail.com: On 2 January 2015 at 18:46, Rickard Strandqvist

Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-03 Thread Sedat Dilek
On Sat, Jan 3, 2015 at 3:45 PM, Rickard Strandqvist rickard_strandqv...@spectrumdigital.se wrote: [ ... ] I just got one more complaining about my subject-line in net: wireless: I use some sed call for this, so it's easy to fix. I will now remove that part hereinafter. I check in

Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-03 Thread Rickard Strandqvist
2015-01-03 15:51 GMT+01:00 Sedat Dilek sedat.di...@gmail.com: On Sat, Jan 3, 2015 at 3:45 PM, Rickard Strandqvist rickard_strandqv...@spectrumdigital.se wrote: [ ... ] I just got one more complaining about my subject-line in net: wireless: I use some sed call for this, so it's easy to fix.

Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-03 Thread Sedat Dilek
On Sat, Jan 3, 2015 at 1:28 PM, Rickard Strandqvist rickard_strandqv...@spectrumdigital.se wrote: 2015-01-02 22:34 GMT+01:00 Rafał Miłecki zaj...@gmail.com: On 2 January 2015 at 18:46, Rickard Strandqvist rickard_strandqv...@spectrumdigital.se wrote: 2015-01-02 13:14 GMT+01:00 Sedat Dilek

Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-02 Thread Rafał Miłecki
On 2 January 2015 at 18:46, Rickard Strandqvist wrote: > 2015-01-02 13:14 GMT+01:00 Sedat Dilek : >> >> On Fri, Jan 2, 2015 at 1:06 PM, Rafał Miłecki wrote: >> > On 2 January 2015 at 13:05, Rafał Miłecki wrote: >> >> On 1 January 2015 at 16:46, Rickard Strandqvist >> >> wrote: >> >>> Remove

Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-02 Thread Sedat Dilek
On Fri, Jan 2, 2015 at 1:06 PM, Rafał Miłecki wrote: > On 2 January 2015 at 13:05, Rafał Miłecki wrote: >> On 1 January 2015 at 16:46, Rickard Strandqvist >> wrote: >>> Remove the function b43legacy_radio_set_tx_iq() that is not used anywhere. >>> >>> This was partially found by using a static

Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-02 Thread Rafał Miłecki
On 2 January 2015 at 13:05, Rafał Miłecki wrote: > On 1 January 2015 at 16:46, Rickard Strandqvist > wrote: >> Remove the function b43legacy_radio_set_tx_iq() that is not used anywhere. >> >> This was partially found by using a static code analysis program called >> cppcheck. > > It seems to be

Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-02 Thread Rafał Miłecki
On 1 January 2015 at 16:46, Rickard Strandqvist wrote: > Remove the function b43legacy_radio_set_tx_iq() that is not used anywhere. > > This was partially found by using a static code analysis program called > cppcheck. It seems to be for A-PHY based hardware (with 0x2060 radio id) which is not

Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-02 Thread Sedat Dilek
On Fri, Jan 2, 2015 at 1:06 PM, Rafał Miłecki zaj...@gmail.com wrote: On 2 January 2015 at 13:05, Rafał Miłecki zaj...@gmail.com wrote: On 1 January 2015 at 16:46, Rickard Strandqvist rickard_strandqv...@spectrumdigital.se wrote: Remove the function b43legacy_radio_set_tx_iq() that is not used

Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-02 Thread Rafał Miłecki
On 2 January 2015 at 13:05, Rafał Miłecki zaj...@gmail.com wrote: On 1 January 2015 at 16:46, Rickard Strandqvist rickard_strandqv...@spectrumdigital.se wrote: Remove the function b43legacy_radio_set_tx_iq() that is not used anywhere. This was partially found by using a static code analysis

Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-02 Thread Rafał Miłecki
On 1 January 2015 at 16:46, Rickard Strandqvist rickard_strandqv...@spectrumdigital.se wrote: Remove the function b43legacy_radio_set_tx_iq() that is not used anywhere. This was partially found by using a static code analysis program called cppcheck. It seems to be for A-PHY based hardware

Re: [PATCH] net: wireless: b43legacy: radio.c: Remove unused function

2015-01-02 Thread Rafał Miłecki
On 2 January 2015 at 18:46, Rickard Strandqvist rickard_strandqv...@spectrumdigital.se wrote: 2015-01-02 13:14 GMT+01:00 Sedat Dilek sedat.di...@gmail.com: On Fri, Jan 2, 2015 at 1:06 PM, Rafał Miłecki zaj...@gmail.com wrote: On 2 January 2015 at 13:05, Rafał Miłecki zaj...@gmail.com wrote: