linux-stable.git/drivers/clocksource/h8300_timer8.c, branch linux-4.5.y Linux kernel stable tree clocksource/drivers/h8300: Use ioread / iowrite 2015-12-16T16:20:13+00:00 Yoshinori Sato ysato@users.sourceforge.jp 2015-12-04T17:48:18+00:00 d33f250af4e67d449f2c748b861ba99d50955469 Signed-off-by: Yoshinori Sato <ysato@users.sourceforge.jp> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org> 
Signed-off-by: Yoshinori Sato <ysato@users.sourceforge.jp>
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
clocksource/drivers/h8300: Initializer cleanup. 2015-12-16T10:33:35+00:00 Yoshinori Sato ysato@users.sourceforge.jp 2015-12-04T17:48:17+00:00 6f2b611db23404426a2b21b343392dc1d9584f92 Signed-off-by: Yoshinori Sato <ysato@users.sourceforge.jp> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org> 
Signed-off-by: Yoshinori Sato <ysato@users.sourceforge.jp>
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
clocksource/drivers/h8300: Simplify delta handling 2015-12-16T10:33:33+00:00 Yoshinori Sato ysato@users.sourceforge.jp 2015-12-04T17:48:16+00:00 f37632d1e9ff8445e0d1ea761f4ce7b98e059d75 Signed-off-by: Yoshinori Sato <ysato@users.sourceforge.jp> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org> 
Signed-off-by: Yoshinori Sato <ysato@users.sourceforge.jp>
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
h8300: Rename ctlr_out/in[bwl] to raw_read/write[bwl] 2015-12-15T09:12:03+00:00 Daniel Lezcano daniel.lezcano@linaro.org 2015-11-08T21:55:12+00:00 751605152b4dbcdf3da2643c965ec1c3b734e11d 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 <daniel.lezcano@linaro.org> 
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 <daniel.lezcano@linaro.org>
clocksource/drivers/h8300_timer8: Retrieve the clock rate at init time 2015-12-15T08:42:20+00:00 Daniel Lezcano daniel.lezcano@linaro.org 2015-11-08T22:24:28+00:00 cce483e0ee70c2c9c6ff08e502b6f0bba3e15f62 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 <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: Daniel Lezcano <daniel.lezcano@linaro.org>
clocksource/drivers/h8300_timer8: Remove irq and lock legacy code 2015-12-15T08:42:18+00:00 Daniel Lezcano daniel.lezcano@linaro.org 2015-11-08T21:39:12+00:00 254d8b5d59516d670111e0ac14d53fed4d61118d 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 <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: Daniel Lezcano <daniel.lezcano@linaro.org>
clocksource/drivers/h8300_timer8: Remove pointless irq re-entrant safe code 2015-12-15T08:42:16+00:00 Daniel Lezcano daniel.lezcano@linaro.org 2015-11-08T17:07:38+00:00 7053fdac76451efe5f16c9e0974dc17fcf29f6d6 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 <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: Daniel Lezcano <daniel.lezcano@linaro.org>
clocksource/drivers/h8300_timer8: Fix irq return value check 2015-12-15T08:42:14+00:00 Daniel Lezcano daniel.lezcano@linaro.org 2015-11-08T16:56:18+00:00 54a0cd5a7e107ba0cf15a4fb876595d0c8a7faab 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 <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: Daniel Lezcano <daniel.lezcano@linaro.org>
clocksource/drivers/h8300_timer8: Remove PERIODIC and ONESHOT macro 2015-12-15T08:42:13+00:00 Daniel Lezcano daniel.lezcano@linaro.org 2015-11-08T16:46:54+00:00 1f058d52b3e32b919742d451c673b3ab71c487dc 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: Daniel 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: Daniel Lezcano <daniel.lezcano@linaro.org>
clocksource/drivers/h8300_timer8: Remove unused macros 2015-12-15T08:42:11+00:00 Daniel Lezcano daniel.lezcano@linaro.org 2015-11-08T16:40:35+00:00 7116ae437fc7380ffc0b9429438bba9de9681024 Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org> 
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>