From patchwork Thu Jun 29 05:45:09 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: =?utf-8?b?U3RhbmxleSBDaGFuZ1vmmIzogrLlvrdd?= X-Patchwork-Id: 114177 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:994d:0:b0:3d9:f83d:47d9 with SMTP id k13csp9435740vqr; Wed, 28 Jun 2023 23:35:56 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ6MhKv01suZ3Dn8dleU9bSM8vP9byLx6lJv9lqdmQ2hLlybgZiEXBw3pkfAy+eV2WTZTSp+ X-Received: by 2002:a17:90a:c90a:b0:263:1e82:2dc7 with SMTP id v10-20020a17090ac90a00b002631e822dc7mr6658423pjt.0.1688020556490; Wed, 28 Jun 2023 23:35:56 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1688020556; cv=none; d=google.com; s=arc-20160816; b=B5Gh5yZOKUUmdYz7BGFScwU6EU83FLTYmAHwmBC3c90jOiNfCBdM25LY9rfSM9AHMa 8oj55lewt64uMnp5OKIVb+VnED/RF2Wlcxh1OSty22Xkpe0KSryVvNv9yDmnKORTnSNo c3vW+0HjEPoVXR/8YpDtjimNGFQqfoM0hWw3Hx8zYOus3oYqa5aU6q2cU8OcZosPf9VG ryTx0CLB+CYDX9ulVL772pgQoaICuZJtlbUBfVJkW3zwLth15ldC5oDhRBL2pRos+gro My2EHNMVvjLCn3xXnFTH5F6t6Nd1R8oQjfrd6hD2AAFaI7WSSJH2reA/qRppVA6Me5IG rRlg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:authenticated-by; bh=VXaDsf4iCO/QPQUUY1QTOzoHYvud+YAoCizfwFoseWY=; fh=BwnLsrH5d3iMWpPchBf3cl3Da1ZTlQpnbNgvgso3TF4=; b=FmUx29X7KGHjCBkc+04Y/vr4ufd7H2+1iiQGWBtfaO70njOzBNObA+wKUAA5yCflux jaxhHRXyAVQmfcTSmjNEAwxlEgwZObsK8A5SLmTOZBhzZg3xVCYRN3R6EdRE9sVSjaTi IDoOYtuKHnfcdImSCA9iaIISQWR2/JBQRLNAevgeXbuh/EdcRjAT8HeZRcz3A2H3itQf 78x0gdYU+xfBMCe1T9yEygwL2tkn6fnBvOIcJTD23FmS9+OxgCJZZhjtSSWWopbYFil1 T9hJVPygN0x4nGEcye28I+4ocbgRgc9uJ9gHejGmNDoWZpF1FHijDCzYAwM4yP2XDaGy wNHw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id x24-20020a17090a8a9800b0024b011026aesi1964728pjn.76.2023.06.28.23.35.40; Wed, 28 Jun 2023 23:35:56 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231422AbjF2Fq7 (ORCPT + 99 others); Thu, 29 Jun 2023 01:46:59 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33312 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230119AbjF2Fqi (ORCPT ); Thu, 29 Jun 2023 01:46:38 -0400 Received: from rtits2.realtek.com.tw (rtits2.realtek.com [211.75.126.72]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id C4EDB1FFA; Wed, 28 Jun 2023 22:46:33 -0700 (PDT) Authenticated-By: X-SpamFilter-By: ArmorX SpamTrap 5.77 with qID 35T5jNc95016165, This message is accepted by code: ctloc85258 Received: from mail.realtek.com (rtexh36506.realtek.com.tw[172.21.6.27]) by rtits2.realtek.com.tw (8.15.2/2.81/5.90) with ESMTPS id 35T5jNc95016165 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=FAIL); Thu, 29 Jun 2023 13:45:23 +0800 Received: from RTEXMBS05.realtek.com.tw (172.21.6.98) by RTEXH36506.realtek.com.tw (172.21.6.27) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.17; Thu, 29 Jun 2023 13:45:24 +0800 Received: from RTEXH36506.realtek.com.tw (172.21.6.27) by RTEXMBS05.realtek.com.tw (172.21.6.98) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.34; Thu, 29 Jun 2023 13:45:23 +0800 Received: from localhost.localdomain (172.21.252.101) by RTEXH36506.realtek.com.tw (172.21.6.27) with Microsoft SMTP Server id 15.1.2507.17 via Frontend Transport; Thu, 29 Jun 2023 13:45:23 +0800 From: Stanley Chang To: Greg Kroah-Hartman CC: Stanley Chang , Vinod Koul , Kishon Vijay Abraham I , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Alan Stern , Flavio Suligoi , Matthias Kaehlcke , Douglas Anderson , Ray Chi , , , , Subject: [PATCH v6 1/5] usb: phy: add usb phy notify port status API Date: Thu, 29 Jun 2023 13:45:09 +0800 Message-ID: <20230629054523.7519-1-stanley_chang@realtek.com> X-Mailer: git-send-email 2.40.1 MIME-Version: 1.0 X-KSE-ServerInfo: RTEXMBS05.realtek.com.tw, 9 X-KSE-AntiSpam-Interceptor-Info: fallback X-KSE-Antivirus-Interceptor-Info: fallback X-KSE-AntiSpam-Interceptor-Info: fallback X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,SPF_HELO_NONE, SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1770017843274689868?= X-GMAIL-MSGID: =?utf-8?q?1770017843274689868?= In Realtek SoC, the parameter of usb phy is designed to can dynamic tuning base on port status. Therefore, add a notify callback of phy driver when usb port status change. The Realtek phy driver is designed to dynamically adjust disconnection level and calibrate phy parameters. When the device connected bit changes and when the disconnected bit changes, do port status change notification: Check if portstatus is USB_PORT_STAT_CONNECTION and portchange is USB_PORT_STAT_C_CONNECTION. 1. The device is connected, the driver lowers the disconnection level and calibrates the phy parameters. 2. The device disconnects, the driver increases the disconnect level and calibrates the phy parameters. When controller to notify connect that device is already ready. If we adjust the disconnection level in notify_connect, the disconnect may have been triggered at this stage. So we need to change that as early as possible. Therefore, we add an api to notify phy the port status changes. Signed-off-by: Stanley Chang --- v5 to v6 change: No change v4 to v5 change: No change v3 to v4 change: Fix the warning for checkpatch with strict. v2 to v3 change: Add more comments about the reason for adding this api v1 to v2 change: No change --- drivers/usb/core/hub.c | 13 +++++++++++++ include/linux/usb/phy.h | 13 +++++++++++++ 2 files changed, 26 insertions(+) diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c index 97a0f8faea6e..10f3364c3fc2 100644 --- a/drivers/usb/core/hub.c +++ b/drivers/usb/core/hub.c @@ -614,6 +614,19 @@ static int hub_ext_port_status(struct usb_hub *hub, int port1, int type, ret = 0; } mutex_unlock(&hub->status_mutex); + + if (!ret) { + struct usb_device *hdev = hub->hdev; + + if (hdev && !hdev->parent) { + struct usb_hcd *hcd = bus_to_hcd(hdev->bus); + + if (hcd->usb_phy) + usb_phy_notify_port_status(hcd->usb_phy, + port1 - 1, *status, *change); + } + } + return ret; } diff --git a/include/linux/usb/phy.h b/include/linux/usb/phy.h index e4de6bc1f69b..b513749582d7 100644 --- a/include/linux/usb/phy.h +++ b/include/linux/usb/phy.h @@ -144,6 +144,10 @@ struct usb_phy { */ int (*set_wakeup)(struct usb_phy *x, bool enabled); + /* notify phy port status change */ + int (*notify_port_status)(struct usb_phy *x, int port, + u16 portstatus, u16 portchange); + /* notify phy connect status change */ int (*notify_connect)(struct usb_phy *x, enum usb_device_speed speed); @@ -316,6 +320,15 @@ usb_phy_set_wakeup(struct usb_phy *x, bool enabled) return 0; } +static inline int +usb_phy_notify_port_status(struct usb_phy *x, int port, u16 portstatus, u16 portchange) +{ + if (x && x->notify_port_status) + return x->notify_port_status(x, port, portstatus, portchange); + else + return 0; +} + static inline int usb_phy_notify_connect(struct usb_phy *x, enum usb_device_speed speed) {