Re: [PATCH v5 5/8] platform/chrome: cros_ec_typec: Update partner altmode active
From: Benson Leung
Date: Wed Dec 18 2024 - 20:30:35 EST
On Fri, Dec 13, 2024 at 03:35:46PM -0800, Abhishek Pandit-Subedi wrote:
> Mux configuration is often the final piece of mode entry and can be used
> to determine whether a partner altmode is active. When mux configuration
> is done, use the active port altmode's SVID to set the partner active
> field for all partner alt modes.
>
> Signed-off-by: Abhishek Pandit-Subedi <abhishekpandit@xxxxxxxxxxxx>
Reviewed-by: Benson Leung <bleung@xxxxxxxxxxxx>
> ---
>
> Changes in v5:
> - Use list_for_each_entry and simplify conditional statement within
>
> drivers/platform/chrome/cros_ec_typec.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/drivers/platform/chrome/cros_ec_typec.c b/drivers/platform/chrome/cros_ec_typec.c
> index ae2f86296954..77f748fc8542 100644
> --- a/drivers/platform/chrome/cros_ec_typec.c
> +++ b/drivers/platform/chrome/cros_ec_typec.c
> @@ -619,6 +619,7 @@ static int cros_typec_configure_mux(struct cros_typec_data *typec, int port_num,
> };
> struct ec_params_usb_pd_mux_ack mux_ack;
> enum typec_orientation orientation;
> + struct cros_typec_altmode_node *node;
> int ret;
>
> ret = cros_ec_cmd(typec->ec, 0, EC_CMD_USB_PD_MUX_INFO,
> @@ -677,6 +678,14 @@ static int cros_typec_configure_mux(struct cros_typec_data *typec, int port_num,
> port->mux_flags);
> }
>
> + /* Iterate all partner alt-modes and set the active alternate mode. */
> + list_for_each_entry(node, &port->partner_mode_list, list) {
> + typec_altmode_update_active(
> + node->amode,
> + port->state.alt &&
> + node->amode->svid == port->state.alt->svid);
> + }
> +
> mux_ack:
> if (!typec->needs_mux_ack)
> return ret;
> --
> 2.47.1.613.gc27f4b7a9f-goog
>
Attachment:
signature.asc
Description: PGP signature