From patchwork Wed Mar 13 03:54:11 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dmitry Baryshkov X-Patchwork-Id: 13590954 Received: from mail-lj1-f169.google.com (mail-lj1-f169.google.com [209.85.208.169]) (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 5F154D52B for ; Wed, 13 Mar 2024 03:54:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.169 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710302066; cv=none; b=U4QmCqkwwDLu1uo4Py4Kqrw6Ly1WSUvkOKJdq4fabM20ugFcZxPJ/o72lhVvbmqZO8F+bfEL0xrHeIv3LWCaMZqvu859O9qnEIPVEueufxh+pDYpXSPDK7pIBKknF1WqF2FvPgxH82PKG280y5IIDSyGTIeZe0JD6l9fCJpRF68= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710302066; c=relaxed/simple; bh=5qUUCTMFfwh0GWLizFk8xctUPdAHmp89Ki8123ktN38=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=k0vcAiY7dB/sLVJNKGR0lP3+qaW0soMpO+NHN3Y7MLxropIvdLiMTIpDVpbkkr1qhXRHMO4ffKEWdeeyhwJIsvvj6vQrs+oPwaGPH0gm0O9X++H4ZV/pPjlShYcCRN1TgVWaa+C1wFxWB0Uap/njx3yNcQ4T4iE/JTySaKWd4ec= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=Vq4sTN5r; arc=none smtp.client-ip=209.85.208.169 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="Vq4sTN5r" Received: by mail-lj1-f169.google.com with SMTP id 38308e7fff4ca-2d228a132acso80456081fa.0 for ; Tue, 12 Mar 2024 20:54:24 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1710302062; x=1710906862; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=RLTPq66GU+FLbpH3nyuA9I8IqdpbGC/dXUxjJTbbTFU=; b=Vq4sTN5resfoKl2ZP0jIXXTxe6kN5nrkl2+af6Nd9sXill94EtJYPjSO7fsRitsmiV f6CvorYT9NujrGwI/6V4vJBc4jRpDHyYWGaw5fUgS9ITYobFPQ8BXMlhLQ6iP09cTUMV NTg88+KUuYj45Qm2ldzBJK9lvWFJ8WLfgOgERsrc3R83nNSygF5h1gXY1pX5KhDx6Auz XUCrzteGKc5oZoH81wGcbXyd3MpC62YrKWWJ6bz3SrNEntu3uKa11D8AepEAXzYYUf5P 1H6a+fVGfQIIV6GbqRtuNMxOwIVUYd3jd9ik0STnHB4jTUtxKxN4vdYk6YrVrsnLCWJ+ iiQg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1710302062; x=1710906862; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=RLTPq66GU+FLbpH3nyuA9I8IqdpbGC/dXUxjJTbbTFU=; b=gmherTwT2UeXoZ1uBmEJxG826La+u4it+JE+AHJACdV0LsffDX1EVMTEl+68i9ynQo kxk7FsowTP7DP9+SKT7Vtv1Gs/EsYhvRcWhclt5XsaUMKrG+h70OCCVA24NYfqqZv0qC CLQgsjvFmSBGyV+IE/4DygTnwCdO8bFf49lWnjGpdp0boihdDHGe++oR+ARRZYqgVsoI 1UvmfYCKk5QnK7hSCRaQ8KOm5oQorPocLr9NiNXrpk1DLTGuCj8aOaRncp8WhblGSAbQ l3RQlTJ80oaLbzvJRNt0ooOof5Xp14JA6GbjMuCsWUYUCSfu3F6PN79XmnpccYmWxUDr cPcQ== X-Forwarded-Encrypted: i=1; AJvYcCWPRHtufS7qKfUml101Stbqb3gS3rTxNg2KaJg0ehQ0JRXHXBqBd/oYjaAsh/Sf9p2IV2hurfxAYousmYJhsEHE6W6RoiuGHOym X-Gm-Message-State: AOJu0YzKXPbCMrXZ5n6H4s+hXPkNos/MYnVOyxzVljOZPy5WoGoFqDvf 2FcfXTDrqAgxLJNcTaP6Dw+J+8qd92gDN2aUWrXcM90xn5A/UeqE+3IZjh+gNvw= X-Google-Smtp-Source: AGHT+IF2gIO8l89DhMPUZnzCGqhK82MhP/4ZMa2FOkGWNOMjWcfpe6eVq0PlHpm1K5Fom8nIThZywQ== X-Received: by 2002:a2e:7812:0:b0:2d4:142:488a with SMTP id t18-20020a2e7812000000b002d40142488amr7666840ljc.8.1710302062209; Tue, 12 Mar 2024 20:54:22 -0700 (PDT) Received: from umbar.lan ([192.130.178.91]) by smtp.gmail.com with ESMTPSA id f25-20020a05651c02d900b002d0acb57c89sm1854319ljo.64.2024.03.12.20.54.20 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 12 Mar 2024 20:54:20 -0700 (PDT) From: Dmitry Baryshkov Date: Wed, 13 Mar 2024 05:54:11 +0200 Subject: [PATCH 1/7] usb: typec: ucsi: fix race condition in connection change ACK'ing Precedence: bulk X-Mailing-List: linux-usb@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Message-Id: <20240313-qcom-ucsi-fixes-v1-1-74d90cb48a00@linaro.org> References: <20240313-qcom-ucsi-fixes-v1-0-74d90cb48a00@linaro.org> In-Reply-To: <20240313-qcom-ucsi-fixes-v1-0-74d90cb48a00@linaro.org> To: Heikki Krogerus , Greg Kroah-Hartman , Guenter Roeck , Bjorn Andersson , Neil Armstrong , Konrad Dybcio Cc: Johan Hovold , linux-usb@vger.kernel.org, linux-arm-msm@vger.kernel.org, stable@vger.kernel.org X-Mailer: b4 0.13.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=2257; i=dmitry.baryshkov@linaro.org; h=from:subject:message-id; bh=5qUUCTMFfwh0GWLizFk8xctUPdAHmp89Ki8123ktN38=; b=owEBbQGS/pANAwAKAYs8ij4CKSjVAcsmYgBl8SNqd16wpFUFHQFpmCrx+aM4x5e+9Cbg/SwXy EdVo+IYGxKJATMEAAEKAB0WIQRMcISVXLJjVvC4lX+LPIo+Aiko1QUCZfEjagAKCRCLPIo+Aiko 1UjFB/4vxPjvb5YNIJgzQJVuz/WdW4MrD6GzN/1jhebh/IPO9C/nP43s77XTsN73+OTlfVNgKj7 KRzqBz1ESGVxQQxhe/2KfA1S5ijpxK+egNcbul3gw/oqD+nGfIZhPoHp7k6GhiX8akiKxqn27LC iJm920Vq03Of8/X5T8XaKKqmaZ26hpUvv9x0bkBWszW0edyW/Nbyj4Ad4nUiQaFlmVS/e7RpQcK Z7HDoO+hI6Oj3xqrjlFHLVrxkw2ozF+vWnWoYPI9QENjgV58QiyN1B+zRyyTNVaIXyrZkuhqa2l FS4ARb4RRh7dauequb0u/GYKVmA1E3ZUQZmDS9/GMRhM9S1M X-Developer-Key: i=dmitry.baryshkov@linaro.org; a=openpgp; fpr=8F88381DD5C873E4AE487DA5199BF1243632046A The code to handle connection change events contains a race: there is an open window for notifications to arrive between clearing EVENT_PENDING bit and sending the ACK_CC_CI command to acknowledge the connection change. This is mostly not an issue, but on Qualcomm platforms when the PPM receives ACK_CC_CI with the ConnectorChange bit set if there is no pending reported Connector Change, it responds with the CommandCompleted + NotSupported notifications, completely breaking UCSI state machine. Fix this by reading out CCI after ACK_CC_CI and scheduling the work if there is a connector change reported. Fixes: bdc62f2bae8f ("usb: typec: ucsi: Simplified registration and I/O API") Cc: stable@vger.kernel.org Signed-off-by: Dmitry Baryshkov Reviewed-by: Heikki Krogerus --- drivers/usb/typec/ucsi/ucsi.c | 20 +++++++++++++++++--- 1 file changed, 17 insertions(+), 3 deletions(-) diff --git a/drivers/usb/typec/ucsi/ucsi.c b/drivers/usb/typec/ucsi/ucsi.c index cf52cb34d285..4abb752c6806 100644 --- a/drivers/usb/typec/ucsi/ucsi.c +++ b/drivers/usb/typec/ucsi/ucsi.c @@ -61,12 +61,28 @@ static int ucsi_acknowledge_command(struct ucsi *ucsi) static int ucsi_acknowledge_connector_change(struct ucsi *ucsi) { + unsigned int con_num; u64 ctrl; + u32 cci; + int ret; ctrl = UCSI_ACK_CC_CI; ctrl |= UCSI_ACK_CONNECTOR_CHANGE; - return ucsi->ops->sync_write(ucsi, UCSI_CONTROL, &ctrl, sizeof(ctrl)); + ret = ucsi->ops->sync_write(ucsi, UCSI_CONTROL, &ctrl, sizeof(ctrl)); + if (ret) + return ret; + + clear_bit(EVENT_PENDING, &ucsi->flags); + ret = ucsi->ops->read(ucsi, UCSI_CCI, &cci, sizeof(cci)); + if (ret) + return ret; + + con_num = UCSI_CCI_CONNECTOR(cci); + if (con_num) + ucsi_connector_change(ucsi, con_num); + + return 0; } static int ucsi_exec_command(struct ucsi *ucsi, u64 command); @@ -1215,8 +1231,6 @@ static void ucsi_handle_connector_change(struct work_struct *work) if (con->status.change & UCSI_CONSTAT_CAM_CHANGE) ucsi_partner_task(con, ucsi_check_altmodes, 1, 0); - clear_bit(EVENT_PENDING, &con->ucsi->flags); - mutex_lock(&ucsi->ppm_lock); ret = ucsi_acknowledge_connector_change(ucsi); mutex_unlock(&ucsi->ppm_lock); From patchwork Wed Mar 13 03:54:12 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dmitry Baryshkov X-Patchwork-Id: 13590956 Received: from mail-lj1-f176.google.com (mail-lj1-f176.google.com [209.85.208.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 5E7B410A2A for ; Wed, 13 Mar 2024 03:54:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.176 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710302068; cv=none; b=QJhP8DM/XUuXNQMS9DUllePoBIEROc/i7kUPgL5HdRF0a6wCDwcyq/EIeV+0yAptOwe8OXojkcF0e5UfhjWiZs7+grQKEeueGwYHLTixzfxM8u5K+uwjfa5pTxAimTgq0Vv4OizG5fXE3fLYcI76JYCXjnrCFs/mugMIUjYMG0U= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710302068; c=relaxed/simple; bh=b4TOdcAfpKP7c8TtSew/74QZD84OuH+/13I9UVvWj6I=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=EEnyhp1rTn9jK8rvbXNkTrYb4chiL9g5JQGlSRxdBe306TLDZ4p/67+wNNSdbzwhJeOZ2Cbnmdri5DG1GmPCtvYg8nbuWEVBCLq3JSLOVu2bW9I+hzZK1KN1yo7GAp7mbRBDC7y+Z1/GjoB864CD1hnWZC9y7neqkrnFhdFxKYM= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=N//dqGSF; arc=none smtp.client-ip=209.85.208.176 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="N//dqGSF" Received: by mail-lj1-f176.google.com with SMTP id 38308e7fff4ca-2d09cf00214so71636561fa.0 for ; Tue, 12 Mar 2024 20:54:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1710302063; x=1710906863; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=gFntvUmfQrQoD9JhHP4a9IiBrKarY81qZUG6+LdJaJU=; b=N//dqGSFTGZvAunpZjEUrPCDR0QAy+0xIvChTBMKfVysAt17fMBVWca1U/DE6hmJFo AZE7RnQOrdAiy1nfH43OuJBJ3loRlKATQS+f3JoY9Ct07oJQ6ACDV54vwx2Ql66uoNHM 27t3+kD7wxgrwF4lL8emztXUM+CY8mnXeHuA3sWunwWVeNp5nR998/WaXMN7IHjPWd/b ACbQOG2tQVVLeBabOfoDjpLyb2du6+7d//2Ltzpw+/5ZPJcQZX3I+DaBfbBZBWm8i9mG iCsGjea1ADQWIO7ehcjkQ06irnS7aw3iQcw1mWKVKbinon62i2gXiyQNrhM7wJVoGfFd Ujrw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1710302063; x=1710906863; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=gFntvUmfQrQoD9JhHP4a9IiBrKarY81qZUG6+LdJaJU=; b=abRxnT38MGjqKyzEpK5Wc36z3/g5KhxFfB0J0gUAQmpe0M6cqU7blanMWac6O26sQn OKelp1sBbR12+Hq/Efl2NtmfjADfnbJDVDN5UPSEH6aCN7t+QP742jIn5y4AwLIXx5OC Q6k2PLdrrROnEJdVBkNlzB/jhv4z0+nYTW/aINtZ0N/aN/7DKLGb8rhEbAZ434obg3sz 9yk63HM1PXnwzc4ylpx+15ET2BB4lUnX52eXyLrfdRMzHBZ1xuSP437G5FMuUIjuI7V2 kdO6frI8Cq9KlnSnb4+Qx+PObxQwda+aeyXy7PIezZs2H+LwdTT8x6WGtgiY1sQ3t3i0 e/fw== X-Forwarded-Encrypted: i=1; AJvYcCVW4U0hV0QBPi2CqCjsEzOJg+FCgBU01ZyjohV/SwT7km+NfmDIx+yiodLsbU5QXPVuwcNQdZzIGGMVAVJEkHSteTQSCQ53rGF2 X-Gm-Message-State: AOJu0YwUAh/6DE+TqdR0eFe2qgKikEyH9KouPuRuQJj2KOBgQ4+4QmBY PDFMg1MdhH3xi8cbAQVuuuYdzwBTijJUR0Nali3r7Y2Ex3Hy9cqF9NV7I0DrMjc= X-Google-Smtp-Source: AGHT+IF7ZptzPxSTf58/lAJjw4Oo4QhbXQNkZBdrDCZziYkkXuk84yKwMDenXbJyp736hIssvaOTAQ== X-Received: by 2002:a2e:8e3c:0:b0:2d3:fca:dae8 with SMTP id r28-20020a2e8e3c000000b002d30fcadae8mr1200502ljk.16.1710302063642; Tue, 12 Mar 2024 20:54:23 -0700 (PDT) Received: from umbar.lan ([192.130.178.91]) by smtp.gmail.com with ESMTPSA id f25-20020a05651c02d900b002d0acb57c89sm1854319ljo.64.2024.03.12.20.54.22 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 12 Mar 2024 20:54:22 -0700 (PDT) From: Dmitry Baryshkov Date: Wed, 13 Mar 2024 05:54:12 +0200 Subject: [PATCH 2/7] usb: typec: ucsi: acknowledge the UCSI_CCI_NOT_SUPPORTED Precedence: bulk X-Mailing-List: linux-usb@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Message-Id: <20240313-qcom-ucsi-fixes-v1-2-74d90cb48a00@linaro.org> References: <20240313-qcom-ucsi-fixes-v1-0-74d90cb48a00@linaro.org> In-Reply-To: <20240313-qcom-ucsi-fixes-v1-0-74d90cb48a00@linaro.org> To: Heikki Krogerus , Greg Kroah-Hartman , Guenter Roeck , Bjorn Andersson , Neil Armstrong , Konrad Dybcio Cc: Johan Hovold , linux-usb@vger.kernel.org, linux-arm-msm@vger.kernel.org, stable@vger.kernel.org X-Mailer: b4 0.13.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=1127; i=dmitry.baryshkov@linaro.org; h=from:subject:message-id; bh=b4TOdcAfpKP7c8TtSew/74QZD84OuH+/13I9UVvWj6I=; b=owEBbQGS/pANAwAKAYs8ij4CKSjVAcsmYgBl8SNqeJcoJB/Iq7ScN3gQPd6nqFh0TtyF56Qo1 NIzkt2f3CaJATMEAAEKAB0WIQRMcISVXLJjVvC4lX+LPIo+Aiko1QUCZfEjagAKCRCLPIo+Aiko 1a3NCACwe1UaBtfp8zYIFp97vnTMJ/Rftih1EHQpFoHrXfy+q8GjM7W55FJWQ0tAJYAHneww/11 rW0r6zoOWO0hGH6L9wScIb2k5fsy8RgV7e5FOj/Yu8QMY/f0FYoQl15l8AUfhF8BWYhgWll8tf/ ipldJqj9i7EgaY0Qr2FdwICG5H2Y2LePyZRb5HljYaW4ABjjn6ojloi0+4f9nExQG7K9bdOkYMb q3EwJ6XFQlbJCSnMjN2nAp+f5lYuWmFria6qyf/FXgkYqO0NJSZ+cETuzQ9B4MG7uYbNwehvcII vndULkoE+/G27inrpaRG3Q2Nyyn9uNwE75AcIJ4co7trqJew X-Developer-Key: i=dmitry.baryshkov@linaro.org; a=openpgp; fpr=8F88381DD5C873E4AE487DA5199BF1243632046A When the PPM reports UCSI_CCI_NOT_SUPPORTED for the command, the flag remains set and no further commands are allowed to be processed until OPM acknowledges failed command completion using ACK_CC_CI. Add missing call to ucsi_acknowledge_command(). Fixes: bdc62f2bae8f ("usb: typec: ucsi: Simplified registration and I/O API") Cc: stable@vger.kernel.org Signed-off-by: Dmitry Baryshkov Reviewed-by: Heikki Krogerus --- drivers/usb/typec/ucsi/ucsi.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/usb/typec/ucsi/ucsi.c b/drivers/usb/typec/ucsi/ucsi.c index 4abb752c6806..bde4f03b9aa2 100644 --- a/drivers/usb/typec/ucsi/ucsi.c +++ b/drivers/usb/typec/ucsi/ucsi.c @@ -167,8 +167,10 @@ static int ucsi_exec_command(struct ucsi *ucsi, u64 cmd) if (!(cci & UCSI_CCI_COMMAND_COMPLETE)) return -EIO; - if (cci & UCSI_CCI_NOT_SUPPORTED) - return -EOPNOTSUPP; + if (cci & UCSI_CCI_NOT_SUPPORTED) { + ret = ucsi_acknowledge_command(ucsi); + return ret ? ret : -EOPNOTSUPP; + } if (cci & UCSI_CCI_ERROR) { if (cmd == UCSI_GET_ERROR_STATUS) From patchwork Wed Mar 13 03:54:13 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dmitry Baryshkov X-Patchwork-Id: 13590955 Received: from mail-lj1-f182.google.com (mail-lj1-f182.google.com [209.85.208.182]) (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 3009D10A36 for ; Wed, 13 Mar 2024 03:54:26 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.182 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710302067; cv=none; b=c29t4G6YlyJWFgaeeoGsuiJVtOZulTdEvzSId/vkU3A/UIdDlpXXQzBv+QcTAvZg0MbBMGtin6iM6A37gCKV7LasCg5dvs/w9jERztzktm8wb3ROZ5gnu5ontyD8JEOIFLGMXob6oxz6AU8iyI3ykaDSk9eEcpyvgxJdLH8D5+U= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710302067; c=relaxed/simple; bh=XsTJRGmLqtfeI4uhYqv88MedEB7+Nrzpk4+GYV5RAlA=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=nwIXqjZgY9iuucO5toSg0XHI1SNp2xIxCct1+NxsJeUAbgeFjdOIZgxol+CZ12SL+i4somJml7BhTlh8FrIui/sX36ZgWnivT+LE4YwlzLSUb7Z6Ws4Ue9fmYPCVmg8nBGBXPIBE8JPNL4q68h9PO3UZ4UHbqJNa3Bf1G1mGnRE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=OP+DHje5; arc=none smtp.client-ip=209.85.208.182 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="OP+DHje5" Received: by mail-lj1-f182.google.com with SMTP id 38308e7fff4ca-2d27184197cso81704551fa.1 for ; Tue, 12 Mar 2024 20:54:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1710302064; x=1710906864; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=bhgYvXY3qfNB9puPl0tbRCud+qYwtmwix0mxjSGycJU=; b=OP+DHje5kxRl9YoabnmKgG0XA6Ndk2Io7E/e1tV3a9VIz1+nK1L5sFn/UnUeq7iwLm mltkMSrDGDBYtqfIlj2Z7UJfhqiQ4HSbYwGPCgutgMSt5qw2p2Sea5lCzN6Dcld9RX8S 61/CoQKA6v9KF89Pbg12yakLp28ZwF+6wJBrkzWOh9+3WdyCJJjxMIc2K3tQVw2TmWnJ xCL44PlQQN0CNr2y0FxRnSYh7sXe3ccj//eIKlIcZjl8U+XBwHZncP7EanREO0MxnoV5 Iz6kt/6aAbP71InEiPByf1D8zGQsO+dGfk3wWFjQgWu/5bQEnKniQSg68epIWeDXQLOT QNGQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1710302064; x=1710906864; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=bhgYvXY3qfNB9puPl0tbRCud+qYwtmwix0mxjSGycJU=; b=Zxu0vmLGRGYLzXabXwP+XJj9Ue5thDUOcNdxduh8KErUWO8Z7OlVivU52ELZf0Rr86 ouIav/Hq8mgkIowvGWxJ+HtLXwjuT0U9yaXlVIcJmOoehMUSNzdXSgplU2WKPjA+wWxb HGdxomFo1aNjoKCQIFCdsJ+z8y9U+to6uru+1nopk0+qfR1hySxwtrNF1OMdapVQJmff mnICBtDxI1SxA2eQTgYGgCZJ5thS3ne5DmYxXrJos9C4Qzu3dHnhx0XBkoZcOQi52U53 4HWUU6W5ILc9GAiFcZUhhyQbj670A0af6aacNGWlQiDgol/n2D/czwH3pHxE5+BLtnBW uPvw== X-Forwarded-Encrypted: i=1; AJvYcCU+N/1KjFSZ4GVmsesemaGxOC38vl40gIpAqPdKW+ioTn8IZUVTFaUIMJHlhNEwqnxZbmihZinCmSyHyYJxI7+OdHhpbeFrb96Q X-Gm-Message-State: AOJu0YyD2aWSV4Ew1+2382WeEdlJ9ngOdBXEIjleooxlRFQMW1rlxJS0 eIblMxLt0lWR1GGRnJh84AkkJyFBD3D7s8t+nvwhSnRkfxkVEG6xSw5++RvlEAk= X-Google-Smtp-Source: AGHT+IGuUJwUxKprfVv1helbjKdC5swNlM5EdpqQUBCZ/j7Fjvo76AQlVaJ8BDDIwbK/p2HrX+FHmw== X-Received: by 2002:a2e:9a8e:0:b0:2d4:6532:f449 with SMTP id p14-20020a2e9a8e000000b002d46532f449mr1716234lji.20.1710302064505; Tue, 12 Mar 2024 20:54:24 -0700 (PDT) Received: from umbar.lan ([192.130.178.91]) by smtp.gmail.com with ESMTPSA id f25-20020a05651c02d900b002d0acb57c89sm1854319ljo.64.2024.03.12.20.54.23 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 12 Mar 2024 20:54:23 -0700 (PDT) From: Dmitry Baryshkov Date: Wed, 13 Mar 2024 05:54:13 +0200 Subject: [PATCH 3/7] usb: typec: ucsi: make ACK_CC_CI rules more obvious Precedence: bulk X-Mailing-List: linux-usb@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Message-Id: <20240313-qcom-ucsi-fixes-v1-3-74d90cb48a00@linaro.org> References: <20240313-qcom-ucsi-fixes-v1-0-74d90cb48a00@linaro.org> In-Reply-To: <20240313-qcom-ucsi-fixes-v1-0-74d90cb48a00@linaro.org> To: Heikki Krogerus , Greg Kroah-Hartman , Guenter Roeck , Bjorn Andersson , Neil Armstrong , Konrad Dybcio Cc: Johan Hovold , linux-usb@vger.kernel.org, linux-arm-msm@vger.kernel.org X-Mailer: b4 0.13.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=1720; i=dmitry.baryshkov@linaro.org; h=from:subject:message-id; bh=XsTJRGmLqtfeI4uhYqv88MedEB7+Nrzpk4+GYV5RAlA=; b=owEBbQGS/pANAwAKAYs8ij4CKSjVAcsmYgBl8SNraClA33CPWuYkPDFf2GlffV2yFp/DWqKa7 JQyl/lMd/uJATMEAAEKAB0WIQRMcISVXLJjVvC4lX+LPIo+Aiko1QUCZfEjawAKCRCLPIo+Aiko 1YfsB/4+Nnmk2AEqncBvGZIyyIxPd3aHD2nZiLyZEb7CBulni/ZUTXCulyCwldLCgywS023JrRG 41Rkn2pHoUQnxjPdpOs2b6fapIoXo32fP4piMRUoCIIFb+sf0+Q2oLd7zKi3Tf35i/v2htftBjJ QwOzuXDH//Ajc+uiqVaB8TNe8ZfprSRp4B/2S7JLikuss7hY2TGbGHwh30liqMDoFrQ6DHyyQtF b7KMWqsOGlwPiHaB1jkX1+ecq0Ecj02YsPEKCPlJHkL9vr3WP1hIOUqiwCFnI2BtOI5GxVmCAKN NJWpkza6/b0rKtM7fQz+TeQlpqIPMDPfoRdcLBaebmpxqK6Y X-Developer-Key: i=dmitry.baryshkov@linaro.org; a=openpgp; fpr=8F88381DD5C873E4AE487DA5199BF1243632046A It is pretty easy to miss a call to usb_acknowledge_command() in the error handling inside ucsi_exec_command(). For example UCSI_CCI_ERROR had this call hidden inside ucsi_read_error(). Move this call and add a comment to make the rules regarding usb_acknowledge_command() calls more obvious. Signed-off-by: Dmitry Baryshkov Reviewed-by: Heikki Krogerus --- drivers/usb/typec/ucsi/ucsi.c | 18 +++++++++++++----- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/drivers/usb/typec/ucsi/ucsi.c b/drivers/usb/typec/ucsi/ucsi.c index bde4f03b9aa2..05a44e346e85 100644 --- a/drivers/usb/typec/ucsi/ucsi.c +++ b/drivers/usb/typec/ucsi/ucsi.c @@ -92,11 +92,6 @@ static int ucsi_read_error(struct ucsi *ucsi) u16 error; int ret; - /* Acknowledge the command that failed */ - ret = ucsi_acknowledge_command(ucsi); - if (ret) - return ret; - ret = ucsi_exec_command(ucsi, UCSI_GET_ERROR_STATUS); if (ret < 0) return ret; @@ -167,14 +162,27 @@ static int ucsi_exec_command(struct ucsi *ucsi, u64 cmd) if (!(cci & UCSI_CCI_COMMAND_COMPLETE)) return -EIO; + /* + * All error cases below must acknowledge the command completion, + * otherwise PPM will be stuck and won't process commands anymore. + * + * In non-error case the command is acknowledged after reading Data + * from the controller. + */ + if (cci & UCSI_CCI_NOT_SUPPORTED) { ret = ucsi_acknowledge_command(ucsi); return ret ? ret : -EOPNOTSUPP; } if (cci & UCSI_CCI_ERROR) { + ret = ucsi_acknowledge_command(ucsi); + if (ret) + return ret; + if (cmd == UCSI_GET_ERROR_STATUS) return -EIO; + return ucsi_read_error(ucsi); } From patchwork Wed Mar 13 03:54:14 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dmitry Baryshkov X-Patchwork-Id: 13590957 Received: from mail-lj1-f175.google.com (mail-lj1-f175.google.com [209.85.208.175]) (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 62C88DDD1 for ; Wed, 13 Mar 2024 03:54:27 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.175 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710302069; cv=none; b=ZZU6xwRnKoZxdNjhqrMJCcxr4kM65qRoRknx7I08xakMMqZ2aN0SqYbwcAz6gonYaiivcG34oW4ueMgU40YTqh8bRvdoorSxMeo0NbpCu5uvuS1aowU3who/pClwRfRkHLezMTFQVOg6pGgFNT3LCElQgltnuqIC1JHRjSSA2dA= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710302069; c=relaxed/simple; bh=6EYmD69sWZmGoX6AKwzh0M4O6uXmEHRrEKh/yt0/KPs=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=oKnQDzHSu4q6loEL1kPXLu4QM4UpKiMQw9Bek4UcdKtyJ+LWnXUTvc7yklmfueXuWXRUG/GOYvjCf8oABBXcbWtq4XWPvApvEQG62euByLvZiZ+tr+Os8LO2l1wNVjTCnCrAwOATsyaKcbrmqKuAxE6rjX7SRzga5grWBUYDQx0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=YZpLyrRt; arc=none smtp.client-ip=209.85.208.175 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="YZpLyrRt" Received: by mail-lj1-f175.google.com with SMTP id 38308e7fff4ca-2d09cf00214so71636771fa.0 for ; Tue, 12 Mar 2024 20:54:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1710302065; x=1710906865; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=Fct0VZjsurRkzQqPpB3JaLxWY0P6FclFVQOW6mKz5w0=; b=YZpLyrRtNe2B1Z74VpEfjNExU+KV+xHRq721Jqup9DucYxpJqxzIFvh9avQ/qKPqDO h1Mz4ZvRom+vPWuVqmXnd3iQE0k7EOZXBKWaYyf9Kb1tr62civn1LRhJomfPYmjH3N38 F/WphJ2ow9iYQ0T9FiGJ2CRVGlSSzVMfLUJmSSBSuTdXrsvHLtmuTVq5Oz3gaInMVfW3 UbqY+KkAI7U5Q4cmBu7GpB7PCFmCT++FOT2B7ywqY/1NgHuaCpfCAQaRxEVWqik8H39w v4+kK+RYX6NbZpBoYBqq07Hsxi4za5v2YwP/p0r6i2nhpCZ4E1rMK3C31ap63M3PFwWq Cc5A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1710302065; x=1710906865; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=Fct0VZjsurRkzQqPpB3JaLxWY0P6FclFVQOW6mKz5w0=; b=OJX9THrtC31ElhbqtT7mUsr0f179E3NB/kDHCfu1lfv737lfd1JUBA45VZtFeqvRKQ qwgC/vJWpy1iQrh48+1rbA1fA6h9ZXe0j5pS//a2ZnPEXk6S7Tg5dCh691Plu8MiC6j4 4iFmXWEHDyCeAS8v4Xr3cBDilvpLc6tBSi+mKxuBGZAbMrtHyzCcql6AYwvRygxpYTSq KDTeSY19/WkmIXIEhdnnVHUO6jxBLVHxnHNt8e8FpIuco3aPN3j1J+Oz3UZKFlW8cvJA J3sQqlS70u+0FuuCkJh7ZAvk3s9BCP0Rek2ZZOGIZ1HrWUNfHu+rAa12jieHmWdgyTOP lEiA== X-Forwarded-Encrypted: i=1; AJvYcCVNpbKXXBHmv+vT26JT3iJmRlrJqGkUji1mILKs9L0/sQylFlOhVG897UPRRTD8WaKyJqg9rn5bYxgbTHHZdKrFn9aMNb8e/XmV X-Gm-Message-State: AOJu0YwGv4NESz+2eNRfhWl5hf796+il9FkeDe3PD0Pb3EgL/xhVnEDz oGNhUUj5BzRNH5nfhLHTTtE/7Cy7EovN+DEZ/gOx3nWru9Od34EDYeaORmTY1go= X-Google-Smtp-Source: AGHT+IGU1nKa9yZZSaZBVwxotiWxvgYGip66g5eNu2UPhAK4cjTwOilx3MCH81jbjbDHlShtOoBQOA== X-Received: by 2002:a2e:350d:0:b0:2d4:16a5:b361 with SMTP id z13-20020a2e350d000000b002d416a5b361mr1431441ljz.25.1710302065397; Tue, 12 Mar 2024 20:54:25 -0700 (PDT) Received: from umbar.lan ([192.130.178.91]) by smtp.gmail.com with ESMTPSA id f25-20020a05651c02d900b002d0acb57c89sm1854319ljo.64.2024.03.12.20.54.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 12 Mar 2024 20:54:24 -0700 (PDT) From: Dmitry Baryshkov Date: Wed, 13 Mar 2024 05:54:14 +0200 Subject: [PATCH 4/7] usb: typec: ucsi: allow non-partner GET_PDOS for Qualcomm devices Precedence: bulk X-Mailing-List: linux-usb@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Message-Id: <20240313-qcom-ucsi-fixes-v1-4-74d90cb48a00@linaro.org> References: <20240313-qcom-ucsi-fixes-v1-0-74d90cb48a00@linaro.org> In-Reply-To: <20240313-qcom-ucsi-fixes-v1-0-74d90cb48a00@linaro.org> To: Heikki Krogerus , Greg Kroah-Hartman , Guenter Roeck , Bjorn Andersson , Neil Armstrong , Konrad Dybcio Cc: Johan Hovold , linux-usb@vger.kernel.org, linux-arm-msm@vger.kernel.org X-Mailer: b4 0.13.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=997; i=dmitry.baryshkov@linaro.org; h=from:subject:message-id; bh=6EYmD69sWZmGoX6AKwzh0M4O6uXmEHRrEKh/yt0/KPs=; b=owEBbQGS/pANAwAKAYs8ij4CKSjVAcsmYgBl8SNreDUqybYN8KU9iG18YedS4ZfehnlaTrZRv 7/A8LzaOIeJATMEAAEKAB0WIQRMcISVXLJjVvC4lX+LPIo+Aiko1QUCZfEjawAKCRCLPIo+Aiko 1cz5B/9rl0hub4Z5WK6m/HRp2u0VhWm7/iLCPxfkgXFaarI7zCOiQf41D8H4NbEWJ6jgcecy/OU nl0B5nbBw52Ix4jTdCe5Gh3sTn6yDoTcxcsNln/T3ncQowxE6g7W8qudfBxEv5AJ5XosAtSWEP8 KiSRh/6WUQOGnwYLtuZXPACSjnp5ObKVHCpiP7RjYov0fO8tmYDaTbzfhf1IuKYkYd6jKjMZGPs KfyunsuG/2tZ36zX8L2X7NVbqltjSEu6X8wuBZLXbOHpilZNv4r+wxTi5EYAHCDqFWuxI+smGY7 0YEgHg2AnKgprVC9jqXv9diDMCum1RYwGNSZp+tQCdHSxmM4 X-Developer-Key: i=dmitry.baryshkov@linaro.org; a=openpgp; fpr=8F88381DD5C873E4AE487DA5199BF1243632046A The name and description of the USB_NO_PARTNER_PDOS quirk specifies that only retrieving PDOS of the attached device is crashing. Retrieving PDOS of the UCSI device works. Fix the condition to limit the workaround only to is_partner cases. Fixes: 1d103d6af241 ("usb: typec: ucsi: fix UCSI on buggy Qualcomm devices") Signed-off-by: Dmitry Baryshkov Reviewed-by: Heikki Krogerus --- drivers/usb/typec/ucsi/ucsi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/usb/typec/ucsi/ucsi.c b/drivers/usb/typec/ucsi/ucsi.c index 05a44e346e85..011d52bf34f6 100644 --- a/drivers/usb/typec/ucsi/ucsi.c +++ b/drivers/usb/typec/ucsi/ucsi.c @@ -641,7 +641,8 @@ static int ucsi_read_pdos(struct ucsi_connector *con, u64 command; int ret; - if (ucsi->quirks & UCSI_NO_PARTNER_PDOS) + if (is_partner && + ucsi->quirks & UCSI_NO_PARTNER_PDOS) return 0; command = UCSI_COMMAND(UCSI_GET_PDOS) | UCSI_CONNECTOR_NUMBER(con->num); From patchwork Wed Mar 13 03:54:15 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dmitry Baryshkov X-Patchwork-Id: 13590958 Received: from mail-lj1-f178.google.com (mail-lj1-f178.google.com [209.85.208.178]) (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 3C8A510A35 for ; Wed, 13 Mar 2024 03:54:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.178 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710302069; cv=none; b=bCAXufkGJZccIZFFoRlSnConSlkdrn3NXKpBixhZaeoHCyc6WCZAdBwkk+uJyiDGqH1TNGon+vVapTzsAPKihh9Zp4xDb8eN7lJL5LKiAC084Np2IUUsQQcu3/b154SUmS38RCFB/Y8aZ2It/GPrSSdGXK6oH6b/8DZR2Opu+uU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710302069; c=relaxed/simple; bh=pwqh+7qUWx0XkGuOfq8GXVfKJVkT8SLDNZBKjtfBxbs=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=QYbuwKWWmbp6yQ69OT/MSNHBGcz6LNJGTAEmbZdHaa6afNCtAVc4xXieuGIwIBC19pY1LvitUzrRoqaYTTUIguOupsF5Y9qWUcLHKxiGTUYggUCfBO//fqix69e50DMDkbtz8cWqoCzHxCy5PjsfpGPgvGWlK5bCzW6JaCuSApQ= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=IUaXGz0r; arc=none smtp.client-ip=209.85.208.178 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="IUaXGz0r" Received: by mail-lj1-f178.google.com with SMTP id 38308e7fff4ca-2d309a23d76so3551531fa.1 for ; Tue, 12 Mar 2024 20:54:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1710302066; x=1710906866; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=uhTvdctsG3UAMNuU+ThWL3PpPImv7wP9zybhV8Y4zOc=; b=IUaXGz0rxDtAD6Cld+luz1aqvemAl0UDawDcjU++IpWlYfvGzG72/Kez6VQDQCIByH ZdBRlw9KC0TIw+OaGGv6MNi+tlEk5alA2DMVVPVU0fhl+VQVV0h3HA2gsql2hNIFSvFF oBcLZEvksHh6n572UgLmKAcCTE/jjHf42gutb2anI8imgDytdXBxs83HfeDHuwO4Ypmf fiJjnj6e1xcFJG7rsaSoB1Jyb+hvO2cdnAiLgsQPXwJ0pEaQjR5H0b3cb8wW47Kc62F1 2W2NdIPaA9uLiTWGIGbPN20KcViSMpaskbOaTsFmbOrsPthND/If5ZhET3VeLPAZsSYM 7Ayw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1710302066; x=1710906866; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=uhTvdctsG3UAMNuU+ThWL3PpPImv7wP9zybhV8Y4zOc=; b=WcbxZyTnH3yxtXlp9turv+XOWzovWQ/H8KIL1q6ReZUQzVjwSmg7p54df021QTRS+W 0gKlVz37+Ma6hQnCdoD1PG8bNvnUp5nKzVdESvd0m/MvNVzlNOOh0qZBTjJl8nPtkS7y Pg2QrPBmwXssie2r4xrtnpb6NL4DUdP+fP19FUR+adeaCpK5TW1lOvaGp+oiX53WhaHb LnonWMDSlEMD8XFWAtFRRzUnDGYDgSJrs3dpoWr1Y98sd3pY/+Oh77r8a13qqYYavQwK j/Aot/RhB9RfPedOv3bsw4m0gbWz5Zx8JbiuNDDer3vIVA2Aqjrx2gvvEX9QT6g6xLxR vB/A== X-Forwarded-Encrypted: i=1; AJvYcCUD2+LQwt//f7BakCcDoToRariFzRQ+ahBsmd1KtCY6/gfvllh+WQ3FF4Ylay8H/r6KEHp7QE7zudMWssjQiWePXNOq/S3nUACc X-Gm-Message-State: AOJu0Yyq4WUf3cMWHEDpP2amyU1ant7UltP7xY52XrF16piAva6Tyu7L kRngcapzmbtO0xO4XU7c64R+54MMMypwVbVwBujTIhjno5ENPyLdvwxxtpd2Ez4= X-Google-Smtp-Source: AGHT+IGUXuKt1xsCYGLcIpNaTG6doYJa88I3f0jTK6Qe7QNycBlM9CkIMQ3hzYDXjsxiQ5roHW7rYQ== X-Received: by 2002:a2e:984e:0:b0:2d4:6c19:67d5 with SMTP id e14-20020a2e984e000000b002d46c1967d5mr203100ljj.18.1710302066638; Tue, 12 Mar 2024 20:54:26 -0700 (PDT) Received: from umbar.lan ([192.130.178.91]) by smtp.gmail.com with ESMTPSA id f25-20020a05651c02d900b002d0acb57c89sm1854319ljo.64.2024.03.12.20.54.25 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 12 Mar 2024 20:54:25 -0700 (PDT) From: Dmitry Baryshkov Date: Wed, 13 Mar 2024 05:54:15 +0200 Subject: [PATCH 5/7] usb: typec: ucsi: limit the UCSI_NO_PARTNER_PDOS even further Precedence: bulk X-Mailing-List: linux-usb@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Message-Id: <20240313-qcom-ucsi-fixes-v1-5-74d90cb48a00@linaro.org> References: <20240313-qcom-ucsi-fixes-v1-0-74d90cb48a00@linaro.org> In-Reply-To: <20240313-qcom-ucsi-fixes-v1-0-74d90cb48a00@linaro.org> To: Heikki Krogerus , Greg Kroah-Hartman , Guenter Roeck , Bjorn Andersson , Neil Armstrong , Konrad Dybcio Cc: Johan Hovold , linux-usb@vger.kernel.org, linux-arm-msm@vger.kernel.org X-Mailer: b4 0.13.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=936; i=dmitry.baryshkov@linaro.org; h=from:subject:message-id; bh=pwqh+7qUWx0XkGuOfq8GXVfKJVkT8SLDNZBKjtfBxbs=; b=owEBbQGS/pANAwAKAYs8ij4CKSjVAcsmYgBl8SNr5RlhRizT3CYUWt8TNerNgnS1//e4G+r31 b4pQpB5/6GJATMEAAEKAB0WIQRMcISVXLJjVvC4lX+LPIo+Aiko1QUCZfEjawAKCRCLPIo+Aiko 1SKzB/9zGp/G8bISVxeCGIhS8uvT6pZELUF3hCqGctUuFrbIpV65Gdq/ez91l1PgETjn6GMCGSW sAKmTpVlfnYDypMn8wZuwVz1yetFoXqRg7XEik/i8PmzxnXJVzzaUoakH/cp4hqBA1kTWgozjnz QRLPZRDHLyOZiQyefK2xfU9z7uZtVOEBUt9Io625dadNsoqHfFgZYsWU7Y7fNlDVusSxtlcr7Mu u21RyF7QSCKpFQtY5OUuNK3HYU/CQWi4Xj0sKJcf2KkWRDPOP06dIVlz7HdPmtf8DxqLTK7qE0H q4ab2aeZAZIYxyMuW8ijEU8rFWVNG2MVGglhuwxHo/9usGyR X-Developer-Key: i=dmitry.baryshkov@linaro.org; a=openpgp; fpr=8F88381DD5C873E4AE487DA5199BF1243632046A Reading Partner Source PDOs for the consumer Connectors appears to be working. Permit getting PDOs in this case in order to populate capabilities of the connected power supply in the sysfs. Signed-off-by: Dmitry Baryshkov Reviewed-by: Heikki Krogerus --- drivers/usb/typec/ucsi/ucsi.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/usb/typec/ucsi/ucsi.c b/drivers/usb/typec/ucsi/ucsi.c index 011d52bf34f6..72d368433b1f 100644 --- a/drivers/usb/typec/ucsi/ucsi.c +++ b/drivers/usb/typec/ucsi/ucsi.c @@ -642,7 +642,9 @@ static int ucsi_read_pdos(struct ucsi_connector *con, int ret; if (is_partner && - ucsi->quirks & UCSI_NO_PARTNER_PDOS) + ucsi->quirks & UCSI_NO_PARTNER_PDOS && + ((con->status.flags & UCSI_CONSTAT_PWR_DIR) || + !is_source(role))) return 0; command = UCSI_COMMAND(UCSI_GET_PDOS) | UCSI_CONNECTOR_NUMBER(con->num); From patchwork Wed Mar 13 03:54:16 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dmitry Baryshkov X-Patchwork-Id: 13590959 Received: from mail-lj1-f171.google.com (mail-lj1-f171.google.com [209.85.208.171]) (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 3EB4E10A3B for ; Wed, 13 Mar 2024 03:54:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.171 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710302070; cv=none; b=CLuC0d5sT2ho32cmWCK60BFXhvsfMJEYour8zPz4E5qGM3LzpEvlveogYwNqxQF/l0XSPehfW+LY7aFx0CUUMVMmZ1iqJff2mGwe0lEP0h6t/H47oHBC9uTp+c1/EFA9+MX2kszU2g+AoOCS6lFsSPWN6AYmBeGGgVZj7sL4HTU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710302070; c=relaxed/simple; bh=hjtNY7rwrzLwExmeSB1V0RfbPf0EFqjWrRvXTMW6u3s=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=JrrxquF5O80SLGwa+pplW9gtubSNWHyFxVSK0HUJ6WZwmUrYwu1HpUco+LUDhaAVJqPGMIveJ3k56eufSTw0w0hKOPv32OHLo2dVGz2AOoA/C1yPpehi2Sa2K11TuUq/EsIP/KCV3I/922egMLmOPPV8tzdnJnvJn+cH92LFD0I= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=ESI5kDfG; arc=none smtp.client-ip=209.85.208.171 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="ESI5kDfG" Received: by mail-lj1-f171.google.com with SMTP id 38308e7fff4ca-2d309a23d76so3551611fa.1 for ; Tue, 12 Mar 2024 20:54:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1710302067; x=1710906867; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=7fFzMCtaUwxqWFGfcU6jyBqJsIQ1qd2eqh/5wTdrbDk=; b=ESI5kDfGw+RUT5wFKUYfTqPRiBshcZynfD7gVREyEH0gMMjcUzHYdiINtls+NVC6Dn PRymzqkxFgrRRdG5Ci1BkbekdPn7rd5+nLRKabSr+nKpIgqhDAGr/71hwxrAzV0XdNUV +NJBrvvuKqbXb0qzkbvQjk7DApPlM5M0sxVV2Rns0VDbLelSImMHsM0CRm8qziQaARzm pkbnqNRIDFACwR1MF/CTvnsF2MJHYGABZKKA5P5SxI9c33+Oilrocdnr4dOH4stsyFCm 0SGvzeBumSk/g5U7Qddi9K4t4MFHClA+Oi9Oa8CVvQnc+AJDbiRXqYzdLmij/D7iz3b/ dMPg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1710302067; x=1710906867; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=7fFzMCtaUwxqWFGfcU6jyBqJsIQ1qd2eqh/5wTdrbDk=; b=ONeDI/e8HITiczb+RVR5LQaVa3A/FVpVHXiByn60lu2Q/yAFDYjlGK9I6wprAcHCxQ hsuMzPC4fhVOuVqjUBVMH0TKFuOxFOhmi5IIHiv4qybE9eFQe0/AijPdULdE3gmTuNPz yUYVJpdLL2ejUwLlRSVtRKc+wxpEqg4i9YfMxXcK3rXj41dKuDjelcWG+UNEt2PyODJp ENkNTrN3IEB/I/l1ISCLTYbDg4DKtVXNM5ZzDZzOVYD69y3E7E2tFn23Bi4pQSZsO8AJ dN4hog8D4qAsK6jxMdi9OZybe9fSBVVVIUM5rTTD9/ReRrzS0FjDmYtFltjst1s3IZsv 20IA== X-Forwarded-Encrypted: i=1; AJvYcCWIEPZOwfpFTZvqvvTLefSKnfhVkfOfS/XU5ib/i/wpEo5J4Gv6iZStSI2AZRUg3p4TaQyAW2uYYNpOzdCRk9N0pXLervdMmjO7 X-Gm-Message-State: AOJu0Ywr6s+tI3Kq4OjhJvgSLs1ogOC5yeBJdULLaq1I7f0Pfvep9m4z Jp4+zJZcV5JwXb4idQ+g/joUO+qXa/PtuWX3cECp7/p5fZgtjqCfpXpJWZFFhkI= X-Google-Smtp-Source: AGHT+IFIf0aayTcXNyruF81FSZALtLVnPMSslXA4xvKeV9oyGrKzjNCVmlS48OOTDYcZ4iyL5usVdA== X-Received: by 2002:a05:651c:105b:b0:2d4:22d9:b01a with SMTP id x27-20020a05651c105b00b002d422d9b01amr477054ljm.11.1710302067679; Tue, 12 Mar 2024 20:54:27 -0700 (PDT) Received: from umbar.lan ([192.130.178.91]) by smtp.gmail.com with ESMTPSA id f25-20020a05651c02d900b002d0acb57c89sm1854319ljo.64.2024.03.12.20.54.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 12 Mar 2024 20:54:26 -0700 (PDT) From: Dmitry Baryshkov Date: Wed, 13 Mar 2024 05:54:16 +0200 Subject: [PATCH 6/7] usb: typec: ucsi: properly register partner's PD device Precedence: bulk X-Mailing-List: linux-usb@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Message-Id: <20240313-qcom-ucsi-fixes-v1-6-74d90cb48a00@linaro.org> References: <20240313-qcom-ucsi-fixes-v1-0-74d90cb48a00@linaro.org> In-Reply-To: <20240313-qcom-ucsi-fixes-v1-0-74d90cb48a00@linaro.org> To: Heikki Krogerus , Greg Kroah-Hartman , Guenter Roeck , Bjorn Andersson , Neil Armstrong , Konrad Dybcio Cc: Johan Hovold , linux-usb@vger.kernel.org, linux-arm-msm@vger.kernel.org X-Mailer: b4 0.13.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=874; i=dmitry.baryshkov@linaro.org; h=from:subject:message-id; bh=hjtNY7rwrzLwExmeSB1V0RfbPf0EFqjWrRvXTMW6u3s=; b=owEBbQGS/pANAwAKAYs8ij4CKSjVAcsmYgBl8SNr/e7wwCwLWXzsNh5AlPrljH8c1bjebYZIn 8rDBmNao7eJATMEAAEKAB0WIQRMcISVXLJjVvC4lX+LPIo+Aiko1QUCZfEjawAKCRCLPIo+Aiko 1ex4B/42jyzjXY1cC5GZEU8UWURS1Dc2rdh13kO5wmShsvrgMpF/oGhA3FRPKAPUOVyGH6odIub /oif7UVdI8wIEjrlxqwQRgwO+HUXZlK8ZQzoAuHcrS37BJQAWTqMHzNkOyEBlpOxkEyf1aRBiSR huxD4SHC9UFToOPFfHdOzJRBM+Jag9tMFliRUzn3KJ+8XidOkgBVdJGatJGJSua7Pe46uuEohXG e/acWjwZX5FROYGG759vSg/l71rEOsx9NRa+CMq2qJH/+1lCPyavH2uibMQSaYhBIN8YB/3ei+y O790+02cRft7tSNsgHZWGKIZ3+k72uiUCke+q6f4LNtdfTfi X-Developer-Key: i=dmitry.baryshkov@linaro.org; a=openpgp; fpr=8F88381DD5C873E4AE487DA5199BF1243632046A Use typec_partner_usb_power_delivery_register() to register PD device for Type-C partner so that the PD device is nested under the partner's device in sysfs. Signed-off-by: Dmitry Baryshkov Reviewed-by: Heikki Krogerus --- drivers/usb/typec/ucsi/ucsi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/typec/ucsi/ucsi.c b/drivers/usb/typec/ucsi/ucsi.c index 72d368433b1f..78e04b7701c8 100644 --- a/drivers/usb/typec/ucsi/ucsi.c +++ b/drivers/usb/typec/ucsi/ucsi.c @@ -833,7 +833,7 @@ static int ucsi_register_partner_pdos(struct ucsi_connector *con) if (con->partner_pd) return 0; - con->partner_pd = usb_power_delivery_register(NULL, &desc); + con->partner_pd = typec_partner_usb_power_delivery_register(con->partner, &desc); if (IS_ERR(con->partner_pd)) return PTR_ERR(con->partner_pd); From patchwork Wed Mar 13 03:54:17 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dmitry Baryshkov X-Patchwork-Id: 13590960 Received: from mail-lj1-f178.google.com (mail-lj1-f178.google.com [209.85.208.178]) (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 71E2214ABC for ; Wed, 13 Mar 2024 03:54:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.178 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710302072; cv=none; b=PORiDk0H5iYgH3Siu1rgYf6xjqtm39u+bHw5mzgYoJVHq4qCJJU9+upeAVArldUCwMU844rrW2nQw/m3x8KkvG+7QN1twWWTA0OktqVaaASBiJOtFHjanEAKyMyCNIm5Kr08n4GIYvfOyFrsbCPM56EDp6aj8TpkS2On9T5D7Qw= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710302072; c=relaxed/simple; bh=S06LW4XbXp4mNZRbX50ghuv/ngjWIPDbWauLWS0Z+FU=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=KKfPibMzfLJaV3fpMkS0FLdYLmaEB0QG2QP6CB4xDaK2KWaqXu8shavdzBsCZrENj9uhN5KaY+jLtKVnxr3Nr2E5b+A5u4sZKoqubtoFakAeQwX6CwRMubYOZqdekQm0/jv8EnAV1TP9CJf/r7RK/iLpMgDuzzAUdXmvO/G9h44= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=Pnov1oIs; arc=none smtp.client-ip=209.85.208.178 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="Pnov1oIs" Received: by mail-lj1-f178.google.com with SMTP id 38308e7fff4ca-2d27fef509eso101134971fa.3 for ; Tue, 12 Mar 2024 20:54:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1710302068; x=1710906868; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=Z632Vl9mGcaUD+E7Cpwdhyjpjxo2r7hLci9lVvPzHlg=; b=Pnov1oIssLpiTJqrAjLlU8ThYua0NravpFbBeOIHDnL1NiU9ugNF89QXqytmXt5rBu nw065NatFw/6ijseQV7HO0JtB3heAaTFJZZ/X4jNvPDvXNr2JIm3wHM5p8hJikLAuv27 IfyvK91T82fWeJCT/myTLmLuJ1erJdDaXeMqZWeB+h4ShQUPGo7cDSh3M8YUkJfd6XMW 9PTMXHT/oaYjCyFBQkCBXAa8C64xdbWJAyqRT7fnTlP0EQwP7EgXyhdoptJFlDCtdbkx 8nxCzSH7uOwfyzkciHoM2W1AVIyulcb4EXXZV1BvTJpVmV5As6E788j1SQmEGVUpimhx YkQQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1710302068; x=1710906868; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=Z632Vl9mGcaUD+E7Cpwdhyjpjxo2r7hLci9lVvPzHlg=; b=Hg/x2UK/z0niAi2XZHIUhfs7ikkM3/cMldqpKYUSgly9W4RmAuyJ8bGseanD7uGlbu lNnHfjJQuRcii+KDr36hj2V9N/Xl9RZrJpziubtsjthu+ItRPfh6a0iBFSrojVwahZsq MWcb9aDBEc+gmQFWBM1dEockv0isoEePqxGNyRrjrWEWgEvKLBUDC1BtIXB8aNWG4Rhx cicEL5Os+Xw0EU/qWq2a2E28xK0ypvO/X4/TZABxY9LNCS2lwBJiQxK32f8/Zm3a0KKW 0azNDtk4oXG6rpQg5CfVquG0jOZ1dwbM0CLAPO/NlMZyDe3IpVbDnSLHP4HvLfzOvNHo 3Ifw== X-Forwarded-Encrypted: i=1; AJvYcCXR3A8FKIe+mhrMOgIMBVbobpQWCurXKPpV2URQdhGA65NzsXNB5e7yFfpilpfXVKYLa9G9TT7Sq7fSRly67Gq0s7KkfgWrhhWr X-Gm-Message-State: AOJu0Yw383Dh2/In6LQ0JIOFCQF/Pt0D+GS0vP31HyTKbWoEzLyfNTQk XyX9eCfcDdmR9XfSiwd7JeXon4tHvgLYtZ9kMAz37j7eD56fnF0AD5PM8bnZk8w= X-Google-Smtp-Source: AGHT+IGT21uwfmDhENLTqEjyobLcyK/klC8iwxAr7x91U3iV4MBVqo6A+D5u/a3myRTNeji9AGY8rA== X-Received: by 2002:a2e:b0f1:0:b0:2d4:4a0d:d48 with SMTP id h17-20020a2eb0f1000000b002d44a0d0d48mr3239650ljl.47.1710302068739; Tue, 12 Mar 2024 20:54:28 -0700 (PDT) Received: from umbar.lan ([192.130.178.91]) by smtp.gmail.com with ESMTPSA id f25-20020a05651c02d900b002d0acb57c89sm1854319ljo.64.2024.03.12.20.54.27 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 12 Mar 2024 20:54:27 -0700 (PDT) From: Dmitry Baryshkov Date: Wed, 13 Mar 2024 05:54:17 +0200 Subject: [PATCH 7/7] soc: qcom: pmic_glink: reenable UCSI on sc8280xp Precedence: bulk X-Mailing-List: linux-usb@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Message-Id: <20240313-qcom-ucsi-fixes-v1-7-74d90cb48a00@linaro.org> References: <20240313-qcom-ucsi-fixes-v1-0-74d90cb48a00@linaro.org> In-Reply-To: <20240313-qcom-ucsi-fixes-v1-0-74d90cb48a00@linaro.org> To: Heikki Krogerus , Greg Kroah-Hartman , Guenter Roeck , Bjorn Andersson , Neil Armstrong , Konrad Dybcio Cc: Johan Hovold , linux-usb@vger.kernel.org, linux-arm-msm@vger.kernel.org X-Mailer: b4 0.13.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=885; i=dmitry.baryshkov@linaro.org; h=from:subject:message-id; bh=S06LW4XbXp4mNZRbX50ghuv/ngjWIPDbWauLWS0Z+FU=; b=owGbwMvMwMXYbdNlx6SpcZXxtFoSQ+pH5exzHOlrKmav+l5ZE3/7pk7Bumje/gT92sakcnOr+ z475i/pZDRmYWDkYpAVU2TxKWiZGrMpOezDjqn1MINYmUCmMHBxCsBEIt6x/9PjOXv3CoPxwnZL /882hZlvIuWPab36PneV8eaX2dtFfV+qacU01FUwytl8t16Z9L3D1vwPa9jl+PWhlYe7+u/Y5hv +Ycz3lnr2I5TNUkfrn9dvwwDe1bJJ+xPebTpwp6k7SS+gM/CiuT73lxj9hCm3JzClFD089Lvg+N 0W68NhrZvCJmf7VAp/8quvXCIqvu2hvEpA49O5wVF57Bbhmre+B4haaAXY2PfoHVomurTcYfVUr zV7PkluCHvymIvZoKJg32P+9bNqjD22OSUWGvd4Gese5Z0jpXjitvapDe8dtu3icbT8s5rzzCen DNvZE77wHinmjjmesnAK7x7RfX9vBd3a8G2KkvccGY31AA== X-Developer-Key: i=dmitry.baryshkov@linaro.org; a=openpgp; fpr=8F88381DD5C873E4AE487DA5199BF1243632046A Now as all UCSI issues have been fixed, reenable UCSI subdevice on the Qualcomm SC8280XP platform. Signed-off-by: Dmitry Baryshkov Reviewed-by: Heikki Krogerus --- drivers/soc/qcom/pmic_glink.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/soc/qcom/pmic_glink.c b/drivers/soc/qcom/pmic_glink.c index f913e9bd57ed..e5a591733a0f 100644 --- a/drivers/soc/qcom/pmic_glink.c +++ b/drivers/soc/qcom/pmic_glink.c @@ -343,7 +343,6 @@ static const unsigned long pmic_glink_sm8450_client_mask = BIT(PMIC_GLINK_CLIENT static const struct of_device_id pmic_glink_of_match[] = { { .compatible = "qcom,sc8180x-pmic-glink", .data = &pmic_glink_sc8180x_client_mask }, - { .compatible = "qcom,sc8280xp-pmic-glink", .data = &pmic_glink_sc8180x_client_mask }, { .compatible = "qcom,pmic-glink", .data = &pmic_glink_sm8450_client_mask }, {} };