Skip to content

Commit 5796849

Browse files
PiyushPatle26acassis
authored andcommitted
espressif/ws2812: include esp_rmt.h in ws2812 headers
esp_ws2812.h exposes esp_ws2812_setup() with a struct rmt_dev_s * argument but did not include esp_rmt.h directly. After guarding the LIRC board registration path with CONFIG_DRIVERS_RC, non-RC ESP configs no longer pulled in the RMT type indirectly and CI started failing with an incomplete struct rmt_dev_s declaration. Include esp_rmt.h in the Xtensa and RISC-V ws2812 headers. Signed-off-by: Piyush Patle <[email protected]>
1 parent 7037268 commit 5796849

2 files changed

Lines changed: 4 additions & 0 deletions

File tree

arch/risc-v/src/common/espressif/esp_ws2812.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,8 @@
3131
#include <debug.h>
3232
#include <stdbool.h>
3333

34+
#include "esp_rmt.h"
35+
3436
#ifndef __ASSEMBLY__
3537
#if defined(__cplusplus)
3638
#define EXTERN extern "C"

arch/xtensa/src/common/espressif/esp_ws2812.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,8 @@
3131
#include <debug.h>
3232
#include <stdbool.h>
3333

34+
#include "esp_rmt.h"
35+
3436
#ifndef __ASSEMBLY__
3537
#if defined(__cplusplus)
3638
#define EXTERN extern "C"

0 commit comments

Comments
 (0)