MIPS: Lemote 2F: cs5536: mfgpt: use raw locks
The lock is taken in the raw irq path and therefore a rawlock should be used instead of a normal spinlock. While here I drop the export symbol on that variable since there are no other users. Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de> Cc: linux-mips@linux-mips.org Cc: Hua Yan <yanh@lemote.com> Cc: Huacai Chen <chenhc@lemote.com> Cc: Alex Smith <alex.smith@imgtec.com> Cc: Hongliang Tao <taohl@lemote.com> Cc: Wu Zhangjin <wuzhangjin@gmail.com> Patchwork: https://patchwork.linux-mips.org/patch/6936/ Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
This commit is contained in:

committed by
Ralf Baechle

parent
bb6c0bd3fd
commit
f02ffb199d
@@ -27,8 +27,7 @@
|
|||||||
|
|
||||||
#include <cs5536/cs5536_mfgpt.h>
|
#include <cs5536/cs5536_mfgpt.h>
|
||||||
|
|
||||||
DEFINE_SPINLOCK(mfgpt_lock);
|
static DEFINE_RAW_SPINLOCK(mfgpt_lock);
|
||||||
EXPORT_SYMBOL(mfgpt_lock);
|
|
||||||
|
|
||||||
static u32 mfgpt_base;
|
static u32 mfgpt_base;
|
||||||
|
|
||||||
@@ -55,7 +54,7 @@ EXPORT_SYMBOL(enable_mfgpt0_counter);
|
|||||||
static void init_mfgpt_timer(enum clock_event_mode mode,
|
static void init_mfgpt_timer(enum clock_event_mode mode,
|
||||||
struct clock_event_device *evt)
|
struct clock_event_device *evt)
|
||||||
{
|
{
|
||||||
spin_lock(&mfgpt_lock);
|
raw_spin_lock(&mfgpt_lock);
|
||||||
|
|
||||||
switch (mode) {
|
switch (mode) {
|
||||||
case CLOCK_EVT_MODE_PERIODIC:
|
case CLOCK_EVT_MODE_PERIODIC:
|
||||||
@@ -79,7 +78,7 @@ static void init_mfgpt_timer(enum clock_event_mode mode,
|
|||||||
/* Nothing to do here */
|
/* Nothing to do here */
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
spin_unlock(&mfgpt_lock);
|
raw_spin_unlock(&mfgpt_lock);
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct clock_event_device mfgpt_clockevent = {
|
static struct clock_event_device mfgpt_clockevent = {
|
||||||
@@ -157,7 +156,7 @@ static cycle_t mfgpt_read(struct clocksource *cs)
|
|||||||
static int old_count;
|
static int old_count;
|
||||||
static u32 old_jifs;
|
static u32 old_jifs;
|
||||||
|
|
||||||
spin_lock_irqsave(&mfgpt_lock, flags);
|
raw_spin_lock_irqsave(&mfgpt_lock, flags);
|
||||||
/*
|
/*
|
||||||
* Although our caller may have the read side of xtime_lock,
|
* Although our caller may have the read side of xtime_lock,
|
||||||
* this is now a seqlock, and we are cheating in this routine
|
* this is now a seqlock, and we are cheating in this routine
|
||||||
@@ -191,7 +190,7 @@ static cycle_t mfgpt_read(struct clocksource *cs)
|
|||||||
old_count = count;
|
old_count = count;
|
||||||
old_jifs = jifs;
|
old_jifs = jifs;
|
||||||
|
|
||||||
spin_unlock_irqrestore(&mfgpt_lock, flags);
|
raw_spin_unlock_irqrestore(&mfgpt_lock, flags);
|
||||||
|
|
||||||
return (cycle_t) (jifs * COMPARE) + count;
|
return (cycle_t) (jifs * COMPARE) + count;
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user