aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/usb/gadget
diff options
context:
space:
mode:
authorFelipe Balbi <balbi@ti.com>2013-03-22 10:58:18 -0400
committerFelipe Balbi <balbi@ti.com>2013-03-27 07:18:59 -0400
commit94a06018dead7fc5f92238ececf31ed601ee528c (patch)
treeca3436d478d6ea209a7b2ba5cc0bb88e3ed9ea92 /drivers/usb/gadget
parent3f8b620124d2f331b71936a8d17e0fa493182632 (diff)
usb: gadget: mv_udc_core: fix sparse warnings
fix the following sparse warnings: drivers/usb/gadget/mv_udc_core.c:1850:6: warning: symbol 'irq_process_reset' was not declared. Should it be static? drivers/usb/gadget/mv_udc_core.c:247:9: warning: context imbalance in 'done' - unexpected unlock drivers/usb/gadget/mv_udc_core.c:1692:25: warning: context imbalance in 'handle_setup_packet' - unexpected unlock Note that there are still other sparse warnings which aren't trivial to fix, so I left them out: drivers/usb/gadget/mv_udc_core.c:848:34: warning: incorrect type in argument 1 (different address spaces) drivers/usb/gadget/mv_udc_core.c:848:34: expected void const volatile [noderef] <asn:2>*addr drivers/usb/gadget/mv_udc_core.c:848:34: got unsigned int *<noident> drivers/usb/gadget/mv_udc_core.c:849:42: warning: incorrect type in argument 2 (different address spaces) drivers/usb/gadget/mv_udc_core.c:849:42: expected void volatile [noderef] <asn:2>*addr drivers/usb/gadget/mv_udc_core.c:849:42: got unsigned int *<noident> Acked-by: Neil Zhang <zhangwm@marvell.com> Signed-off-by: Felipe Balbi <balbi@ti.com>
Diffstat (limited to 'drivers/usb/gadget')
-rw-r--r--drivers/usb/gadget/mv_udc_core.c6
1 files changed, 5 insertions, 1 deletions
diff --git a/drivers/usb/gadget/mv_udc_core.c b/drivers/usb/gadget/mv_udc_core.c
index 9a68c051a5a8..88be74e6297d 100644
--- a/drivers/usb/gadget/mv_udc_core.c
+++ b/drivers/usb/gadget/mv_udc_core.c
@@ -212,6 +212,8 @@ static int process_ep_req(struct mv_udc *udc, int index,
212 * request is still in progress. 212 * request is still in progress.
213 */ 213 */
214static void done(struct mv_ep *ep, struct mv_req *req, int status) 214static void done(struct mv_ep *ep, struct mv_req *req, int status)
215 __releases(&ep->udc->lock)
216 __acquires(&ep->udc->lock)
215{ 217{
216 struct mv_udc *udc = NULL; 218 struct mv_udc *udc = NULL;
217 unsigned char stopped = ep->stopped; 219 unsigned char stopped = ep->stopped;
@@ -1651,6 +1653,8 @@ out:
1651 1653
1652static void handle_setup_packet(struct mv_udc *udc, u8 ep_num, 1654static void handle_setup_packet(struct mv_udc *udc, u8 ep_num,
1653 struct usb_ctrlrequest *setup) 1655 struct usb_ctrlrequest *setup)
1656 __releases(&ep->udc->lock)
1657 __acquires(&ep->udc->lock)
1654{ 1658{
1655 bool delegate = false; 1659 bool delegate = false;
1656 1660
@@ -1847,7 +1851,7 @@ static void irq_process_tr_complete(struct mv_udc *udc)
1847 } 1851 }
1848} 1852}
1849 1853
1850void irq_process_reset(struct mv_udc *udc) 1854static void irq_process_reset(struct mv_udc *udc)
1851{ 1855{
1852 u32 tmp; 1856 u32 tmp;
1853 unsigned int loops; 1857 unsigned int loops;