Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Xu Yang <xu.yang_2@nxp.com>
To:     Luca Ceresoli <luca.ceresoli@bootlin.com>,
        Francesco Dolcini <francesco@dolcini.it>
CC:     Jun Li <jun.li@nxp.com>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "festevam@gmail.com" <festevam@gmail.com>,
        "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
        "kernel@pengutronix.de" <kernel@pengutronix.de>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        dl-linux-imx <linux-imx@nxp.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "s.hauer@pengutronix.de" <s.hauer@pengutronix.de>,
        "shawnguo@kernel.org" <shawnguo@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        Francesco Dolcini <francesco.dolcini@toradex.com>
Subject: RE: [EXT] Re: [PATCH v2 2/3] usb: chipidea: imx: support disabling
 runtime-pm
Thread-Topic: [EXT] Re: [PATCH v2 2/3] usb: chipidea: imx: support disabling
 runtime-pm
Thread-Index: AQHZkukSER6EMlEOvUGUKvCWQIHv2K+swg6AgBG0WYCAAQQm4A==
Date:   Tue, 18 Jul 2023 08:31:48 +0000
Message-ID: <DB7PR04MB450573F8EA936E049F053A258C38A@DB7PR04MB4505.eurprd04.prod.outlook.com>
References: <20230504162312.1506763-1-luca.ceresoli@bootlin.com>
        <ZFPiRvoF5l8uGzzZ@francesco-nb.int.toradex.com>
        <PA4PR04MB96403377F5E37C12AD8C25B389729@PA4PR04MB9640.eurprd04.prod.outlook.com>
        <20230505120618.2f4cf22c@booty>
        <ZFThyn/D/dDK9nk3@francesco-nb.int.toradex.com>
        <PA4PR04MB96405EE2468555EA900B340189739@PA4PR04MB9640.eurprd04.prod.outlook.com>
        <ZFjaNzY32x8o2XG7@francesco-nb.int.toradex.com>
        <20230508151756.785ec07e@booty> <20230529121825.71e9b6d6@booty>
        <PA4PR04MB96405138465D215C34285F02894B9@PA4PR04MB9640.eurprd04.prod.outlook.com>
        <ZKaWL3+ClI7iNr/4@francesco-nb.int.toradex.com>
 <20230717184537.6d6ed607@booty>
