Re: [PATCH] brcm80211: brcmsmac: dma: Remove some unused functions

2015-01-09 Thread Rickard Strandqvist
2015-01-07 9:58 GMT+01:00 Arend van Spriel : > On 01/07/15 07:29, Julia Lawall wrote: >> >> >> >> On Wed, 7 Jan 2015, Rickard Strandqvist wrote: >> >>> 2015-01-05 12:06 GMT+01:00 Arend van Spriel: On 01/05/15 11:49, Kalle Valo wrote: > > > Rickard Strandqvist writes: > >

Re: [PATCH] brcm80211: brcmsmac: dma: Remove some unused functions

2015-01-07 Thread Arend van Spriel
On 01/07/15 07:29, Julia Lawall wrote: On Wed, 7 Jan 2015, Rickard Strandqvist wrote: 2015-01-05 12:06 GMT+01:00 Arend van Spriel: On 01/05/15 11:49, Kalle Valo wrote: Rickard Strandqvist writes: As I hope you can see I have made some changes regarding the subject-line. Thought it was

Re: [PATCH] brcm80211: brcmsmac: dma: Remove some unused functions

2015-01-07 Thread Arend van Spriel
On 01/07/15 00:33, Rickard Strandqvist wrote: 2015-01-05 12:06 GMT+01:00 Arend van Spriel: On 01/05/15 11:49, Kalle Valo wrote: Rickard Strandqvist writes: As I hope you can see I have made some changes regarding the subject-line. Thought it was an advantage to be able to see which file I

Re: [PATCH] brcm80211: brcmsmac: dma: Remove some unused functions

2015-01-06 Thread Julia Lawall
On Wed, 7 Jan 2015, Rickard Strandqvist wrote: > 2015-01-05 12:06 GMT+01:00 Arend van Spriel : > > On 01/05/15 11:49, Kalle Valo wrote: > >> > >> Rickard Strandqvist writes: > >> > >>> As I hope you can see I have made some changes regarding the > >>> subject-line. Thought it was an advantage t

Re: [PATCH] brcm80211: brcmsmac: dma: Remove some unused functions

2015-01-06 Thread Rickard Strandqvist
2015-01-05 12:06 GMT+01:00 Arend van Spriel : > On 01/05/15 11:49, Kalle Valo wrote: >> >> Rickard Strandqvist writes: >> >>> As I hope you can see I have made some changes regarding the >>> subject-line. Thought it was an advantage to be able to see which file >>> I actually removed something fro

Re: [PATCH] brcm80211: brcmsmac: dma: Remove some unused functions

2015-01-05 Thread Arend van Spriel
On 01/05/15 11:49, Kalle Valo wrote: Rickard Strandqvist writes: As I hope you can see I have made some changes regarding the subject-line. Thought it was an advantage to be able to see which file I actually removed something from. There seems to be a big focus on getting right on subject-line

Re: [PATCH] brcm80211: brcmsmac: dma: Remove some unused functions

2015-01-05 Thread Kalle Valo
Rickard Strandqvist writes: > As I hope you can see I have made some changes regarding the > subject-line. Thought it was an advantage to be able to see which file > I actually removed something from. There seems to be a big focus on > getting right on subject-line right in recent weeks. > > I wo

Re: [PATCH] brcm80211: brcmsmac: dma: Remove some unused functions

2015-01-05 Thread Kalle Valo
Larry Finger writes: > On 01/03/2015 06:47 PM, Rickard Strandqvist wrote: >> Removes some functions that are not used anywhere: >> dma_txflush() dma_txsuspended() >> >> This was partially found by using a static code analysis program called >> cppcheck. >> >> Signed-off-by: Rickard Strandqvist

Re: [PATCH] brcm80211: brcmsmac: dma: Remove some unused functions

2015-01-04 Thread Rickard Strandqvist
2015-01-04 7:21 GMT+01:00 Larry Finger : > On 01/03/2015 06:47 PM, Rickard Strandqvist wrote: >> >> Removes some functions that are not used anywhere: >> dma_txflush() dma_txsuspended() >> >> This was partially found by using a static code analysis program called >> cppcheck. >> >> Signed-off-by: R

Re: [PATCH] brcm80211: brcmsmac: dma: Remove some unused functions

2015-01-03 Thread Larry Finger
On 01/03/2015 06:47 PM, Rickard Strandqvist wrote: Removes some functions that are not used anywhere: dma_txflush() dma_txsuspended() This was partially found by using a static code analysis program called cppcheck. Signed-off-by: Rickard Strandqvist --- drivers/net/wireless/brcm80211/brcms

[PATCH] brcm80211: brcmsmac: dma: Remove some unused functions

2015-01-03 Thread Rickard Strandqvist
Removes some functions that are not used anywhere: dma_txflush() dma_txsuspended() This was partially found by using a static code analysis program called cppcheck. Signed-off-by: Rickard Strandqvist --- drivers/net/wireless/brcm80211/brcmsmac/dma.c | 19 --- drivers/net/wire