pru - calm cppcheck

orgua · Oct 1, 2024 · d784c6d · d784c6d
1 parent b67e863
commit d784c6d
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 5 deletions.
diff --git a/software/firmware/pru0-shepherd-fw/Makefile b/software/firmware/pru0-shepherd-fw/Makefile
@@ -25,7 +25,10 @@ ASM_SOURCES = $(notdir $(wildcard ./*.asm))
 # ... currently empty
 
 ifeq ($(TYPE), HRV)  # switch to HRV with "make TYPE=HRV"
-CFLAGS += -D HRV_SUPPORT
+CFLAGS += -DHRV_SUPPORT
+endif
+ifeq ($(TYPE), EMU)
+CFLAGS += -DEMU_SUPPORT
 endif
 
 # look for gcc, but use cgt as fallback / default

diff --git a/software/firmware/pru0-shepherd-fw/include/sampling.h b/software/firmware/pru0-shepherd-fw/include/sampling.h
@@ -3,9 +3,17 @@
 
 #include "commons.h"
 
-void     sample_init(const volatile struct SharedMem *shared_mem);
-void     sample(volatile struct SharedMem *shared_mem, struct SampleBuffer *current_buffer_far,
-                enum ShepherdMode mode);
+void sample_init(const volatile struct SharedMem *shared_mem);
+
+#ifdef EMU_SUPPORT
+void sample(volatile struct SharedMem *const shared_mem,
+            struct SampleBuffer *const current_buffer_far, const enum ShepherdMode mode);
+#else
+void sample(const volatile struct SharedMem *const shared_mem,
+            struct SampleBuffer *const current_buffer_far, const enum ShepherdMode mode);
+#endif
+
+
 uint32_t sample_dbg_adc(uint32_t channel_num);
 void     sample_dbg_dac(uint32_t value);
 

diff --git a/software/firmware/pru0-shepherd-fw/sampling.c b/software/firmware/pru0-shepherd-fw/sampling.c
@@ -102,7 +102,7 @@ static inline void sample_hrv_ADCs(struct SampleBuffer *const buffer, const uint
 void sample(volatile struct SharedMem *const shared_mem,
             struct SampleBuffer *const current_buffer_far, const enum ShepherdMode mode)
 #else
-void sample(volatile const struct SharedMem *const shared_mem,
+void sample(const volatile struct SharedMem *const shared_mem,
             struct SampleBuffer *const current_buffer_far, const enum ShepherdMode mode)
 #endif
 {