In-Reply-To: <20230717184537.6d6ed607@booty>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?o2HOsGGGy8DKIlR2q9Y4vPZpUQyOeC/vLUfQgYPdSkzopmH9vaXmBlYi42Cl?=
 =?us-ascii?Q?zGVrTMRXJJnCy26+Bz0WBUUOU5nnm03UN1D/wkJm9Fp94FDDx6sTYLWDQrrh?=
 =?us-ascii?Q?vUMpc19N0b01SUNd9ZfeTqoJWoi5m4yh2RGFIQic3019UCZf6hga6If8Yh1Q?=
 =?us-ascii?Q?o3Z5000pC0ONifQK5SgJbwbBUAjDSGyldpOLVxQsj44rxN6bHQG/s15HffYO?=
 =?us-ascii?Q?9VOn8cOAsUizhQEbZ4DdId5WU1OEeGvDXAcRI16UyOtLcmx3MlTe0tji6doJ?=
 =?us-ascii?Q?DHtJWWUtPiksEyKDEUNeJ3HvKskj9W3olHblV6dtehoNpGlyIaP4iaiwL6L8?=
 =?us-ascii?Q?o3UewyHpIXzRxUKWMApoDfn5cRLl8sIKmbRTr1oaYoX/y6FmCHkdN8hExrcK?=
 =?us-ascii?Q?kZeGwSeI7LSpfWPTXDyzqcOSNlb3kUkLeeeNXc/QaOKBly4OLi4PdH45959v?=
 =?us-ascii?Q?CjLMWv6xHAB5wMoXT/EmMcdjjb8hJv4GlsM6q4jQcGH/5wx580n+86rFSEvl?=
 =?us-ascii?Q?EgPc0Awm660sYm4ZSZP8+b9+SQN8CVVm9C8NbIrojx6ylah1uYJVM8Mpero+?=
 =?us-ascii?Q?2xg1sGnN6EZC5Uuw3Xflf5rFpb8t8evlSP1xXDHoe47h8XxFQ7JZgWTky8L7?=
 =?us-ascii?Q?WRMZ0tKRZ6nKTYsO/IdzOEneAW+36zl+8er+FjawNNNzaFawoPotQofq/tNG?=
 =?us-ascii?Q?WzFLKGWQuKCbBe/JtRMTChgpxFZp3p2v3meS9w9017yo6G0joeIrU3agHqFF?=
 =?us-ascii?Q?ZlwwHnNmG1XxLho33N0k2k7vMlCQmAiieGIQQrZVKKQz9Ldd8LAV+sHMuZYQ?=
 =?us-ascii?Q?IdKIJtV2egEnZu+yHyXQGPKMPNBbNmq+nvAkAvki6RXXDEdSNMUrAVr0qmeu?=
 =?us-ascii?Q?2N/4hNOOd+79i3uJQLImeEycFi7CPWaNihJFdlNQ9+xSH5nMrVMBAufnGs6D?=
 =?us-ascii?Q?6yL/hRb47EeQckM9bcI7oMawmVxUxctAb0SheIItmHsq/6bClKeHTpKqopKe?=
 =?us-ascii?Q?o4yUtVxAxTUp7IC9CiOV0CsuuBGbMuX/bBmeJpN1F9z4dpspBJiIpQZyGyJW?=
 =?us-ascii?Q?Yh+zDro2pxvWPqh/ZluadlXagNhQR7F1SPabDGTmPKUnriFr4X6r50bM8T7B?=
 =?us-ascii?Q?/crLsLPj3esrIMlZJqwq7yirnpAOveFZB7Y8M1bQQqRLaXiVcap07OR3/ZoQ?=
 =?us-ascii?Q?GnpnuqmT5dESiHp6bFAYiAMjpYhcbdQXpqZ089sB4rZbwefv6Re0Djkf0ru1?=
 =?us-ascii?Q?kM2wbHIctDhOmIVZEuOJRLdNW+XeDlueExnHInhYuohRzkvUmCROp0/G7F95?=
 =?us-ascii?Q?uLUb2ZZSETrMwy/w/QseLL4lQuWHaILLFzbeharf6vp+QnPiJ2OE7NySv928?=
 =?us-ascii?Q?wxh8RPHcf8NaThgDclRSjpgqV2QeFzvz387g4TnJsfglCUwAoDi+8ULoZiZL?=
 =?us-ascii?Q?8VsQfrOjBfwHqu9kV+lY7lmxQq6l7FqPsjLdATkeoPIIY/F0rNcdvLum4GAP?=
 =?us-ascii?Q?pa52NMUY2RHsmjCvowAEivl+FDEWzKrZIo8PP1tS7kWPrtF1CI9Gly6OGy7R?=
 =?us-ascii?Q?F4AtdRWDtA11MfLQxok=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DB7PR04MB4505.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 305beb94-a709-422c-3ce3-08db87696da7
X-MS-Exchange-CrossTenant-originalarrivaltime: 18 Jul 2023 08:31:48.3876
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: KPgj9e6bSpBM2SRp4ZgrzWlUNzCClmNlVAMynffXQjw3YwUFGTWVCiD2EvzBGY7ioJD4DQka9yIPH5qYgypu7A==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR04MB8881
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE,
        URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


> -----Original Message-----
>=20
> Ciao Francesco,
>=20
> On Thu, 6 Jul 2023 12:23:43 +0200
> Francesco Dolcini <francesco@dolcini.it> wrote:
>=20
> > Hello Luca,
> >
> > On Tue, May 30, 2023 at 11:22:51AM +0000, Jun Li wrote:
> > > Yes, your understanding is correct, talked with Xu(in CC), he will ta=
ke this
> > > soon.
> >
> > A series was posted
> >
> > I had no time to try or look at it yet.
>=20
> Thanks for keeping me up to date on this topic, which is still totally
> relevant to me.
>=20
> I looked at the series, but it does not seem to be addressing the
> problem with USB host not detecting new devices when VBUS is not
> directly connected, e.g. in the Colibri imx6ull SoM.
>=20
> Xu, do you confirm the series at the link is _not_ solving the problem
> being discussed here?

Have you tried this patchset? The upstream driver couldn't get correct
USB role from HW_USBPHY_CTRL register when the ID pin is float. This is
what this patchset is trying to fix. With this patch, condition=20
"(!vbus_is_on && !mxs_phy_is_otg_host(mxs_phy)" will always be false when
controller acts as host role, then __mxs_phy_disconnect_line(phy, true)
will never be called. So I think it doesn't matter whether VBUS is connecte=
d
or not when act as host mode. If you still have issue after apply this patc=
hset,
please let me know.

Thanks,
Xu Yang

>=20
> Best regards,
>=20
> Luca
>=20