[PATCH v2 48/48] usb: gadget: epautoconf: cleanup dead code
Sergei Shtylyov
sergei.shtylyov at cogentembedded.com
Tue Jul 14 04:16:27 PDT 2015
Hello.
On 7/14/2015 12:39 PM, Robert Baldyga wrote:
> Function find_ep() is no longer needed here, so we can remove it.
> We also don't use anything from gadget_chips.h header any longer.
> Signed-off-by: Robert Baldyga <r.baldyga at samsung.com>
> ---
> drivers/usb/gadget/epautoconf.c | 14 --------------
> 1 file changed, 14 deletions(-)
> diff --git a/drivers/usb/gadget/epautoconf.c b/drivers/usb/gadget/epautoconf.c
> index e9a8682..9a80925 100644
> --- a/drivers/usb/gadget/epautoconf.c
> +++ b/drivers/usb/gadget/epautoconf.c
> @@ -20,20 +20,6 @@
> #include <linux/usb/ch9.h>
> #include <linux/usb/gadget.h>
>
> -#include "gadget_chips.h"
> -
> -static struct usb_ep *
> -find_ep (struct usb_gadget *gadget, const char *name)
> -{
> - struct usb_ep *ep;
> -
> - list_for_each_entry (ep, &gadget->ep_list, ep_list) {
> - if (0 == strcmp (ep->name, name))
> - return ep;
> - }
> - return NULL;
> -}
> -
I don't think duplicating the function in each driver that needs it is
better than turniong this function into public.
WBR, Sergei
More information about the linux-arm-kernel
mailing list