Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Artur Petrosyan <Arthur.Petrosyan@synopsys.com>
To:     Marek Szyprowski <m.szyprowski@samsung.com>,
        "linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
        "linux-samsung-soc@vger.kernel.org" 
        <linux-samsung-soc@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
CC:     Minas Harutyunyan <Minas.Harutyunyan@synopsys.com>,
        Felipe Balbi <balbi@kernel.org>,
        Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>,
        Krzysztof Kozlowski <krzk@kernel.org>
Subject: Re: [PATCH 1/2] usb: dwc2: rename DWC2_POWER_DOWN_PARAM_NONE state
Thread-Topic: [PATCH 1/2] usb: dwc2: rename DWC2_POWER_DOWN_PARAM_NONE state
Thread-Index: AQHXiSYbmD399Z0pn0uw+0FNmkO/Sg==
Date:   Thu, 5 Aug 2021 13:49:24 +0000
Message-ID: <CH0PR12MB5265A61756A08A29362D8F77A7F29@CH0PR12MB5265.namprd12.prod.outlook.com>
References: <CGME20210804114433eucas1p134417b605abeb57728d358fc2f42162b@eucas1p1.samsung.com>
 <20210804114421.10282-1-m.szyprowski@samsung.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?s5nMrfu04ndN2JZsoMejBpzRWckMjyuXJFLMvOYm59WC6OZ1LM681/pSe4pB?=
 =?us-ascii?Q?UJTCY5qjqsLibCLepHVxP4lJrLxlcrgpR97uBGf6/w7Kcsm5Nb4EwyEX/v9y?=
 =?us-ascii?Q?26tEfs9vOBl5ZhTb4yWG+0zJZ0hArqe2CHX6dBMljUzRhcy/9BaU8u1Qj0LD?=
 =?us-ascii?Q?MN9Cliax5ahUpWm9MCoaPovzkgM/+ak5mcJ24idzCPmfMCwg1Nsh8lCWYfct?=
 =?us-ascii?Q?FUgihDDO49flmL1pKlyJLS7ZJwWeEqntO08xw2NJpV3vhR7sW1rDjBUikBLk?=
 =?us-ascii?Q?clWNdN61wVtt0bBM1jcawhjZyy3MWBc8dJzEtfQ0ybS6TLih7FWyjfKGkVET?=
 =?us-ascii?Q?C200vy/XALLvTox3VV76DnGgXTURIVY0MoneL3KTg5+4rrIxvg2wSJE+1DkT?=
 =?us-ascii?Q?wmMa+e1RdIDP7hLuh36utn9uxm3yXLD4Y3XwvKs6Wud4OJNsAFOx6L6q4Why?=
 =?us-ascii?Q?T5KOtl+LbnShF4KThWWxY3yny9SmCe8sV2VlU9dWAEioV5gF2jeXsnR4/upe?=
 =?us-ascii?Q?DnVZLk7mOa3BCq9B1m6XzqCwBwSo4ukaLbNPkbptskoW9IgNBO7i3rtO/6zI?=
 =?us-ascii?Q?nY4ANPy1y6XYY5DbkYAzUcl/bucK8N0VQbcI/v9IdBZAhrV23B59AgH/apz2?=
 =?us-ascii?Q?3rrL5uiTzgzg8zb/GbnrQ8vTCIOhLTe/g+0Wn9oCaOG8CLEzA5tD3wnukoSQ?=
 =?us-ascii?Q?G/Xu50grj5BhVG0WGaLVqANpWiaEuMDnyO/7Byz1Xsapvt6DYSadSvZrAXgT?=
 =?us-ascii?Q?7zYbqiEAcbuBcNfhy1NIOosWrsAntqOfUBsEVXUYrwP6JL7orORKTTYcsB2a?=
 =?us-ascii?Q?toBKoIMIvNy+lpnqpQfuWHdTyzxp8NYhgDKEM6JsRCwG274iud2YoMjI8CFA?=
 =?us-ascii?Q?XokGijVzjfDjRuTC58otLPsk0i+urhi0Lnb/d8GQdqrB3LB/b6aZmAk3HObP?=
 =?us-ascii?Q?RRTET4zZQRZjJSF1iVD4uk6pB/Ec8XS5lzwQpTRdcXiHUzt0XlbsGR2X6gBO?=
 =?us-ascii?Q?EIjO/FyatzAf9xvuEFBfcdJZ5l4NDseqCsOOEiWvY4Q5TOHIslYqzGc9dyB7?=
 =?us-ascii?Q?16eiNpuzjc6CVNovKkmYve12HdU0WzKU+Msirz+dO8Hy4WjGK+SUzWNG7R3O?=
 =?us-ascii?Q?DpYBVT5Wwd5GkrPPURh4RqlXoOuiyAc1vP4SlO/uj/DyrXm+Z8QmG1znPBEg?=
 =?us-ascii?Q?mbW/dvo3EM/CniGPBfTq9sFl4oE8ysOyTm570dejFg7wjPqizdQnKjuUVddl?=
 =?us-ascii?Q?XlGrQzzYHgGX8AdB+QZ+TjITBdUV8xl1tuWEkYr9Ac6WYU8Ss1fdTEg2XDff?=
 =?us-ascii?Q?294leVo+kZDabH0Cv6z69F/e?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: synopsys.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: CH0PR12MB5265.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 16e9c81c-dcd0-4f6d-3afd-08d95817d621
