Split out LED report processing from the host into a seperate routine in the LowLevel KeyboardMouse device demo, to avoid duplicate code.

Demos/Device/LowLevel/KeyboardMouse/KeyboardMouse.c

@@ -180,21 +180,8 @@ void EVENT_USB_Device_UnhandledControlRequest(void)
 					  return;
 				}
 
-				/* Read in the LED report from the host */
+				/* Read in and process the LED report from the host */
-				uint8_t LEDStatus = Endpoint_Read_Byte();
+				Keyboard_ProcessLEDReport(Endpoint_Read_Byte());
-				uint8_t LEDMask   = LEDS_LED2;
-				
-				if (LEDStatus & KEYBOARD_LED_NUMLOCK)
-				  LEDMask |= LEDS_LED1;
-				
-				if (LEDStatus & KEYBOARD_LED_CAPSLOCK)
-				  LEDMask |= LEDS_LED3;
-
-				if (LEDStatus & KEYBOARD_LED_SCROLLLOCK)
-				  LEDMask |= LEDS_LED4;
-
-				/* Set the status LEDs to the current HID LED status */
-				LEDs_SetAllLEDs(LEDMask);
 
 				/* Clear the endpoint data */
 				Endpoint_ClearOUT();
@@ -206,6 +193,28 @@ void EVENT_USB_Device_UnhandledControlRequest(void)
 	}
 }
 
+/** Processes a given Keyboard LED report from the host, and sets the board LEDs to match. Since the Keyboard
+ *  LED report can be sent through either the control endpoint (via a HID SetReport request) or the HID OUT
+ *  endpoint, the processing code is placed here to avoid duplicating it and potentially having different
+ *  behaviour depending on the method used to sent it.
+ */
+void Keyboard_ProcessLEDReport(const uint8_t LEDStatus)
+{
+	uint8_t LEDMask = LEDS_LED2;
+	
+	if (LEDStatus & KEYBOARD_LED_NUMLOCK)
+	  LEDMask |= LEDS_LED1;
+	
+	if (LEDStatus & KEYBOARD_LED_CAPSLOCK)
+	  LEDMask |= LEDS_LED3;
+
+	if (LEDStatus & KEYBOARD_LED_SCROLLLOCK)
+	  LEDMask |= LEDS_LED4;
+
+	/* Set the status LEDs to the current Keyboard LED status */
+	LEDs_SetAllLEDs(LEDMask);
+}
+
 /** Keyboard task. This generates the next keyboard HID report for the host, and transmits it via the
  *  keyboard IN endpoint when the host is ready for more data. Additionally, it processes host LED status
  *  reports sent to the device via the keyboard OUT reporting endpoint.
@@ -260,21 +269,8 @@ void Keyboard_HID_Task(void)
 	/* Check if Keyboard LED Endpoint Ready for Read/Write */
 	if (Endpoint_IsReadWriteAllowed())
 	{		
-		/* Read in the LED report from the host */
+		/* Read in and process the LED report from the host */
-		uint8_t LEDStatus = Endpoint_Read_Byte();
+		Keyboard_ProcessLEDReport(Endpoint_Read_Byte());
-		uint8_t LEDMask   = LEDS_LED2;
-		
-		if (LEDStatus & KEYBOARD_LED_NUMLOCK)
-		  LEDMask |= LEDS_LED1;
-		
-		if (LEDStatus & KEYBOARD_LED_CAPSLOCK)
-		  LEDMask |= LEDS_LED3;
-
-		if (LEDStatus & KEYBOARD_LED_SCROLLLOCK)
-		  LEDMask |= LEDS_LED4;
-
-		/* Set the status LEDs to the current Keyboard LED status */
-		LEDs_SetAllLEDs(LEDMask);
 
 		/* Handshake the OUT Endpoint - clear endpoint and ready for next report */
 		Endpoint_ClearOUT();

Demos/Device/LowLevel/KeyboardMouse/KeyboardMouse.h

@@ -131,6 +131,7 @@
 			
 	/* Function Prototypes: */
 		void SetupHardware(void);
+		void Keyboard_ProcessLEDReport(const uint8_t LEDStatus);
 		void Keyboard_HID_Task(void);
 		void Mouse_HID_Task(void);