Hi,
The goal of this series was to improve the alt mode handling in the
driver, but now it seems that we can use the "poll worker" that was
introduced for that to handle other tasks better as well.
Ulrich reported some problems that are caused by the second
GET_CONNECTOR_STATUS right after the first one that was introduced in
217504a05532 ("usb: typec: ucsi: Work around PPM losing change
information"). In the last patch I try to improve that workaround by
extracting it out of the generic event handler into its own task and
executing it only when it's really needed. That seems to improve the
situation.
These patches definitely improve the quality of the driver by making
it a bit more readable, but they also appear to make the behaviour a
bit more predictably and uniform on different platforms.
Benjamin, can you test these?
thanks,
Heikki Krogerus (7):
usb: typec: ucsi: Always cancel the command if PPM reports BUSY
condition
usb: typec: ucsi: Don't stop alt mode registration on busy condition
usb: typec: ucsi: Add polling mechanism for partner tasks like alt
mode checking
usb: typec: ucsi: acpi: Reduce the command completion timeout
usb: typec: ucsi: Check the partner alt modes always if there is PD
contract
usb: typec: ucsi: Read the PDOs in separate work
usb: typec: ucsi: Better fix for missing unplug events issue
drivers/usb/typec/ucsi/ucsi.c | 337 ++++++++++++++---------------
drivers/usb/typec/ucsi/ucsi.h | 3 +-
drivers/usb/typec/ucsi/ucsi_acpi.c | 2 +-
3 files changed, 167 insertions(+), 175 deletions(-)
--
2.33.0
