Added new HOST_STATE_WaitForDeviceRemoval host state machine state for non-blocking disabling of device communications until the device has been removed (for use when an error occurs or communications with the device have completed). Changed over all host mode demos to use the new state.

Added verbose documentation for each of the USB Host state machine states.

8 files changed, 52 insertions, 29 deletions

diff --git a/Demos/Host/LowLevel/CDCHost/CDCHost.c b/Demos/Host/LowLevel/CDCHost/CDCHost.c
index b3e067418..de7ab88c6 100644
--- a/Demos/Host/LowLevel/CDCHost/CDCHost.c
+++ b/Demos/Host/LowLevel/CDCHost/CDCHost.c
@@ -148,7 +148,7 @@ void CDC_Host_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 			

@@ -162,7 +162,7 @@ void CDC_Host_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 				

diff --git a/Demos/Host/LowLevel/GenericHIDHost/GenericHIDHost.c b/Demos/Host/LowLevel/GenericHIDHost/GenericHIDHost.c
index 9f7da5998..a1b62ac62 100644
--- a/Demos/Host/LowLevel/GenericHIDHost/GenericHIDHost.c
+++ b/Demos/Host/LowLevel/GenericHIDHost/GenericHIDHost.c
@@ -249,7 +249,7 @@ void HID_Host_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 

@@ -263,7 +263,7 @@ void HID_Host_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 				

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 			

diff --git a/Demos/Host/LowLevel/KeyboardHost/KeyboardHost.c b/Demos/Host/LowLevel/KeyboardHost/KeyboardHost.c
index 734ef2a57..2517848b0 100644
--- a/Demos/Host/LowLevel/KeyboardHost/KeyboardHost.c
+++ b/Demos/Host/LowLevel/KeyboardHost/KeyboardHost.c
@@ -213,7 +213,7 @@ void Keyboard_HID_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 		

@@ -227,7 +227,7 @@ void Keyboard_HID_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 				

@@ -257,7 +257,7 @@ void Keyboard_HID_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 				

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 

diff --git a/Demos/Host/LowLevel/KeyboardHostWithParser/KeyboardHostWithParser.c b/Demos/Host/LowLevel/KeyboardHostWithParser/KeyboardHostWithParser.c
index 4e78b0de9..2ef15e7b8 100644
--- a/Demos/Host/LowLevel/KeyboardHostWithParser/KeyboardHostWithParser.c
+++ b/Demos/Host/LowLevel/KeyboardHostWithParser/KeyboardHostWithParser.c
@@ -148,7 +148,7 @@ void Keyboard_HID_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 

@@ -163,7 +163,7 @@ void Keyboard_HID_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 				

@@ -182,7 +182,7 @@ void Keyboard_HID_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 				

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;	

 			}

 

diff --git a/Demos/Host/LowLevel/MassStorageHost/MassStorageHost.c b/Demos/Host/LowLevel/MassStorageHost/MassStorageHost.c
index 2f84d0fb8..1bb94f6c8 100644
--- a/Demos/Host/LowLevel/MassStorageHost/MassStorageHost.c
+++ b/Demos/Host/LowLevel/MassStorageHost/MassStorageHost.c
@@ -154,7 +154,7 @@ void MassStorage_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 

@@ -168,7 +168,7 @@ void MassStorage_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 				

