Merge pull request #15 from Tectu/master

Merge Tectu Changes

drivers/tdisp/HD44780/tdisp_lld.c

@@ -115,6 +115,39 @@ void tdisp_lld_home(void) {
 	tdisp_lld_write_cmd(0x02);
 }
 
+void tdisp_lld_control(uint16_t what, void *value) {
+	(void)what;
+	(void)value;
+/*
+	switch(attributes) {
+		case TDISP_ON:
+			_displaycontrol |= 0x04;
+			tdisp_lld_write_cmd(0x08 | _displaycontrol);
+			break;
+		case TDISP_OFF:
+			_displaycontrol &=~ 0x04;
+			tdisp_lld_write_cmd(0x08 | _displaycontrol);
+			break;
+		case TDISP_CURSOR_ON:
+			_displaycontrol |= 0x02;
+			tdisp_lld_write_cmd(0x08 | _displaycontrol);
+			break;
+		case TDISP_CURSOR_OFF:
+			_displaycontrol &=~ 0x02;
+			tdisp_lld_write_cmd(0x08 | _displaycontrol);
+			break;
+		case TDISP_CURSOR_BLINK_ON:
+			_displaycontrol |= 0x00;
+			tdisp_lld_write_cmd(0x08 | _displaycontrol);
+			break;
+		case TDISP_CURSOR_BLINK_OFF:
+			_displaycontrol &=~ 0x00;
+			tdisp_lld_write_cmd(0x08 | _displaycontrol);
+			break;
+	}
+*/
+}
+
 #endif /* GFX_USE_TDISP */
 /** @} */
 

include/tdisp/lld/tdisp_lld.h

@@ -44,6 +44,7 @@ extern void 	tdisp_lld_set_cursor(coord_t col, coord_t row);
 extern void		tdisp_lld_create_char(uint8_t address, char *charmap);
 extern void		tdisp_lld_clear(void);
 extern void		tdisp_lld_home(void);
+extern void		tdisp_lld_control(uint16_t what, void *value);
 
 #ifdef __cplusplus
 }

include/tdisp/tdisp.h

@@ -67,12 +67,13 @@ bool_t tdispInit(void);
 
 /**
  * @brief	Control different display properties
- * @note	Multiple attributes can be passed using the OR operator.
- * @note	Example: tdispSetAttributes(TDISP_DISPLAY_ON | TDISP_CURSOR_BLINK)
+ * @note	A wrapper macro exists for each option, please use them
+ *			instead of this function manually.
  *
- * @param[in] attributes	The attributes
+ * @param[in] what		What you want to control
+ * @param[in] value		The value to be assigned
  */
-void tdispSetAttributes(uint8_t attributes);
+void tdispControl(uint16_t what, void *value);
 
 /**
  * @brief	Clears the display

src/tdisp/tdisp.c

@@ -41,33 +41,8 @@ bool_t tdispInit(void) {
 	return ret;
 }
 
-void tdispSetAttributes(uint8_t attributes) {
-	switch(attributes) {
-		case TDISP_ON:
-			_displaycontrol |= 0x04;
-			tdisp_lld_write_cmd(0x08 | _displaycontrol);
-			break;
-		case TDISP_OFF:
-			_displaycontrol &=~ 0x04;
-			tdisp_lld_write_cmd(0x08 | _displaycontrol);
-			break;
-		case TDISP_CURSOR_ON:
-			_displaycontrol |= 0x02;
-			tdisp_lld_write_cmd(0x08 | _displaycontrol);
-			break;
-		case TDISP_CURSOR_OFF:
-			_displaycontrol &=~ 0x02;
-			tdisp_lld_write_cmd(0x08 | _displaycontrol);
-			break;
-		case TDISP_CURSOR_BLINK_ON:
-			_displaycontrol |= 0x00;
-			tdisp_lld_write_cmd(0x08 | _displaycontrol);
-			break;
-		case TDISP_CURSOR_BLINK_OFF:
-			_displaycontrol &=~ 0x00;
-			tdisp_lld_write_cmd(0x08 | _displaycontrol);
-			break;
-	}
+void tdispControl(uint16_t what, void *value) {
+	tdisp_lld_control(what, value);
 }
 
 void tdispClear(void) {