Hi Michael,
On Thu, Sep 30, 2021 at 12:27:15PM +0200, Michael Grzeschik wrote:
> If the streaming endpoint is not enabled, the worker has nothing to do.
> In the case it still got enqueued, this patch ensueres that it will bail
> out without handling any data.
>
> Signed-off-by: Michael Grzeschik <m.grzeschik@xxxxxxxxxxxxxx>
Reviewed-by: Paul Elder <paul.elder@xxxxxxxxxxxxxxxx>
> ---
> drivers/usb/gadget/function/uvc_video.c | 7 +++++--
> 1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/usb/gadget/function/uvc_video.c b/drivers/usb/gadget/function/uvc_video.c
> index ccee35177411d..152647495fa61 100644
> --- a/drivers/usb/gadget/function/uvc_video.c
> +++ b/drivers/usb/gadget/function/uvc_video.c
> @@ -335,12 +335,12 @@ static void uvcg_video_pump(struct work_struct *work)
> {
> struct uvc_video *video = container_of(work, struct uvc_video, pump);
> struct uvc_video_queue *queue = &video->queue;
> - struct usb_request *req;
> + struct usb_request *req = NULL;
> struct uvc_buffer *buf;
> unsigned long flags;
> int ret;
>
> - while (1) {
> + while (video->ep->enabled) {
> /* Retrieve the first available USB request, protected by the
> * request lock.
> */
> @@ -390,6 +390,9 @@ static void uvcg_video_pump(struct work_struct *work)
> video->req_int_count++;
> }
>
> + if (!req)
> + return;
> +
> spin_lock_irqsave(&video->req_lock, flags);
> list_add_tail(&req->list, &video->req_free);
> spin_unlock_irqrestore(&video->req_lock, flags);
> --
> 2.30.2
>
