Re: [PATCH] usb: typec: ucsi: Handle connectors already attached at init
William Bright <[email protected]>
| Newsgroups | org.kernel.vger.linux-usb,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <aoLmG1nTBJ8bd7ok@will-Legion-Slim-5-16APH8> |
On Sun, Aug 16, 2026 at 03:35:59PM +0200, Christian A. Ehrhardt wrote: > > Hi William, > > On Thu, Aug 13, 2026 at 10:38:40AM +0100, William Bright wrote: > > After a cold boot the PPM may already have a partner attached without a > > pending connector change to report. On a CYPD6125 the CCI register reads > > back 0 in ucsi_init(), so ucsi_connector_change() is never called and the > > Type-C mux is left unconfigured for the already-attached port. > > > > Fix this by additionally checking the connector status of every > > connector and call ucsi_connector_change() for connectors that are > > already attached. Whilst also continuing to call ucsi_connector_change() > > for connectors that report a change via the CCI register. > > I'm not sure if this is the right thing to do. > > First, there is code in ucsi_register_port() that is supposed to > handle the case where a connection is already present. I wonder > why that does not work for you? > Hi Christian, Thanks for your feedback, I had another go this morning at trying to reproduce the issue I was seeing last week where I was just seeing only high-speed on the host instead of superspeed and I am struggling to reproduce it even when doing a load of cold-boots. Can we drop this patch-series for now? I need to find a more reliable way of reproducing this issue and if I manage to then I will need to come up with a fix that is on the ucsi_register_port() path as you suggested. Regards Will