@@ -187,6 +187,8 @@ void MassStorage_Task(void)
 			if ((ErrorCode = MassStore_GetMaxLUN(&MassStore_MaxLUNIndex)) != HOST_SENDCONTROL_Successful)

 			{	

 				ShowDiskReadError(PSTR("Get Max LUN"), false, ErrorCode);

+

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 			

@@ -197,6 +199,8 @@ void MassStorage_Task(void)
 			if ((ErrorCode = MassStore_MassStorageReset()) != HOST_SENDCONTROL_Successful)

 			{

 				ShowDiskReadError(PSTR("Mass Storage Reset"), false, ErrorCode);

+				

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 			

@@ -213,6 +217,8 @@ void MassStorage_Task(void)
 			if (((ErrorCode = MassStore_PreventAllowMediumRemoval(0, true)) != 0) || (SCSICommandStatus.Status != Command_Pass))

 			{

 				ShowDiskReadError(PSTR("Prevent/Allow Medium Removal"), (SCSICommandStatus.Status != Command_Pass), ErrorCode);

+				

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 

@@ -221,6 +227,8 @@ void MassStorage_Task(void)
 			if (((ErrorCode = MassStore_Inquiry(0, &InquiryData)) != 0) || (SCSICommandStatus.Status != Command_Pass))

 			{

 				ShowDiskReadError(PSTR("Inquiry"), (SCSICommandStatus.Status != Command_Pass), ErrorCode);

+				

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 

@@ -237,7 +245,9 @@ void MassStorage_Task(void)
 				if ((ErrorCode = MassStore_TestUnitReady(0)) != 0)

 				{

 					ShowDiskReadError(PSTR("Test Unit Ready"), false, ErrorCode);

-					break;				

+

+					USB_HostState = HOST_STATE_WaitForDeviceRemoval;

+					break;

 				}

 			}

 			while ((SCSICommandStatus.Status != Command_Pass) && USB_IsConnected);

@@ -255,6 +265,8 @@ void MassStorage_Task(void)
 			if (((ErrorCode = MassStore_ReadCapacity(0, &DiskCapacity)) != 0) || (SCSICommandStatus.Status != Command_Pass))

 			{

 				ShowDiskReadError(PSTR("Read Capacity"), (SCSICommandStatus.Status != Command_Pass), ErrorCode);

+				

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 			

@@ -269,6 +281,8 @@ void MassStorage_Task(void)
 			    (SCSICommandStatus.Status != Command_Pass))

 			{

 				ShowDiskReadError(PSTR("Read Device Block"), (SCSICommandStatus.Status != Command_Pass), ErrorCode);

+				

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 			

@@ -319,6 +333,8 @@ void MassStorage_Task(void)
 				    (SCSICommandStatus.Status != Command_Pass))

 				{

 					ShowDiskReadError(PSTR("Read Device Block"), (SCSICommandStatus.Status != Command_Pass), ErrorCode);

+					

+					USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 					break;

 				}

 

@@ -339,8 +355,7 @@ void MassStorage_Task(void)
 			LEDs_SetAllLEDs(LEDMASK_USB_READY);

 			

 			/* Wait until USB device disconnected */

-			while (USB_IsConnected);

-			

+			USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 			break;

 	}

 }

@@ -372,7 +387,4 @@ void ShowDiskReadError(char* CommandString, bool FailedAtSCSILayer, uint8_t Erro
 

 	/* Indicate device error via the status LEDs */

 	LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

-

-	/* Wait until USB device disconnected */

-	while (USB_IsConnected);

 }

diff --git a/Demos/Host/LowLevel/MouseHost/MouseHost.c b/Demos/Host/LowLevel/MouseHost/MouseHost.c
index 2a2e1b994..d5c4bf941 100644
--- a/Demos/Host/LowLevel/MouseHost/MouseHost.c
+++ b/Demos/Host/LowLevel/MouseHost/MouseHost.c
@@ -209,7 +209,7 @@ void Mouse_HID_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 

@@ -223,7 +223,7 @@ void Mouse_HID_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 				

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 			

@@ -253,7 +253,7 @@ void Mouse_HID_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 				

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 

diff --git a/Demos/Host/LowLevel/MouseHostWithParser/MouseHostWithParser.c b/Demos/Host/LowLevel/MouseHostWithParser/MouseHostWithParser.c
index d7a947105..1e931232c 100644
--- a/Demos/Host/LowLevel/MouseHostWithParser/MouseHostWithParser.c
+++ b/Demos/Host/LowLevel/MouseHostWithParser/MouseHostWithParser.c
@@ -149,7 +149,7 @@ void Mouse_HID_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 		

@@ -163,7 +163,7 @@ void Mouse_HID_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 				

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 			

@@ -182,7 +182,7 @@ void Mouse_HID_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 				

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;			

 			}

 

diff --git a/Demos/Host/LowLevel/StillImageHost/StillImageHost.c b/Demos/Host/LowLevel/StillImageHost/StillImageHost.c
index 524387123..0f511ad7a 100644
--- a/Demos/Host/LowLevel/StillImageHost/StillImageHost.c
+++ b/Demos/Host/LowLevel/StillImageHost/StillImageHost.c
@@ -149,7 +149,7 @@ void StillImage_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 

@@ -162,7 +162,7 @@ void StillImage_Task(void)
 				LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

 

 				/* Wait until USB device disconnected */

-				while (USB_IsConnected);

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 				

@@ -195,6 +195,8 @@ void StillImage_Task(void)
 			if ((ErrorCode = SImage_RecieveBlockHeader()) != PIPE_RWSTREAM_NoError)

 			{

 				ShowCommandError(ErrorCode, false);

+				

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 			

@@ -246,6 +248,8 @@ void StillImage_Task(void)
 			if ((ErrorCode = SImage_RecieveBlockHeader()) != PIPE_RWSTREAM_NoError)

 			{

 				ShowCommandError(ErrorCode, false);

+				

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 			

@@ -253,6 +257,8 @@ void StillImage_Task(void)
 			if ((PIMA_ReceivedBlock.Type != CType_ResponseBlock) || (PIMA_ReceivedBlock.Code != PIMA_RESPONSE_OK))

 			{

 				ShowCommandError(PIMA_ReceivedBlock.Code, true);

+				

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 			

@@ -274,6 +280,8 @@ void StillImage_Task(void)
 			if ((ErrorCode = SImage_RecieveBlockHeader()) != PIPE_RWSTREAM_NoError)

 			{

 				ShowCommandError(ErrorCode, false);

+				

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 			

@@ -281,6 +289,8 @@ void StillImage_Task(void)
 			if ((PIMA_ReceivedBlock.Type != CType_ResponseBlock) || (PIMA_ReceivedBlock.Code != PIMA_RESPONSE_OK))

 			{

 				ShowCommandError(PIMA_ReceivedBlock.Code, true);

+				

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 

@@ -302,6 +312,8 @@ void StillImage_Task(void)
 			if ((ErrorCode = SImage_RecieveBlockHeader()) != PIPE_RWSTREAM_NoError)

 			{

 				ShowCommandError(ErrorCode, false);

+				

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 

@@ -309,6 +321,8 @@ void StillImage_Task(void)
 			if ((PIMA_ReceivedBlock.Type != CType_ResponseBlock) || (PIMA_ReceivedBlock.Code != PIMA_RESPONSE_OK))

 			{

 				ShowCommandError(PIMA_ReceivedBlock.Code, true);

+				

+				USB_HostState = HOST_STATE_WaitForDeviceRemoval;

 				break;

 			}

 

@@ -363,7 +377,4 @@ void ShowCommandError(uint8_t ErrorCode, bool ResponseCodeError)
 			

 	/* Indicate error via status LEDs */

 	LEDs_SetAllLEDs(LEDMASK_USB_ERROR);

-				

-	/* Wait until USB device disconnected */

-	while (USB_IsConnected);

 }