From nobody Fri Dec 19 16:18:25 2025 Received: from mail-pl1-f176.google.com (mail-pl1-f176.google.com [209.85.214.176]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id CCDBB398FBA for ; Fri, 5 Dec 2025 05:50:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.176 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764913816; cv=none; b=GFzqEI9F5QgUo8g0CExDYb/RRgaDK9HTbEHGzgwP6C2wU9d6ttnRLrZceBq7g9Er/jDNaB3x0+IECiQ6qgTdKpmz+/7/RVGnFAyx5zsLhrnzQIrBYgUiL8av7qu2MNCkiXrLpWKVncNUPn9c+62uhXSHuusRpZCttI+CpJ/WTS8= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764913816; c=relaxed/simple; bh=I5y1anxpbGUPuI7+N6DhRy5eW8OvaQHyYusJeN62NX4=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:To:Cc; b=EuAr2DZKyf+OlRqKtboVNkRhYkqKIWMREbwy2pdToNyiBnBBOhBsuoBlQZfBZHx9X3jOX7ibRqYuIABFhwrTCV500pOVsdrwnG+sklGCUimmQTJ+drPkz9+a0kSbLMu+CMldpkBjQK7NFPQ5nLSlckS6g6QlUD3nUK/R59JzV0E= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=chromium.org; spf=pass smtp.mailfrom=chromium.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b=FASgR0Rw; arc=none smtp.client-ip=209.85.214.176 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=chromium.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=chromium.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b="FASgR0Rw" Received: by mail-pl1-f176.google.com with SMTP id d9443c01a7336-297f35be2ffso26715745ad.2 for ; Thu, 04 Dec 2025 21:50:14 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; t=1764913814; x=1765518614; darn=vger.kernel.org; h=cc:to:message-id:content-transfer-encoding:mime-version:subject :date:from:from:to:cc:subject:date:message-id:reply-to; bh=8AUVbX1TOmwb16DX97d8X9EDhxuQuPh2onLfTtVa/1M=; b=FASgR0Rwvk6R554Ab7yKGIjByzXSkKqr4sS/ZymaadJG1B7F/mNNOh8tdnue8BU5RI 9cxVpFSBaD1xUeQTgrSO2YPk4rWPoMK6oz81ZpXuJp2+IbGpqBhgKQgNtAAim911TVhy 5VMbhORTts3KLng69FCpW8J9piyTark90A9WQ= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1764913814; x=1765518614; h=cc:to:message-id:content-transfer-encoding:mime-version:subject :date:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=8AUVbX1TOmwb16DX97d8X9EDhxuQuPh2onLfTtVa/1M=; b=uUYrsQJhyLYDhmuGAc1NEQIzksC47rHhCORrjw15GMg8MTTZEKKY1jVzq9FO7KcRXq OMgiFNxjAARt4GWHmeQGTXU3W6sE5BEXWvo6pQ35lFzbGFtgER8kmNmfBPa0vNMIAZsv 9ItzTBEPecyEfucd7kW8/NuT79xKTxRWOfy7bMLZOhk+8KH8UjDaJ9pyoq+63Ifo5+ej FHJCP8Ibj4+Z3z8yABB5gtrHSPnjApUkuBVZvbPxmtkIoqikSwcTNjCs9ahHXu4fnOCA SR4N1OrSUUrhGLuKXXjSkp3Mo42Hu4ntWS4yxW5apWRnH1nx0U0iVDaRTO54gspUMVL9 9KJw== X-Forwarded-Encrypted: i=1; AJvYcCUsZm5vEHCU5rJcnzqV95tC72qj0Xre6iew6+7ORLWJzesGBUpTu8b+ehxJGL2XKorhPvSB87KZY3mwe7U=@vger.kernel.org X-Gm-Message-State: AOJu0YwZZMcVldtXrlwunPNaZ+15q4eDU0ZlFUyaYWuo15XZW1d32yhT Uc4QZqNBwUXOUvy2xOtwCcdQWkNeMNsx+08Jf62MTztK30Ok1hoiNXoJFgF8p5BapA== X-Gm-Gg: ASbGncsTCNiGY/FbqvFAy9JvPwf7HVLEPb9ctlVU8kyqpJeINnHeIdCoTk/GRv6AuRZ HsHEEr/GDSQMU6qdnYo1yRU4RvCWTvf0+ZneJ/RP+xQsfHppmWQAA7cWEOubTInBx+UReM1G/N9 QOuEgQQehQ7JHELnT41L7/rFZeH7bPnyuyRVSWdHISS5r5wBkscpJrAymvMTKEObiOthJ/TmMmO dfqBBgHlXQJwA6qtOCW2Dc66axDvi6pnuvpyaHRDovzM8xmm3XBI8QlDq9RaylDQdF0Yj0l1ncn 6tI1nDiLdGoCmi9LArMM88ULftsy6WBDj0NsJ0iAGgoJGp5MUcH81skiIf08rxkzKEbDYBaBb9g ymaRCadIp5P7hLWI7x1LYqLaD9Wu0A+xKYtgoElyoydV1jDPmv2P1XVGOGFlo0AMXDV61OogJni eyzDkRfwE9YrsyrdraM98HqRDJrxj7LROwGj3KOicdxI2gUgb9Qcs+TqajtTq8ip+ptvaEDEipk sNVyPO/jEBG X-Google-Smtp-Source: AGHT+IFnigvKzzQUuNFalavhwjtmJL5Oxu5Fg6CyLssv2l+bE0VS3DXPfhB7aTEK96V7/ZhObRhJtQ== X-Received: by 2002:a17:903:1b50:b0:295:2276:6704 with SMTP id d9443c01a7336-29d683d8d2emr93303425ad.51.1764913814114; Thu, 04 Dec 2025 21:50:14 -0800 (PST) Received: from yuanhsinte-p620-1.tpe.corp.google.com ([2a00:79e0:201d:8:fc0:6b2a:7c91:70fd]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-29daeae6d75sm36621985ad.98.2025.12.04.21.50.12 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 04 Dec 2025 21:50:13 -0800 (PST) From: Hsin-Te Yuan Date: Fri, 05 Dec 2025 13:50:10 +0800 Subject: [PATCH v5] usb: typec: ucsi: Get connector status after enable notifications Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20251205-ucsi-v5-1-488eb89bc9b8@chromium.org> X-B4-Tracking: v=1; b=H4sIAJFyMmkC/23MTQ6DIBCG4asY1rVh+FHaVe/RdIEDKgu1gUraG O9eNC6MMaw+Ms87kWC9s4Hcs4l4G11wQ5+GvGQEW903NncmbcIokwBQ5iMGlyMztVVIaWFKkk7 f3tbuu2aer7RbFz6D/63VCMvvIRAhT89ghdLqQrDqga0fOjd218E3ZGlEtndqcyw5w0EYSTnnl p04vnMMNseTq4CKEjUvoVInTuyd3JxITuFNyEIZo7U8uHme/6Mx76hDAQAA X-Change-ID: 20251117-ucsi-c2dfe8c006d7 To: Heikki Krogerus , Greg Kroah-Hartman , Guenter Roeck Cc: linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, Hsin-Te Yuan X-Mailer: b4 0.14.2 Originally, the notification for connector change will be enabled after the first read of the connector status. Therefore, if the event happens during this window, it will be missing and make the status unsynced. Get the connector status only after enabling the notification for connector change to ensure the status is synced. Fixes: c1b0bc2dabfa ("usb: typec: Add support for UCSI interface") Signed-off-by: Hsin-Te Yuan --- Changes in v5: - Hold the lock of each connector during the initialization to avoid race condition between initialization and other event handler - Add Fixes tag - Link to v4: https://lore.kernel.org/r/20251125-ucsi-v4-1-8c94568ddaa5@chr= omium.org Changes in v4: - Handle a single connector in ucsi_init_port() and call it in a loop - Link to v3: https://lore.kernel.org/r/20251121-ucsi-v3-1-b1047ca371b8@chr= omium.org Changes in v3: - Seperate the status checking part into a new function called ucsi_init_port() and call it after enabling the notifications - Link to v2: https://lore.kernel.org/r/20251118-ucsi-v2-1-d314d50333e2@chr= omium.org Changes in v2: - Remove unnecessary braces. - Link to v1: https://lore.kernel.org/r/20251117-ucsi-v1-1-1dcbc5ea642b@chr= omium.org --- drivers/usb/typec/ucsi/ucsi.c | 126 +++++++++++++++++++++++---------------= ---- 1 file changed, 69 insertions(+), 57 deletions(-) diff --git a/drivers/usb/typec/ucsi/ucsi.c b/drivers/usb/typec/ucsi/ucsi.c index 3f568f790f39b0271667e80816270274b8dd3008..9073b005c5ba92fcb6ed1fdf7dd= 65090832c7058 100644 --- a/drivers/usb/typec/ucsi/ucsi.c +++ b/drivers/usb/typec/ucsi/ucsi.c @@ -1560,11 +1560,70 @@ static struct fwnode_handle *ucsi_find_fwnode(struc= t ucsi_connector *con) return NULL; } =20 +static void ucsi_init_port(struct ucsi *ucsi, struct ucsi_connector *con) +{ + enum usb_role u_role =3D USB_ROLE_NONE; + int ret; + + /* Get the status */ + ret =3D ucsi_get_connector_status(con, false); + if (ret) { + dev_err(ucsi->dev, "con%d: failed to get status\n", con->num); + return; + } + + if (ucsi->ops->connector_status) + ucsi->ops->connector_status(con); + + switch (UCSI_CONSTAT(con, PARTNER_TYPE)) { + case UCSI_CONSTAT_PARTNER_TYPE_UFP: + case UCSI_CONSTAT_PARTNER_TYPE_CABLE_AND_UFP: + u_role =3D USB_ROLE_HOST; + fallthrough; + case UCSI_CONSTAT_PARTNER_TYPE_CABLE: + typec_set_data_role(con->port, TYPEC_HOST); + break; + case UCSI_CONSTAT_PARTNER_TYPE_DFP: + u_role =3D USB_ROLE_DEVICE; + typec_set_data_role(con->port, TYPEC_DEVICE); + break; + default: + break; + } + + /* Check if there is already something connected */ + if (UCSI_CONSTAT(con, CONNECTED)) { + typec_set_pwr_role(con->port, UCSI_CONSTAT(con, PWR_DIR)); + ucsi_register_partner(con); + ucsi_pwr_opmode_change(con); + ucsi_port_psy_changed(con); + if (con->ucsi->cap.features & UCSI_CAP_GET_PD_MESSAGE) + ucsi_get_partner_identity(con); + if (con->ucsi->cap.features & UCSI_CAP_CABLE_DETAILS) + ucsi_check_cable(con); + } + + /* Only notify USB controller if partner supports USB data */ + if (!(UCSI_CONSTAT(con, PARTNER_FLAG_USB))) + u_role =3D USB_ROLE_NONE; + + ret =3D usb_role_switch_set_role(con->usb_role_sw, u_role); + if (ret) + dev_err(ucsi->dev, "con:%d: failed to set usb role:%d\n", + con->num, u_role); + + if (con->partner && UCSI_CONSTAT(con, PWR_OPMODE) =3D=3D UCSI_CONSTAT_PWR= _OPMODE_PD) { + ucsi_register_device_pdos(con); + ucsi_get_src_pdos(con); + ucsi_check_altmodes(con); + ucsi_check_connector_capability(con); + } +} + static int ucsi_register_port(struct ucsi *ucsi, struct ucsi_connector *co= n) { struct typec_capability *cap =3D &con->typec_cap; enum typec_accessory *accessory =3D cap->accessory; - enum usb_role u_role =3D USB_ROLE_NONE; u64 command; char *name; int ret; @@ -1659,62 +1718,6 @@ static int ucsi_register_port(struct ucsi *ucsi, str= uct ucsi_connector *con) goto out; } =20 - /* Get the status */ - ret =3D ucsi_get_connector_status(con, false); - if (ret) { - dev_err(ucsi->dev, "con%d: failed to get status\n", con->num); - goto out; - } - - if (ucsi->ops->connector_status) - ucsi->ops->connector_status(con); - - switch (UCSI_CONSTAT(con, PARTNER_TYPE)) { - case UCSI_CONSTAT_PARTNER_TYPE_UFP: - case UCSI_CONSTAT_PARTNER_TYPE_CABLE_AND_UFP: - u_role =3D USB_ROLE_HOST; - fallthrough; - case UCSI_CONSTAT_PARTNER_TYPE_CABLE: - typec_set_data_role(con->port, TYPEC_HOST); - break; - case UCSI_CONSTAT_PARTNER_TYPE_DFP: - u_role =3D USB_ROLE_DEVICE; - typec_set_data_role(con->port, TYPEC_DEVICE); - break; - default: - break; - } - - /* Check if there is already something connected */ - if (UCSI_CONSTAT(con, CONNECTED)) { - typec_set_pwr_role(con->port, UCSI_CONSTAT(con, PWR_DIR)); - ucsi_register_partner(con); - ucsi_pwr_opmode_change(con); - ucsi_port_psy_changed(con); - if (con->ucsi->cap.features & UCSI_CAP_GET_PD_MESSAGE) - ucsi_get_partner_identity(con); - if (con->ucsi->cap.features & UCSI_CAP_CABLE_DETAILS) - ucsi_check_cable(con); - } - - /* Only notify USB controller if partner supports USB data */ - if (!(UCSI_CONSTAT(con, PARTNER_FLAG_USB))) - u_role =3D USB_ROLE_NONE; - - ret =3D usb_role_switch_set_role(con->usb_role_sw, u_role); - if (ret) { - dev_err(ucsi->dev, "con:%d: failed to set usb role:%d\n", - con->num, u_role); - ret =3D 0; - } - - if (con->partner && UCSI_CONSTAT(con, PWR_OPMODE) =3D=3D UCSI_CONSTAT_PWR= _OPMODE_PD) { - ucsi_register_device_pdos(con); - ucsi_get_src_pdos(con); - ucsi_check_altmodes(con); - ucsi_check_connector_capability(con); - } - trace_ucsi_register_port(con->num, con); =20 out: @@ -1823,6 +1826,10 @@ static int ucsi_init(struct ucsi *ucsi) goto err_unregister; } =20 + /* Delay other interactions with each connector until ucsi_init_port is d= one */ + for (i =3D 0; i < ucsi->cap.num_connectors; i++) + mutex_lock(&connector[i].lock); + /* Enable all supported notifications */ ntfy =3D ucsi_get_supported_notifications(ucsi); command =3D UCSI_SET_NOTIFICATION_ENABLE | ntfy; @@ -1833,6 +1840,11 @@ static int ucsi_init(struct ucsi *ucsi) ucsi->connector =3D connector; ucsi->ntfy =3D ntfy; =20 + for (i =3D 0; i < ucsi->cap.num_connectors; i++) { + ucsi_init_port(ucsi, &connector[i]); + mutex_unlock(&connector[i].lock); + } + mutex_lock(&ucsi->ppm_lock); ret =3D ucsi->ops->read_cci(ucsi, &cci); mutex_unlock(&ucsi->ppm_lock); --- base-commit: 2061f18ad76ecaddf8ed17df81b8611ea88dbddd change-id: 20251117-ucsi-c2dfe8c006d7 Best regards, --=20 Hsin-Te Yuan