NeoPixel clear() method wont clear pixels about adafruit_seesaw

[ladyada]

call show()

Adafruit_Seesaw/seesaw_neopixel.cpp

Line 102 in ddd6956

void seesaw_NeoPixel::show(void) {

[mic159]

@ladyada the show() function only sends the SEESAW_NEOPIXEL_SHOW command to the NeoTrellis/Seesaw, but the buffer on the NeoTrellis/Seesaw still has the original pixel data, so the LEDs stay lit up as they were.

Adafruit_Seesaw/seesaw_neopixel.cpp

Line 114 in ddd6956

this->writeEmpty(SEESAW_NEOPIXEL_BASE, SEESAW_NEOPIXEL_SHOW);

Simple reproduce case:

#include "Adafruit_NeoTrellis.h"

Adafruit_NeoTrellis pad;

void setup() {
  pad.begin();
}

void loop() {
  pad.pixels.setPixelColor(0, 255, 0, 0);
  pad.pixels.setPixelColor(15, 255, 0, 0);
  pad.pixels.show();
  delay(1000);
  pad.pixels.clear();
  pad.pixels.show();
  delay(1000);
}

This should blink the pixels, but it just stays solid.

[ladyada]

ooooooooooooooh - yeah thats a bug. sorry no fix at this time because the firmware is pre-burned :( you have to write to each pixel!

[ladyada]

wanna try adding a
this->write(SEESAW_NEOPIXEL_BASE, SEESAW_NEOPIXEL_BUF, writeBuf, len + 2 );
type call after the memset in clear()? that should then send that data buffer off, may need adjustment for variable names n such - i dont have a seesaw on hand right now to test

like... https://learn.adafruit.com/adafruit-seesaw-atsamd09-breakout/neopixel#buf-0x04-32-bytes-write-only-12-3

[mic159]

@ladyada yes, it looks like we can use SEESAW_NEOPIXEL_BUF a few times to clear the buffer on the remote end. Unfortunately the buffer is larger than the 32 byte maximum of TwoWire, so it has to be called in a loop.

Check #21