Re: [uml-user] [PATCH 2/2] clocksource: Make clocksource register functions void

2014-01-23 Thread Hans-Christian Egtvedt
Around Thu 23 Jan 2014 15:12:46 +0800 or thereabout, Yijing Wang wrote: > Currently, clocksource_register() and __clocksource_register_scale() > functions always return 0, it's pointless, make functions void. > And remove the dead code that check the clocksource_register_hz() > return value. > > S

Re: [uml-user] [PATCH 2/2] clocksource: Make clocksource register functions void

2014-01-23 Thread Yijing Wang
On 2014/1/23 16:04, Tony Prisk wrote: > On 23/01/14 20:58, Geert Uytterhoeven wrote: >> On Thu, Jan 23, 2014 at 8:45 AM, Tony Prisk wrote: -static inline int clocksource_register_hz(struct clocksource *cs, u32 hz) +static inline void clocksource_register_hz(struct clocksource *cs

Re: [uml-user] [PATCH 2/2] clocksource: Make clocksource register functions void

2014-01-23 Thread Daniel Lezcano
On 01/23/2014 08:12 AM, Yijing Wang wrote: > Currently, clocksource_register() and __clocksource_register_scale() > functions always return 0, it's pointless, make functions void. > And remove the dead code that check the clocksource_register_hz() > return value. > > Signed-off-by: Yijing Wang We

[uml-user] [PATCH 2/2] clocksource: Make clocksource register functions void

2014-01-23 Thread Yijing Wang
Currently, clocksource_register() and __clocksource_register_scale() functions always return 0, it's pointless, make functions void. And remove the dead code that check the clocksource_register_hz() return value. Signed-off-by: Yijing Wang --- arch/arm/mach-davinci/time.c|

[uml-user] [PATCH 1/2] clocksource: Remove outdated comments

2014-01-23 Thread Yijing Wang
clocksource_register() and __clocksource_register_scale() always return 0, so the comment is just pointless, it's outdated, remove it. Signed-off-by: Yijing Wang --- kernel/time/clocksource.c |3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git a/kernel/time/clocksource.c b/k

Re: [uml-user] [PATCH 2/2] clocksource: Make clocksource register functions void

2014-01-23 Thread Yijing Wang
On 2014/1/23 16:40, Daniel Lezcano wrote: > On 01/23/2014 08:12 AM, Yijing Wang wrote: >> Currently, clocksource_register() and __clocksource_register_scale() >> functions always return 0, it's pointless, make functions void. >> And remove the dead code that check the clocksource_register_hz() >> r

Re: [uml-user] [PATCH 2/2] clocksource: Make clocksource register functions void

2014-01-23 Thread David Laight
From: Linuxppc-dev Tony Prisk > On 23/01/14 20:12, Yijing Wang wrote: > > Currently, clocksource_register() and __clocksource_register_scale() > > functions always return 0, it's pointless, make functions void. > > And remove the dead code that check the clocksource_register_hz() > > return value.

Re: [uml-user] [PATCH 2/2] clocksource: Make clocksource register functions void

2014-01-23 Thread Tony Prisk
On 23/01/14 20:58, Geert Uytterhoeven wrote: > On Thu, Jan 23, 2014 at 8:45 AM, Tony Prisk wrote: >>>-static inline int clocksource_register_hz(struct clocksource *cs, u32 >>> hz) >>> +static inline void clocksource_register_hz(struct clocksource *cs, u32 >>> hz) >>>{ >>> return _

Re: [uml-user] [PATCH 2/2] clocksource: Make clocksource register functions void

2014-01-23 Thread Tony Prisk
On 23/01/14 20:12, Yijing Wang wrote: > Currently, clocksource_register() and __clocksource_register_scale() > functions always return 0, it's pointless, make functions void. > And remove the dead code that check the clocksource_register_hz() > return value. > > Signed-off-by: Yijing Wang .. >

Re: [uml-user] [PATCH 2/2] clocksource: Make clocksource register functions void

2014-01-23 Thread Geert Uytterhoeven
On Thu, Jan 23, 2014 at 8:45 AM, Tony Prisk wrote: >> -static inline int clocksource_register_hz(struct clocksource *cs, u32 >> hz) >> +static inline void clocksource_register_hz(struct clocksource *cs, u32 >> hz) >> { >> return __clocksource_register_scale(cs, 1, hz); >> } > > > Thi