Added new Drivers/USB/LowLevel/Device.c file to house Device mode specific functions that are more complicated than simple macros. Moved USB_Device_SendRemoteWakeup() to the new Device.c source file and corrected it to unfreeze and restart the USB controller clock before issuing a Remote Wakeup request.

Removed the USB_Device_IsRemoteWakeupSent() and USB_Device_IsUSBSuspended() macros, as they are now obsolete.

3 files changed, 72 insertions, 50 deletions

diff --git a/LUFA/Drivers/USB/LowLevel/Device.c b/LUFA/Drivers/USB/LowLevel/Device.c
new file mode 100644
index 000000000..b21e6574d
--- /dev/null
+++ b/LUFA/Drivers/USB/LowLevel/Device.c
@@ -0,0 +1,52 @@
+/*

+             LUFA Library

+     Copyright (C) Dean Camera, 2010.

+              

+  dean [at] fourwalledcubicle [dot] com

+      www.fourwalledcubicle.com

+*/

+

+/*

+  Copyright 2010  Dean Camera (dean [at] fourwalledcubicle [dot] com)

+

+  Permission to use, copy, modify, distribute, and sell this 

+  software and its documentation for any purpose is hereby granted

+  without fee, provided that the above copyright notice appear in 

+  all copies and that both that the copyright notice and this

+  permission notice and warranty disclaimer appear in supporting 

+  documentation, and that the name of the author not be used in 

+  advertising or publicity pertaining to distribution of the 

+  software without specific, written prior permission.

+

+  The author disclaim all warranties with regard to this

+  software, including all implied warranties of merchantability

+  and fitness.  In no event shall the author be liable for any

+  special, indirect or consequential damages or any damages

+  whatsoever resulting from loss of use, data or profits, whether

+  in an action of contract, negligence or other tortious action,

+  arising out of or in connection with the use or performance of

+  this software.

+*/

+

+#define  __INCLUDE_FROM_USB_DRIVER

+#include "../HighLevel/USBMode.h"

+

+#if defined(USB_CAN_BE_DEVICE)

+

+#include "Device.h"

+

+void USB_Device_SendRemoteWakeup(void)

+{

+	if (!(USB_Options & USB_OPT_MANUAL_PLL))

+	{

+		USB_PLL_On();

+		while (!(USB_PLL_IsReady()));

+	}

+

+	USB_CLK_Unfreeze();

+

+	UDCON |= (1 << RMWKUP);

+	while (!(UDCON & (1 << RMWKUP)));

+}

+

+#endif

diff --git a/LUFA/Drivers/USB/LowLevel/Device.h b/LUFA/Drivers/USB/LowLevel/Device.h
index 7cd2d8b06..b595564a6 100644
--- a/LUFA/Drivers/USB/LowLevel/Device.h
+++ b/LUFA/Drivers/USB/LowLevel/Device.h
@@ -85,49 +85,26 @@
 			 *  USB interface should be initialized in full speed (12Mb/s) mode.
 			 */
 			#define USB_DEVICE_OPT_FULLSPEED               (0 << 0)
-			
+
+		/* Function Prototypes: */
+			/** Sends a Remote Wakeup request to the host. This signals to the host that the device should
+			 *  be taken out of suspended mode, and communications should resume.
+			 *
+			 *  Typically, this is implemented so that HID devices (mice, keyboards, etc.) can wake up the
+			 *  host computer when the host has suspended all USB devices to enter a low power state.
+			 *
+			 *  \note This macro should only be used if the device has indicated to the host that it
+			 *        supports the Remote Wakeup feature in the device descriptors, and should only be
+			 *        issued if the host is currently allowing remote wakeup events from the device (i.e.,
+			 *        the \ref USB_RemoteWakeupEnabled flag is set). When the NO_DEVICE_REMOTE_WAKEUP compile
+			 *        time option is used, this macro is unavailable.
+			 *
+			 *  \see \ref Group_Descriptors for more information on the RMWAKEUP feature and device descriptors.
+			 */
+			void USB_Device_SendRemoteWakeup(void);
+							
 		/* Pseudo-Function Macros: */
