linux-toradex.git/arch/sparc/kernel/leon_kernel.c, branch v4.10

sparc: leon: Fix a retry loop in leon_init_timers()

2016-12-12T02:15:49+00:00

The original code causes a static checker warning because it has a
continue inside a do { } while (0); loop.  In that context, a continue
and a break are equivalent.  The intent was to go back to the start of
the loop so the continue was a bug.

I've added a retry label at the start and changed the continue to a goto
retry.  Then I removed the do { } while (0) loop and pulled the code in
one indent level.

Fixes: 2791c1a43900 ("SPARC/LEON: added support for selecting Timer Core and Timer within core")
Signed-off-by: Dan Carpenter 
Signed-off-by: David S. Miller

sparc: Fix misspellings in comments.

2016-03-21T04:28:58+00:00

Signed-off-by: Adam Buchbinder 
Reviewed-by: Julian Calaby 
Signed-off-by: David S. Miller

genirq: Remove irq argument from irq flow handlers

2015-09-16T13:47:51+00:00

Most interrupt flow handlers do not use the irq argument. Those few
which use it can retrieve the irq number from the irq descriptor.

Remove the argument.

Search and replace was done with coccinelle and some extra helper
scripts around it. Thanks to Julia for her help!

Signed-off-by: Thomas Gleixner 
Cc: Julia Lawall 
Cc: Jiang Liu

sparc/irq: Use access helper irq_data_get_affinity_mask()

2015-07-31T20:20:05+00:00

This is a preparatory patch for moving irq_data struct members.

Signed-off-by: Jiang Liu 
Cc: David S. Miller 
Link: http://lkml.kernel.org/r/1433145945-789-27-git-send-email-jiang.liu@linux.intel.com
Signed-off-by: Thomas Gleixner

sparc: leon: Fix race condition between leon_cycles_offset and timer_interrupt

2014-09-09T23:39:10+00:00

This makes sure that leon_cycles_offset takes the pending bit into
account and that leon_clear_clock_irq clears the pending bit. Otherwise,
if leon_cycles_offset is executed after the timer has wrapped but before
timer_interrupt has increased timer_cs_internal_counter, time can be
perceived to go backwards.

Signed-off-by: Andreas Larsson 
Signed-off-by: David S. Miller

sparc32: fix sparse warnings in time_32.c

2014-05-19T02:01:27+00:00

Fix following warnings:
time_32.c:63:1: warning: symbol 'rtc_lock' was not declared. Should it be static?
time_32.c:357:13: warning: symbol 'time_init' was not declared. Should it be static?
time_32.c:148:16: warning: dereference of noderef expression

Add extern definition of rtc_lock in mc146818rtc.h.
time_init() is called from init/main.c - add prototype to kernel.h.
Use proper u32 __iomem * for master_l10_counter.
Fix all users.

Signed-off-by: Sam Ravnborg 
Signed-off-by: David S. Miller

sparc32: fix sparse warnings in leon_kernel.c

2014-04-29T05:12:27+00:00

Fix following warnings:
leon_kernel.c:40:15: warning: symbol 'leon3_gptimer_idx' was not declared. Should it be static?
leon_kernel.c:68:6: warning: symbol 'leon_eirq_setup' was not declared. Should it be static?
leon_kernel.c:273:13: warning: symbol 'leon_percpu_timer_ce_interrupt' was not declared. Should it be static?

Define symbols as static.

Signed-off-by: Sam Ravnborg 
Cc: Daniel Hellstrom 
Signed-off-by: David S. Miller

sparc32: fix sparse warnings in sun4m_irq.c and sun4d_irq.c

2014-04-29T05:12:25+00:00

Fix following warnings:

sun4m_irq.c:308:6: warning: symbol 'sun4m_nmi' was not declared. Should it be static?
sun4m_irq.c:396:28: warning: incorrect type in assignment (different address spaces)
sun4m_irq.c:396:28:    expected unsigned int volatile *extern [addressable] [toplevel] master_l10_counter
sun4d_irq.c:469:28: warning: incorrect type in assignment (different address spaces)
sun4d_irq.c:469:28:    expected unsigned int volatile *extern [addressable] [toplevel] master_l10_counter

master_l10_counter is a pointer to __iomem - add annotations.

Signed-off-by: Sam Ravnborg 
Signed-off-by: David S. Miller

sparc32, leon: Remove separate "ticker" timer for SMP

2013-06-19T09:10:29+00:00

This reduces the need from two timers to one timer.

Moreover, without this patch, when the "ticker" timer triggers timer_cs_read via
tick_periodic it reads the value of the usual timer it can get an wrapped timer
value without timer_cs_internal_counter having been updated leading to the clock
going backwards. This effectively hangs one cpu that gets stuck in
update_wall_time with an offset slightly smaller than 0xffffffffffffffff.

Signed-off-by: Andreas Larsson 
Signed-off-by: David S. Miller

sparc32, leon: Do not overwrite previously set irq flow handlers

2013-05-05T00:38:18+00:00

This is needed because when scan_of_devices finds the GAISLER_GPTIMER
core that corresponds to the SMP "ticker" timer, the previously set
proper irq flow handler gets overwritten with an incorrect one. This
leads to very flaky timer interrupt handling on some hardware. Proper
updates to handlers can still be done using leon_update_virq_handling.

Signed-off-by: Andreas Larsson 
Signed-off-by: David S. Miller