Re: [PATCH] clocksource: sirf: remove unused variable

2015-01-13 Thread Arnd Bergmann
On Tuesday 13 January 2015 23:24:24 Barry Song wrote: > > yes, arnd. i also asked Daniel to apply "clocksource: marco: Rename > marco to atlas7" rebased on this fix here: > https://git.linaro.org/people/daniel.lezcano/linux.git/commit/81428bbe1d8a073bb3741678ef1d3c1016119c77 Ok > so this one is

Re: [PATCH] clocksource: sirf: remove unused variable

2015-01-13 Thread Daniel Lezcano
On 01/13/2015 04:24 PM, Arnd Bergmann wrote: On Tuesday 13 January 2015 15:27:05 Daniel Lezcano wrote: On 01/13/2015 03:22 PM, Arnd Bergmann wrote: A recent rework of the driver left an obviously unused variable around, and now the compiler complains: clocksource/timer-marco.c: In function 'si

Re: [PATCH] clocksource: sirf: remove unused variable

2015-01-13 Thread Barry Song
2015-01-13 22:27 GMT+08:00 Daniel Lezcano : > On 01/13/2015 03:22 PM, Arnd Bergmann wrote: >> >> A recent rework of the driver left an obviously unused variable >> around, and now the compiler complains: >> >> clocksource/timer-marco.c: In function 'sirfsoc_marco_timer_init': >> clocksource/timer-m

Re: [PATCH] clocksource: sirf: remove unused variable

2015-01-13 Thread Arnd Bergmann
On Tuesday 13 January 2015 15:27:05 Daniel Lezcano wrote: > On 01/13/2015 03:22 PM, Arnd Bergmann wrote: > > A recent rework of the driver left an obviously unused variable > > around, and now the compiler complains: > > > > clocksource/timer-marco.c: In function 'sirfsoc_marco_timer_init': > > clo

Re: [PATCH] clocksource: sirf: remove unused variable

2015-01-13 Thread Daniel Lezcano
On 01/13/2015 03:22 PM, Arnd Bergmann wrote: A recent rework of the driver left an obviously unused variable around, and now the compiler complains: clocksource/timer-marco.c: In function 'sirfsoc_marco_timer_init': clocksource/timer-marco.c:260:6: warning: unused variable 'timer_div' [-Wunused

Re: [PATCH] clocksource: sirf: Remove unused variable

2015-01-11 Thread Daniel Lezcano
On 01/11/2015 04:58 PM, Barry Song wrote: 2014-12-16 0:17 GMT+08:00 Daniel Lezcano : On 12/15/2014 04:59 PM, Thierry Reding wrote: From: Thierry Reding Commit ef89af1f4380 ("clocksource: sirf: Remove hard-coded clock rate") removes all uses of the timer_div variable. Since the variable is no

Re: [PATCH] clocksource: sirf: Remove unused variable

2015-01-11 Thread Barry Song
2014-12-16 0:17 GMT+08:00 Daniel Lezcano : > On 12/15/2014 04:59 PM, Thierry Reding wrote: >> >> From: Thierry Reding >> >> Commit ef89af1f4380 ("clocksource: sirf: Remove hard-coded clock rate") >> removes all uses of the timer_div variable. Since the variable is no >> longer used it should be re

Re: [PATCH] clocksource: sirf: Remove unused variable

2015-01-11 Thread Barry Song
2014-12-24 5:33 GMT+08:00 Fabio Estevam : > Remove 'timer_div' variable to fix the following build warning: > > drivers/clocksource/timer-marco.c: In function 'sirfsoc_marco_timer_init': > drivers/clocksource/timer-marco.c:260:6: warning: unused variable 'timer_div' > [-Wunused-variable] > > Signe

Re: [PATCH] clocksource: sirf: Remove unused variable

2014-12-15 Thread Daniel Lezcano
On 12/15/2014 04:59 PM, Thierry Reding wrote: From: Thierry Reding Commit ef89af1f4380 ("clocksource: sirf: Remove hard-coded clock rate") removes all uses of the timer_div variable. Since the variable is no longer used it should be removed. Signed-off-by: Thierry Reding --- Applied to my t