-			#if defined(__DOXYGEN__)
-				/** Sends a Remote Wakeup request to the host. This signals to the host that the device should
-				 *  be taken out of suspended mode, and communications should resume.
-				 *
-				 *  Typically, this is implemented so that HID devices (mice, keyboards, etc.) can wake up the
-				 *  host computer when the host has suspended all USB devices to enter a low power state.
-				 *
-				 *  \note This macro should only be used if the device has indicated to the host that it
-				 *        supports the Remote Wakeup feature in the device descriptors, and should only be
-				 *        issued if the host is currently allowing remote wakeup events from the device (i.e.,
-				 *        the \ref USB_RemoteWakeupEnabled flag is set). When the NO_DEVICE_REMOTE_WAKEUP compile
-				 *        time option is used, this macro is unavailable.
-				 *
-				 *  \see \ref Group_Descriptors for more information on the RMWAKEUP feature and device descriptors.
-				 */
-				static inline void USB_Device_SendRemoteWakeup(void);
-				
-				/** Indicates if a Remote Wakeup request is being sent to the host. This returns true if a
-				 *  remote wakeup is currently being sent, false otherwise.
-				 *
-				 *  This can be used in conjunction with the \ref USB_Device_IsUSBSuspended() macro to determine if
-				 *  a sent RMWAKEUP request was accepted or rejected by the host.
-				 *
-				 *  \note This macro should only be used if the device has indicated to the host that it
-				 *        supports the Remote Wakeup feature in the device descriptors. When the NO_DEVICE_REMOTE_WAKEUP
-				 *        compile time option is used, this macro is unavailable.
-				 *
-				 *  \see \ref Group_Descriptors for more information on the RMWAKEUP feature and device descriptors.
-				 *
-				 *  \return Boolean true if no Remote Wakeup request is currently being sent, false otherwise
-				 */
-				static inline bool USB_Device_IsRemoteWakeupSent(void);
-				
-				/** Indicates if the device is currently suspended by the host. Whilst suspended, the device is
-				 *  to enter a low power state until resumed by the host. No USB traffic to or from the device
-				 *  can occur (except for Remote Wakeup requests) during suspension by the host.
-				 *
-				 *  \return Boolean true if the USB communications have been suspended by the host, false otherwise.
-				 */
-				static inline bool USB_Device_IsUSBSuspended(void);
-				
+			#if defined(__DOXYGEN__)				
 				/** Enables the device mode Start Of Frame events. When enabled, this causes the
 				 *  \ref EVENT_USB_Device_StartOfFrame() event to fire once per millisecond, synchronized to the USB bus,
 				 *  at the start of each USB frame when enumerated in device mode.
@@ -139,14 +116,6 @@
 				 */
 				static inline bool USB_Device_DisableSOFEvents(void);
 			#else
-				#if !defined(NO_DEVICE_REMOTE_WAKEUP)
-					#define USB_Device_SendRemoteWakeup()   MACROS{ UDCON |= (1 << RMWKUP); }MACROE
-
-					#define USB_Device_IsRemoteWakeupSent()       ((UDCON &  (1 << RMWKUP)) ? false : true)
-				#endif
-				
-				#define USB_Device_IsUSBSuspended()           ((UDINT &  (1 << SUSPI)) ? true : false)
-				
 				#define USB_Device_EnableSOFEvents()    MACROS{ USB_INT_Enable(USB_INT_SOFI); }MACROE
 
 				#define USB_Device_DisableSOFEvents()   MACROS{ USB_INT_Disable(USB_INT_SOFI); }MACROE
diff --git a/LUFA/Drivers/USB/USB.h b/LUFA/Drivers/USB/USB.h
index 234d6f9e3..74d1cdfc7 100644
--- a/LUFA/Drivers/USB/USB.h
+++ b/LUFA/Drivers/USB/USB.h
@@ -42,6 +42,7 @@
  *  \section Sec_Dependencies Module Source Dependencies
  *  The following files must be built with any user project that uses this module:
  *    - LUFA/Drivers/USB/LowLevel/DevChapter9.c
+ *    - LUFA/Drivers/USB/LowLevel/Device.c
  *    - LUFA/Drivers/USB/LowLevel/Endpoint.c
  *    - LUFA/Drivers/USB/LowLevel/Host.c
  *    - LUFA/Drivers/USB/LowLevel/HostChapter9.c