aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorfangxiaozhi <huananhu@huawei.com>2013-02-07 15:32:07 +0800
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2013-02-28 06:59:07 -0800
commit12b64a247ef45dcd865a98d63bbae7355e51633b (patch)
tree644786180afcb75e9a1c25aae0cb4dbd8d8515d8
parent30361160bc12da86b9ce4c7c60bd68ff930eda76 (diff)
downloadlinux-linaro-stable-12b64a247ef45dcd865a98d63bbae7355e51633b.tar.gz
USB: storage: properly handle the endian issues of idProduct
commit cd060956c5e97931c3909e4a808508469c0bb9f6 upstream. 1. The idProduct is little endian, so make sure its value to be compatible with the current CPU. Make no break on big endian processors. Signed-off-by: fangxiaozhi <huananhu@huawei.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r--drivers/usb/storage/initializers.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/usb/storage/initializers.c b/drivers/usb/storage/initializers.c
index 16b0bf055eeb..7ab9046ae0ec 100644
--- a/drivers/usb/storage/initializers.c
+++ b/drivers/usb/storage/initializers.c
@@ -147,7 +147,7 @@ static int usb_stor_huawei_dongles_pid(struct us_data *us)
int idProduct;
idesc = &us->pusb_intf->cur_altsetting->desc;
- idProduct = us->pusb_dev->descriptor.idProduct;
+ idProduct = le16_to_cpu(us->pusb_dev->descriptor.idProduct);
/* The first port is CDROM,
* means the dongle in the single port mode,
* and a switch command is required to be sent. */
@@ -169,7 +169,7 @@ int usb_stor_huawei_init(struct us_data *us)
int result = 0;
if (usb_stor_huawei_dongles_pid(us)) {
- if (us->pusb_dev->descriptor.idProduct >= 0x1446)
+ if (le16_to_cpu(us->pusb_dev->descriptor.idProduct) >= 0x1446)
result = usb_stor_huawei_scsi_init(us);
else
result = usb_stor_huawei_feature_init(us);