'broadcast' is used just once and we can use tick_resume_broadcast() directly instead. Also it changes the code a bit to get rid of extra indentation level which was forcing us to break function calls into two lines.
Signed-off-by: Viresh Kumar viresh.kumar@linaro.org --- kernel/time/tick-common.c | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-)
diff --git a/kernel/time/tick-common.c b/kernel/time/tick-common.c index 85ee9fd..2dc6822 100644 --- a/kernel/time/tick-common.c +++ b/kernel/time/tick-common.c @@ -350,17 +350,16 @@ void tick_resume(void) { struct tick_device *td = tick_get_cpu_device(); struct clock_event_device *dev = td->evtdev; - int broadcast = tick_resume_broadcast();
clockevents_set_mode(dev, CLOCK_EVT_MODE_RESUME);
- if (!broadcast) { - if (td->mode == TICKDEV_MODE_PERIODIC) - tick_setup_periodic(dev, 0); - else - tick_setup_oneshot(dev, dev->event_handler, - ktime_get()); - } + if (tick_resume_broadcast()) + return; + + if (td->mode == TICKDEV_MODE_PERIODIC) + tick_setup_periodic(dev, 0); + else + tick_setup_oneshot(dev, dev->event_handler, ktime_get()); }
/**