Mirrors_Framework/nuttx
1
0
Fork 0
mirror of https://github.com/apache/nuttx.git synced 2026-06-02 01:21:26 +08:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

arch: cxd56xx: Use spinlock API in cxd56_gpioint.c

Browse Source 
Summary:
- This commit improves cxd56_gpioint performance in SMP mode.

Impact:
- This commit affects SMP mode only.

Testing:
- Tested with spresense:wifi_smp

Signed-off-by: Masayuki Ishikawa <Masayuki.Ishikawa@jp.sony.com>
This commit is contained in:
Masayuki Ishikawa
2020-09-11 16:36:02 +09:00
committed by Xiang Xiao
parent 58317b1430
commit 154d6bc556
1 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
arch/arm/src/cxd56xx/cxd56_gpioint.c
+9 -9
View File
@@ -128,7 +128,7 @@ static int alloc_slot(int pin, bool isalloc)
: CXD56_TOPREG_IOCAPP_INTSEL0; : CXD56_TOPREG_IOCAPP_INTSEL0;
int offset = (pin < PIN_IS_CLK) ? 1 : 56; int offset = (pin < PIN_IS_CLK) ? 1 : 56;
flags = enter_critical_section(); flags = spin_lock_irqsave();
for (slot = 0; slot < MAX_SYS_SLOT; slot++) for (slot = 0; slot < MAX_SYS_SLOT; slot++)
{ {
@@ -158,12 +158,12 @@ static int alloc_slot(int pin, bool isalloc)
} }
else else
{ {
leave_critical_section(flags); spin_unlock_irqrestore(flags);
return -ENXIO; /* no space */ return -ENXIO; /* no space */
} }
} }
leave_critical_section(flags); spin_unlock_irqrestore(flags);
if (PIN_IS_CLK <= pin) if (PIN_IS_CLK <= pin)
{ {
@@ -323,13 +323,13 @@ static void invert_irq(int irq)
irqstate_t flags; irqstate_t flags;
uint32_t val; uint32_t val;
flags = enter_critical_section(); flags = spin_lock_irqsave();
val = getreg32(CXD56_INTC_INVERT); val = getreg32(CXD56_INTC_INVERT);
val ^= (1 << (irq - CXD56_IRQ_EXTINT)); val ^= (1 << (irq - CXD56_IRQ_EXTINT));
putreg32(val, CXD56_INTC_INVERT); putreg32(val, CXD56_INTC_INVERT);
leave_critical_section(flags); spin_unlock_irqrestore(flags);
} }
static bool inverted_irq(int irq) static bool inverted_irq(int irq)
@@ -445,9 +445,9 @@ int cxd56_gpioint_config(uint32_t pin, uint32_t gpiocfg, xcpt_t isr,
irq_attach(irq, NULL, NULL); irq_attach(irq, NULL, NULL);
g_isr[slot] = NULL; g_isr[slot] = NULL;
flags = enter_critical_section(); flags = spin_lock_irqsave();
g_bothedge &= ~(1 << slot); g_bothedge &= ~(1 << slot);
leave_critical_section(flags); spin_unlock_irqrestore(flags);
return irq; return irq;
} }
@@ -461,9 +461,9 @@ int cxd56_gpioint_config(uint32_t pin, uint32_t gpiocfg, xcpt_t isr,
{ {
/* set GPIO pseudo both edge interrupt */ /* set GPIO pseudo both edge interrupt */
flags = enter_critical_section(); flags = spin_lock_irqsave();
g_bothedge |= (1 << slot); g_bothedge |= (1 << slot);
leave_critical_section(flags); spin_unlock_irqrestore(flags);
/* detect the change from the current signal */ /* detect the change from the current signal */