linux-toradex.git/drivers/clocksource/h8300_timer8.c, branch v4.9.16

clocksources: Switch back to the clksrc table

2016-06-28T08:19:35+00:00

All the clocksource drivers's init function are now converted to return
an error code. CLOCKSOURCE_OF_DECLARE is no longer used as well as the
clksrc-of table.

Let's convert back the names:
 - CLOCKSOURCE_OF_DECLARE_RET => CLOCKSOURCE_OF_DECLARE
 - clksrc-of-ret              => clksrc-of

Signed-off-by: Daniel Lezcano 

For exynos_mct and samsung_pwm_timer:
Acked-by: Krzysztof Kozlowski 

For arch/arc:
Acked-by: Vineet Gupta 

For mediatek driver:
Acked-by: Matthias Brugger 

For the Rockchip-part
Acked-by: Heiko Stuebner 

For STi :
Acked-by: Patrice Chotard 

For the mps2-timer.c and versatile.c changes:
Acked-by: Liviu Dudau 

For the OXNAS part :
Acked-by: Neil Armstrong 

For LPC32xx driver:
Acked-by: Sylvain Lemieux 

For Broadcom Kona timer change:
Acked-by: Ray Jui 

For Sun4i and Sun5i:
Acked-by: Chen-Yu Tsai 

For Meson6:
Acked-by: Carlo Caione 

For Keystone:
Acked-by: Santosh Shilimkar 

For NPS:
Acked-by: Noam Camus 

For bcm2835:
Acked-by: Eric Anholt

clocksource/drivers/h8300_timer8: Convert init function to return error

2016-06-28T08:19:23+00:00

The init functions do not return any error. They behave as the following:

  - panic, thus leading to a kernel crash while another timer may work and
       make the system boot up correctly

  or

  - print an error and let the caller unaware if the state of the system

Change that by converting the init functions to return an error conforming
to the CLOCKSOURCE_OF_RET prototype.

Proper error handling (rollback, errno value) will be changed later case
by case, thus this change just return back an error or success in the init
function.

Signed-off-by: Daniel Lezcano

clocksource/drivers/h8300: Use ioread / iowrite

2015-12-16T16:20:13+00:00

Signed-off-by: Yoshinori Sato 
Signed-off-by: Daniel Lezcano

clocksource/drivers/h8300: Initializer cleanup.

2015-12-16T10:33:35+00:00

Signed-off-by: Yoshinori Sato 
Signed-off-by: Daniel Lezcano

clocksource/drivers/h8300: Simplify delta handling

2015-12-16T10:33:33+00:00

Signed-off-by: Yoshinori Sato 
Signed-off-by: Daniel Lezcano

h8300: Rename ctlr_out/in[bwl] to raw_read/write[bwl]

2015-12-15T09:12:03+00:00

For the sake of consistency, let rename all ctrl_out/in calls to the write/read
calls so we have the same API consistent with the other architectures hence
open the door for the increasing of the test compilation coverage.

The unsigned long coercive cast is removed because all variables are set to
the right type "void __iomem *".

Signed-off-by: Daniel Lezcano

clocksource/drivers/h8300_timer8: Retrieve the clock rate at init time

2015-12-15T08:42:20+00:00

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: Daniel Lezcano

clocksource/drivers/h8300_timer8: Remove irq and lock legacy code

2015-12-15T08:42:18+00:00

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: Daniel Lezcano

clocksource/drivers/h8300_timer8: Remove pointless irq re-entrant safe code

2015-12-15T08:42:16+00:00

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: Daniel Lezcano

clocksource/drivers/h8300_timer8: Fix irq return value check

2015-12-15T08:42:14+00:00

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: Daniel Lezcano