diff options
author | Felipe Balbi <balbi@ti.com> | 2013-03-22 10:58:18 -0400 |
---|---|---|
committer | Felipe Balbi <balbi@ti.com> | 2013-03-27 07:18:59 -0400 |
commit | 94a06018dead7fc5f92238ececf31ed601ee528c (patch) | |
tree | ca3436d478d6ea209a7b2ba5cc0bb88e3ed9ea92 /drivers/usb/gadget | |
parent | 3f8b620124d2f331b71936a8d17e0fa493182632 (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.c | 6 |
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 | */ |
214 | static void done(struct mv_ep *ep, struct mv_req *req, int status) | 214 | static 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 | ||
1652 | static void handle_setup_packet(struct mv_udc *udc, u8 ep_num, | 1654 | static 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 | ||
1850 | void irq_process_reset(struct mv_udc *udc) | 1854 | static void irq_process_reset(struct mv_udc *udc) |
1851 | { | 1855 | { |
1852 | u32 tmp; | 1856 | u32 tmp; |
1853 | unsigned int loops; | 1857 | unsigned int loops; |