Hi Heiko,
fix warnings: drivers/usb/gadget/at91_udc.c:1344:12: warning: 'at91rm9200_udc_init' defined but not used [-Wunused-function] drivers/usb/gadget/at91_udc.c:1379:13: warning: 'at91rm9200_udc_pullup' defined but not used [-Wunused-function] drivers/usb/gadget/at91_udc.c:1476:12: warning: 'at91sam9263_udc_init' defined but not used [-Wunused-function]
Signed-off-by: Heiko Schocher hs@denx.de
drivers/usb/gadget/at91_udc.c | 30 +++--------------------------- 1 file changed, 3 insertions(+), 27 deletions(-)
diff --git a/drivers/usb/gadget/at91_udc.c b/drivers/usb/gadget/at91_udc.c index 01a5907..85bfbf5 100644 --- a/drivers/usb/gadget/at91_udc.c +++ b/drivers/usb/gadget/at91_udc.c @@ -1341,7 +1341,7 @@ static int at91_stop(struct usb_gadget *gadget)
/*-------------------------------------------------------------------------*/
-static int at91rm9200_udc_init(struct at91_udc *udc) +static __maybe_unused int at91rm9200_udc_init(struct at91_udc *udc) { struct at91_ep *ep; int ret; @@ -1376,7 +1376,8 @@ static int at91rm9200_udc_init(struct at91_udc *udc) return 0; }
-static void at91rm9200_udc_pullup(struct at91_udc *udc, int is_on) +static __maybe_unused +void at91rm9200_udc_pullup(struct at91_udc *udc, int is_on) { int active = !udc->board.pullup_active_low;
@@ -1473,31 +1474,6 @@ static const struct at91_udc_caps at91sam9261_udc_caps = { }; #endif
-static int at91sam9263_udc_init(struct at91_udc *udc)
I'm a bit puzzled...
For the at91rm9200 related boards it is enough to add __maybe_unused and leave the unused function in the source code.
Hence, I'm wondering why do you remove the function for at91sam9263?
Shouldn't all be removed or have added __maybe_unused ?
-{
- struct at91_ep *ep;
- int i;
- for (i = 0; i < NUM_ENDPOINTS; i++) {
ep = &udc->ep[i];switch (i) {case 0:case 1:case 2:case 3:ep->maxpacket = 64;break;case 4:case 5:ep->maxpacket = 256;break;}- }
- return 0;
-}
int usb_gadget_handle_interrupts(int index) { struct at91_udc *udc = controller;
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@denx.de