++ Felipe

On 7/28/2016 1:27 AM, Bhaktipriya Shridhar wrote:
> alloc_ordered_workqueue replaces the deprecated
> create_singlethread_workqueue.
> 
> There are multiple work items on the work queue, which require
> ordering. Hence, an ordered workqueue has been used.
> 
> The workqueue "wq_otg" is not being used on a memory reclaim path.
> Hence, WQ_MEM_RECLAIM has not been set.
> 
> Signed-off-by: Bhaktipriya Shridhar <[email protected]>
> ---
>  drivers/usb/dwc2/hcd.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/usb/dwc2/hcd.c b/drivers/usb/dwc2/hcd.c
> index 2df3d04..df5a065 100644
> --- a/drivers/usb/dwc2/hcd.c
> +++ b/drivers/usb/dwc2/hcd.c
> @@ -5040,7 +5040,7 @@ int dwc2_hcd_init(struct dwc2_hsotg *hsotg, int irq)
> 
>       /* Create new workqueue and init work */
>       retval = -ENOMEM;
> -     hsotg->wq_otg = create_singlethread_workqueue("dwc2");
> +     hsotg->wq_otg = alloc_ordered_workqueue("dwc2", 0);
>       if (!hsotg->wq_otg) {
>               dev_err(hsotg->dev, "Failed to create workqueue\n");
>               goto error2;
> --
> 2.1.4
> 
> 


Acked-by: John Youn <[email protected]>

John

Reply via email to