Extract updateUsbBuffer() from UsbCommand_ApplyConfig()

This commit is contained in:
László Monda
2017-11-05 22:56:51 +01:00
parent df0b0c33b5
commit 1c9e350006
2 changed files with 10 additions and 9 deletions
@@ -6,6 +6,13 @@
#include "usb_protocol_handler.h" #include "usb_protocol_handler.h"
#include "keymap.h" #include "keymap.h"
void updateUsbBuffer(uint8_t usbStatusCode, uint16_t parserOffset, parser_stage_t parserStage)
{
SET_USB_BUFFER_UINT8(0, usbStatusCode);
SET_USB_BUFFER_UINT16(1, parserOffset);
SET_USB_BUFFER_UINT8(3, parserStage);
}
void UsbCommand_ApplyConfig(void) void UsbCommand_ApplyConfig(void)
{ {
// Validate the staging configuration. // Validate the staging configuration.
@@ -13,10 +20,7 @@ void UsbCommand_ApplyConfig(void)
ParserRunDry = true; ParserRunDry = true;
StagingUserConfigBuffer.offset = 0; StagingUserConfigBuffer.offset = 0;
uint8_t parseConfigStatus = ParseConfig(&StagingUserConfigBuffer); uint8_t parseConfigStatus = ParseConfig(&StagingUserConfigBuffer);
updateUsbBuffer(parseConfigStatus, StagingUserConfigBuffer.offset, ParsingStage_Validate);
SET_USB_BUFFER_UINT8(0, parseConfigStatus);
SET_USB_BUFFER_UINT16(1, StagingUserConfigBuffer.offset);
SET_USB_BUFFER_UINT8(3, ParsingStage_Validate);
if (parseConfigStatus != UsbStatusCode_Success) { if (parseConfigStatus != UsbStatusCode_Success) {
return; return;
@@ -36,10 +40,7 @@ void UsbCommand_ApplyConfig(void)
ParserRunDry = false; ParserRunDry = false;
ValidatedUserConfigBuffer.offset = 0; ValidatedUserConfigBuffer.offset = 0;
parseConfigStatus = ParseConfig(&ValidatedUserConfigBuffer); parseConfigStatus = ParseConfig(&ValidatedUserConfigBuffer);
updateUsbBuffer(parseConfigStatus, ValidatedUserConfigBuffer.offset, ParsingStage_Apply);
SET_USB_BUFFER_UINT8(0, parseConfigStatus);
SET_USB_BUFFER_UINT16(1, ValidatedUserConfigBuffer.offset);
SET_USB_BUFFER_UINT8(3, ParsingStage_Apply);
if (parseConfigStatus != UsbStatusCode_Success) { if (parseConfigStatus != UsbStatusCode_Success) {
return; return;
@@ -6,7 +6,7 @@
typedef enum { typedef enum {
ParsingStage_Validate, ParsingStage_Validate,
ParsingStage_Apply, ParsingStage_Apply,
} parsing_stage_t; } parser_stage_t;
// Functions: // Functions: