Skip to content

Commit

Permalink
cope with I$ miss on teensy
Browse files Browse the repository at this point in the history
if the instruction cache is completely flushed *even before* the counter
is read, under specific situations (presumably when relevant parts of
show() are on different cache lines), the first bit would not be sent
(but discarded), often leading to a change in colour.

prevent compiler optimisation across counter reads,
order the delays so that they are easier to reason about.
  • Loading branch information
Volker Eckert committed Oct 3, 2024
1 parent aa798ff commit 840121e
Showing 1 changed file with 27 additions and 5 deletions.
32 changes: 27 additions & 5 deletions Adafruit_NeoPixel.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,11 @@

#include "Adafruit_NeoPixel.h"

#undef REPRODUCE_TEENSY_TIMING
#ifdef REPRODUCE_TEENSY_TIMING
#include "imxrt.h"
#endif

#if defined(TARGET_LPC1768)
#include <time.h>
#endif
Expand Down Expand Up @@ -1912,16 +1917,32 @@ void Adafruit_NeoPixel::show(void) {
ARM_DEMCR |= ARM_DEMCR_TRCENA;
ARM_DWT_CTRL |= ARM_DWT_CTRL_CYCCNTENA;

#ifdef REPRODUCE_TEENSY_TIMING
// reproducer for the timing issue
{
static uint8_t ct;

//if (!(ct & 1)) {
if (true) {
asm("dsb");
asm("isb");
SCB_CACHE_ICIALLU = 0UL;
asm("dsb");
asm("isb");
}
ct++;
}
#endif

#if defined(NEO_KHZ400) // 800 KHz check needed only if 400 KHz support enabled
if (is800KHz) {
#endif
cyc = ARM_DWT_CYCCNT + CYCLES_800;
cyc = ARM_DWT_CYCCNT; // make sure it's initialised..
while (p < end) {
pix = *p++;
for (mask = 0x80; mask; mask >>= 1) {
while (ARM_DWT_CYCCNT - cyc < CYCLES_800)
;
cyc = ARM_DWT_CYCCNT;
barrier();
*set = msk;
if (pix & mask) {
while (ARM_DWT_CYCCNT - cyc < CYCLES_800_T1H)
Expand All @@ -1931,10 +1952,11 @@ void Adafruit_NeoPixel::show(void) {
;
}
*clr = msk;
barrier();
while (ARM_DWT_CYCCNT - cyc < CYCLES_800)
;
}
}
while (ARM_DWT_CYCCNT - cyc < CYCLES_800)
;
#if defined(NEO_KHZ400)
} else { // 400 kHz bitstream
cyc = ARM_DWT_CYCCNT + CYCLES_400;
Expand Down

0 comments on commit 840121e

Please sign in to comment.