aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authormrfaptastic <12006953+mrfaptastic@users.noreply.github.com>2021-08-18 16:33:59 +0100
committermrfaptastic <12006953+mrfaptastic@users.noreply.github.com>2021-08-18 16:33:59 +0100
commite12eaf15b5598703ff76ded68d5b575e64a7476a (patch)
tree37928bb39e34e7d95b1d653b40c38fac3863c228
parent3b13645e7ff4427694a8daa448e66d77882d5f92 (diff)
Revert "Attempted clearScreen() syncronisation fix"
This reverts commit f6584dd321fcf5e54ffd4429ab6b06ab7adb2b34.
-rw-r--r--ESP32-HUB75-MatrixPanel-I2S-DMA.cpp6
-rw-r--r--ESP32-HUB75-MatrixPanel-I2S-DMA.h7
2 files changed, 5 insertions, 8 deletions
diff --git a/ESP32-HUB75-MatrixPanel-I2S-DMA.cpp b/ESP32-HUB75-MatrixPanel-I2S-DMA.cpp
index 0e3b058..d9d4ca9 100644
--- a/ESP32-HUB75-MatrixPanel-I2S-DMA.cpp
+++ b/ESP32-HUB75-MatrixPanel-I2S-DMA.cpp
@@ -532,9 +532,9 @@ void IRAM_ATTR MatrixPanel_I2S_DMA::updateMatrixDMABuffer(int16_t x_coord, int16
/* Update the entire buffer with a single specific colour - quicker */
-bool MatrixPanel_I2S_DMA::updateMatrixDMABuffer(uint8_t red, uint8_t green, uint8_t blue)
+void MatrixPanel_I2S_DMA::updateMatrixDMABuffer(uint8_t red, uint8_t green, uint8_t blue)
{
- if ( !initialized ) return false;
+ if ( !initialized ) return;
/* https://ledshield.wordpress.com/2012/11/13/led-brightness-to-your-eye-gamma-correction-no/ */
#ifndef NO_CIE1931
@@ -585,8 +585,6 @@ bool MatrixPanel_I2S_DMA::updateMatrixDMABuffer(uint8_t red, uint8_t green, uint
} while(matrix_frame_parallel_row); // end row iteration
} // colour depth loop (8)
-
- return true;
} // updateMatrixDMABuffer (full frame paint)
/**
diff --git a/ESP32-HUB75-MatrixPanel-I2S-DMA.h b/ESP32-HUB75-MatrixPanel-I2S-DMA.h
index 4750b0b..f8648a1 100644
--- a/ESP32-HUB75-MatrixPanel-I2S-DMA.h
+++ b/ESP32-HUB75-MatrixPanel-I2S-DMA.h
@@ -422,10 +422,9 @@ class MatrixPanel_I2S_DMA {
virtual void fillScreen(uint16_t color); // overwrite adafruit implementation
/**
- * A wrapper to fill whatever selected DMA buffer / screen with black.
- * Returns true when operation is completed for syncronisation reasons.
+ * A wrapper to fill whatever selected DMA buffer / screen with black
*/
- inline bool clearScreen(){ return updateMatrixDMABuffer(0,0,0); };
+ inline void clearScreen(){ updateMatrixDMABuffer(0,0,0); };
#ifndef NO_FAST_FUNCTIONS
/**
@@ -595,7 +594,7 @@ class MatrixPanel_I2S_DMA {
void updateMatrixDMABuffer(int16_t x, int16_t y, uint8_t red, uint8_t green, uint8_t blue);
/* Update the entire DMA buffer (aka. The RGB Panel) a certain colour (wipe the screen basically) */
- bool updateMatrixDMABuffer(uint8_t red, uint8_t green, uint8_t blue);
+ void updateMatrixDMABuffer(uint8_t red, uint8_t green, uint8_t blue);
/**
* wipes DMA buffer(s) and reset all color/service bits