On Thu, 28 Mar 2024 at 00:46, Christian A. Ehrhardt <lk@c--e.de> wrote:
>
> As briefly discussed here
> https://lore.kernel.org/lkml/Zf1XUrG1UbVJWzoz@kuha.fi.intel.com/
> acknowledge connector change events along with the first command
> in ucsi_handle_connector_change(). The connector lock should be
> sufficient to protect the rest of the function and the partner
> tasks.
>
> This allows us to remove the Dell quirk in ucsi_acpi.c.
> Additionally, this reduces the number of commands that are sent
> with an un-acknowleged connector change event.
>
> Christian A. Ehrhardt (3):
> usb: typec: ucsi: Stop abuse of bit definitions from ucsi.h
> usb: typec: ucsi: Never send a lone connector change ack
> usb: typec: ucsi_acpi: Remove Dell quirk
>
> drivers/usb/typec/ucsi/ucsi.c | 48 ++++++++++-------------
> drivers/usb/typec/ucsi/ucsi.h | 2 -
> drivers/usb/typec/ucsi/ucsi_acpi.c | 56 ++-------------------------
> drivers/usb/typec/ucsi/ucsi_stm32g0.c | 1 +
> 4 files changed, 25 insertions(+), 82 deletions(-)
Tested-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> #
SM8450-HDK, sc8180x-primus
--
With best wishes
Dmitry