From patchwork Sun May 31 15:54:46 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Minas Harutyunyan X-Patchwork-Id: 11580687 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 85865139A for ; Sun, 31 May 2020 15:54:57 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 6D2BC206DA for ; Sun, 31 May 2020 15:54:57 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=synopsys.com header.i=@synopsys.com header.b="LQ5LYso2" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727906AbgEaPy4 (ORCPT ); Sun, 31 May 2020 11:54:56 -0400 Received: from smtprelay-out1.synopsys.com ([149.117.87.133]:51288 "EHLO smtprelay-out1.synopsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725912AbgEaPy4 (ORCPT ); Sun, 31 May 2020 11:54:56 -0400 Received: from mailhost.synopsys.com (sv1-mailhost2.synopsys.com [10.205.2.132]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) by smtprelay-out1.synopsys.com (Postfix) with ESMTPS id 4A18EC03BD; Sun, 31 May 2020 15:54:54 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=synopsys.com; s=mail; t=1590940495; bh=FmXT1IiqXKu1gChXx1xxX4ZT/ZqBPC9zQxHpkew9WMU=; h=Date:From:Subject:To:Cc:From; b=LQ5LYso20l5AbojWB6ZQMSDJuwG5K+kiWnEghkBAJkL2t/y/95jKyaYgyYEIW5Pks AhLwBuqU+HzngcXikUc7570gyQ3CW+IyA25olWdBSY769WnJvuBIvMDuH3FKrnMk8t enHQzWpsIw58YTDCV9s50AGk7ZNHYrhtl8t92MR1AWQ1ne95+tDwnhqrUPKZ1wlkYi nShWINl9YRIoWzzLBYq4/qWLm09+GAnRtGfUc790NvNoU1aCkLwHQ8YgmWrwCYoRyp 5CKoSprzhy/5y27rcPgMCDKjk2eqgWsFOLObQgUm7sGXUIMCB3fmoG2eSOqNQL8RwR xHLvWwz8eB6hQ== Received: from hminas-z420 (hminas-z420.internal.synopsys.com [10.116.126.211]) (using TLSv1 with cipher AES128-SHA (128/128 bits)) (No client certificate requested) by mailhost.synopsys.com (Postfix) with ESMTPSA id 754AAA006F; Sun, 31 May 2020 15:54:49 +0000 (UTC) Received: by hminas-z420 (sSMTP sendmail emulation); Sun, 31 May 2020 19:54:46 +0400 Date: Sun, 31 May 2020 19:54:46 +0400 Message-Id: <867ae31fcdfde93e45f12bcab23d6055ebd4024c.1590940209.git.hminas@synopsys.com> X-SNPS-Relay: synopsys.com From: Minas Harutyunyan Subject: [PATCH v3] usb: dwc2: Postponed gadget registration to the udc class driver To: Felipe Balbi , Greg Kroah-Hartman , Minas Harutyunyan , linux-usb@vger.kernel.org Cc: John Youn , stable@vger.kernel.org, Marek Vasut Sender: linux-usb-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-usb@vger.kernel.org During dwc2 driver probe, after gadget registration to the udc class driver, if exist any builtin function driver it immediately bound to dwc2 and after init host side (dwc2_hcd_init()) stucked in host mode. Patch postpone gadget registration after host side initialization done. Cc: stable@vger.kernel.org Fixes: 117777b2c3bb9 ("usb: dwc2: Move gadget probe function into platform code") Reported-by: kbuild test robot Tested-by: Marek Vasut Signed-off-by: Minas Harutyunyan --- Changes in V2: - added module configuration check Changed in V3: - added tag "Reported-by: kbuild test robot drivers/usb/dwc2/gadget.c | 6 ------ drivers/usb/dwc2/platform.c | 11 +++++++++++ 2 files changed, 11 insertions(+), 6 deletions(-) diff --git a/drivers/usb/dwc2/gadget.c b/drivers/usb/dwc2/gadget.c index 12b98b466287..7faf5f8c056d 100644 --- a/drivers/usb/dwc2/gadget.c +++ b/drivers/usb/dwc2/gadget.c @@ -4920,12 +4920,6 @@ int dwc2_gadget_init(struct dwc2_hsotg *hsotg) epnum, 0); } - ret = usb_add_gadget_udc(dev, &hsotg->gadget); - if (ret) { - dwc2_hsotg_ep_free_request(&hsotg->eps_out[0]->ep, - hsotg->ctrl_req); - return ret; - } dwc2_hsotg_dump(hsotg); return 0; diff --git a/drivers/usb/dwc2/platform.c b/drivers/usb/dwc2/platform.c index e571c8ae65ec..c347d93eae64 100644 --- a/drivers/usb/dwc2/platform.c +++ b/drivers/usb/dwc2/platform.c @@ -575,6 +575,17 @@ static int dwc2_driver_probe(struct platform_device *dev) if (hsotg->dr_mode == USB_DR_MODE_PERIPHERAL) dwc2_lowlevel_hw_disable(hsotg); +#if IS_ENABLED(CONFIG_USB_DWC2_PERIPHERAL) || \ + IS_ENABLED(CONFIG_USB_DWC2_DUAL_ROLE) + /* Postponed adding a new gadget to the udc class driver list */ + if (hsotg->gadget_enabled) { + retval = usb_add_gadget_udc(hsotg->dev, &hsotg->gadget); + if (retval) { + dwc2_hsotg_remove(hsotg); + goto error_init; + } + } +#endif /* CONFIG_USB_DWC2_PERIPHERAL || CONFIG_USB_DWC2_DUAL_ROLE */ return 0; error_init: