From nobody Tue Apr 7 04:45:25 2026 Received: from SEYPR02CU001.outbound.protection.outlook.com (mail-koreacentralazon11023121.outbound.protection.outlook.com [40.107.44.121]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0B03820468E; Mon, 16 Mar 2026 06:48:36 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.44.121 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773643720; cv=fail; b=QY0Mvb/vRjDhRUBHtxQ7LD3jBBFOlkjwMGfbli0DOD88y4aJszL+q9E9qLsaj5+dzXb9SphiNQFNYW2KmAhvXu+rhL8w/ETj1uP1eFeb17EK0EI9riIAQH3Ji8Kk+Wu1n5cpi7eEbsk8SxP3FbCmDbJc72n6B3a8n08zOOEMRTc= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773643720; c=relaxed/simple; bh=Z3GdLqnrFsh38eRhnmEKYt+MRYvjfHGtAVF/ReQqvMU=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=nrK+PAEw34S4ZHSXXcygaxAavss81+VwOKFqtopQxZz3TME035IqbC04cpU/VkChgMonTUiQQGy3xxqw2n6Qc67dNFSW02FHdkc2FiiSe9afm1+Hre8aUmUOmFxMYLm/gegSd1H79yZt2GvxwpqSoyktZsM53SyVuVZrvqT21z4= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=cixtech.com; spf=pass smtp.mailfrom=cixtech.com; arc=fail smtp.client-ip=40.107.44.121 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=cixtech.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=cixtech.com ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=GX1dgIoROvC2+jTtLjrK9adB9QrGgavNTp4OeDeUo82fBAMAPB1hY68WsBetRd6iVkjkwZMN/r6Of2YJkZJG3tnfMh8tkQEaT2VryhlsTtrJiYY4vMAAQ8EG/AUH1Me1j0fBbKGpyt5dZ+z9OP8oWlzq0sMUGDtEpF7228EiT2pPgGVkefjwEAOJOUX7ioLA3H/gIDkJ009vyLkfCw8tNnbGwclwXxxeSYZ+vR0pIvqu4W5tby99TinhZF6ff2EFeePDJtZBIM3nBXBfjfESsQiZaXOUrFKZyF+92jO3ykXuqIWqn/6qmImp8Rbhc7IQNrIkbeLYUXf9IRLvgPonEw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=sXDSnA+Ere+2fWtuh5VlU/p2cZ40qwmiFd6570xMffc=; b=dgGN0jem0hy9KQ+6CeWExpvqNCIbbvwwF9/DDmGPyJvmikWAhCDbLa+ZrdUdX4YrePaJ0DHp1KaflDdhnkkCAISGHKh5SAI91OECmJhXDvGrEzlXHXqLiwHxmUVoKvhAKV8rYWUQTFWGiX/6ALkTGlR3Wi+k2oyb5puAoOiLzHpVJ/YWwbx69YLlOHLN9wjcjQAX2Hq2XH6PlomOit2upBzWD2ydzBV7pDtVy/vjWTZts7vJtXB3f8i/V2WaEJ3YvDWQht0j338BradEk6GfyNC0cmzYVcd7rpJLb7NUENw1Ny2wgvmcimZBkmLfTg/ceAHmHmE1LTx+LlkuzFBJ7w== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 222.71.101.198) smtp.rcpttodomain=cadence.com smtp.mailfrom=cixtech.com; dmarc=bestguesspass action=none header.from=cixtech.com; dkim=none (message not signed); arc=none (0) Received: from TYCP286CA0170.JPNP286.PROD.OUTLOOK.COM (2603:1096:400:3c6::11) by SEYPR06MB6484.apcprd06.prod.outlook.com (2603:1096:101:171::5) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9700.24; Mon, 16 Mar 2026 06:48:33 +0000 Received: from TY2PEPF0000AB87.apcprd03.prod.outlook.com (2603:1096:400:3c6:cafe::7a) by TYCP286CA0170.outlook.office365.com (2603:1096:400:3c6::11) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.9700.24 via Frontend Transport; Mon, 16 Mar 2026 06:48:31 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 222.71.101.198) smtp.mailfrom=cixtech.com; dkim=none (message not signed) header.d=none;dmarc=bestguesspass action=none header.from=cixtech.com; Received-SPF: Pass (protection.outlook.com: domain of cixtech.com designates 222.71.101.198 as permitted sender) receiver=protection.outlook.com; client-ip=222.71.101.198; helo=smtprelay.cixcomputing.com; pr=C Received: from smtprelay.cixcomputing.com (222.71.101.198) by TY2PEPF0000AB87.mail.protection.outlook.com (10.167.253.6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9700.17 via Frontend Transport; Mon, 16 Mar 2026 06:48:32 +0000 Received: from nchen-desktop (unknown [172.16.64.25]) by smtprelay.cixcomputing.com (Postfix) with ESMTPSA id 728AB4126F87; Mon, 16 Mar 2026 14:48:31 +0800 (CST) From: Peter Chen To: robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, gregkh@linuxfoundation.org, pawell@cadence.com, rogerq@kernel.org Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org, cix-kernel-upstream@cixtech.com, Peter Chen Subject: [PATCH v2 1/2] dt-bindings: usb: cdns,usb3: document USBSSP controller support Date: Mon, 16 Mar 2026 14:48:30 +0800 Message-ID: <20260316064831.274865-2-peter.chen@cixtech.com> X-Mailer: git-send-email 2.50.1 In-Reply-To: <20260316064831.274865-1-peter.chen@cixtech.com> References: <20260316064831.274865-1-peter.chen@cixtech.com> 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 X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: TY2PEPF0000AB87:EE_|SEYPR06MB6484:EE_ X-MS-Office365-Filtering-Correlation-Id: 9028ad1b-9473-42b1-1563-08de83280a56 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|36860700016|82310400026|1800799024|376014|18002099003|22082099003|56012099003|7053199007; X-Microsoft-Antispam-Message-Info: EzNRkBIHikWLjTb+TpMnzbnO+9GMo8DzKoensDQwZPoV3lzkqwkibsdjEjpslLMVqifEpusDgExQJByRwmfCP730SrSqLoNKQ4ppavp1/LD28RiSsoLebeLojAV661XUo7hkZ96jcNVZQ82/Phd8Zq230/SQiiuWPHPJetqojdkUaMCaSySwlulZ8efz7quDfp/gTLVJNWOyHfziWqqWGwOHoIZMQ9DV60oYAk2rSZ0s5BB9jEWdfothfzgHFG5X+1goWQ69kPioYTHljbNul8LBQuNVYyRCo+NOzncft4rBLUo45EJ1H7sPK/lsEcUAwHaZtbFU1RmtqeDiDU0Ei8Mall7YdqgOmmIEAP/74zF+CXlOen8ZzgZab8ZvKt/4dJU7R2R6v2hjNeTzuLq95a6EUGzeNabeff8NvIk/cbF6mIwDuN8G4YEzcS0dzuBubyDM+uCZp0sevzQkoRrmfAXs4vfZBkRSDCTz7Rxz9OhoTnzyeUGOFn3FK01cgeqlIWyCUwHga+rbkrFEWXMtwpVby/2M5ib6zBE7MTGt0zy0Cc0TdoMcGlVr0GI1ea8rnaiyNHDOQ2LRQebVbACME5uEvWZKTS55M/KXG6CV3zddik736A/6JE5NMyd3tnA+cijFXsQSkJZzv9aArQtmjSMnm6FA2LLJTkcqDbmgo/Rm0T4Wl9s5Qgoky04vCv4xr6GhgKmIrqduXnwfJk7ZRkr8XUU4b13y0ucvzxyAPTo= X-Forefront-Antispam-Report: CIP:222.71.101.198;CTRY:CN;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:smtprelay.cixcomputing.com;PTR:ErrorRetry;CAT:NONE;SFS:(13230040)(36860700016)(82310400026)(1800799024)(376014)(18002099003)(22082099003)(56012099003)(7053199007);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: hua1lM5yQx4aDDRIwDl29LIjxh9In5Y1uX+5AHmLbrB9PzwRHfTcMo4F8OBy7K0/W2g2aoRH/WdJm92fvRHi/LqBrNbio/KPTqB18PSxThYV5ycMh2X9/awFAmaLjqxjdKvINY4dW4f0Id6OZLSDElGE78puvOXlDOBxPVacn8VobbBk4g6Bqt8B3gtZlomCNQ9/JINLp7Ea3lUOf50HaL5U7PkFnIgSQHA8orSywMHLqgRefa0e99cr6GrgTAf40D1QJjhY6OoORqeTnhWvg4teO7FjlqXbxRlJeIs1Qp0QHdCVVfoStTz3DsZ9PcA7m+CwW2l1gUh7IDru+vj3rowzLBVowVI0mfbnqAarCFQ82ShC/4zeRV6AHtsL5RmtW/qrjypmJPkDANnvch+ruppvUoYMkGhN2Nhes2Kaq8nUs652I+iRGgzPIjg/bnQ9 X-OriginatorOrg: cixtech.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Mar 2026 06:48:32.5893 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 9028ad1b-9473-42b1-1563-08de83280a56 X-MS-Exchange-CrossTenant-Id: 0409f77a-e53d-4d23-943e-ccade7cb4811 X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=0409f77a-e53d-4d23-943e-ccade7cb4811;Ip=[222.71.101.198];Helo=[smtprelay.cixcomputing.com] X-MS-Exchange-CrossTenant-AuthSource: TY2PEPF0000AB87.apcprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: SEYPR06MB6484 Update the Cadence USBSS DRD binding to document that it also covers the USBSSP (SuperSpeed Plus, USB 3.1 gen2x1) controller. Both USBSS and USBSSP share the same DRD/OTG register interface, so the driver auto-detects the controller version at runtime =E2=80=94 no additional compatible string is needed. Changes to the binding: - Update title and add description - maximum-speed: add super-speed-plus This patch is Assisted-by: Cursor:claude-4.6-opus Signed-off-by: Peter Chen Acked-by: Rob Herring (Arm) --- Documentation/devicetree/bindings/usb/cdns,usb3.yaml | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/usb/cdns,usb3.yaml b/Documen= tation/devicetree/bindings/usb/cdns,usb3.yaml index f454ddd9bbaa..ad5acf956edf 100644 --- a/Documentation/devicetree/bindings/usb/cdns,usb3.yaml +++ b/Documentation/devicetree/bindings/usb/cdns,usb3.yaml @@ -4,11 +4,17 @@ $id: http://devicetree.org/schemas/usb/cdns,usb3.yaml# $schema: http://devicetree.org/meta-schemas/core.yaml# =20 -title: Cadence USBSS-DRD controller +title: Cadence USBSS and USBSSP DRD controller =20 maintainers: - Pawel Laszczak =20 +description: + Cadence USB dual-role controller. Covers USBSS (SuperSpeed, USB 3.0) and + USBSSP (SuperSpeed Plus, USB 3.1 gen2x1). Both variants share the same + DRD/OTG register interface, so the driver auto-detects the controller + version at runtime. + properties: compatible: const: cdns,usb3 @@ -49,7 +55,7 @@ properties: cdns3 to type C connector. =20 maximum-speed: - enum: [super-speed, high-speed, full-speed] + enum: [super-speed-plus, super-speed, high-speed, full-speed] =20 phys: minItems: 1 --=20 2.50.1 From nobody Tue Apr 7 04:45:25 2026 Received: from TYDPR03CU002.outbound.protection.outlook.com (mail-japaneastazon11023135.outbound.protection.outlook.com [52.101.127.135]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6D50A5CDF1; Mon, 16 Mar 2026 06:48:37 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.127.135 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773643720; cv=fail; b=qtmiLrdfp8EAdUw8zoAJOIqaHrfXTszxEn3fWjSYJTafZyzA6Ac1rwbKlJpauCTyxDcZ6c/ogz5JmwdcW/0UCRszJRomHZudbVZf7zIO9KljqB2hTpwWTjneWTqe6XcFy2ZwINhRMYDQZU/WTkJ6pWbClXv8SEaPFGfd12gtIBM= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773643720; c=relaxed/simple; bh=ZoClpGwvwZwLmenzgXZ98iPCkLRG73o7JbwgHSQV1GI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=NWFX8E90e790BgLK2eeQFDakK9J4PtynzGnr2yYz86DdetvNgoIHojlY8y6hnrHQKgGgOIEIr/7v5n1Dle03X7kyboIZwzh930DxOcJooNHHVfVpU7bpK2I9H7xeNU/gphzEKtVJMGms8g3I1jQBrODlqR+K9BOknIY1fZ1MEQc= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=cixtech.com; spf=pass smtp.mailfrom=cixtech.com; arc=fail smtp.client-ip=52.101.127.135 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=cixtech.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=cixtech.com ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=dsNB6bp/rhmsX1Ep8TKipaOiE+VXI9ED3/DOTklP3GMIV1V/xqm6xHSy0fSZ3uPU/4AfZ3R+xYLp4JYQuGAerwfNTSDurxrpLD2a037z5u8uORsUHD8tHfI28BoxrEruISDBH+mKx3unbLWShmNaiiG9YMgagH1wx4VaUMZMyIa5Cupyul8OdQLEjhzcCK1KFBYHv7xWSnlozxFKfyeQHj3ccGLXmgrOIlptCFae8GkSw667kCTShnWadc/hZCj1UOzsnGQurPjcKSRdIWH3Rt0GGTVkBLJRVlc1Iqc5KyrFHSyvXp9e25ZGZSEQiNqE5s0BzIdrPN3O02F9biA5hA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=jX9Nkp5RpXDmU/Abq4XNFHzmCy1OCvZgTYQAuLcUrgQ=; b=JRd7ghg2hgEvRTLNlfVz0CJre1y9euyDVpJIC8/7yqe11St6WlRfuG5s/cxM+zlLimf50sw/O4z7on36BNz1uT9bN6lppTCbsm84MSOnZqgngRNRjVyVzGDf1/RnHpvpQ+tCsWA6M0iAlHD17pyarj5/8qU1EWLxWydRqtUu3har6LHR5doNIxMa2+KYIlgUWt3CGjTteOCY5QPbm6RNLnpFeN9ox7clTZnOtdYrjHMz6vgZrqCX/1t259kJEiazKBT6oMhz5WA8rdqHdIkYwljFQwopHHA4Wixjcxq5HtWUuTBVawpGO2djVEe/yyIBOuAYmIPOUzwyJUf3+K1adA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 222.71.101.198) smtp.rcpttodomain=cadence.com smtp.mailfrom=cixtech.com; dmarc=bestguesspass action=none header.from=cixtech.com; dkim=none (message not signed); arc=none (0) Received: from SG2PR01CA0198.apcprd01.prod.exchangelabs.com (2603:1096:4:189::7) by SEZPR06MB6549.apcprd06.prod.outlook.com (2603:1096:101:17e::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9700.22; Mon, 16 Mar 2026 06:48:34 +0000 Received: from OSA0EPF000000C6.apcprd02.prod.outlook.com (2603:1096:4:189:cafe::37) by SG2PR01CA0198.outlook.office365.com (2603:1096:4:189::7) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.9700.24 via Frontend Transport; Mon, 16 Mar 2026 06:48:29 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 222.71.101.198) smtp.mailfrom=cixtech.com; dkim=none (message not signed) header.d=none;dmarc=bestguesspass action=none header.from=cixtech.com; Received-SPF: Pass (protection.outlook.com: domain of cixtech.com designates 222.71.101.198 as permitted sender) receiver=protection.outlook.com; client-ip=222.71.101.198; helo=smtprelay.cixcomputing.com; pr=C Received: from smtprelay.cixcomputing.com (222.71.101.198) by OSA0EPF000000C6.mail.protection.outlook.com (10.167.240.52) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9700.17 via Frontend Transport; Mon, 16 Mar 2026 06:48:32 +0000 Received: from nchen-desktop (unknown [172.16.64.25]) by smtprelay.cixcomputing.com (Postfix) with ESMTPSA id 8137B4126F88; Mon, 16 Mar 2026 14:48:31 +0800 (CST) From: Peter Chen To: robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, gregkh@linuxfoundation.org, pawell@cadence.com, rogerq@kernel.org Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org, cix-kernel-upstream@cixtech.com, Peter Chen Subject: [PATCH v2 2/2] usb: cdns3: Add USBSSP platform driver support Date: Mon, 16 Mar 2026 14:48:31 +0800 Message-ID: <20260316064831.274865-3-peter.chen@cixtech.com> X-Mailer: git-send-email 2.50.1 In-Reply-To: <20260316064831.274865-1-peter.chen@cixtech.com> References: <20260316064831.274865-1-peter.chen@cixtech.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: OSA0EPF000000C6:EE_|SEZPR06MB6549:EE_ X-MS-Office365-Filtering-Correlation-Id: d5fc61d9-1d89-4082-9ee4-08de83280a3b X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|1800799024|376014|82310400026|36860700016|18002099003|22082099003|56012099003; X-Microsoft-Antispam-Message-Info: uEbDewr5f0WfM1Qc5ZQ1g1U4pi6tqx00AFfOzx4WhzLiQG06IYEY/sGu/tEE6k7pRRMl3w2aQMSf3PNpv43fyr3WNdrZtW8HOHUCnivBM4C3bX4OFo60iQh4SHm1rRCssBEb9X6FOeM2dj+0MI6VYPBStl52riWAjhkKtTJVnnOTUv6xA5JXb3EP62rqkmQG9ZNtn6PNm8JT01bSUjLbr1VBv3Ct9DdrMdMRG0g1d+5YfjIGvSBCCjKNLlS6yuQKk+6I9xfg6DuCujcplBdfpz2EpSC3Q/xG2xlRXI7Up8qwVyS5rRBZ5WOYy8NJOfSXhelZQDzCBMv58DEFCTXzRP30Akb+uK1ZBz2s3fjPGWonr8AZsT4X5CckXoCfuThTRsyWX9qJgc7SmbMEmlj875VN8EJ8JgrdSHLWGFB2Mk4guzBI0hSaheaVj8pxJC2FeJZpwHJ4lQVKE+SZ9eyzTEO8KnqgK/ncjkSCBWMzEGoiA9fUnYkGi1rhar2udub+PiyyZIRguAZqiO0DksY767wcz+1noQHxjQIcsAEpG+s8rhgXzSzQ3EJAJzXQD6TZRz8tEJ3SxSBZEujUws835vVkO3YDe5WHNSedFqqMT1dLu7/ywlqAP03sCUQALiJd/7vZo/tkgn8tAP5QYMuZCIo8F+5EOyT5WtN5XxAmyVk8PSlV6krJ1tlIdRlFIgismjgUIH0Z5++F82KrzpMESKmxPTzLB+7tKVRNdQVVqylY2ZXGI7D17T9CxI9dr4BOWcGluc1GiAh2VNA74WD1ow== X-Forefront-Antispam-Report: CIP:222.71.101.198;CTRY:CN;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:smtprelay.cixcomputing.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(1800799024)(376014)(82310400026)(36860700016)(18002099003)(22082099003)(56012099003);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: F07kB1V7hXt5eisYORmMmRgv2eU/AR2MaIb1SyX2JGcfPrLfjTKP+GdYYDMQtpgkwWiCFEq/tkCVwHBZ6/Lbfnd33NhN2RwWz0Boa3+px2ZE+hZdInce6hieBEcHjQoEcOvSO+lTlxRITMsjZjc60z0J/+z4mp5SW72CRSdFfpUvlGuPLnbKVBT44jOjTBfVNiwkg6EKaD+c4aAMQfhS7XA/3RVdT8rQC0lFHaay+nLBNHdTekf53hsLn/n5SBq/77ygcgz1ZPQcgDUF0B6NGnIZtm3gq1GWUlr5eq5jF2pwYc8VbcVdCjlqjBgzhrJWWM1G7ZWptaQ6nyRwSOzYpOkqKSTp5CyfD9V+xBG4yUKAZE5xUyxHNLEMfQv7MbYAmOEbdldiuvxnwmqR13yVNqU2GlBdSCxoYFocC3oRHPgbDY1bdbQ9Hlu7sCSH2i7t X-OriginatorOrg: cixtech.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Mar 2026 06:48:32.4349 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: d5fc61d9-1d89-4082-9ee4-08de83280a3b X-MS-Exchange-CrossTenant-Id: 0409f77a-e53d-4d23-943e-ccade7cb4811 X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=0409f77a-e53d-4d23-943e-ccade7cb4811;Ip=[222.71.101.198];Helo=[smtprelay.cixcomputing.com] X-MS-Exchange-CrossTenant-AuthSource: OSA0EPF000000C6.apcprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: SEZPR06MB6549 Content-Type: text/plain; charset="utf-8" The Cadence USBSSP (CDNSP) controller was previously only accessible through PCI, coupling the gadget driver with the PCI glue layer into a single monolithic module (cdnsp-udc-pci). This prevented using the CDNSP IP on SoC/platform designs that expose the controller through device tree. It restructures the driver to decouple the CDNSP gadget from PCI. - Introduce CONFIG_USB_CDNSP as a standalone tristate (analogous to CONFIG_USB_CDNS3), with USB_CDNSP_GADGET and USB_CDNSP_HOST as bool sub-options. The gadget code builds as a separate cdnsp.ko module. - Regroup USBSSP and CDNS3 Kconfig options under the USB_CDNS_SUPPORT menu so they appear properly grouped in menuconfig. - Refactor cdnsp-pci.c into a thin PCI-to-platform wrapper (similar to cdns3-pci-wrap.c) that registers a platform device and passes PCI resources and platform data to the common platform driver. - Auto-detect the controller version (USBSS vs USBSSP) at runtime by reading the DRD/OTG Device ID register in cdns_drd_init(), and select the appropriate gadget init function (cdns3_gadget_init or cdnsp_gadget_init) based on cdns->version. This follows the same pattern already used for host initialization. - Fix gadget-export.h to use IS_REACHABLE() keyed on the tristate module config (CONFIG_USB_CDNS3/CONFIG_USB_CDNSP) instead of IS_ENABLED() on the bool gadget config. The bool configs are always 'y' when enabled, causing IS_ENABLED/IS_REACHABLE to always return true and resulting in link errors when cdns-usb-common is built-in but the gadget module is loadable. - Add missing MODULE_LICENSE()/MODULE_DESCRIPTION() and EXPORT_SYMBOL_GPL() to the cdns3 and cdnsp gadget modules, required by modpost. - Pass override_apb_timeout through cdns3_platform_data so the PCI wrapper can communicate PCI-specific APB timeout values to the common driver. This patch is Assisted-by: Cursor:claude-4.6-opus Signed-off-by: Peter Chen --- drivers/usb/cdns3/Kconfig | 50 ++++--- drivers/usb/cdns3/Makefile | 30 ++--- drivers/usb/cdns3/cdns3-gadget.c | 4 + drivers/usb/cdns3/cdns3-plat.c | 17 ++- drivers/usb/cdns3/cdnsp-gadget.c | 4 + drivers/usb/cdns3/cdnsp-pci.c | 217 +++++++++++++----------------- drivers/usb/cdns3/core.c | 11 +- drivers/usb/cdns3/core.h | 5 +- drivers/usb/cdns3/gadget-export.h | 4 +- 9 files changed, 164 insertions(+), 178 deletions(-) diff --git a/drivers/usb/cdns3/Kconfig b/drivers/usb/cdns3/Kconfig index 0a514b591527..97fa84dddbca 100644 --- a/drivers/usb/cdns3/Kconfig +++ b/drivers/usb/cdns3/Kconfig @@ -20,10 +20,6 @@ config USB_CDNS3 Say Y here if your system has a Cadence USB3 dual-role controller. It supports: dual-role switch, Host-only, and Peripheral-only. =20 - If you choose to build this driver is a dynamically linked - as module, the module will be called cdns3.ko. -endif - if USB_CDNS3 =20 config USB_CDNS3_GADGET @@ -89,29 +85,27 @@ config USB_CDNS3_STARFIVE =20 If you choose to build this driver as module it will be dynamically linked and module will be called cdns3-starfive.ko -endif =20 -if USB_CDNS_SUPPORT +endif # USB_CDNS3 =20 -config USB_CDNSP_PCI - tristate "Cadence CDNSP Dual-Role Controller" - depends on USB_CDNS_SUPPORT && USB_PCI && ACPI +config USB_CDNSP + tristate "Cadence USBSSP Dual-Role Controller" + depends on USB_CDNS_SUPPORT help - Say Y here if your system has a Cadence CDNSP dual-role controller. - It supports: dual-role switch Host-only, and Peripheral-only. - - If you choose to build this driver is a dynamically linked - module, the module will be called cdnsp.ko. -endif + Say Y here if your system has a Cadence USBSSP dual-role controller. + It supports: dual-role switch, Host-only, and Peripheral-only. + Cadence CDNSP Controller device mode is very similar to XHCI controller. + Therefore some algorithms used has been taken from xHCI driver. + Host controller is compliant with XHCI so it uses standard XHCI driver. =20 -if USB_CDNSP_PCI +if USB_CDNSP =20 config USB_CDNSP_GADGET - bool "Cadence CDNSP device controller" - depends on USB_GADGET=3Dy || USB_GADGET=3DUSB_CDNSP_PCI + bool "Cadence USBSSP device controller" + depends on USB_GADGET=3Dy || USB_GADGET=3DUSB_CDNSP help Say Y here to enable device controller functionality of the - Cadence CDNSP-DEV driver. + Cadence USBSSP-DEV driver. =20 Cadence CDNSP Device Controller in device mode is very similar to XHCI controller. Therefore some algorithms @@ -120,8 +114,8 @@ config USB_CDNSP_GADGET It doesn't support LS. =20 config USB_CDNSP_HOST - bool "Cadence CDNSP host controller" - depends on USB=3Dy || USB=3DUSB_CDNSP_PCI + bool "Cadence USBSSP host controller" + depends on USB=3Dy || USB=3DUSB_CDNSP select USB_CDNS_HOST help Say Y here to enable host controller functionality of the @@ -130,4 +124,16 @@ config USB_CDNSP_HOST Host controller is compliant with XHCI so it uses standard XHCI driver. =20 -endif +config USB_CDNSP_PCI + tristate "Cadence USBSSP support on PCIe-based platforms" + depends on USB_PCI && ACPI + help + If you're using the USBSSP Core IP with a PCIe, please say + 'Y' or 'M' here. + + If you choose to build this driver as module it will + be dynamically linked and module will be called cdnsp-pci.ko + +endif # USB_CDNSP + +endif # USB_CDNS_SUPPORT diff --git a/drivers/usb/cdns3/Makefile b/drivers/usb/cdns3/Makefile index 48dfae75b5aa..63484f145bb9 100644 --- a/drivers/usb/cdns3/Makefile +++ b/drivers/usb/cdns3/Makefile @@ -4,41 +4,33 @@ CFLAGS_cdns3-trace.o :=3D -I$(src) CFLAGS_cdnsp-trace.o :=3D -I$(src) =20 cdns-usb-common-y :=3D core.o drd.o -cdns3-y :=3D cdns3-plat.o =20 ifeq ($(CONFIG_USB),m) obj-m +=3D cdns-usb-common.o -obj-m +=3D cdns3.o +obj-m +=3D cdns3-plat.o else obj-$(CONFIG_USB_CDNS_SUPPORT) +=3D cdns-usb-common.o -obj-$(CONFIG_USB_CDNS3) +=3D cdns3.o +obj-$(CONFIG_USB_CDNS_SUPPORT) +=3D cdns3-plat.o endif =20 cdns-usb-common-$(CONFIG_USB_CDNS_HOST) +=3D host.o -cdns3-$(CONFIG_USB_CDNS3_GADGET) +=3D cdns3-gadget.o cdns3-ep0.o =20 +# For CDNS3 gadget ifneq ($(CONFIG_USB_CDNS3_GADGET),) +cdns3-y :=3D cdns3-gadget.o cdns3-ep0.o cdns3-$(CONFIG_TRACING) +=3D cdns3-trace.o +obj-$(CONFIG_USB_CDNS3) +=3D cdns3.o endif - obj-$(CONFIG_USB_CDNS3_PCI_WRAP) +=3D cdns3-pci-wrap.o obj-$(CONFIG_USB_CDNS3_TI) +=3D cdns3-ti.o obj-$(CONFIG_USB_CDNS3_IMX) +=3D cdns3-imx.o obj-$(CONFIG_USB_CDNS3_STARFIVE) +=3D cdns3-starfive.o =20 -cdnsp-udc-pci-y :=3D cdnsp-pci.o - -ifdef CONFIG_USB_CDNSP_PCI -ifeq ($(CONFIG_USB),m) -obj-m +=3D cdnsp-udc-pci.o -else -obj-$(CONFIG_USB_CDNSP_PCI) +=3D cdnsp-udc-pci.o -endif -endif - -cdnsp-udc-pci-$(CONFIG_USB_CDNSP_GADGET) +=3D cdnsp-ring.o cdnsp-gadget.o \ - cdnsp-mem.o cdnsp-ep0.o - +# For CDNSP gadget ifneq ($(CONFIG_USB_CDNSP_GADGET),) -cdnsp-udc-pci-$(CONFIG_TRACING) +=3D cdnsp-trace.o +cdnsp-y :=3D cdnsp-ring.o cdnsp-gadget.o \ + cdnsp-mem.o cdnsp-ep0.o +cdnsp-$(CONFIG_TRACING) +=3D cdnsp-trace.o +obj-$(CONFIG_USB_CDNSP) +=3D cdnsp.o endif +obj-$(CONFIG_USB_CDNSP_PCI) +=3D cdnsp-pci.o diff --git a/drivers/usb/cdns3/cdns3-gadget.c b/drivers/usb/cdns3/cdns3-gad= get.c index d59a60a16ec7..b800bd1bedd4 100644 --- a/drivers/usb/cdns3/cdns3-gadget.c +++ b/drivers/usb/cdns3/cdns3-gadget.c @@ -3508,3 +3508,7 @@ int cdns3_gadget_init(struct cdns *cdns) =20 return 0; } +EXPORT_SYMBOL_GPL(cdns3_gadget_init); + +MODULE_LICENSE("GPL"); +MODULE_DESCRIPTION("Cadence USBSS DRD Driver - gadget"); diff --git a/drivers/usb/cdns3/cdns3-plat.c b/drivers/usb/cdns3/cdns3-plat.c index 735df88774e4..71c612e27b73 100644 --- a/drivers/usb/cdns3/cdns3-plat.c +++ b/drivers/usb/cdns3/cdns3-plat.c @@ -44,6 +44,14 @@ static void set_phy_power_off(struct cdns *cdns) phy_power_off(cdns->usb2_phy); } =20 +static int cdns3_plat_gadget_init(struct cdns *cdns) +{ + if (cdns->version < CDNSP_CONTROLLER_V2) + return cdns3_gadget_init(cdns); + else + return cdnsp_gadget_init(cdns); +} + /** * cdns3_plat_probe - probe for cdns3 core device * @pdev: Pointer to cdns3 core platform device @@ -64,6 +72,8 @@ static int cdns3_plat_probe(struct platform_device *pdev) =20 cdns->dev =3D dev; cdns->pdata =3D dev_get_platdata(dev); + if (cdns->pdata && cdns->pdata->override_apb_timeout) + cdns->override_apb_timeout =3D cdns->pdata->override_apb_timeout; =20 platform_set_drvdata(pdev, cdns); =20 @@ -143,12 +153,15 @@ static int cdns3_plat_probe(struct platform_device *p= dev) if (ret) goto err_phy_power_on; =20 - cdns->gadget_init =3D cdns3_gadget_init; - ret =3D cdns_init(cdns); if (ret) goto err_cdns_init; =20 + cdns->gadget_init =3D cdns3_plat_gadget_init; + ret =3D cdns_core_init_role(cdns); + if (ret) + goto err_cdns_init; + device_set_wakeup_capable(dev, true); pm_runtime_set_active(dev); pm_runtime_enable(dev); diff --git a/drivers/usb/cdns3/cdnsp-gadget.c b/drivers/usb/cdns3/cdnsp-gad= get.c index 6b3815f8a6e5..8db7eee528a1 100644 --- a/drivers/usb/cdns3/cdnsp-gadget.c +++ b/drivers/usb/cdns3/cdnsp-gadget.c @@ -2075,3 +2075,7 @@ int cdnsp_gadget_init(struct cdns *cdns) =20 return 0; } +EXPORT_SYMBOL_GPL(cdnsp_gadget_init); + +MODULE_LICENSE("GPL"); +MODULE_DESCRIPTION("Cadence CDNSP DRD Driver - gadget"); diff --git a/drivers/usb/cdns3/cdnsp-pci.c b/drivers/usb/cdns3/cdnsp-pci.c index 566d94e49102..432007cfe695 100644 --- a/drivers/usb/cdns3/cdnsp-pci.c +++ b/drivers/usb/cdns3/cdnsp-pci.c @@ -1,6 +1,6 @@ // SPDX-License-Identifier: GPL-2.0 /* - * Cadence PCI Glue driver. + * Cadence USBSSP PCI Glue driver. * * Copyright (C) 2019 Cadence. * @@ -16,7 +16,19 @@ #include =20 #include "core.h" -#include "gadget-export.h" + +struct cdnsp_wrap { + struct platform_device *plat_dev; + struct resource dev_res[6]; + int devfn; +}; + +#define RES_IRQ_HOST_ID 0 +#define RES_IRQ_PERIPHERAL_ID 1 +#define RES_IRQ_OTG_ID 2 +#define RES_HOST_ID 3 +#define RES_DEV_ID 4 +#define RES_DRD_ID 5 =20 #define PCI_BAR_HOST 0 #define PCI_BAR_OTG 0 @@ -26,16 +38,16 @@ #define PCI_DEV_FN_OTG 1 =20 #define PCI_DRIVER_NAME "cdns-pci-usbssp" -#define PLAT_DRIVER_NAME "cdns-usbssp" +#define PLAT_DRIVER_NAME "cdns-usb3" =20 -#define CHICKEN_APB_TIMEOUT_VALUE 0x1C20 +#define CHICKEN_APB_TIMEOUT_VALUE 0x1C20 =20 static struct pci_dev *cdnsp_get_second_fun(struct pci_dev *pdev) { /* * Gets the second function. - * Platform has two function. The fist keeps resources for - * Host/Device while the secon keeps resources for DRD/OTG. + * Platform has two function. The first keeps resources for + * Host/Device while the second keeps resources for DRD/OTG. */ if (pdev->device =3D=3D PCI_DEVICE_ID_CDNS_USBSSP) return pci_get_device(pdev->vendor, PCI_DEVICE_ID_CDNS_USBSS, NULL); @@ -48,11 +60,12 @@ static struct pci_dev *cdnsp_get_second_fun(struct pci_= dev *pdev) static int cdnsp_pci_probe(struct pci_dev *pdev, const struct pci_device_id *id) { - struct device *dev =3D &pdev->dev; - struct pci_dev *func; + struct platform_device_info plat_info; + static struct cdns3_platform_data pdata; + struct cdnsp_wrap *wrap; struct resource *res; - struct cdns *cdnsp; - int ret; + struct pci_dev *func; + int ret =3D 0; =20 /* * For GADGET/HOST PCI (devfn) function number is 0, @@ -79,146 +92,105 @@ static int cdnsp_pci_probe(struct pci_dev *pdev, } =20 pci_set_master(pdev); + if (pci_is_enabled(func)) { - cdnsp =3D pci_get_drvdata(func); + wrap =3D pci_get_drvdata(func); } else { - cdnsp =3D kzalloc_obj(*cdnsp); - if (!cdnsp) { + wrap =3D kzalloc_obj(*wrap); + if (!wrap) { ret =3D -ENOMEM; goto put_pci; } } =20 - /* For GADGET device function number is 0. */ - if (pdev->devfn =3D=3D 0) { - resource_size_t rsrc_start, rsrc_len; - - /* Function 0: host(BAR_0) + device(BAR_1).*/ - dev_dbg(dev, "Initialize resources\n"); - rsrc_start =3D pci_resource_start(pdev, PCI_BAR_DEV); - rsrc_len =3D pci_resource_len(pdev, PCI_BAR_DEV); - res =3D devm_request_mem_region(dev, rsrc_start, rsrc_len, "dev"); - if (!res) { - dev_dbg(dev, "controller already in use\n"); - ret =3D -EBUSY; - goto free_cdnsp; - } - - cdnsp->dev_regs =3D devm_ioremap(dev, rsrc_start, rsrc_len); - if (!cdnsp->dev_regs) { - dev_dbg(dev, "error mapping memory\n"); - ret =3D -EFAULT; - goto free_cdnsp; - } - - cdnsp->dev_irq =3D pdev->irq; - dev_dbg(dev, "USBSS-DEV physical base addr: %pa\n", - &rsrc_start); - - res =3D &cdnsp->xhci_res[0]; - res->start =3D pci_resource_start(pdev, PCI_BAR_HOST); - res->end =3D pci_resource_end(pdev, PCI_BAR_HOST); - res->name =3D "xhci"; - res->flags =3D IORESOURCE_MEM; - dev_dbg(dev, "USBSS-XHCI physical base addr: %pa\n", - &res->start); - - /* Interrupt for XHCI, */ - res =3D &cdnsp->xhci_res[1]; - res->start =3D pdev->irq; - res->name =3D "host"; - res->flags =3D IORESOURCE_IRQ; + res =3D wrap->dev_res; + + if (pdev->devfn =3D=3D PCI_DEV_FN_HOST_DEVICE) { + /* Function 0: host(BAR_0) + device(BAR_2). */ + dev_dbg(&pdev->dev, "Initialize Device resources\n"); + res[RES_DEV_ID].start =3D pci_resource_start(pdev, PCI_BAR_DEV); + res[RES_DEV_ID].end =3D pci_resource_end(pdev, PCI_BAR_DEV); + res[RES_DEV_ID].name =3D "dev"; + res[RES_DEV_ID].flags =3D IORESOURCE_MEM; + dev_dbg(&pdev->dev, "USBSSP-DEV physical base addr: %pa\n", + &res[RES_DEV_ID].start); + + res[RES_HOST_ID].start =3D pci_resource_start(pdev, PCI_BAR_HOST); + res[RES_HOST_ID].end =3D pci_resource_end(pdev, PCI_BAR_HOST); + res[RES_HOST_ID].name =3D "xhci"; + res[RES_HOST_ID].flags =3D IORESOURCE_MEM; + dev_dbg(&pdev->dev, "USBSSP-XHCI physical base addr: %pa\n", + &res[RES_HOST_ID].start); + + /* Interrupt for XHCI */ + wrap->dev_res[RES_IRQ_HOST_ID].start =3D pdev->irq; + wrap->dev_res[RES_IRQ_HOST_ID].name =3D "host"; + wrap->dev_res[RES_IRQ_HOST_ID].flags =3D IORESOURCE_IRQ; + + /* Interrupt for device. It's the same as for HOST. */ + wrap->dev_res[RES_IRQ_PERIPHERAL_ID].start =3D pdev->irq; + wrap->dev_res[RES_IRQ_PERIPHERAL_ID].name =3D "peripheral"; + wrap->dev_res[RES_IRQ_PERIPHERAL_ID].flags =3D IORESOURCE_IRQ; } else { - res =3D &cdnsp->otg_res; - res->start =3D pci_resource_start(pdev, PCI_BAR_OTG); - res->end =3D pci_resource_end(pdev, PCI_BAR_OTG); - res->name =3D "otg"; - res->flags =3D IORESOURCE_MEM; - dev_dbg(dev, "CDNSP-DRD physical base addr: %pa\n", - &res->start); + res[RES_DRD_ID].start =3D pci_resource_start(pdev, PCI_BAR_OTG); + res[RES_DRD_ID].end =3D pci_resource_end(pdev, PCI_BAR_OTG); + res[RES_DRD_ID].name =3D "otg"; + res[RES_DRD_ID].flags =3D IORESOURCE_MEM; + dev_dbg(&pdev->dev, "CDNSP-DRD physical base addr: %pa\n", + &res[RES_DRD_ID].start); =20 /* Interrupt for OTG/DRD. */ - cdnsp->otg_irq =3D pdev->irq; + wrap->dev_res[RES_IRQ_OTG_ID].start =3D pdev->irq; + wrap->dev_res[RES_IRQ_OTG_ID].name =3D "otg"; + wrap->dev_res[RES_IRQ_OTG_ID].flags =3D IORESOURCE_IRQ; } =20 - /* - * Cadence PCI based platform require some longer timeout for APB - * to fixes domain clock synchronization issue after resuming - * controller from L1 state. - */ - cdnsp->override_apb_timeout =3D CHICKEN_APB_TIMEOUT_VALUE; - pci_set_drvdata(pdev, cdnsp); - if (pci_is_enabled(func)) { - cdnsp->dev =3D dev; - cdnsp->gadget_init =3D cdnsp_gadget_init; - - ret =3D cdns_init(cdnsp); - if (ret) - goto free_cdnsp; + /* set up platform device info */ + pdata.override_apb_timeout =3D CHICKEN_APB_TIMEOUT_VALUE; + memset(&plat_info, 0, sizeof(plat_info)); + plat_info.parent =3D &pdev->dev; + plat_info.fwnode =3D pdev->dev.fwnode; + plat_info.name =3D PLAT_DRIVER_NAME; + plat_info.id =3D pdev->devfn; + plat_info.res =3D wrap->dev_res; + plat_info.num_res =3D ARRAY_SIZE(wrap->dev_res); + plat_info.dma_mask =3D pdev->dma_mask; + plat_info.data =3D &pdata; + plat_info.size_data =3D sizeof(pdata); + wrap->devfn =3D pdev->devfn; + /* register platform device */ + wrap->plat_dev =3D platform_device_register_full(&plat_info); + if (IS_ERR(wrap->plat_dev)) { + ret =3D PTR_ERR(wrap->plat_dev); + kfree(wrap); + goto put_pci; + } } =20 - device_wakeup_enable(&pdev->dev); - if (pci_dev_run_wake(pdev)) - pm_runtime_put_noidle(&pdev->dev); - - return 0; - -free_cdnsp: - if (!pci_is_enabled(func)) - kfree(cdnsp); - + pci_set_drvdata(pdev, wrap); put_pci: pci_dev_put(func); - return ret; } =20 static void cdnsp_pci_remove(struct pci_dev *pdev) { - struct cdns *cdnsp; + struct cdnsp_wrap *wrap; struct pci_dev *func; =20 func =3D cdnsp_get_second_fun(pdev); - cdnsp =3D (struct cdns *)pci_get_drvdata(pdev); + wrap =3D pci_get_drvdata(pdev); =20 - if (pci_dev_run_wake(pdev)) - pm_runtime_get_noresume(&pdev->dev); + if (wrap->devfn =3D=3D pdev->devfn) + platform_device_unregister(wrap->plat_dev); =20 - if (pci_is_enabled(func)) { - cdns_remove(cdnsp); - } else { - kfree(cdnsp); - } + if (!pci_is_enabled(func)) + kfree(wrap); =20 pci_dev_put(func); } =20 -static int __maybe_unused cdnsp_pci_suspend(struct device *dev) -{ - struct cdns *cdns =3D dev_get_drvdata(dev); - - return cdns_suspend(cdns); -} - -static int __maybe_unused cdnsp_pci_resume(struct device *dev) -{ - struct cdns *cdns =3D dev_get_drvdata(dev); - unsigned long flags; - int ret; - - spin_lock_irqsave(&cdns->lock, flags); - ret =3D cdns_resume(cdns); - spin_unlock_irqrestore(&cdns->lock, flags); - cdns_set_active(cdns, 1); - - return ret; -} - -static const struct dev_pm_ops cdnsp_pci_pm_ops =3D { - SET_SYSTEM_SLEEP_PM_OPS(cdnsp_pci_suspend, cdnsp_pci_resume) -}; - static const struct pci_device_id cdnsp_pci_ids[] =3D { { PCI_DEVICE(PCI_VENDOR_ID_CDNS, PCI_DEVICE_ID_CDNS_USBSSP), .class =3D PCI_CLASS_SERIAL_USB_DEVICE }, @@ -230,13 +202,10 @@ static const struct pci_device_id cdnsp_pci_ids[] =3D= { }; =20 static struct pci_driver cdnsp_pci_driver =3D { - .name =3D "cdnsp-pci", + .name =3D PCI_DRIVER_NAME, .id_table =3D cdnsp_pci_ids, .probe =3D cdnsp_pci_probe, .remove =3D cdnsp_pci_remove, - .driver =3D { - .pm =3D &cdnsp_pci_pm_ops, - } }; =20 module_pci_driver(cdnsp_pci_driver); @@ -245,4 +214,4 @@ MODULE_DEVICE_TABLE(pci, cdnsp_pci_ids); MODULE_ALIAS("pci:cdnsp"); MODULE_AUTHOR("Pawel Laszczak "); MODULE_LICENSE("GPL v2"); -MODULE_DESCRIPTION("Cadence CDNSP PCI driver"); +MODULE_DESCRIPTION("Cadence CDNSP PCI wrapper"); diff --git a/drivers/usb/cdns3/core.c b/drivers/usb/cdns3/core.c index f0e32227c0b7..10f00b6c3c83 100644 --- a/drivers/usb/cdns3/core.c +++ b/drivers/usb/cdns3/core.c @@ -80,7 +80,7 @@ static void cdns_exit_roles(struct cdns *cdns) * * Returns 0 on success otherwise negative errno */ -static int cdns_core_init_role(struct cdns *cdns) +int cdns_core_init_role(struct cdns *cdns) { struct device *dev =3D cdns->dev; enum usb_dr_mode best_dr_mode; @@ -197,11 +197,14 @@ static int cdns_core_init_role(struct cdns *cdns) goto err; } =20 + dev_dbg(dev, "Cadence USB3 core: probe succeed\n"); + return 0; err: cdns_exit_roles(cdns); return ret; } +EXPORT_SYMBOL_GPL(cdns_core_init_role); =20 /** * cdns_hw_role_state_machine - role switch state machine based on hw eve= nts. @@ -469,14 +472,8 @@ int cdns_init(struct cdns *cdns) if (ret) goto init_failed; =20 - ret =3D cdns_core_init_role(cdns); - if (ret) - goto init_failed; - spin_lock_init(&cdns->lock); =20 - dev_dbg(dev, "Cadence USB3 core: probe succeed\n"); - return 0; init_failed: cdns_drd_exit(cdns); diff --git a/drivers/usb/cdns3/core.h b/drivers/usb/cdns3/core.h index 801be9e61340..dc8c4137de15 100644 --- a/drivers/usb/cdns3/core.h +++ b/drivers/usb/cdns3/core.h @@ -45,6 +45,7 @@ struct cdns3_platform_data { unsigned long quirks; #define CDNS3_DEFAULT_PM_RUNTIME_ALLOW BIT(0) #define CDNS3_DRD_SUSPEND_RESIDENCY_ENABLE BIT(1) + u32 override_apb_timeout; /* 0 =3D use default (e.g. for PCI) */ }; =20 /** @@ -119,14 +120,14 @@ struct cdns { struct cdns3_platform_data *pdata; spinlock_t lock; struct xhci_plat_priv *xhci_plat_data; - u32 override_apb_timeout; - int (*gadget_init)(struct cdns *cdns); + u32 override_apb_timeout; }; =20 int cdns_hw_role_switch(struct cdns *cdns); int cdns_init(struct cdns *cdns); int cdns_remove(struct cdns *cdns); +int cdns_core_init_role(struct cdns *cdns); =20 #ifdef CONFIG_PM_SLEEP int cdns_resume(struct cdns *cdns); diff --git a/drivers/usb/cdns3/gadget-export.h b/drivers/usb/cdns3/gadget-e= xport.h index c37b6269b001..0cb600e2b5d2 100644 --- a/drivers/usb/cdns3/gadget-export.h +++ b/drivers/usb/cdns3/gadget-export.h @@ -10,7 +10,7 @@ #ifndef __LINUX_CDNS3_GADGET_EXPORT #define __LINUX_CDNS3_GADGET_EXPORT =20 -#if IS_ENABLED(CONFIG_USB_CDNSP_GADGET) +#if defined(CONFIG_USB_CDNSP_GADGET) && IS_REACHABLE(CONFIG_USB_CDNSP) =20 int cdnsp_gadget_init(struct cdns *cdns); #else @@ -22,7 +22,7 @@ static inline int cdnsp_gadget_init(struct cdns *cdns) =20 #endif /* CONFIG_USB_CDNSP_GADGET */ =20 -#if IS_ENABLED(CONFIG_USB_CDNS3_GADGET) +#if defined(CONFIG_USB_CDNS3_GADGET) && IS_REACHABLE(CONFIG_USB_CDNS3) =20 int cdns3_gadget_init(struct cdns *cdns); #else --=20 2.50.1