Merge pull request #7233 from c1728p9/usb_cleanup

Fix various USB warnings

Author: Cruz Monrreal

TESTS/usb_device/basic/USBTester.cpp

@@ -41,8 +41,9 @@
 
 
 USBTester::USBTester(USBPhy *phy, uint16_t vendor_id, uint16_t product_id, uint16_t product_release):
-                        USBDevice(phy, vendor_id, product_id, product_release), reset_count(0), suspend_count(0),
-                        resume_count(0), interface_0_alt_set(NONE), interface_1_alt_set(NONE), configuration_set(NONE)
+                        USBDevice(phy, vendor_id, product_id, product_release), interface_0_alt_set(NONE),
+                        interface_1_alt_set(NONE), configuration_set(NONE), reset_count(0),
+                        suspend_count(0), resume_count(0)
 {
 
     EndpointResolver resolver(endpoint_table());

TESTS/usb_device/basic/USBTester.h

@@ -38,7 +38,7 @@ class USBTester: public USBDevice {
     */
     USBTester(USBPhy *phy, uint16_t vendor_id, uint16_t product_id, uint16_t product_release);
 
-    ~USBTester();
+    virtual ~USBTester();
 
     /*
     *

usb/device/USBAudio/USBAudio.cpp

@@ -342,7 +342,7 @@ float USBAudio::get_volume()
 {
     lock();
 
-    float ret = _mute ? 0.0 : _volume;
+    float ret = _mute ? 0.0f : _volume;
 
     unlock();
     return ret;

usb/device/USBDevice/EndpointResolver.cpp

@@ -122,7 +122,7 @@ int EndpointResolver::next_index(usb_ep_type_t type, bool in_not_out)
             continue;
         }
 
-        if (shared && (1 << other)) {
+        if (shared && (_used & (1 << other))) {
             // This endpoint can only be one direction at a time and is in
             // use by the other direction
             continue;

usb/device/USBHID/USBHID.cpp

@@ -26,7 +26,7 @@ class USBHID::AsyncSend: public AsyncOp {
 
     }
 
-    ~AsyncSend()
+    virtual ~AsyncSend()
     {
 
     }
@@ -58,7 +58,7 @@ class USBHID::AsyncRead: public AsyncOp {
 
     }
 
-    ~AsyncRead()
+    virtual ~AsyncRead()
     {
 
     }
@@ -90,7 +90,7 @@ class USBHID::AsyncWait: public AsyncOp {
 
     }
 
-    ~AsyncWait()
+    virtual ~AsyncWait()
     {
 
     }