Re: [PATCH 2/2] usb: gadget: f_loopfack: fix function params handling

From: Robert Baldyga
Date: Thu Sep 24 2015 - 11:18:29 EST


Hi Felipe,

I see that Krzysztof Opasiak made the same change in patch
https://lkml.org/lkml/2015/9/22/578, so please just ignore my patch.

My first patch is still needed - I will resend it with CC: stable.

Thanks,
Robert Baldyga

On 09/24/2015 05:02 PM, Robert Baldyga wrote:
> Move function parameters to struct f_loopback to make them per instance
> instead of having them as global variables. Since we can have multiple
> instances of USB function we also want to have separate set of parameters
> for each instance.
>
> Signed-off-by: Robert Baldyga <r.baldyga@xxxxxxxxxxx>
> ---
> drivers/usb/gadget/function/f_loopback.c | 22 ++++++++++++----------
> 1 file changed, 12 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/usb/gadget/function/f_loopback.c b/drivers/usb/gadget/function/f_loopback.c
> index 300e601..8d2dac8 100644
> --- a/drivers/usb/gadget/function/f_loopback.c
> +++ b/drivers/usb/gadget/function/f_loopback.c
> @@ -34,6 +34,9 @@ struct f_loopback {
>
> struct usb_ep *in_ep;
> struct usb_ep *out_ep;
> +
> + unsigned qlen;
> + unsigned buflen;
> };
>
> static inline struct f_loopback *func_to_loop(struct usb_function *f)
> @@ -41,9 +44,6 @@ static inline struct f_loopback *func_to_loop(struct usb_function *f)
> return container_of(f, struct f_loopback, function);
> }
>
> -static unsigned qlen;
> -static unsigned buflen;
> -
> /*-------------------------------------------------------------------------*/
>
> static struct usb_interface_descriptor loopback_intf = {
> @@ -251,7 +251,7 @@ static void loopback_complete(struct usb_ep *ep, struct usb_request *req)
> }
>
> /* queue the buffer for some later OUT packet */
> - req->length = buflen;
> + req->length = loop->buflen;
> status = usb_ep_queue(ep, req, GFP_ATOMIC);
> if (status == 0)
> return;
> @@ -288,7 +288,9 @@ static void disable_loopback(struct f_loopback *loop)
>
> static inline struct usb_request *lb_alloc_ep_req(struct usb_ep *ep, int len)
> {
> - return alloc_ep_req(ep, len, buflen);
> + struct f_loopback *loop = ep->driver_data;
> +
> + return alloc_ep_req(ep, len, loop->buflen);
> }
>
> static int enable_endpoint(struct usb_composite_dev *cdev, struct f_loopback *loop,
> @@ -315,7 +317,7 @@ static int enable_endpoint(struct usb_composite_dev *cdev, struct f_loopback *lo
> * we buffer at most 'qlen' transfers; fewer if any need more
> * than 'buflen' bytes each.
> */
> - for (i = 0; i < qlen && result == 0; i++) {
> + for (i = 0; i < loop->qlen && result == 0; i++) {
> req = lb_alloc_ep_req(ep, 0);
> if (!req)
> goto fail1;
> @@ -388,10 +390,10 @@ static struct usb_function *loopback_alloc(struct usb_function_instance *fi)
> lb_opts->refcnt++;
> mutex_unlock(&lb_opts->lock);
>
> - buflen = lb_opts->bulk_buflen;
> - qlen = lb_opts->qlen;
> - if (!qlen)
> - qlen = 32;
> + loop->buflen = lb_opts->bulk_buflen;
> + loop->qlen = lb_opts->qlen;
> + if (!loop->qlen)
> + loop->qlen = 32;
>
> loop->function.name = "loopback";
> loop->function.bind = loopback_bind;
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/