implemented hal_read_led(), removed debug symbols bloating the library

4 files changed, 21 insertions, 5 deletions

diff --git a/Makefile b/Makefile
index 4436af4..cfb7766 100644
--- a/Makefile
+++ b/Makefile
@@ -24,15 +24,15 @@ CC = arm-none-eabi-gcc
 LD = arm-none-eabi-gcc
 OBJCOPY = arm-none-eabi-objcopy
 
-CFLAGS  = -Os -g -Wall -I.\
+CFLAGS  = -Os -Wall -I.\
 -D_STM32F103RBT6_  -D_STM3x_  -D_STM32x_ -mthumb -mcpu=cortex-m3 \
 -fsigned-char  -DSTM32F10X_MD -DUSE_STDPERIPH_DRIVER -DHSE_VALUE=6000000UL \
--DCMSIS -DUSE_GLOBAL_CONFIG   -g3 -ffunction-sections -std=c99  -mlittle-endian \
+-DCMSIS -DUSE_GLOBAL_CONFIG -ffunction-sections -std=c99  -mlittle-endian \
 $(INCLUDES) -o
 
 LDSCRIPT = stm32_flash.ld
 
-LDFLAGS += -T$(LDSCRIPT) -u _start -u _Minimum_Stack_Size  -mcpu=cortex-m3 -mthumb -specs=nano.specs -specs=nosys.specs -nostdlib -Wl,-static  -g3 -N -nostartfiles -Wl,--gc-sections
+LDFLAGS += -T$(LDSCRIPT) -u _start -u _Minimum_Stack_Size  -mcpu=cortex-m3 -mthumb -specs=nano.specs -specs=nosys.specs -nostdlib -Wl,-static -N -nostartfiles -Wl,--gc-sections
 
 all: $(SYX)
 
diff --git a/include/app.h b/include/app.h
index 96801eb..b06f7a6 100644
--- a/include/app.h
+++ b/include/app.h
@@ -71,6 +71,19 @@
 void hal_plot_led(u8 type, u8 index, u8 red, u8 green, u8 blue);
 
 /**
+ * Read the RGB value of an LED.  This function is safe to call from any
+ * of the app functions below, at any time. Result is undefined if an invalid address is passed
+ * for any of the red, green or blue components.
+ *
+ * @param type - TYPEPAD to address any pad or button, TYPESETUP to address the Setup LED
+ * @param index - The index of the button, as above
+ * @param red - address to read red colour value, in [0, MAXLED].
+ * @param green - address to read green colour value, in [0, MAXLED]
+ * @param blue - address to read blue colour value, in [0, MAXLED]
+ */
+void hal_read_led(u8 type, u8 index, u8 *red, u8 *green, u8 *blue);
+
+/**
  * Send a MIDI message to either USB port or to the DIN output.
  * 
  * @param port - which port to send the message to - can be USBSTANDALONE, USBMIDI or DINMIDI.
diff --git a/lib/launchpad_pro.a b/lib/launchpad_pro.a
index 383c9c6..70b470c 100644
--- a/lib/launchpad_pro.a
+++ b/lib/launchpad_pro.a
diff --git a/tools/simulator.c b/tools/simulator.c
index ab4eee1..cec1ede 100644
--- a/tools/simulator.c
+++ b/tools/simulator.c
@@ -41,8 +41,11 @@
 
 void hal_plot_led(u8 type, u8 index, u8 red, u8 green, u8 blue)
 {
-        // wire this up to MIDI out...
-        printf("...hal_plot_led(%d, %d, %d, %d, %d);\n", type, index, red, green, blue);
+    // wire this up to MIDI out...?
+}
+
+void hal_read_led(u8 type, u8 index, u8 *red, u8 *green, u8 *blue)
+{
 }
 
 void hal_send_midi(u8 port, u8 status, u8 d1, u8 d2)