Re: [PATCH] usb: typec: ucsi: Handle connectors already attached at init
From: William Bright
Date: Mon Aug 17 2026 - 06:45:00 EST
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