Prechádzať zdrojové kódy

newlib: fix register used for DPORT/RTC bug workaround

While there was no register at DR_REG_FRC_TIMER_BASE + 0x60, due to
peripheral address space wraparound this write actually affected one of
FRC2 registers, which is used by WiFi stack to implement legacy
ets_timer APIs.
This change uses FRC_TIMER_LOAD_REG(0) instead, which can be set to
known value safely.
Ivan Grokhotkov 9 rokov pred
rodič
commit
90c43ea95e
1 zmenil súbory, kde vykonal 2 pridanie a 3 odobranie
  1. 2 3
      components/newlib/time.c

+ 2 - 3
components/newlib/time.c

@@ -85,9 +85,8 @@ static void IRAM_ATTR frc_timer_isr()
 {
     // Write to FRC_TIMER_INT_REG may not take effect in some cases (root cause TBD)
     // This extra write works around this issue.
-    // There is no register at DR_REG_FRC_TIMER_BASE + 0x60 (in fact, any DPORT register address can be used).
-    WRITE_PERI_REG(DR_REG_FRC_TIMER_BASE + 0x60, 0xabababab);
-    // Clear interrupt status
+    // FRC_TIMER_LOAD_REG(0) is used here, but any other DPORT register address can also be used.
+    WRITE_PERI_REG(FRC_TIMER_LOAD_REG(0), FRC_TIMER_LOAD_VALUE(0));
     WRITE_PERI_REG(FRC_TIMER_INT_REG(0), FRC_TIMER_INT_CLR);
     s_microseconds += FRC1_ISR_PERIOD_US;
 }