us_ticker_irq_handler call optimization

maciejbocianski · maciejbocianski · commit f81ed8e60d36 · 2018-02-23T09:04:11.000+01:00
diff --git a/targets/TARGET_NORDIC/TARGET_NRF5/us_ticker.c b/targets/TARGET_NORDIC/TARGET_NRF5/us_ticker.c
@@ -77,11 +77,7 @@ void COMMON_RTC_IRQ_HANDLER(void)
 
     rtc_ovf_event_check();
 
-    if (m_common_sw_irq_flag & US_TICKER_SW_IRQ_MASK) {
-        m_common_sw_irq_flag &= ~US_TICKER_SW_IRQ_MASK;
-        us_ticker_irq_handler();
-    }
-    if (nrf_rtc_event_pending(COMMON_RTC_INSTANCE, US_TICKER_EVENT)) {
+    if ((m_common_sw_irq_flag & US_TICKER_SW_IRQ_MASK) || nrf_rtc_event_pending(COMMON_RTC_INSTANCE, US_TICKER_EVENT)) {
         us_ticker_irq_handler();
     }
 
@@ -297,6 +293,7 @@ void us_ticker_disable_interrupt(void)
 
 void us_ticker_clear_interrupt(void)
 {
+    m_common_sw_irq_flag &= ~US_TICKER_SW_IRQ_MASK;
     nrf_rtc_event_clear(COMMON_RTC_INSTANCE, US_TICKER_EVENT);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -77,11 +77,7 @@ void COMMON_RTC_IRQ_HANDLER(void)`
`77`	`77`
`78`	`78`	`rtc_ovf_event_check();`
`79`	`79`
`80`		`- if (m_common_sw_irq_flag & US_TICKER_SW_IRQ_MASK) {`
`81`		`- m_common_sw_irq_flag &= ~US_TICKER_SW_IRQ_MASK;`
`82`		`- us_ticker_irq_handler();`
`83`		`- }`
`84`		`- if (nrf_rtc_event_pending(COMMON_RTC_INSTANCE, US_TICKER_EVENT)) {`
	`80`	`+ if ((m_common_sw_irq_flag & US_TICKER_SW_IRQ_MASK) \|\| nrf_rtc_event_pending(COMMON_RTC_INSTANCE, US_TICKER_EVENT)) {`
`85`	`81`	`us_ticker_irq_handler();`
`86`	`82`	`}`
`87`	`83`
`@@ -297,6 +293,7 @@ void us_ticker_disable_interrupt(void)`
`297`	`293`
`298`	`294`	`void us_ticker_clear_interrupt(void)`
`299`	`295`	`{`
	`296`	`+ m_common_sw_irq_flag &= ~US_TICKER_SW_IRQ_MASK;`
`300`	`297`	`nrf_rtc_event_clear(COMMON_RTC_INSTANCE, US_TICKER_EVENT);`
`301`	`298`	`}`
`302`	`299`