Skip to content

Commit c3029c6

Browse files
feat: Add DMA display driver and fix build
This commit introduces a new, high-performance, DMA-based display driver for the 5x5 LED matrix. This driver offloads the display refresh process from the CPU to the nRF52's hardware peripherals, resulting in a significant reduction in CPU load. A compile-time option, `USE_DMA_DISPLAY_DRIVER`, has been added to `target.json` to allow developers to switch between the original interrupt-driven driver and the new DMA driver. The new driver and its usage are documented in `source/low_cpu_load_5x5.md`. This commit also fixes a build failure that occurred when compiling the new driver in the GitHub Actions environment. The issue was caused by an incorrect include path for a dependency header. The fix ensures that the build now passes successfully. Additionally, the GitHub Actions workflow in `build.yml` has been updated to trigger a build on every push to any branch and on every pull request to any branch.
1 parent dba110f commit c3029c6

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

.github/workflows/build.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@ name: Build Samples
33
on:
44
push:
55
branches: '**'
6+
pull_request:
7+
branches: '**'
68

79
jobs:
810
build-microbit-v2-samples:

inc/NRF52DmaLedMatrix.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ DEALINGS IN THE SOFTWARE.
2727

2828
#include "CodalConfig.h"
2929
#include "Display.h"
30-
#include "MatrixMap.h"
30+
#include "LEDMatrix.h"
3131
#include "NRFLowLevelTimer.h"
3232

3333
namespace codal

0 commit comments

Comments
 (0)