lkml.org 
[lkml]   [2013]   [Feb]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Date
Subject[PATCH 1/1]linux-usb: fix the product IDs to be little endian in initializers.c
From: fangxiaozhi <huananhu@huawei.com>

1. The idProduct is little endian, so make the product ID's value to be little endian. Make no break on big endian processors.

Signed-off-by: fangxiaozhi <huananhu@huawei.com>
--------------------------------------------------------------------

diff -uprN linux-3.8-rc6_orig/drivers/usb/storage/initializers.c linux-3.8-rc6/drivers/usb/storage/initializers.c
--- linux-3.8-rc6_orig/drivers/usb/storage/initializers.c 2013-02-06 14:48:51.564355283 +0800
+++ linux-3.8-rc6/drivers/usb/storage/initializers.c 2013-02-06 15:11:40.925434289 +0800
@@ -152,12 +152,15 @@ static int usb_stor_huawei_dongles_pid(s
* means the dongle in the single port mode,
* and a switch command is required to be sent. */
if (idesc && idesc->bInterfaceNumber == 0) {
- if ((idProduct == 0x1001)
- || (idProduct == 0x1003)
- || (idProduct == 0x1004)
- || (idProduct >= 0x1401 && idProduct <= 0x1500)
- || (idProduct >= 0x1505 && idProduct <= 0x1600)
- || (idProduct >= 0x1c02 && idProduct <= 0x2202)) {
+ if ((idProduct == cpu_to_le16(0x1001))
+ || (idProduct == cpu_to_le16(0x1003))
+ || (idProduct == cpu_to_le16(0x1004))
+ || (idProduct >= cpu_to_le16(0x1401)
+ && idProduct <= cpu_to_le16(0x1500))
+ || (idProduct >= cpu_to_le16(0x1505)
+ && idProduct <= cpu_to_le16(0x1600))
+ || (idProduct >= cpu_to_le16(0x1c02)
+ && idProduct <= cpu_to_le16(0x2202))) {
return 1;
}
}
@@ -169,7 +172,7 @@ int usb_stor_huawei_init(struct us_data
int result = 0;

if (usb_stor_huawei_dongles_pid(us)) {
- if (us->pusb_dev->descriptor.idProduct >= 0x1446)
+ if (us->pusb_dev->descriptor.idProduct >= cpu_to_le16(0x1446))
result = usb_stor_huawei_scsi_init(us);
else
result = usb_stor_huawei_feature_init(us);



\
 
 \ /
  Last update: 2013-02-06 11:01    [W:0.233 / U:0.056 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site