usb: gadget: should use u16 type variable to store MaxPower
authorDu, ChangbinX <changbinx.du@intel.com>
Tue, 17 Dec 2013 11:47:42 +0000 (11:47 +0000)
committerFelipe Balbi <balbi@ti.com>
Thu, 19 Dec 2013 15:27:43 +0000 (09:27 -0600)
From 7e827a0d300e084f74c65122baa5e3193f9a7f18 Mon Sep 17 00:00:00 2001
From: "Du, Changbin" <changbinx.du@intel.com>
Date: Mon, 16 Dec 2013 20:32:13 +0800
Subject: [PATCH] usb/gadget: should use u16 type variable to store MaxPower

The MaxPower field is of u16 type. So using u8 type variable can break
data (high byte lost).

Signed-off-by: Du, Changbin <changbinx.du@intel.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>
drivers/usb/gadget/composite.c

index 43cbd76fca06cd319d5b55e10759c7c706bcbf06..d742bed7a5fa604ee7144f3b2c17cd9f6e06bc80 100644 (file)
@@ -1728,7 +1728,7 @@ composite_resume(struct usb_gadget *gadget)
 {
        struct usb_composite_dev        *cdev = get_gadget_data(gadget);
        struct usb_function             *f;
-       u                             maxpower;
+       u16                             maxpower;
 
        /* REVISIT:  should we have config level
         * suspend/resume callbacks?