The current code to initialize, register and read the clocksource is
already factored out in mmio.c via the clocksource_mmio_init function.

The only difference is the readl vs readl_relaxed.

Factor out the code with the clocksource_mmio_init function.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>
Acked-by: NMaxime Ripard <maxime.ripard@free-electrons.com>

The lock in the timer16_clocksource_read is not needed, remove it.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

The function irq_of_parse_and_map returns zero in case of failure.

Fix the return code test to check against zero.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

The fields are not used in the code, remove them.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

The macros are no longer used in the code, remove them.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

The headers are not needed, remove them.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

The current code retrieves the rate value when the timer is enabled which
occurs each time a timer is re-armed. Except if the clock frequency has changed
magically I don't see why this should be done each time.

Retrieve the clock rate value at init time only.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

The time framawork takes care of disabling the interrupts and takes a lock
to prevent races.

Remove the legacy code in the driver taking care of the races.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

The current code assumes the interrupt function is re-entrant.

That is not correct. An interrupt handler is never invoked concurrently. The
interrupt line is masked on all processors.

Remove the chewing flags in the code.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

The value returned in case of error for the 'irq_of_parse_and_map' function is
zero in case of error. Fix the check in the init code.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Specify the delta as parameter for the timer8_clock_event_start function
instead of using a macro to tell PERIODIC or ONESHOT.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Some macros are unused, delete them.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

The dev_warn is using the platform driver which was removed in the previous
patch.

Let's replace dev_warn by pr_warn.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Remove some legacy code and replace it by the clksrc-of code.

Do some cleanup and code consolidation.
Signed-off-by: NYoshinori Sato <ysato@users.sourceforge.jp>
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Implement an ARM delay timer to be used for udelay(). This allows us to
skip the delay loop calibration at boot on Marvell BG2, BG2Q, BG2CD
platforms. And after this patch, udelay() will be unaffected by CPU
frequency changes.

Note: Although in case there are several possible delay timers, we may
not select the "best" delay timer. Take one Marvell Berlin platform for
example: we have arch timer and dw-apb timer. The arch timer freq is
25MHZ while the dw-apb timer freq is 100MHZ, current selection would
choose the dw-apb timer. But the dw apb timer is on the APB bus while
arch timer sits in CPU, the cost of accessing the apb timer is higher
than the arch timer. We could introduce "rating" concept to delay
timer, but this approach "brings a lot of complexity and workarounds
in the code for a small benefit" as pointed out by Daniel.

Later, Arnd pointed out "However, we could argue that this actually
doesn't matter at all, because the entire point of the ndelay()/
udelay()/mdelay() functions is to waste CPU cycles doing not much at
all, so we can just as well waste them reading the timer register
than spinning on the CPU reading the arch timer more often.", so we
just simply register the dw apb base delay timer.
Signed-off-by: NJisheng Zhang <jszhang@marvell.com>
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

In order to compile on all arch without error with 'allyesconfig' make
sure the platform selected the GENERIC_CLOCKEVENTS. Without this patch
the new added drivers will prevent the kernel to compile on PARISC.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Increase the compilation test coverage by adding the COMPILE_TEST option.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Increase the compilation test coverage by adding the COMPILE_TEST option.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Let the platform's Kconfig to select the clock instead of having a reverse
dependency from the driver to the platform options.

Add the COMPILE_TEST option for the compilation test coverage. Due to the
non portable 'delay' code, this driver is only compilable on ARM.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>
Tested-by: NKrzysztof Kozlowski <k.kozlowski@samsung.com>
Reviewed-by: NKrzysztof Kozlowski <k.kozlowski@samsung.com>
Reviewed-by: NChanwoo Choi <cw00.choi@samsung.com>

Let the platform's Kconfig to select the clock instead of having a reverse
dependency from the driver to the platform options.

Add the COMPILE_TEST option for the compilation test coverage.

This change is debatable as the option itself in the Kconfig allows to
select the driver for the platform or not. This change will make the prcmu
timer always selected.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>
Acked-by: NLinus Walleij <linus.walleij@linaro.org>

Increase the compilation test coverage by adding the COMPILE_TEST option.

Due to the non portable code for the delay timer, this option is only
available for the ARM architecture.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Increase the compilation test coverage by adding the COMPILE_TEST option.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Increase the compilation test coverage by adding the COMPILE_TEST option.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Increase the compilation test coverage by adding the COMPILE_TEST option.

The driver depends on the common clock framework, thus the dependency added
on COMMON_CLK.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Increase the compilation test coverage by adding the COMPILE_TEST option.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Remove the <asm/time.h> header inclusion which is pointless.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Increase the compilation test coverage by adding the COMPILE_TEST option.

Due to the non portable code for the delay timer, this option is only
available for the ARM architecture.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Increase the compilation test coverage by adding the COMPILE_TEST option.

The driver depends on the common clock framework, thus the dependency added
on COMMON_CLK.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Increase the compilation test coverage by adding the COMPILE_TEST option.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Increase the compilation test coverage by adding the COMPILE_TEST option.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Increase the compilation test coverage by adding the COMPILE_TEST option.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Increase the compilation test coverage by adding the COMPILE_TEST option.

The driver is using the atomic_io API which is not portable, so the
compilation is restricted to ARM only.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Increase the compilation test coverage by adding the COMPILE_TEST option.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Increase the compilation test coverage by adding the COMPILE_TEST option.

Due to the non portable 'delay' code, the compilation is restricted to the
ARM architecture only.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Increase the compilation test coverage by adding the COMPILE_TEST option.

Due to the dsb() usage in the driver, this driver is only compilable on
ARM and ARM64.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Increase the compilation test coverage by adding the COMPILE_TEST option.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>

Increase the compilation test coverage by adding the COMPILE_TEST option.
Signed-off-by: NDaniel Lezcano <daniel.lezcano@linaro.org>