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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.