Browse Source

usb: lvstest: Fix sparse warnings generated by kbuild test bot

Following sparse warnings were reported by kbuild test bot

drivers/usb/misc/lvstest.c:314:28: sparse: incorrect type in assignment (different base types)
   drivers/usb/misc/lvstest.c:314:28:    expected unsigned short [unsigned] [usertype] portchange
   drivers/usb/misc/lvstest.c:314:28:    got restricted __le16 [usertype] wPortChange
drivers/usb/misc/lvstest.c:332:40: sparse: restricted __le16 degrades to integer

This patch fixes above warnings.

Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Pratyush Anand <pratyush.anand@st.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Pratyush Anand 11 years ago
parent
commit
b1bd3f1a39
1 changed files with 2 additions and 2 deletions
  1. 2 2
      drivers/usb/misc/lvstest.c

+ 2 - 2
drivers/usb/misc/lvstest.c

@@ -311,7 +311,7 @@ static void lvs_rh_work(struct work_struct *work)
 		if (ret < 4)
 			continue;
 
-		portchange = port_status->wPortChange;
+		portchange = le16_to_cpu(port_status->wPortChange);
 
 		if (portchange & USB_PORT_STAT_C_LINK_STATE)
 			lvs_rh_clear_port_feature(hdev, i,
@@ -329,7 +329,7 @@ static void lvs_rh_work(struct work_struct *work)
 			lvs_rh_clear_port_feature(hdev, i,
 					USB_PORT_FEAT_C_CONNECTION);
 
-			if (port_status->wPortStatus &
+			if (le16_to_cpu(port_status->wPortStatus) &
 					USB_PORT_STAT_CONNECTION) {
 				lvs->present = true;
 				lvs->portnum = i;