X-MS-Exchange-CrossTenant-originalarrivaltime: 05 Aug 2021 13:49:24.8216
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: c33c9f88-1eb7-4099-9700-16013fd9e8aa
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: MWOyCq/3tgSslIL10n/4obNkgxLL3Rju4BqdP45YAHkbP6tiCyvM6ZIxHz7ZRFsVINF9g58rQxVBJ/xVoyrWcg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH2PR12MB5516
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Marek,=0A=
=0A=
On 8/4/2021 3:44 PM, Marek Szyprowski wrote:=0A=
> DWC2_POWER_DOWN_PARAM_NONE really means that the driver still uses clock=
=0A=
> gating to save power when hardware is not used. Rename the state name to=
=0A=
> DWC2_POWER_DOWN_PARAM_CLOCK_GATING to match the driver behavior.=0A=
> =0A=
> Suggested-by: Minas Harutyunyan <Minas.Harutyunyan@synopsys.com>=0A=
> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>=0A=
> ---=0A=
> This is a follow-up of this discussion:=0A=
> https://urldefense.com/v3/__https://lore.kernel.org/linux-usb/26099de1-82=
6f-42bf-0de7-759a47faf4a0@samsung.com/__;!!A4F2R9G_pg!MIGkDa9hfT_3k8EATqQs_=
UYCb7yXMN18CC-gsrNOMI8NqZzAok3DZA6G04fgkS_4H52snpA$=0A=
> =0A=
> This should be applied on top of v5.14-rc3.=0A=
> ---=0A=
>   drivers/usb/dwc2/core.h      |  4 ++--=0A=
>   drivers/usb/dwc2/core_intr.c |  8 ++++----=0A=
>   drivers/usb/dwc2/hcd.c       | 10 +++++-----=0A=
>   drivers/usb/dwc2/params.c    | 22 +++++++++++-----------=0A=
>   drivers/usb/dwc2/platform.c  |  2 +-=0A=
>   5 files changed, 23 insertions(+), 23 deletions(-)=0A=
> =0A=
> diff --git a/drivers/usb/dwc2/core.h b/drivers/usb/dwc2/core.h=0A=
> index cb9059a8444b..38b6733d26ec 100644=0A=
> --- a/drivers/usb/dwc2/core.h=0A=
> +++ b/drivers/usb/dwc2/core.h=0A=
> @@ -382,7 +382,7 @@ enum dwc2_ep0_state {=0A=
>    *			If power_down is enabled, the controller will enter=0A=
>    *			power_down in both peripheral and host mode when=0A=
>    *			needed.=0A=
> - *			0 - No (default)=0A=
> + *			0 - Clock gating (default)=0A=
>    *			1 - Partial power down=0A=
>    *			2 - Hibernation=0A=
>    * @no_clock_gating:	Specifies whether to avoid clock gating feature.=
=0A=
> @@ -482,7 +482,7 @@ struct dwc2_core_params {=0A=
>   	bool external_id_pin_ctl;=0A=
>   =0A=
>   	int power_down;=0A=
> -#define DWC2_POWER_DOWN_PARAM_NONE		0=0A=
> +#define DWC2_POWER_DOWN_PARAM_CLOCK_GATING	0=0A=
>   #define DWC2_POWER_DOWN_PARAM_PARTIAL		1=0A=
>   #define DWC2_POWER_DOWN_PARAM_HIBERNATION	2=0A=
>   	bool no_clock_gating;=0A=
> diff --git a/drivers/usb/dwc2/core_intr.c b/drivers/usb/dwc2/core_intr.c=
=0A=
> index a5c52b237e72..660abff1d42b 100644=0A=
> --- a/drivers/usb/dwc2/core_intr.c=0A=
> +++ b/drivers/usb/dwc2/core_intr.c=0A=
> @@ -327,7 +327,7 @@ static void dwc2_handle_session_req_intr(struct dwc2_=
hsotg *hsotg)=0A=
>   =0A=
>   			/* Exit gadget mode clock gating. */=0A=
>   			if (hsotg->params.power_down =3D=3D=0A=
> -			    DWC2_POWER_DOWN_PARAM_NONE && hsotg->bus_suspended)=0A=
> +			    DWC2_POWER_DOWN_PARAM_CLOCK_GATING && hsotg->bus_suspended)=0A=
>   				dwc2_gadget_exit_clock_gating(hsotg, 0);=0A=
>   		}=0A=
>   =0A=
> @@ -438,7 +438,7 @@ static void dwc2_handle_wakeup_detected_intr(struct d=
wc2_hsotg *hsotg)=0A=
>   =0A=
>   			/* Exit gadget mode clock gating. */=0A=
>   			if (hsotg->params.power_down =3D=3D=0A=
> -			    DWC2_POWER_DOWN_PARAM_NONE && hsotg->bus_suspended)=0A=
> +			    DWC2_POWER_DOWN_PARAM_CLOCK_GATING && hsotg->bus_suspended)=0A=
>   				dwc2_gadget_exit_clock_gating(hsotg, 0);=0A=
>   		} else {=0A=
>   			/* Change to L0 state */=0A=
> @@ -455,7 +455,7 @@ static void dwc2_handle_wakeup_detected_intr(struct d=
wc2_hsotg *hsotg)=0A=
>   			}=0A=
>   =0A=
>   			if (hsotg->params.power_down =3D=3D=0A=
> -			    DWC2_POWER_DOWN_PARAM_NONE && hsotg->bus_suspended)=0A=
> +			    DWC2_POWER_DOWN_PARAM_CLOCK_GATING && hsotg->bus_suspended)=0A=
>   				dwc2_host_exit_clock_gating(hsotg, 1);=0A=
>   =0A=
>   			/*=0A=
> @@ -551,7 +551,7 @@ static void dwc2_handle_usb_suspend_intr(struct dwc2_=
hsotg *hsotg)=0A=
>   					dev_err(hsotg->dev,=0A=
>   						"enter hibernation failed\n");=0A=
>   				break;=0A=
> -			case DWC2_POWER_DOWN_PARAM_NONE:=0A=
> +			case DWC2_POWER_DOWN_PARAM_CLOCK_GATING:=0A=
>   				/*=0A=
>   				 * If neither hibernation nor partial power down are supported,=0A=
>   				 * clock gating is used to save power.=0A=
> diff --git a/drivers/usb/dwc2/hcd.c b/drivers/usb/dwc2/hcd.c=0A=
> index 2a7828971d05..067f2770c2ef 100644=0A=
> --- a/drivers/usb/dwc2/hcd.c=0A=
> +++ b/drivers/usb/dwc2/hcd.c=0A=
> @@ -3333,7 +3333,7 @@ int dwc2_port_suspend(struct dwc2_hsotg *hsotg, u16=
 windex)=0A=
>   			dev_err(hsotg->dev, "enter hibernation failed.\n");=0A=
>   		spin_lock_irqsave(&hsotg->lock, flags);=0A=
>   		break;=0A=
> -	case DWC2_POWER_DOWN_PARAM_NONE:=0A=
> +	case DWC2_POWER_DOWN_PARAM_CLOCK_GATING:=0A=
>   		/*=0A=
>   		 * If not hibernation nor partial power down are supported,=0A=
>   		 * clock gating is used to save power.=0A=
> @@ -3701,7 +3701,7 @@ static int dwc2_hcd_hub_control(struct dwc2_hsotg *=
hsotg, u16 typereq,=0A=
>   			}=0A=
>   =0A=
>   			if (hsotg->params.power_down =3D=3D=0A=
> -			    DWC2_POWER_DOWN_PARAM_NONE && hsotg->bus_suspended)=0A=
> +			    DWC2_POWER_DOWN_PARAM_CLOCK_GATING && hsotg->bus_suspended)=0A=
>   				dwc2_host_exit_clock_gating(hsotg, 0);=0A=
>   =0A=
>   			pcgctl =3D dwc2_readl(hsotg, PCGCTL);=0A=
> @@ -4398,7 +4398,7 @@ static int _dwc2_hcd_suspend(struct usb_hcd *hcd)=
=0A=
>   		/* After entering suspend, hardware is not accessible */=0A=
>   		clear_bit(HCD_FLAG_HW_ACCESSIBLE, &hcd->flags);=0A=
>   		break;=0A=
> -	case DWC2_POWER_DOWN_PARAM_NONE:=0A=
> +	case DWC2_POWER_DOWN_PARAM_CLOCK_GATING:=0A=
>   		/*=0A=
>   		 * If not hibernation nor partial power down are supported,=0A=
>   		 * clock gating is used to save power.=0A=
> @@ -4482,7 +4482,7 @@ static int _dwc2_hcd_resume(struct usb_hcd *hcd)=0A=
>   		 */=0A=
>   		set_bit(HCD_FLAG_HW_ACCESSIBLE, &hcd->flags);=0A=
>   		break;=0A=
> -	case DWC2_POWER_DOWN_PARAM_NONE:=0A=
> +	case DWC2_POWER_DOWN_PARAM_CLOCK_GATING:=0A=
>   		/*=0A=
>   		 * If not hibernation nor partial power down are supported,=0A=
>   		 * port resume is done using the clock gating programming flow.=0A=
> @@ -4680,7 +4680,7 @@ static int _dwc2_hcd_urb_enqueue(struct usb_hcd *hc=
d, struct urb *urb,=0A=
>   				"exit partial_power_down failed\n");=0A=
>   	}=0A=
>   =0A=
> -	if (hsotg->params.power_down =3D=3D DWC2_POWER_DOWN_PARAM_NONE &&=0A=
> +	if (hsotg->params.power_down =3D=3D DWC2_POWER_DOWN_PARAM_CLOCK_GATING =
&&=0A=
>   	    hsotg->bus_suspended) {=0A=
>   		if (dwc2_is_device_mode(hsotg))=0A=
>   			dwc2_gadget_exit_clock_gating(hsotg, 0);=0A=
> diff --git a/drivers/usb/dwc2/params.c b/drivers/usb/dwc2/params.c=0A=
> index 59e119345994..dac26410b575 100644=0A=
> --- a/drivers/usb/dwc2/params.c=0A=
> +++ b/drivers/usb/dwc2/params.c=0A=
> @@ -68,14 +68,14 @@ static void dwc2_set_his_params(struct dwc2_hsotg *hs=
otg)=0A=
>   	p->ahbcfg =3D GAHBCFG_HBSTLEN_INCR16 <<=0A=
>   		GAHBCFG_HBSTLEN_SHIFT;=0A=
>   	p->change_speed_quirk =3D true;=0A=
> -	p->power_down =3D DWC2_POWER_DOWN_PARAM_NONE;=0A=
> +	p->power_down =3D DWC2_POWER_DOWN_PARAM_CLOCK_GATING;=0A=
>   }=0A=
>   =0A=
>   static void dwc2_set_s3c6400_params(struct dwc2_hsotg *hsotg)=0A=
>   {=0A=
>   	struct dwc2_core_params *p =3D &hsotg->params;=0A=
>   =0A=
> -	p->power_down =3D DWC2_POWER_DOWN_PARAM_NONE;=0A=
> +	p->power_down =3D DWC2_POWER_DOWN_PARAM_CLOCK_GATING;=0A=
>   	p->no_clock_gating =3D true;=0A=
>   	p->phy_utmi_width =3D 8;=0A=
>   }=0A=
> @@ -90,7 +90,7 @@ static void dwc2_set_rk_params(struct dwc2_hsotg *hsotg=
)=0A=
>   	p->host_perio_tx_fifo_size =3D 256;=0A=
>   	p->ahbcfg =3D GAHBCFG_HBSTLEN_INCR16 <<=0A=
>   		GAHBCFG_HBSTLEN_SHIFT;=0A=
> -	p->power_down =3D DWC2_POWER_DOWN_PARAM_NONE;=0A=
> +	p->power_down =3D DWC2_POWER_DOWN_PARAM_CLOCK_GATING;=0A=
>   }=0A=
>   =0A=
>   static void dwc2_set_ltq_params(struct dwc2_hsotg *hsotg)=0A=
> @@ -120,7 +120,7 @@ static void dwc2_set_amlogic_params(struct dwc2_hsotg=
 *hsotg)=0A=
>   	p->phy_type =3D DWC2_PHY_TYPE_PARAM_UTMI;=0A=
>   	p->ahbcfg =3D GAHBCFG_HBSTLEN_INCR8 <<=0A=
>   		GAHBCFG_HBSTLEN_SHIFT;=0A=
> -	p->power_down =3D DWC2_POWER_DOWN_PARAM_NONE;=0A=
> +	p->power_down =3D DWC2_POWER_DOWN_PARAM_CLOCK_GATING;=0A=
>   }=0A=
>   =0A=
>   static void dwc2_set_amlogic_g12a_params(struct dwc2_hsotg *hsotg)=0A=
> @@ -179,7 +179,7 @@ static void dwc2_set_stm32mp15_fsotg_params(struct dw=
c2_hsotg *hsotg)=0A=
>   	p->activate_stm_fs_transceiver =3D true;=0A=
>   	p->activate_stm_id_vb_detection =3D true;=0A=
>   	p->ahbcfg =3D GAHBCFG_HBSTLEN_INCR16 << GAHBCFG_HBSTLEN_SHIFT;=0A=
> -	p->power_down =3D DWC2_POWER_DOWN_PARAM_NONE;=0A=
> +	p->power_down =3D DWC2_POWER_DOWN_PARAM_CLOCK_GATING;=0A=
>   	p->host_support_fs_ls_low_power =3D true;=0A=
>   	p->host_ls_low_power_phy_clk =3D true;=0A=
>   }=0A=
> @@ -194,7 +194,7 @@ static void dwc2_set_stm32mp15_hsotg_params(struct dw=
c2_hsotg *hsotg)=0A=
>   	p->host_nperio_tx_fifo_size =3D 256;=0A=
>   	p->host_perio_tx_fifo_size =3D 256;=0A=
>   	p->ahbcfg =3D GAHBCFG_HBSTLEN_INCR16 << GAHBCFG_HBSTLEN_SHIFT;=0A=
> -	p->power_down =3D DWC2_POWER_DOWN_PARAM_NONE;=0A=
> +	p->power_down =3D DWC2_POWER_DOWN_PARAM_CLOCK_GATING;=0A=
>   	p->lpm =3D false;=0A=
>   	p->lpm_clock_gating =3D false;=0A=
>   	p->besl =3D false;=0A=
> @@ -339,7 +339,7 @@ static void dwc2_set_param_power_down(struct dwc2_hso=
tg *hsotg)=0A=
>   	else if (hsotg->hw_params.power_optimized)=0A=
>   		val =3D DWC2_POWER_DOWN_PARAM_PARTIAL;=0A=
>   	else=0A=
> -		val =3D DWC2_POWER_DOWN_PARAM_NONE;=0A=
> +		val =3D DWC2_POWER_DOWN_PARAM_CLOCK_GATING;=0A=
>   =0A=
>   	hsotg->params.power_down =3D val;=0A=
>   }=0A=
> @@ -585,27 +585,27 @@ static void dwc2_check_param_power_down(struct dwc2=
_hsotg *hsotg)=0A=
>   	int param =3D hsotg->params.power_down;=0A=
>   =0A=
>   	switch (param) {=0A=
> -	case DWC2_POWER_DOWN_PARAM_NONE:=0A=
> +	case DWC2_POWER_DOWN_PARAM_CLOCK_GATING:=0A=
>   		break;=0A=
>   	case DWC2_POWER_DOWN_PARAM_PARTIAL:=0A=
>   		if (hsotg->hw_params.power_optimized)=0A=
>   			break;=0A=
>   		dev_dbg(hsotg->dev,=0A=
>   			"Partial power down isn't supported by HW\n");=0A=
> -		param =3D DWC2_POWER_DOWN_PARAM_NONE;=0A=
> +		param =3D DWC2_POWER_DOWN_PARAM_CLOCK_GATING;=0A=
>   		break;=0A=
>   	case DWC2_POWER_DOWN_PARAM_HIBERNATION:=0A=
>   		if (hsotg->hw_params.hibernation)=0A=
>   			break;=0A=
>   		dev_dbg(hsotg->dev,=0A=
>   			"Hibernation isn't supported by HW\n");=0A=
> -		param =3D DWC2_POWER_DOWN_PARAM_NONE;=0A=
> +		param =3D DWC2_POWER_DOWN_PARAM_CLOCK_GATING;=0A=
>   		break;=0A=
>   	default:=0A=
>   		dev_err(hsotg->dev,=0A=
>   			"%s: Invalid parameter power_down=3D%d\n",=0A=
>   			__func__, param);=0A=
> -		param =3D DWC2_POWER_DOWN_PARAM_NONE;=0A=
> +		param =3D DWC2_POWER_DOWN_PARAM_CLOCK_GATING;=0A=
>   		break;=0A=
>   	}=0A=
>   =0A=
> diff --git a/drivers/usb/dwc2/platform.c b/drivers/usb/dwc2/platform.c=0A=
> index c8f18f3ba9e3..7bd8fb6c1378 100644=0A=
> --- a/drivers/usb/dwc2/platform.c=0A=
> +++ b/drivers/usb/dwc2/platform.c=0A=
> @@ -342,7 +342,7 @@ static int dwc2_driver_remove(struct platform_device =
*dev)=0A=
>   	}=0A=
>   =0A=
>   	/* Exit clock gating when driver is removed. */=0A=
> -	if (hsotg->params.power_down =3D=3D DWC2_POWER_DOWN_PARAM_NONE &&=0A=
> +	if (hsotg->params.power_down =3D=3D DWC2_POWER_DOWN_PARAM_CLOCK_GATING =
&&=0A=
>   	    hsotg->bus_suspended) {=0A=
>   		if (dwc2_is_device_mode(hsotg))=0A=
>   			dwc2_gadget_exit_clock_gating(hsotg, 0);=0A=
> =0A=
=0A=
In function dwc2_port_resume() you didn't rename the =0A=
"DWC2_POWER_DOWN_PARAM_NONE" to "DWC2_POWER_DOWN_PARAM_CLOCK_GATING" and =
=0A=
also didn't add a case for "DWC2_POWER_DOWN_PARAM_NONE".=0A=
=0A=
In any case when we tested the patchset we encountered a problem when =0A=
using an external HUB connection to root HUB.=0A=
=0A=
If there is no device connected to the external HUB bus will be =0A=
suspended by the autosuspend. If this situation by connecting any device =
=0A=
to the external hub ports we don't see dwc2_port_resume() function call =0A=
and device remains not functional.=0A=
=0A=
Could you please perform the same test on your side and let us know the =0A=
results?=0A=
=0A=
Regards,=0A=
Artur=0A=