diff options
author | Nathan Chancellor <natechancellor@gmail.com> | 2019-03-27 01:01:27 -0400 |
---|---|---|
committer | Daniel Lezcano <daniel.lezcano@linaro.org> | 2019-04-16 08:26:54 -0400 |
commit | 4d86c9f73c5a9a7c3c0661e922509c2c51801671 (patch) | |
tree | 79f6972398ca649c4c79689d79c32aeea46fa352 /drivers/clocksource | |
parent | fbc87aa0f7c429999dc31f1bac3b2615008cac32 (diff) |
clocksource/drivers/timer-ti-dm: Remove omap_dm_timer_set_load_start
Commit 008258d995a6 ("clocksource/drivers/timer-ti-dm: Make
omap_dm_timer_set_load_start() static") made omap_dm_time_set_load_start
static because its prototype was not defined in a header. Unfortunately,
this causes a build warning on multi_v7_defconfig because this function
is not used anywhere in this translation unit:
drivers/clocksource/timer-ti-dm.c:589:12: error: unused function
'omap_dm_timer_set_load_start' [-Werror,-Wunused-function]
In fact, omap_dm_timer_set_load_start hasn't been used anywhere since
commit f190be7f39a5 ("staging: tidspbridge: remove driver") and the
prototype was removed in commit 592ea6bd1fad ("clocksource: timer-ti-dm:
Make unexported functions static"), which is probably where this should
have happened.
Fixes: 592ea6bd1fad ("clocksource: timer-ti-dm: Make unexported functions static")
Fixes: 008258d995a6 ("clocksource/drivers/timer-ti-dm: Make omap_dm_timer_set_load_start() static")
Signed-off-by: Nathan Chancellor <natechancellor@gmail.com>
Acked-by: Tony Lindgren <tony@atomide.com>
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Diffstat (limited to 'drivers/clocksource')
-rw-r--r-- | drivers/clocksource/timer-ti-dm.c | 28 |
1 files changed, 0 insertions, 28 deletions
diff --git a/drivers/clocksource/timer-ti-dm.c b/drivers/clocksource/timer-ti-dm.c index 3352da6ed61f..ee8ec5a8cb16 100644 --- a/drivers/clocksource/timer-ti-dm.c +++ b/drivers/clocksource/timer-ti-dm.c | |||
@@ -585,34 +585,6 @@ static int omap_dm_timer_set_load(struct omap_dm_timer *timer, int autoreload, | |||
585 | return 0; | 585 | return 0; |
586 | } | 586 | } |
587 | 587 | ||
588 | /* Optimized set_load which removes costly spin wait in timer_start */ | ||
589 | static int omap_dm_timer_set_load_start(struct omap_dm_timer *timer, | ||
590 | int autoreload, unsigned int load) | ||
591 | { | ||
592 | u32 l; | ||
593 | |||
594 | if (unlikely(!timer)) | ||
595 | return -EINVAL; | ||
596 | |||
597 | omap_dm_timer_enable(timer); | ||
598 | |||
599 | l = omap_dm_timer_read_reg(timer, OMAP_TIMER_CTRL_REG); | ||
600 | if (autoreload) { | ||
601 | l |= OMAP_TIMER_CTRL_AR; | ||
602 | omap_dm_timer_write_reg(timer, OMAP_TIMER_LOAD_REG, load); | ||
603 | } else { | ||
604 | l &= ~OMAP_TIMER_CTRL_AR; | ||
605 | } | ||
606 | l |= OMAP_TIMER_CTRL_ST; | ||
607 | |||
608 | __omap_dm_timer_load_start(timer, l, load, timer->posted); | ||
609 | |||
610 | /* Save the context */ | ||
611 | timer->context.tclr = l; | ||
612 | timer->context.tldr = load; | ||
613 | timer->context.tcrr = load; | ||
614 | return 0; | ||
615 | } | ||
616 | static int omap_dm_timer_set_match(struct omap_dm_timer *timer, int enable, | 588 | static int omap_dm_timer_set_match(struct omap_dm_timer *timer, int enable, |
617 | unsigned int match) | 589 | unsigned int match) |
618 | { | 590 | { |