From nobody Wed Oct 1 22:31:20 2025 Received: from DM5PR21CU001.outbound.protection.outlook.com (mail-centralusazon11011048.outbound.protection.outlook.com [52.101.62.48]) (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 13DDB299AB3; Fri, 26 Sep 2025 10:54:16 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.62.48 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758884059; cv=fail; b=Q3WrKtYa5cqntgOaGm0TWPL0ZBFzRuZ4fmGvxewTBPDsFA3zkcJKEuMpURzfkV5fSSbB9g3gtYLzgCPv/d2yTcnqpOcFhAGNCDSyp9Fw5PztF3Q7GsF5wvshrUCJbQq7rxZq/xl/6/xGpwPyCJE5MJk9HlLzStPgYIdlRmjI1eM= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758884059; c=relaxed/simple; bh=sSqr/oUbOSFIc25Rs5dZOBHBJh3QW6Zpxi1JKBKdIHI=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=FyCQAzh8/MvrggOq9NeB7/+ECht2gDusJMRXUtp7gxOmkRyGoBC96jdH03Z1cO5nG81X7oqplC2RBxg9sUBXhHZkZGBFx+bYna1U3bh/VDnHkS3BqGMtsfbdWP1zzGchYuJTqkT18CQURSD39C3rAb40BvT3ycNOVERltOWXi7M= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com; spf=fail smtp.mailfrom=amd.com; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b=nn4DiteV; arc=fail smtp.client-ip=52.101.62.48 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="nn4DiteV" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=unMMwTxpTJGLSkhcS7oPhH6MJSGagrZPjhAVQ6d0sc55jOru3h0dO/vFudj9GBPFtMUpUS0Q6KfUmV+XBvyv4WZj85LSU+YCxz6ImlJzb53WFLMssRemnr8mdolVxapnVDOhrdvKGU6gdGeIUKzj81Evd/YTYFv88aS5TCuV32SdCji+bxqQ/s75c+jZ3Zg0pMHGOf15U9XwDUbn/5IAnLiqMeplhurRcKgc1pttLmoGCp6Kq1Y3SJrC9aonDDHVu53ijL7W7ep3aVzkq/dklZN9bSkgDsGTwvMpKLkzd3LMSG/yhBEFtsxlsMDq71fuyC3Q/7CsC5vqv+RUlleH9Q== 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=XIlS8VG9BhAuPq8+q2wxIohgg8u2w+HSy/2b+JV/3Ac=; b=X/6pLUKiBviXz2wQd+pLL67THTjdQ85k+TFRLguXe9Mpa+Uj72qQ0UopmQ08j8GSHKJWEYKZRD+7jMHupNJbgvJ07CF6UU+Hj0to82JgpEyk3RNMy3WH//qs6DGg9EfbtM3CTm0OlZ39l0eSBwJ7MPDm8Nt1lzDjDnhrK41wWSO9B0n9MLfo1uXFMNwx9WrBcRZYXmnfH7+PJj0K0aCAWBysRxOKY+Ymqt1tY1lbChV3dkQa++9l5c1sqaNmsmzWwIGiWIiVGUn/6Fq62uZwKNmyw2FVifmgJLeuxcWaNwpqbd2c5VhM6bi04y0Hr0gYpRld7XB9ILsTha5f59saBg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=gmail.com smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=XIlS8VG9BhAuPq8+q2wxIohgg8u2w+HSy/2b+JV/3Ac=; b=nn4DiteVCeXmePzO4JG3eQNkt810tl68HEwRUkuxBv2I9TS2wI8/3BYODXd8DOITzAuE42KE4CsB6Uz5VN7Q4uV3a5GncwEeByKGojcCdSK1947v1krksxnu9lAEz8cxkDkgzHxSACLlUhl6T5Tv6c+054nSeewbq/l+tpba5mM= Received: from BL0PR05CA0029.namprd05.prod.outlook.com (2603:10b6:208:91::39) by PH7PR12MB6954.namprd12.prod.outlook.com (2603:10b6:510:1b7::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9160.10; Fri, 26 Sep 2025 10:54:10 +0000 Received: from BL02EPF0002992E.namprd02.prod.outlook.com (2603:10b6:208:91:cafe::71) by BL0PR05CA0029.outlook.office365.com (2603:10b6:208:91::39) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.9182.6 via Frontend Transport; Fri, 26 Sep 2025 10:54:10 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=satlexmb08.amd.com; pr=C Received: from satlexmb08.amd.com (165.204.84.17) by BL02EPF0002992E.mail.protection.outlook.com (10.167.249.59) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9160.9 via Frontend Transport; Fri, 26 Sep 2025 10:54:10 +0000 Received: from SATLEXMB06.amd.com (10.181.40.147) by satlexmb08.amd.com (10.181.42.217) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.2.2562.17; Fri, 26 Sep 2025 03:54:09 -0700 Received: from satlexmb08.amd.com (10.181.42.217) by SATLEXMB06.amd.com (10.181.40.147) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Fri, 26 Sep 2025 05:54:08 -0500 Received: from xhdradheys41.xilinx.com (10.180.168.240) by satlexmb08.amd.com (10.181.42.217) with Microsoft SMTP Server id 15.2.2562.17 via Frontend Transport; Fri, 26 Sep 2025 03:54:01 -0700 From: Manikanta Guntupalli To: , , , , , , , , , , , , , , , , , , , , , , , CC: , , , , Manikanta Guntupalli Subject: [PATCH V8 1/5] dt-bindings: i3c: Add AMD I3C master controller support Date: Fri, 26 Sep 2025 16:23:45 +0530 Message-ID: <20250926105349.2932952-2-manikanta.guntupalli@amd.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20250926105349.2932952-1-manikanta.guntupalli@amd.com> References: <20250926105349.2932952-1-manikanta.guntupalli@amd.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: BL02EPF0002992E:EE_|PH7PR12MB6954:EE_ X-MS-Office365-Filtering-Correlation-Id: d6b2568b-9039-4c02-2b6c-08ddfceb0626 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|82310400026|7416014|376014|36860700013|1800799024|13003099007|921020; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?UtHl8W6p/cNIMpFW8hyh8MZLiU/j90fBbsKQ3ZsHgDG17ZJsJ6anayiU0EVz?= =?us-ascii?Q?H9tGRv9t6CFMFCNjfQr7DIx6BBnkokcxWzxsKxqRo/itxn6eeFsjSB7/m8du?= =?us-ascii?Q?jdy/i8M1y4GUk6YhevKDbtviERBBLNyCOkVac9/+yBvlSPxG6GsViOP4eoSp?= =?us-ascii?Q?8M+RqdSP3BbqLYIXFPhklYUlxIs91DgJD9A5NmwiKxuH7VbcAy+y0JHZuHOk?= =?us-ascii?Q?nH2sSgKRkW9gLZCzd9lk4tRsSkxghSGZC0mXpWr0JyzuT3Pv7/aq3FhdFn9O?= =?us-ascii?Q?m9R7kMPyyJLWgmpT5NfTdPyF+MnVGNBrKffRwyNZMIM7TDHTSNOKCFB6rKMn?= =?us-ascii?Q?Lq7nC8GrpwC4I0PCKwy8YOMjAMas1vmWzRcWOsdjltzMGpgGlczyWsiVJYIN?= =?us-ascii?Q?0RsHNVotdx6vkkX7BiRVw5UXuCb7JcyrSC4Cc885am9jwMsrDlBoSiPkgngQ?= =?us-ascii?Q?9SydnWdNOxD4kHnL4aaiSQUr6hD8AzEJSHgtDQXW8nLdq1kFCEIn1v7uIPuo?= =?us-ascii?Q?Bp+QXAD+llJbaVR7f0i/Q9Y10EDuO3VzBxHkROLmFiXPqZUsua5uexKEOHPo?= =?us-ascii?Q?3cqb6breOf19/z7fkrfrfRYdYVVQE/IKZKK4kJZzOd+/zISKmnKJMUbrJukU?= =?us-ascii?Q?t26Qwec259cmTrGBIH/IW7qOu3JkV59YRxRqnaDwSO7/EvJyINkZlxOm4scT?= =?us-ascii?Q?VD6NXf1F4cFWSufGLFKntLtA728aHt5fQscNXeMYzNhnB63L72PI8NBagZYz?= =?us-ascii?Q?0nU1aJoiAxjlkslAiDcLpk8nQDE/j/+LEjePlx6jwGFCqlBDbI8h41grClB/?= =?us-ascii?Q?GfOo55K9Pg3bj27OOhtSXNomJnU93naT6ikhL+dG8EO6EwJRCnlBJYv1mTp2?= =?us-ascii?Q?OPkgY0wVJUyqlFNzEgl7xfgVwXXz3/WnJLMpgvs3Gbs22RuVata0F63li1U2?= =?us-ascii?Q?AFZltz11Wvy7YSXt/O0I1IYqT7PQGgGB2tMW2SSsBy/jOSLYCwUBZJly0rRA?= =?us-ascii?Q?Ruf8433V4/PiHrAO8QRWzcd1tpxUiBpH9fEpRabSQ7/PnKtXdtXqyUNIlskA?= =?us-ascii?Q?ZzbP7i9ul5nMmXGVbiJFraEFAJzmggjfNliL/Wp8D95MQiAM+VKwTdxbFG7b?= =?us-ascii?Q?fDy9ENvyWjpWofkE310GJF9YyP+QmNOFeKZ2a8uMValcnB0Ws/5WyCDaRX6j?= =?us-ascii?Q?bNX0Vu+JI1+GTDgussl3BX0xoeEKWc9mEeKdVGmRw89bHY6DabdXCSFx+JeS?= =?us-ascii?Q?zExGIfax8pxBJ8HnDQ/2UHlm17Pg0pleOydNzG4j679bVlE0d+NU2JCqFdDt?= =?us-ascii?Q?Jd8fjfHGUfcgM9LpNI/fbd9g7/IjtorlRM01cKeqAMzEQtdmaVmSZbkgju1D?= =?us-ascii?Q?HR851XwOmM3FH3lxyW0n1tSWryYhnEpkRPYoQ4QichUlFmpoCE9JsZE+2s71?= =?us-ascii?Q?+Yh1jJdXXVWsCSmLunlFzJS8bPK0mVb6nhzDa/eKZs1h6UYxKnQyDnStmq4m?= =?us-ascii?Q?KohqNgvwd3kH1hMoLwqDewjYPD/yz4H5WeWCX81yLTk2IMWmpZ3h86eyWA?= =?us-ascii?Q?=3D=3D?= X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:satlexmb08.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(82310400026)(7416014)(376014)(36860700013)(1800799024)(13003099007)(921020);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Sep 2025 10:54:10.6722 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: d6b2568b-9039-4c02-2b6c-08ddfceb0626 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[satlexmb08.amd.com] X-MS-Exchange-CrossTenant-AuthSource: BL02EPF0002992E.namprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR12MB6954 Content-Type: text/plain; charset="utf-8" Add device tree binding documentation for the AMD I3C master controller version 1.0. Signed-off-by: Manikanta Guntupalli Reviewed-by: Rob Herring (Arm) --- Changes for V2: Updated commit subject and description. Moved allOf to after required. Removed xlnx,num-targets property. Changes for V3: Updated commit description. Corrected the order of properties and removed resets property. Added compatible to required list. Added interrupts to example. Changes for V4: Added h/w documentation details. Changes for V5: Renamed the xlnx,axi-i3c.yaml file into xlnx,axi-i3c-1.0.yaml. Changes for V6: Corrected the file name for $id in yaml to fix the dtschema warning. Changes for V7: Added i3c controller version details to commit description. Changes for V8: None. --- .../bindings/i3c/xlnx,axi-i3c-1.0.yaml | 55 +++++++++++++++++++ 1 file changed, 55 insertions(+) create mode 100644 Documentation/devicetree/bindings/i3c/xlnx,axi-i3c-1.0.= yaml diff --git a/Documentation/devicetree/bindings/i3c/xlnx,axi-i3c-1.0.yaml b/= Documentation/devicetree/bindings/i3c/xlnx,axi-i3c-1.0.yaml new file mode 100644 index 000000000000..17c63807dbcf --- /dev/null +++ b/Documentation/devicetree/bindings/i3c/xlnx,axi-i3c-1.0.yaml @@ -0,0 +1,55 @@ +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/i3c/xlnx,axi-i3c-1.0.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: AMD I3C master + +maintainers: + - Manikanta Guntupalli + +description: + The AXI-I3C IP is an I3C Controller with an AXI4-Lite interface, compati= ble + with the MIPI I3C Specification v1.1.1. The design includes bidirectiona= l I/O + buffers that implement open collector drivers for the SDA and SCL signal= s. + External pull-up resistors are required to properly hold the bus at a Lo= gic-1 + level when the drivers are released. + + For more details, please see https://docs.amd.com/r/en-US/pg439-axi-i3c + +properties: + compatible: + const: xlnx,axi-i3c-1.0 + + reg: + maxItems: 1 + + clocks: + maxItems: 1 + + interrupts: + maxItems: 1 + +required: + - compatible + - reg + - clocks + +allOf: + - $ref: i3c.yaml# + +unevaluatedProperties: false + +examples: + - | + i3c@80000000 { + compatible =3D "xlnx,axi-i3c-1.0"; + reg =3D <0x80000000 0x10000>; + clocks =3D <&zynqmp_clk 71>; + interrupt-parent =3D <&imux>; + interrupts =3D <0 89 4>; + #address-cells =3D <3>; + #size-cells =3D <0>; + }; +... --=20 2.34.1 From nobody Wed Oct 1 22:31:20 2025 Received: from PH7PR06CU001.outbound.protection.outlook.com (mail-westus3azon11010002.outbound.protection.outlook.com [52.101.201.2]) (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 CF1C11A0728; Fri, 26 Sep 2025 10:59:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.201.2 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758884392; cv=fail; b=Q1NBckKF+AmijLQNNW6HFnhfkKrooROSXrq3d8+Q04xv/R9EMBFJPcyNUDYWtDW82P0dk5kOcqm8Qht0sRbf0pSzlaaSE3lS51ng6107NxmxPs5LeVWZ98rmzXLCEjtw2KJqfN2Fr3MZ2kHUiUWbPGBBoZRf2MBbe2pNW7HHyEw= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758884392; c=relaxed/simple; bh=M6gFHvaJVpiZKJqHjNf4N7eJIblRQjW489JnTU3J9zE=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=lJeFSFZKxaj5L+uSNZoSoH1npQnXG7RdPWL7dG6H9o2nk08WEqrjnrBSCBOj1+x0rOPg8VgXQUc5S7kXTsIYkoxNWqpjzAxx0Y9L0AHJmvGLMY7uMzKIqWfGEHvDifwmHfQm1SqH2TwvSd26zWscDFAVUPDLi8OzpOE+rAxWJHg= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com; spf=fail smtp.mailfrom=amd.com; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b=DNde8Hmz; arc=fail smtp.client-ip=52.101.201.2 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="DNde8Hmz" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=PtuzHUHggKE38dd4hCrVJH98U8o46GezL85H1LHoYOQZM/BElpTPg50zwFQ07DD2CIgvpidQGH6ni3RIICsAvih3qbwQzgc7phnKtivox51PjKdBczb7pTtD+y3wiShwgJ55EFMFfMJLjiJ//mBY3kZ9tTlns46JSB6jQw1ZAt8zwqlTDfmOXFKA52q0JB9BMg0ke4a/rI6NwsqhoXem6aVWYIidXafqDu+mZuHcabmhtxPosAfXZRCb7ks5EGOWS7kShw85ZrrbJ1Bw4bSPTuRMlN8zJ5J7d7KP6DjOGQ349XOL0yNDlSJX/acivMeseN63s5U3sBfELGtkQUKIjw== 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=E3hBq9Fzr7MFqIOU0wU2Fuhlzau0yLKsxJp0QqKOawU=; b=fNPcb+rQpoQjlKb1xOaNTtg6ZqCqyXq+UMoNtZrUYNRzNuSqVu5q6ZjgE6bV3XVGUgQJlV7EcOobj5kgD5TAG4v2kLf1cF8pejax1jAPzu6jGfb67n0VAxKWJhNE8XZKj7TQf2ceE5emlBM9HCGkvB+gKtdyU+GR/XgBP10XrANWhwb00RaaH5M5744ePZYAwVrAEC803au6zlPrHV4OGOHfzXDm5t0MaL6CmU29/ib8RalMJineBmjk05etHR15YGsRikqPRdf8H84MtbqEX1TnCiH5wfBwW/QDCUlyzXodxs6woRvizDfrTG0xpXGwurwX0hBUmZTWLLYOGRuSgw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=gmail.com smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=E3hBq9Fzr7MFqIOU0wU2Fuhlzau0yLKsxJp0QqKOawU=; b=DNde8Hmz95ugLSKC5r2AfQJP3J2raZ2tZo8PXaKmn9lvpdZykDhDC5e2HVsguxr5BF3DaVZa7Rbr8Yz9/r4ajjONtkrOzuxPhxPt51Sd/6AbuWcME1a6rSwKdyh50e/CUyHOR8+EGEUarbDyBVSAhE7Rczv68iK/3QhtBbvv3S0= Received: from BL1PR13CA0062.namprd13.prod.outlook.com (2603:10b6:208:2b8::7) by DS0PR12MB8785.namprd12.prod.outlook.com (2603:10b6:8:14c::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9160.10; Fri, 26 Sep 2025 10:54:19 +0000 Received: from BL02EPF0002992A.namprd02.prod.outlook.com (2603:10b6:208:2b8:cafe::e0) by BL1PR13CA0062.outlook.office365.com (2603:10b6:208:2b8::7) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.9182.4 via Frontend Transport; Fri, 26 Sep 2025 10:54:12 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=satlexmb08.amd.com; pr=C Received: from satlexmb08.amd.com (165.204.84.17) by BL02EPF0002992A.mail.protection.outlook.com (10.167.249.55) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9160.9 via Frontend Transport; Fri, 26 Sep 2025 10:54:18 +0000 Received: from SATLEXMB06.amd.com (10.181.40.147) by satlexmb08.amd.com (10.181.42.217) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.2.2562.17; Fri, 26 Sep 2025 03:54:18 -0700 Received: from satlexmb08.amd.com (10.181.42.217) by SATLEXMB06.amd.com (10.181.40.147) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Fri, 26 Sep 2025 05:54:17 -0500 Received: from xhdradheys41.xilinx.com (10.180.168.240) by satlexmb08.amd.com (10.181.42.217) with Microsoft SMTP Server id 15.2.2562.17 via Frontend Transport; Fri, 26 Sep 2025 03:54:10 -0700 From: Manikanta Guntupalli To: , , , , , , , , , , , , , , , , , , , , , , , CC: , , , , Manikanta Guntupalli Subject: [PATCH V8 2/5] asm-generic/io.h: Add big-endian MMIO accessors Date: Fri, 26 Sep 2025 16:23:46 +0530 Message-ID: <20250926105349.2932952-3-manikanta.guntupalli@amd.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20250926105349.2932952-1-manikanta.guntupalli@amd.com> References: <20250926105349.2932952-1-manikanta.guntupalli@amd.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: BL02EPF0002992A:EE_|DS0PR12MB8785:EE_ X-MS-Office365-Filtering-Correlation-Id: 1cf62e7f-5023-40f9-c62c-08ddfceb0af5 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|1800799024|36860700013|82310400026|7416014|376014|921020; X-Microsoft-Antispam-Message-Info: =?utf-8?B?VmNIdXFqNGxxWS9McVdWck5yN0FUaTY4cVFSRlZ6ekV6M0JrY3VXL2FUMWkr?= =?utf-8?B?Vmw1ek1oS005NXpYNEFTZ2FhSExFNnkrQnVPRVVSY0dwNEVETVp6ZlVMeUpM?= =?utf-8?B?bS8vL0JWTTZ1dXBpdkU1RitqRmNQaW1EeU9Cdmg5L2JNaGEvb2RxVENMUnNN?= =?utf-8?B?bkV1MTMxY1dBc2NZK1Z0eldBaHVoek4xaEtXUlRxeGR0MWZKM3V2dHN5UC9L?= =?utf-8?B?ZjE1QUh0YU9ldEZtSUJPby9kN0JNMlQzZFlVaXhTWW1qL2hkN2NSOThGY1I1?= =?utf-8?B?QTNoMklVQW9ScytWMXh0QkVkc001a29ZbzRxZmY5ek9iY3Z6VUVTUkUvaUl4?= =?utf-8?B?YkV6TEE1TXFkOWtkU0JZMGpSRnpocXFHcXpoZDRKQ1hLc25HeVNHWG9Yb09V?= =?utf-8?B?a1NvRHorY2JmdjU2RDJycWF4WTduVHRTSzU0aTNTZEJEMldxT0M1akd2b0M3?= =?utf-8?B?WUFwNWhmcW5UWFAyVEwrL3FTUHpQNmpLQ0xqZFM5WlNUbGQ5SnZmcjRyT3Fx?= =?utf-8?B?Tlcwa1V4bThvL2JEc2w1YThDczBMbjQ1T3l3dlZySUFMbXREcEZsLytvR0ky?= =?utf-8?B?N0NrN3oxd3ZZeFpWZHhJMzF1YWczS2ZuRHlXcjUvUjFKY001b0RkWVA1TC9T?= =?utf-8?B?WmUyMSs2QkNITFJ5dVhINDArT3Y3c0MrZm50M3dLZGVZQ2JtU0hLcFNoekMr?= =?utf-8?B?VTRJY3BZRGJIZmhIbTUySTUyZTA4MXFtdGkyMnVVQWVoRXpQWVI4OUVjRU45?= =?utf-8?B?bHpBNXJSRFBaYklyUEtPYUJtaTJLanhmUHNudDRETHVIY2xVL0hvSmR2Uktl?= =?utf-8?B?Q25nSmE0YThPclBTVHRHcjVkOURoRGVrSE5wdWlGb3BNUVdDNUt3SWRpME9k?= =?utf-8?B?bUdsOE1aMWw5aFN4NHVTQ2Q3cEhZZ3FKZHRYdEUxODhhOGRKWWk2SzZZSmww?= =?utf-8?B?bDRqbHc4ZXc5ank0S3hQb1ZsdXQ1aGtLYTlhUU5VWGhhVXRENDFXSFVsSCs3?= =?utf-8?B?OU0yRmhURTZNRTRqL2t6Ujc1SDV3Y1cxY0lab0pFOW1CNlhCYjV5YVZMNDFu?= =?utf-8?B?SFBsYm9BL2g2YnBGZTVORHVsTS9JbFRkR2NvUTFmeVZSUUh3OFBWZWpRWkx4?= =?utf-8?B?MldJY0tja1BwZTVlN1V1elF5cEpUSk10V2hRTFN5K3NLWkhlMFBrSkp5bUZZ?= =?utf-8?B?azRCckoxc01sNlhLWTVkTjdGTFlkSVFpbjlBbVBFR21vRTU0TUVtVllMdDRk?= =?utf-8?B?REVRYngvaTdKb0dpakRKN1FTMnhWeHRWVEtaQlk2UkRjUFZyWXpkREExQWsx?= =?utf-8?B?WmJjdFQvRmQ0dDNTWnNycXZXaEplZTltcTFDUy81dzQyamVnYlRKZFRWQWRo?= =?utf-8?B?eWxrU3lxK05PMStpeGx6d1U5dlZLWitmQ2xnU1V1Kzl6L2VXY2xsY2JBS09x?= =?utf-8?B?aVNuR2xVeEt1YzdzTXUwVUs3WWRxRjd0YzBuTkdCb0gxekFNRjRrS2p1bU5p?= =?utf-8?B?dXBhRHNpU1lhdlViNmRocUtRUVJvQXdyZC8yQTFQNlRqMTdGa0dFM3ZTNEdV?= =?utf-8?B?am1hSGFtbndqaWNYcTNQaFJBazNoQVNDSldtdGphVDNnc3piYlMyeW5mWWtW?= =?utf-8?B?QWpNWXFITEQxVkg4TXRjMG5rbGErTHk5TTlhWkNZYUUvMHhONGN0Zlk3N1U0?= =?utf-8?B?MTdDSXN3aEY2Y2ZVMSt4ZjNPQksyWWx5Qk5Ta2dVcDFUdzhITUlmeCtmbUNp?= =?utf-8?B?MEVNck1ZL2ZvM24rdVJlL2NRUzh3eis2SGh4blJValRoRXgwaWlHbzZiUHhH?= =?utf-8?B?QW9PN1pMeXFXZFdDNzVsZXk1eDdQNk1xVlJISjlwdTlUV0tkdExJakdRRFhY?= =?utf-8?B?NnBJcUIzZklxNElvOEo1MitLdmNYN1RweXNoSE1Zd2dPZVdtbTBUQnBPMFhv?= =?utf-8?B?TE1IZE84Q2FnZ25PT0hidXhxNDY4disyc3JMa2E1dExwNEtVVHpvSFZOZEVo?= =?utf-8?B?Z2ZITDFRWWN6QW4wWWVYM0NRbld2ZkZCcEt2dVZjb05hUFJNd2xtL0NNTkJp?= =?utf-8?B?U1JXVG1nNHZ6cjlkWWtiVXlIRkJyaEtCUVlMZz09?= X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:satlexmb08.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(1800799024)(36860700013)(82310400026)(7416014)(376014)(921020);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Sep 2025 10:54:18.7445 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 1cf62e7f-5023-40f9-c62c-08ddfceb0af5 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[satlexmb08.amd.com] X-MS-Exchange-CrossTenant-AuthSource: BL02EPF0002992A.namprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR12MB8785 Add MMIO accessors to support big-endian memory operations. These helpers include {read, write}{w, l, q}_be() and {read, write}s{w, l, q}_be(), which allows to access big-endian memory regions while returning the results in the CPU=E2=80=99s native endianness. This provides a consistent interface to interact with hardware using big-endian register layouts. Signed-off-by: Manikanta Guntupalli Reviewed-by: Frank Li --- Changes since V7: This patch introduced in V7. Changes for V8: None. --- include/asm-generic/io.h | 202 +++++++++++++++++++++++++++++++++++++++ 1 file changed, 202 insertions(+) diff --git a/include/asm-generic/io.h b/include/asm-generic/io.h index 11abad6c87e1..d18a8ca6c06c 100644 --- a/include/asm-generic/io.h +++ b/include/asm-generic/io.h @@ -295,6 +295,96 @@ static inline void writeq(u64 value, volatile void __i= omem *addr) #endif #endif /* CONFIG_64BIT */ =20 +/* + * {read,write}{w,l,q}_be() access big endian memory and return result + * in native endianness. + */ + +#ifndef readw_be +#define readw_be readw_be +static inline u16 readw_be(const volatile void __iomem *addr) +{ + u16 val; + + log_read_mmio(16, addr, _THIS_IP_, _RET_IP_); + __io_br(); + val =3D __be16_to_cpu((__be16 __force)__raw_readw(addr)); + __io_ar(val); + log_post_read_mmio(val, 16, addr, _THIS_IP_, _RET_IP_); + return val; +} +#endif + +#ifndef readl_be +#define readl_be readl_be +static inline u32 readl_be(const volatile void __iomem *addr) +{ + u32 val; + + log_read_mmio(32, addr, _THIS_IP_, _RET_IP_); + __io_br(); + val =3D __be32_to_cpu((__be32 __force)__raw_readl(addr)); + __io_ar(val); + log_post_read_mmio(val, 32, addr, _THIS_IP_, _RET_IP_); + return val; +} +#endif + +#ifdef CONFIG_64BIT +#ifndef readq_be +#define readq_be readq_be +static inline u64 readq_be(const volatile void __iomem *addr) +{ + u64 val; + + log_read_mmio(64, addr, _THIS_IP_, _RET_IP_); + __io_br(); + val =3D __be64_to_cpu((__be64 __force)__raw_readq(addr)); + __io_ar(val); + log_post_read_mmio(val, 64, addr, _THIS_IP_, _RET_IP_); + return val; +} +#endif +#endif /* CONFIG_64BIT */ + +#ifndef writew_be +#define writew_be writew_be +static inline void writew_be(u16 value, volatile void __iomem *addr) +{ + log_write_mmio(value, 16, addr, _THIS_IP_, _RET_IP_); + __io_bw(); + __raw_writew((u16 __force)__cpu_to_be16(value), addr); + __io_aw(); + log_post_write_mmio(value, 16, addr, _THIS_IP_, _RET_IP_); +} +#endif + +#ifndef writel_be +#define writel_be writel_be +static inline void writel_be(u32 value, volatile void __iomem *addr) +{ + log_write_mmio(value, 32, addr, _THIS_IP_, _RET_IP_); + __io_bw(); + __raw_writel((u32 __force)__cpu_to_be32(value), addr); + __io_aw(); + log_post_write_mmio(value, 32, addr, _THIS_IP_, _RET_IP_); +} +#endif + +#ifdef CONFIG_64BIT +#ifndef writeq_be +#define writeq_be writeq_be +static inline void writeq_be(u64 value, volatile void __iomem *addr) +{ + log_write_mmio(value, 64, addr, _THIS_IP_, _RET_IP_); + __io_bw(); + __raw_writeq((u64 __force)__cpu_to_be64(value), addr); + __io_aw(); + log_post_write_mmio(value, 64, addr, _THIS_IP_, _RET_IP_); +} +#endif +#endif /* CONFIG_64BIT */ + /* * {read,write}{b,w,l,q}_relaxed() are like the regular version, but * are not guaranteed to provide ordering against spinlocks or memory @@ -524,6 +614,118 @@ static inline void writesq(volatile void __iomem *add= r, const void *buffer, #endif #endif /* CONFIG_64BIT */ =20 +/* + * {read,write}s{w,l,q}_be() repeatedly access the same memory address + * in big endianness in 16-, 32- or 64-bit chunks (@count times) and + * return result in native endianness. + */ + +#ifndef readsw_be +#define readsw_be readsw_be +static inline void readsw_be(const volatile void __iomem *addr, + void *buffer, + unsigned int count) +{ + if (count) { + u16 *buf =3D buffer; + + do { + u16 x =3D __be16_to_cpu((__be16 __force)__raw_readw(addr)); + *buf++ =3D x; + } while (--count); + } +} +#endif + +#ifndef readsl_be +#define readsl_be readsl_be +static inline void readsl_be(const volatile void __iomem *addr, + void *buffer, + unsigned int count) +{ + if (count) { + u32 *buf =3D buffer; + + do { + u32 x =3D __be32_to_cpu((__be32 __force)__raw_readl(addr)); + *buf++ =3D x; + } while (--count); + } +} +#endif + +#ifdef CONFIG_64BIT +#ifndef readsq_be +#define readsq_be readsq_be +static inline void readsq_be(const volatile void __iomem *addr, + void *buffer, + unsigned int count) +{ + if (count) { + u64 *buf =3D buffer; + + do { + u64 x =3D __be64_to_cpu((__be64 __force)__raw_readq(addr)); + *buf++ =3D x; + } while (--count); + } +} +#endif +#endif /* CONFIG_64BIT */ + +#ifndef writesw_be +#define writesw_be writesw_be +static inline void writesw_be(volatile void __iomem *addr, + const void *buffer, + unsigned int count) +{ + if (count) { + const u16 *buf =3D buffer; + + do { + __raw_writew((u16 __force)__cpu_to_be16(*buf), addr); + buf++; + } while (--count); + } +} +#endif + +#ifndef writesl_be +#define writesl_be writesl_be +static inline void writesl_be(volatile void __iomem *addr, + const void *buffer, + unsigned int count) +{ + if (count) { + const u32 *buf =3D buffer; + + do { + __raw_writel((u32 __force)__cpu_to_be32(*buf), addr); + buf++; + } while (--count); + } +} +#endif + +#ifdef CONFIG_64BIT +#ifndef writesq_be +#define writesq_be writesq_be +static inline void writesq_be(volatile void __iomem *addr, + const void *buffer, + unsigned int count) +{ + if (count) { + const u64 *buf =3D buffer; + + do { + __raw_writeq((u64 __force)__cpu_to_be64(*buf), addr); + buf++; + } while (--count); + } +} +#endif +#endif /* CONFIG_64BIT */ + #ifndef PCI_IOBASE #define PCI_IOBASE ((void __iomem *)0) #endif --=20 2.34.1 From nobody Wed Oct 1 22:31:20 2025 Received: from CY7PR03CU001.outbound.protection.outlook.com (mail-westcentralusazon11010019.outbound.protection.outlook.com [40.93.198.19]) (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 1E016299AB3; Fri, 26 Sep 2025 10:54:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.93.198.19 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758884073; cv=fail; b=Q3Is/ErpiKEkVDXATyso1OFuOfmnqMYdgbYH3LKp/z1pSTh+cm12GCcDKLopWootvaPs56O8RHAmuYRpFgF1easTZ8dhT1ZCYkfametOW83YfXf9DbV3M0fmJhAIcvZOATHeV94hXuP5zQQtJ2LFHrtkqurA+QOaWWbNvkB7tV4= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758884073; c=relaxed/simple; bh=klxEfI85D8o5MI8GVdUw1flLZJAiGcxDzjKmdYA96CI=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=sHciyRRF0rqx4Dz61vs+GaIPW/5IZJ3qWZ80+KWSXDr0GvJ7UJG1+B0kxgDUi5SGxgXz0dmcIKaQNY9+MVDTp5tVLbA5iYT51KHbn7sPUHJ9aNssANXd17q6Js2CMnLWN91scrAq++hHRqU5+3ceZh7buMNFhRe1/B+ptm2Nu6Y= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com; spf=fail smtp.mailfrom=amd.com; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b=1uFwcob8; arc=fail smtp.client-ip=40.93.198.19 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="1uFwcob8" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=TGaJCCexntiHEbcYXhRlgQp03CaylhD6VES7Ji2no0JE6vUkN84l2kg7M1RdzyLEebB5iWXGahrjL4NMkxBdcSRbLDLVbOqkDdOv53MNmLdh2bjjKOrCaYthpYWWJXBogiVBTTvwrNM84CHogRzUvAv7kCITmVnbrbYFKyB9fHOKgpl+uPhkv8WobAHJJuDmz2rQOCZwGSndi6Bwu8VTK435/ZuxzfiSDCngC/Ru8naLHOAK2bAiL17VhgeV5bUAqJTV0CYBBU3LweTHqZHLVEE/ZYGEQ1Rs7IsCvHcbEz7yxYn2XifZKQrVSNxE79A/+Pd0uS0PDo+nL8MtzWWNag== 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=I4WA3NmV9P64Ewa9wuaSx8ufMiI8qcoNuxhGM7GFX8w=; b=Kx6uYfMIfOQT9qugszV3CUaDkshp4+Q6Hi5JQaluTMBLtbxKgnFztqctR/xl/LP/VEbqvPWUJKlmWKCev9IqH7pMPtes6e6YJQPPu1Y5q6VTroOMThpVV4Xjbr3ckUblssg+EEGoiZWNUFceq8fIEI9HN4PpuS9+4V4dfoJvtd3MCDBJCeHDgsshs53c+AWK3Uvgw44by+CJ76tC5ZAOVm3xhD83d7ggJJ3kGDwKMHVJTgh5LZwh4uGQ4mh8Ors2KLLn4kQgrYIPg5gjl6el96reDokSYRYhf4kYwdYOKyoN0929712EY5Ju6RuEfvX52xY0PpHzdvpajsMCvuJvXg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=gmail.com smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=I4WA3NmV9P64Ewa9wuaSx8ufMiI8qcoNuxhGM7GFX8w=; b=1uFwcob8k7edM6aY4vuD1OrPXscugJlS8shpmOm2r0mx3oDI3eAihX/a3UsTDygY51FViu77ngQ4NiOkz9PT0go+FNbPG/8i35fW96mvhaSqwX3u10IsQMiQpaybwWamKzqlRdLkDFNiT9MJc8t1XH2mfK/T2bYbjJ5ZGj7lkYE= Received: from MN0PR04CA0016.namprd04.prod.outlook.com (2603:10b6:208:52d::8) by BY5PR12MB4116.namprd12.prod.outlook.com (2603:10b6:a03:210::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9137.19; Fri, 26 Sep 2025 10:54:27 +0000 Received: from BL02EPF0001A103.namprd05.prod.outlook.com (2603:10b6:208:52d:cafe::b1) by MN0PR04CA0016.outlook.office365.com (2603:10b6:208:52d::8) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.9160.12 via Frontend Transport; Fri, 26 Sep 2025 10:54:26 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=satlexmb07.amd.com; pr=C Received: from satlexmb07.amd.com (165.204.84.17) by BL02EPF0001A103.mail.protection.outlook.com (10.167.241.133) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9160.9 via Frontend Transport; Fri, 26 Sep 2025 10:54:26 +0000 Received: from satlexmb08.amd.com (10.181.42.217) by satlexmb07.amd.com (10.181.42.216) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.17; Fri, 26 Sep 2025 03:54:25 -0700 Received: from xhdradheys41.xilinx.com (10.180.168.240) by satlexmb08.amd.com (10.181.42.217) with Microsoft SMTP Server id 15.2.2562.17 via Frontend Transport; Fri, 26 Sep 2025 03:54:19 -0700 From: Manikanta Guntupalli To: , , , , , , , , , , , , , , , , , , , , , , , CC: , , , , Manikanta Guntupalli Subject: [PATCH V8 3/5] i3c: fix big-endian FIFO transfers Date: Fri, 26 Sep 2025 16:23:47 +0530 Message-ID: <20250926105349.2932952-4-manikanta.guntupalli@amd.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20250926105349.2932952-1-manikanta.guntupalli@amd.com> References: <20250926105349.2932952-1-manikanta.guntupalli@amd.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: BL02EPF0001A103:EE_|BY5PR12MB4116:EE_ X-MS-Office365-Filtering-Correlation-Id: 6ac641e1-bc5a-4c7e-b576-08ddfceb0fba X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|7416014|376014|36860700013|82310400026|1800799024|921020|13003099007; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?YH7NaNRvajucV114ew1oLgIUWIkr0ColUYjWW+XLfnaZ28w7uDh21+ftYa1j?= =?us-ascii?Q?Ga3fqfHaONhRSAne+oNMiJ/6BZ0QrMYNK9hn4zyW4TYTcItpYljIt0PCcNIX?= =?us-ascii?Q?3Z1/6C0hSoscGYh7PTuFn6tDFKJOU/CbQa/nlHYhz+0vbuUNBlw5Mc5p0EFZ?= =?us-ascii?Q?FFiAo6EPU6DWgeSCZ213Pe0bQR5K3ZeyR7F7uvfV2V4rUwai8siLyFtp/1ck?= =?us-ascii?Q?ik9aidptXHXR9vRV8qqd3KJy82+aMwTDY6G1Y1bUds6BZR4iBaYYCo3Sx6DV?= =?us-ascii?Q?qzwvkJtgFzP3NA9itXVFyIFca14xJ1ZbCgipplRQSe1HASHkXHmRnTQ/32tu?= =?us-ascii?Q?zI2pz3XDQ2D/GCdfGxf/vacIq31U78BykMD7vopZkMvsCgpN4G+iulRFtfBw?= =?us-ascii?Q?KY2hHHZ9FPfLsiyXCFWt8LgtW7CpRDP6kP4IEaNOn7ILAWDQJ9qIGm0EgL78?= =?us-ascii?Q?a7T1f3D0yPkghGIZblI4HXweQJKbIkDdlOVPaglmjt4f6EBg4HAMOfsgM9Vv?= =?us-ascii?Q?oDTzF5lVKapsPxl1XsDfcoF8Jj6SDYqL0dQE8nzCF/mxTqTnU/mjhPPKxfxI?= =?us-ascii?Q?33Yg9LiHpWYyNGqIFOwHwL7jDwL1QVGeDnz79iIKDZzaHtlzfCvisN1sz2DP?= =?us-ascii?Q?dnKw741GwTNHWEBBb9SJpqCAVNPb8uk1NaWlMOkgBhTnvrx0G1CvH2YNyAwH?= =?us-ascii?Q?XyGzngSpN+sR4RUchHzBaD+hEVmtRo7dyMi3Ghxg4RLJK7ofNMV3CqkQi3HI?= =?us-ascii?Q?1LOGGT5koiLDimO1XV8KD7dmQJOnX8wfYnRGXQ4UQALjuusUhbV5u5nYP22M?= =?us-ascii?Q?hsHDnLyE5WoldzZI4ISmhrFUE9/AcwkZ3Xh69pWiuuAXoZEyzO0U6pUqT/Qq?= =?us-ascii?Q?I0xgMUyBRTCCsSahmoO607SGIhQNCJYxOeHhQGa3vObzzQywy9+e7e37Nowt?= =?us-ascii?Q?NapMP8MoDiRfH+pBosyoG/8Vy6cyMKegRGC9A3D+y8HDePeShwgsMJ7Q22Tw?= =?us-ascii?Q?J1h6g9fefP5GG1ypeYsqbEyF3VzEwFL77V7ZFBHLdxvgcD+HtD8VVJj4ntU9?= =?us-ascii?Q?FYE3MgeBFxxgsx1GtIWzfbf5b3hN/Uz1VUa96b3lSD5wFzLHdQvPXWvTlL2+?= =?us-ascii?Q?CsYxsckAzjfEi5/0J5/QeWslRRMgqvoA9LEt0W0GY3bgZBqn1iKUjlx3gwVx?= =?us-ascii?Q?JJpRe5UhwjhtxDxvj6EOUeEACMO2O13i/PXTrvn4mFleq/QrNMw0drmyaNX3?= =?us-ascii?Q?D/H4UkPDkoptdDHA5amgf2XtfR6YrYiiZ9WedIGx+W8NymnpTplqy/veOy55?= =?us-ascii?Q?S1By9O4/2vUGWvBmwJRTxVY7yu1QQ9Nogl06P5C3uu8xYLfwtInKU6V5XdAj?= =?us-ascii?Q?kh3D6OWWi0lQZxN7rsOAev8GpobR9EfDBbRp9XMEWFRAwAvvL8e1S+3ChYg9?= =?us-ascii?Q?0f/vYNRe3NfAnwZckVd8EybvSmDlgbwqeXFTdFZoAjpsAc+OYtTxSQTm2N25?= =?us-ascii?Q?eKWEgtKTvcTwpHkb9o3zOMBeWWCh9OxqoL6M?= X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:satlexmb07.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(7416014)(376014)(36860700013)(82310400026)(1800799024)(921020)(13003099007);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Sep 2025 10:54:26.7450 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 6ac641e1-bc5a-4c7e-b576-08ddfceb0fba X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[satlexmb07.amd.com] X-MS-Exchange-CrossTenant-AuthSource: BL02EPF0001A103.namprd05.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR12MB4116 Content-Type: text/plain; charset="utf-8" From: Arnd Bergmann Short MMIO transfers that are not a multiple of four bytes in size need a special case for the final bytes, however the existing implementation is not endian-safe and introduces an incorrect byteswap on big-endian kernels. This usually does not cause problems because most systems are little-endian and most transfers are multiple of four bytes long, but still needs to be fixed to avoid the extra byteswap. Change the special case for both i3c_writel_fifo() and i3c_readl_fifo() to use non-byteswapping writesl() and readsl() with a single element instead of the byteswapping writel()/readl() that are meant for individual MMIO registers. As data is copied between a FIFO and a memory buffer, the writesl()/readsl() loops are typically based on __raw_readl()/ __raw_writel(), resulting in the order of bytes in the FIFO to match the order in the buffer, regardless of the CPU endianess. The earlier versions in the dw-i3c and i3c-master-cdns had a correct implementation, but the generic version that was recently added broke it. Fixes: 733b439375b4 ("i3c: master: Add inline i3c_readl_fifo() and i3c_writ= el_fifo()") Cc: Manikanta Guntupalli Signed-off-by: Arnd Bergmann Reviewed-by: Jorge Marques Signed-off-by: Manikanta Guntupalli --- Changes since V8: This patch has been included in this series as the current series depends o= n it. https://patchwork.kernel.org/project/linux-i3c/patch/20250924201837.3691486= -1-arnd@kernel.org/ --- drivers/i3c/internals.h | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/drivers/i3c/internals.h b/drivers/i3c/internals.h index 0d857cc68cc5..77d56415cb99 100644 --- a/drivers/i3c/internals.h +++ b/drivers/i3c/internals.h @@ -38,7 +38,11 @@ static inline void i3c_writel_fifo(void __iomem *addr, c= onst void *buf, u32 tmp =3D 0; =20 memcpy(&tmp, buf + (nbytes & ~3), nbytes & 3); - writel(tmp, addr); + /* + * writesl() instead of writel() to keep FIFO byte orderer to match + * the order in the buffer regardless of the CPU endianess. + */ + writesl(addr, &tmp, 1); } } =20 @@ -55,7 +59,11 @@ static inline void i3c_readl_fifo(const void __iomem *ad= dr, void *buf, if (nbytes & 3) { u32 tmp; =20 - tmp =3D readl(addr); + /* + * readsl() instead of readl() to keep FIFO byte orderer to match + * the order in the buffer regardless of the CPU endianess. + */ + readsl(addr, &tmp, 1); memcpy(buf + (nbytes & ~3), &tmp, nbytes & 3); } } --=20 2.34.1 From nobody Wed Oct 1 22:31:20 2025 Received: from MW6PR02CU001.outbound.protection.outlook.com (mail-westus2azon11012038.outbound.protection.outlook.com [52.101.48.38]) (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 25673299AB3; Fri, 26 Sep 2025 10:54:40 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.48.38 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758884082; cv=fail; b=Cnj5GNsxpl4inpiZzEUFcvp+5onw4BDAFXy7+44I8DNQlxibox20H/jbbuJsIWWXdbDkCADStIBBaLdqLNT3e8C9e82EoCdUFF8siA03Fm3JlVpVBEg9r/T5wZFfiNqGa05fyHnrtFQu9E+rCULUWxDGYk4jdsSl+1zqjQGe6sc= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758884082; c=relaxed/simple; bh=DFyJC7STV+17J1YWBhrKUGvUrnZLNgHRTuQ08vU/+Js=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=oBPTkst7Wsodb6hrNYFEPvatH1mokygl14J/Jnpi3MTZsYbpqk1pvzCGp5YkXCV8UJxat4FUG1qGSse+WLafOAY0muovxMZtgZdRXzLWN/LfK2LkEME+6LienScDCgPPi5bVuXLgurBxCXu6usK3H60PicMNg9ekF7krHofdG7Q= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com; spf=fail smtp.mailfrom=amd.com; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b=nEMT0oe2; arc=fail smtp.client-ip=52.101.48.38 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="nEMT0oe2" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=A0pswIPbOQYEswlIH97Zyh7+Vp8BaRFQa43PoWfB9sQWXMvc2h6avnzHtr/4u29qrp9IM5YsXQn2WQ1mSTwSp0hplPpBBby0/Qz+sTcusRT6EIh+Smbi42hhEBEguyGuZDBesIAfKF8bkTdklYpnJesN6RB9TzmCk2drS46DWUJy8ohf5T3MigEMv4vBcEByLYUVRLkCD31jFB9VycecLVXaXXnMp2lY7Mi11M9Hz5/d6upKyf9qczNy03X+x0nCFs0P3VbMtYchruc+WeS+iiEW+e2HcjU5CdTygi76hpyH8IStjCGXQRnO75Ckp728ZDljbL3YoTvnH32798EjBg== 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=W+nGQnzoYt8BiKll2m81rvMUAkQ4MfOUHiR5pL8D268=; b=fnb8r1MF4Fk2IRGOeLu8Q2bZSD4wxs/bxOhAXKD9kHB6W329dKn1mIozFT0NYmFyIpDopvxk6gISwXxSWEKKM4ZPqB5rZEFEmVP1RKeaGUTAiS9cx6DdJIgj7776aeD7RwhsDUoE2UR4c19xT2RW8ql3hAwUb/9FFlvgo3OuB4Afei1k8ZwKpHKNNpX6XKnUJ+Hc5U47mqZ3/UvBXoC1s1DwDVopDooWNbgkeKUjYjKmUiaQSsz8FXNJG1r/PyK7o/pNiWTVCcTy2MDdoOoQoLSfcYAd7olsMZ89o3VVMn5CrxjllszwDWZovZlCQCcV0rscDMooesBBY0MMHamumw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=gmail.com smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=W+nGQnzoYt8BiKll2m81rvMUAkQ4MfOUHiR5pL8D268=; b=nEMT0oe26Yr+vddmKB2Dnl/TFuyKDmtxWODNXAtPJbf7FvuvCmno/bjDdrkDXsTfgZvlUddrlYEBqF5a4Uz2Gq9gbxq+TYftb9X42ehjdcrhHc/KjbFoh2AzqmhgC2eUtx7fKxodVepwWYv6kTsnI/bb5CFC/96AXG44W7ckQxs= Received: from BN0PR04CA0041.namprd04.prod.outlook.com (2603:10b6:408:e8::16) by BY5PR12MB4100.namprd12.prod.outlook.com (2603:10b6:a03:200::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9160.13; Fri, 26 Sep 2025 10:54:36 +0000 Received: from BL02EPF0002992D.namprd02.prod.outlook.com (2603:10b6:408:e8:cafe::3e) by BN0PR04CA0041.outlook.office365.com (2603:10b6:408:e8::16) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.9160.13 via Frontend Transport; Fri, 26 Sep 2025 10:54:35 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=satlexmb08.amd.com; pr=C Received: from satlexmb08.amd.com (165.204.84.17) by BL02EPF0002992D.mail.protection.outlook.com (10.167.249.58) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9160.9 via Frontend Transport; Fri, 26 Sep 2025 10:54:35 +0000 Received: from satlexmb10.amd.com (10.181.42.219) by satlexmb08.amd.com (10.181.42.217) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.17; Fri, 26 Sep 2025 03:54:35 -0700 Received: from satlexmb08.amd.com (10.181.42.217) by satlexmb10.amd.com (10.181.42.219) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.17; Fri, 26 Sep 2025 03:54:34 -0700 Received: from xhdradheys41.xilinx.com (10.180.168.240) by satlexmb08.amd.com (10.181.42.217) with Microsoft SMTP Server id 15.2.2562.17 via Frontend Transport; Fri, 26 Sep 2025 03:54:27 -0700 From: Manikanta Guntupalli To: , , , , , , , , , , , , , , , , , , , , , , , CC: , , , , Manikanta Guntupalli Subject: [PATCH V8 4/5] i3c: master: Add endianness support for i3c_readl_fifo() and i3c_writel_fifo() Date: Fri, 26 Sep 2025 16:23:48 +0530 Message-ID: <20250926105349.2932952-5-manikanta.guntupalli@amd.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20250926105349.2932952-1-manikanta.guntupalli@amd.com> References: <20250926105349.2932952-1-manikanta.guntupalli@amd.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: BL02EPF0002992D:EE_|BY5PR12MB4100:EE_ X-MS-Office365-Filtering-Correlation-Id: 40f85e83-db9b-4cd8-d938-08ddfceb1501 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|1800799024|82310400026|376014|7416014|36860700013|921020; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?0u2VA92aSvOYvzx0u14r82m61YuvpSiLMCCVEyVlxuW2xjXIqz2AonLqfs/o?= =?us-ascii?Q?9Tx2ztK8f8pq/y/koPrDXg9Nt/g45c7tQfXV9Trm/ZBN79FuzEhEjQtj22BG?= =?us-ascii?Q?Jnyugzl3WJhrwRoekuUyGT/j+z6ym39ujMn07Q45gH6BYg6dEAdnHPc4Nmuo?= =?us-ascii?Q?g3cqd3RhS3NC9E90oJ1zuwF4hsuN+p60tgfh9TYahFvcwtnNJbbpl6f80GD3?= =?us-ascii?Q?fVCZmhw21zHC22RTnaHoK24bUu3hPkFw3A5HrzxO/0rhz4raC+8T9o4oYdUz?= =?us-ascii?Q?/dUnwlDM4NC9JT/4pv8EKwUKftVvW/je+nFDQScAUUwBXt/HsQUf2ez6hyZ0?= =?us-ascii?Q?yJ7mUFDxMRA4QfViZy1KX413n36+mjrRhh67gFfY9zJnIVpkjATEzc9hc9tW?= =?us-ascii?Q?H6TbJX9qflJqv6SYfs4Gr7FR427Qt6JTRMMszO7XuKV3Wipucm7w1Dwv3KLM?= =?us-ascii?Q?PmjXHwtcepyRSGARFjLF3ecnVEmYHcBv7c0GTA9PkKf7Sy3IKlYS39eQhDNI?= =?us-ascii?Q?QM7/r8qlsqziic7ZgRV+qAF5TerWoQSV//5ax+jymMao44JCzz2Zu5azFTmD?= =?us-ascii?Q?TPzpbEjvQ13a99FVEtuUFh1g6ycqe3w1UyvXjwvCOP+EAQWr3RKgAqQy264U?= =?us-ascii?Q?37TmROTIlpnGOLp8xaNBprIHDcPb+FqzDGjvl+ryNBokhBkfdRRRRs0Fy9K7?= =?us-ascii?Q?Y7g1ipVJP6AvyZ9VU2fMd+f3sAuA09Qann5KqB/SpufISZKTeEnOwvtO3Ua6?= =?us-ascii?Q?TuOzQiWPh9iHYo4MYCAvekBtkkNaIc5dT/fSpADMVKZ6qd03m1GoNVDRdV8f?= =?us-ascii?Q?sDQ5VvjGjOrhaoQpXOYfGm/Y0g3P6uDAl+H2ovwloFhW+1/WmNkswrmeONMH?= =?us-ascii?Q?j3Ad8anHyxXIVSGcpmQTLEkS/a8UMHEm6ZzB2M1Fj5Krvh7I3GIRVzEzR8iS?= =?us-ascii?Q?blj8t3jpfj1qqk4wI+gV+D+B0M2OiqAJOM3qaIBI6vUN4IkovyjgYgODCldm?= =?us-ascii?Q?9YOfC9TQqwKzDutun8su9V7Ufh7JX6TPP70GiBLqGdfRdQTMFucnNktGuG7A?= =?us-ascii?Q?oruJAYG4Vj5Yi2lMeT1AWP487z+T90bg/TxyX3ZiESjoNPAujvhAWWk+CDZl?= =?us-ascii?Q?ggvMjTLVpTzZTCB6bBDE/qo5aDmZTSlvH5I4Id4wHKYRSI8BYlDkjTiBh/RH?= =?us-ascii?Q?3K2f77UJZt3rmzsY2GfF+8maDrtAY7cFKv2Y6DBmI7ZYvBo83JWQW8D6bi5G?= =?us-ascii?Q?r/l/1DSR6s4nEdGdnh3ad4eOQywTp2flBc5XpM94v7P3eXD2xS1EhzpvCXae?= =?us-ascii?Q?gCxY3GfbASYAixW47BPJv3WsSLB/M1v52gSDgsFRqZLMhvfiLwN48SUnzT2/?= =?us-ascii?Q?W0DUSm5ocYV7i3+NTzSY5mG7W3KAXjPpxukuVhEOA5cy+4h1J1M1xCTUPSTY?= =?us-ascii?Q?0YckYtOaJdgdtPZJ3fPwAyBm9LFY2Euli/hw4XhDBUj+pueuiO0w6ZY5hgcj?= =?us-ascii?Q?vSbVqGBDda8fIc7Z6wTxP/dnk7KiTRhfWb8citZpGmFQ73EWuHXJ4OKabRHQ?= =?us-ascii?Q?xCjL5cCGVEavMLkm67XMWsayF/kW+WSmwRc0uIes?= X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:satlexmb08.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(1800799024)(82310400026)(376014)(7416014)(36860700013)(921020);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Sep 2025 10:54:35.6010 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 40f85e83-db9b-4cd8-d938-08ddfceb1501 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[satlexmb08.amd.com] X-MS-Exchange-CrossTenant-AuthSource: BL02EPF0002992D.namprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR12MB4100 Content-Type: text/plain; charset="utf-8" Add endianness handling to the FIFO access helpers i3c_readl_fifo() and i3c_writel_fifo(). This ensures correct data transfers on platforms where the FIFO registers are expected to be accessed in either big-endian or little-endian format. Update the Synopsys, Cadence, and Renesas I3C master controller drivers to pass the appropriate endianness argument to these helpers. Signed-off-by: Manikanta Guntupalli Reviewed-by: Frank Li --- Changes since V7: This patch introduced in V7. Changes for V8: Resolved conflicts with "i3c: fix big-endian FIFO transfers". Updated description. --- drivers/i3c/internals.h | 45 +++++++++++++++++++++------- drivers/i3c/master/dw-i3c-master.c | 9 ++++-- drivers/i3c/master/i3c-master-cdns.c | 9 ++++-- drivers/i3c/master/renesas-i3c.c | 12 +++++--- 4 files changed, 55 insertions(+), 20 deletions(-) diff --git a/drivers/i3c/internals.h b/drivers/i3c/internals.h index 77d56415cb99..d3630e9129ae 100644 --- a/drivers/i3c/internals.h +++ b/drivers/i3c/internals.h @@ -24,25 +24,40 @@ int i3c_dev_request_ibi_locked(struct i3c_dev_desc *dev, const struct i3c_ibi_setup *req); void i3c_dev_free_ibi_locked(struct i3c_dev_desc *dev); =20 +enum i3c_fifo_endian { + I3C_FIFO_LITTLE_ENDIAN, + I3C_FIFO_BIG_ENDIAN, +}; + /** * i3c_writel_fifo - Write data buffer to 32bit FIFO * @addr: FIFO Address to write to * @buf: Pointer to the data bytes to write * @nbytes: Number of bytes to write + * @endian: Endianness of FIFO write */ static inline void i3c_writel_fifo(void __iomem *addr, const void *buf, - int nbytes) + int nbytes, enum i3c_fifo_endian endian) { - writesl(addr, buf, nbytes / 4); + if (endian) + writesl_be(addr, buf, nbytes / 4); + else + writesl(addr, buf, nbytes / 4); + if (nbytes & 3) { u32 tmp =3D 0; =20 memcpy(&tmp, buf + (nbytes & ~3), nbytes & 3); + /* - * writesl() instead of writel() to keep FIFO byte orderer to match - * the order in the buffer regardless of the CPU endianess. + * writesl_be()/writesl() instead of writel_be()/writel() to keep + * FIFO byte orderer to match the order in the buffer regardless + * of the CPU endianess. */ - writesl(addr, &tmp, 1); + if (endian) + writesl_be(addr, &tmp, 1); + else + writesl(addr, &tmp, 1); } } =20 @@ -51,19 +66,29 @@ static inline void i3c_writel_fifo(void __iomem *addr, = const void *buf, * @addr: FIFO Address to read from * @buf: Pointer to the buffer to store read bytes * @nbytes: Number of bytes to read + * @endian: Endianness of FIFO read */ static inline void i3c_readl_fifo(const void __iomem *addr, void *buf, - int nbytes) + int nbytes, enum i3c_fifo_endian endian) { - readsl(addr, buf, nbytes / 4); + if (endian) + readsl_be(addr, buf, nbytes / 4); + else + readsl(addr, buf, nbytes / 4); + if (nbytes & 3) { u32 tmp; =20 /* - * readsl() instead of readl() to keep FIFO byte orderer to match - * the order in the buffer regardless of the CPU endianess. + * readsl_be()/readsl() instead of readl_be()/readl() to keep + * FIFO byte orderer to match the order in the buffer regardless + * of the CPU endianess. */ - readsl(addr, &tmp, 1); + if (endian) + readsl_be(addr, &tmp, 1); + else + readsl(addr, &tmp, 1); + memcpy(buf + (nbytes & ~3), &tmp, nbytes & 3); } } diff --git a/drivers/i3c/master/dw-i3c-master.c b/drivers/i3c/master/dw-i3c= -master.c index 974122b2d20e..5d723ac041c2 100644 --- a/drivers/i3c/master/dw-i3c-master.c +++ b/drivers/i3c/master/dw-i3c-master.c @@ -337,19 +337,22 @@ static int dw_i3c_master_get_free_pos(struct dw_i3c_m= aster *master) static void dw_i3c_master_wr_tx_fifo(struct dw_i3c_master *master, const u8 *bytes, int nbytes) { - i3c_writel_fifo(master->regs + RX_TX_DATA_PORT, bytes, nbytes); + i3c_writel_fifo(master->regs + RX_TX_DATA_PORT, bytes, nbytes, + I3C_FIFO_LITTLE_ENDIAN); } =20 static void dw_i3c_master_read_rx_fifo(struct dw_i3c_master *master, u8 *bytes, int nbytes) { - i3c_readl_fifo(master->regs + RX_TX_DATA_PORT, bytes, nbytes); + i3c_readl_fifo(master->regs + RX_TX_DATA_PORT, bytes, nbytes, + I3C_FIFO_LITTLE_ENDIAN); } =20 static void dw_i3c_master_read_ibi_fifo(struct dw_i3c_master *master, u8 *bytes, int nbytes) { - i3c_readl_fifo(master->regs + IBI_QUEUE_STATUS, bytes, nbytes); + i3c_readl_fifo(master->regs + IBI_QUEUE_STATUS, bytes, nbytes, + I3C_FIFO_LITTLE_ENDIAN); } =20 static struct dw_i3c_xfer * diff --git a/drivers/i3c/master/i3c-master-cdns.c b/drivers/i3c/master/i3c-= master-cdns.c index 97b151564d3d..de3b5e894b4b 100644 --- a/drivers/i3c/master/i3c-master-cdns.c +++ b/drivers/i3c/master/i3c-master-cdns.c @@ -428,13 +428,15 @@ to_cdns_i3c_master(struct i3c_master_controller *mast= er) static void cdns_i3c_master_wr_to_tx_fifo(struct cdns_i3c_master *master, const u8 *bytes, int nbytes) { - i3c_writel_fifo(master->regs + TX_FIFO, bytes, nbytes); + i3c_writel_fifo(master->regs + TX_FIFO, bytes, nbytes, + I3C_FIFO_LITTLE_ENDIAN); } =20 static void cdns_i3c_master_rd_from_rx_fifo(struct cdns_i3c_master *master, u8 *bytes, int nbytes) { - i3c_readl_fifo(master->regs + RX_FIFO, bytes, nbytes); + i3c_readl_fifo(master->regs + RX_FIFO, bytes, nbytes, + I3C_FIFO_LITTLE_ENDIAN); } =20 static bool cdns_i3c_master_supports_ccc_cmd(struct i3c_master_controller = *m, @@ -1319,7 +1321,8 @@ static void cdns_i3c_master_handle_ibi(struct cdns_i3= c_master *master, buf =3D slot->data; =20 nbytes =3D IBIR_XFER_BYTES(ibir); - i3c_readl_fifo(master->regs + IBI_DATA_FIFO, buf, nbytes); + i3c_readl_fifo(master->regs + IBI_DATA_FIFO, buf, nbytes, + I3C_FIFO_LITTLE_ENDIAN); =20 slot->len =3D min_t(unsigned int, IBIR_XFER_BYTES(ibir), dev->ibi->max_payload_len); diff --git a/drivers/i3c/master/renesas-i3c.c b/drivers/i3c/master/renesas-= i3c.c index 174d3dc5d276..5610cf71740e 100644 --- a/drivers/i3c/master/renesas-i3c.c +++ b/drivers/i3c/master/renesas-i3c.c @@ -835,7 +835,8 @@ static int renesas_i3c_priv_xfers(struct i3c_dev_desc *= dev, struct i3c_priv_xfer } =20 if (!i3c_xfers[i].rnw && i3c_xfers[i].len > 4) { - i3c_writel_fifo(i3c->regs + NTDTBP0, cmd->tx_buf, cmd->len); + i3c_writel_fifo(i3c->regs + NTDTBP0, cmd->tx_buf, cmd->len, + I3C_FIFO_LITTLE_ENDIAN); if (cmd->len > NTDTBP0_DEPTH * sizeof(u32)) renesas_set_bit(i3c->regs, NTIE, NTIE_TDBEIE0); } @@ -1021,7 +1022,8 @@ static irqreturn_t renesas_i3c_tx_isr(int irq, void *= data) /* Clear the Transmit Buffer Empty status flag. */ renesas_clear_bit(i3c->regs, NTST, NTST_TDBEF0); } else { - i3c_writel_fifo(i3c->regs + NTDTBP0, cmd->tx_buf, cmd->len); + i3c_writel_fifo(i3c->regs + NTDTBP0, cmd->tx_buf, cmd->len, + I3C_FIFO_LITTLE_ENDIAN); } } =20 @@ -1061,7 +1063,8 @@ static irqreturn_t renesas_i3c_resp_isr(int irq, void= *data) if (NDBSTLV0_RDBLV(renesas_readl(i3c->regs, NDBSTLV0)) && !cmd->err) bytes_remaining =3D data_len - cmd->rx_count; =20 - i3c_readl_fifo(i3c->regs + NTDTBP0, cmd->rx_buf, bytes_remaining); + i3c_readl_fifo(i3c->regs + NTDTBP0, cmd->rx_buf, bytes_remaining, + I3C_FIFO_LITTLE_ENDIAN); renesas_clear_bit(i3c->regs, NTIE, NTIE_RDBFIE0); break; default: @@ -1203,7 +1206,8 @@ static irqreturn_t renesas_i3c_rx_isr(int irq, void *= data) cmd->i2c_bytes_left--; } else { read_bytes =3D NDBSTLV0_RDBLV(renesas_readl(i3c->regs, NDBSTLV0)) * siz= eof(u32); - i3c_readl_fifo(i3c->regs + NTDTBP0, cmd->rx_buf, read_bytes); + i3c_readl_fifo(i3c->regs + NTDTBP0, cmd->rx_buf, read_bytes, + I3C_FIFO_LITTLE_ENDIAN); cmd->rx_count =3D read_bytes; } =20 --=20 2.34.1 From nobody Wed Oct 1 22:31:20 2025 Received: from CH4PR04CU002.outbound.protection.outlook.com (mail-northcentralusazon11013041.outbound.protection.outlook.com [40.107.201.41]) (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 D105A299AB3; Fri, 26 Sep 2025 10:54:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.201.41 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758884094; cv=fail; b=ogl4AxlP/61HrCTEaWSNpkyvRxlY9OludVrMJeIwIXdfCuhlt8NVml0YW/myuMhbvHTIc2eUprigSLpWsksTm+PeO13hWAjGnmkErWSqyZDP4+8YPiKLNnS/0OnJK33ILkgZpbwlP/qSohLw8sNfwvTlBHSiRNkMlIqhCl4783Y= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758884094; c=relaxed/simple; bh=YClS2I/vXf6jBlqU2/Vqei/pwu8238A3DmBBaCLZufw=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=X9s3VkcZ3RF2vt8TRHCxLhXyX9EXst2kf7SafwHHyU6vGBlWT3Itu2I5F9O0iO37yE43QmWo5Q6tQ4J6hzBcZRTuz8T1ULqA+vJzEIrWODpdDMHkBa+9EgLI3T+1AjMEd6kFczPsri/zlWELSL+vf9y421CyAnnsUSawKJrmx+o= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com; spf=fail smtp.mailfrom=amd.com; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b=MwntlRPh; arc=fail smtp.client-ip=40.107.201.41 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="MwntlRPh" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=HLDH2S7+x4GyiXoNcx6U2svy9Tdd+2vJEUb2R45wDKKmRy/N6AgnEi/rSechQuYOJnS8HLX8/Te+q+5aU8mEmuWHRMeujjHS0DqclNbCrrhyoOpo6AAhER3MLU5HWyeAR4ytumORWhMD6VYAh0oxA21yre1GQcy37eE8s6aOoCM+xtf3Ux2R/sbtPU6DfnvJevwEMclJgI9sBC4PyVCekPr4QdDgHebFRAw01d6FI8d4xf7RF0ni9Iuli6wzPjS9AriqRWVivCDjdANh+JgA496xDBRLhT5zJ6J4/58gkZUeIUIH+RD87IpaaOJXfC7DVwa37uaWOmk/zB3H9ultsw== 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=kDz3KJhSRs3DCVTaokp/xDMkBDoXBD5yP+6r5WbSsKs=; b=xikShT0R+z3ur1exMctBJyCGoAGk5ANT6GAiOtsdVak8mQCwbMHU8rOYqFOewzq7sn4jwsIId36jt1aOkKKRWSTPoRE2GFAMl241xSC8lVGzSMVtNW5bljGFg1ywXe/s+7FHA1FCfCnNuHkfYzKg7oiyVjcZzqDjaedeIcKxs7JCSLPmfXPD21ttL4yoj+xZdWz7cbTAQFPHs41NGpB7Ku4RYghq7AbhS4fV6b6j9Irr3mFQXMqgXbH5CKfwscEAoOglGLj3y+igAn7DENixaQa5fM9YY9WxEGIhFPFpIVti+P1kiEVuR8UbbUO6UOnuUgXOeQYarjzUH3+lzuF2/A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=gmail.com smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=kDz3KJhSRs3DCVTaokp/xDMkBDoXBD5yP+6r5WbSsKs=; b=MwntlRPhtPhMkJ66+6MfyJduu7mUJp4uTro4yA+80I34Csuoyfz2ObjxE8rkN06DwBqypvdfWdVdnbKBxukxyPb4uYmccOtbwUBmaLS2kMSNgJBI1ILNkkZQV5As7Gm16ZAoz7Jvz1YvQp6N9eqpgLTSu+SuKddZRDX+Bcwr/9o= Received: from BL1PR13CA0075.namprd13.prod.outlook.com (2603:10b6:208:2b8::20) by DS7PR12MB6239.namprd12.prod.outlook.com (2603:10b6:8:95::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9160.11; Fri, 26 Sep 2025 10:54:45 +0000 Received: from BL02EPF0002992A.namprd02.prod.outlook.com (2603:10b6:208:2b8:cafe::b7) by BL1PR13CA0075.outlook.office365.com (2603:10b6:208:2b8::20) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.9160.10 via Frontend Transport; Fri, 26 Sep 2025 10:54:45 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=satlexmb08.amd.com; pr=C Received: from satlexmb08.amd.com (165.204.84.17) by BL02EPF0002992A.mail.protection.outlook.com (10.167.249.55) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9160.9 via Frontend Transport; Fri, 26 Sep 2025 10:54:45 +0000 Received: from SATLEXMB03.amd.com (10.181.40.144) by satlexmb08.amd.com (10.181.42.217) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.2.2562.17; Fri, 26 Sep 2025 03:54:45 -0700 Received: from satlexmb08.amd.com (10.181.42.217) by SATLEXMB03.amd.com (10.181.40.144) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Fri, 26 Sep 2025 05:54:43 -0500 Received: from xhdradheys41.xilinx.com (10.180.168.240) by satlexmb08.amd.com (10.181.42.217) with Microsoft SMTP Server id 15.2.2562.17 via Frontend Transport; Fri, 26 Sep 2025 03:54:36 -0700 From: Manikanta Guntupalli To: , , , , , , , , , , , , , , , , , , , , , , , CC: , , , , Manikanta Guntupalli Subject: [PATCH V8 5/5] i3c: master: Add AMD I3C bus controller driver Date: Fri, 26 Sep 2025 16:23:49 +0530 Message-ID: <20250926105349.2932952-6-manikanta.guntupalli@amd.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20250926105349.2932952-1-manikanta.guntupalli@amd.com> References: <20250926105349.2932952-1-manikanta.guntupalli@amd.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 Received-SPF: None (SATLEXMB03.amd.com: manikanta.guntupalli@amd.com does not designate permitted sender hosts) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BL02EPF0002992A:EE_|DS7PR12MB6239:EE_ X-MS-Office365-Filtering-Correlation-Id: fff90245-ae9c-4a59-3290-08ddfceb1b12 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|36860700013|7416014|376014|82310400026|1800799024|921020; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?qHFTmwMZcQhX+7wPQc0m+55dpYVAdBDW1kBWQYUSIRDMKEaRFfefyWJbLG7G?= =?us-ascii?Q?zjN0plVsCVnLBYCv312tIrh3XvH/amzSGVDl+pqeNc/r8m72ljY2xpD9yyzl?= =?us-ascii?Q?zNqoMjdtOOoPk1g26ZwfU36pzeIMxFVEDWsFIche9h+owwVgT0Kmm2TJuAlS?= =?us-ascii?Q?tAmPaIm8+kvabhcxZHPgDraOxWNTIcveSZLxn6uFFl9hAVekFvjiTpirtzK6?= =?us-ascii?Q?T0fimU1TepMjylYMGMv7lLRtVCgf9356FSMVvPsqHeG+EQUfIWJpVa5N2Yff?= =?us-ascii?Q?ng6rukfcZVUmUxi2LrbJ494QE5Cb+m+tBCIFWQ0/deQZqFDAPO+5NX+iFHg8?= =?us-ascii?Q?Jz2UL4xWyCiB6iRaNqyfwMOV1smkgvnGQ54cv8t8Qb1+zqwLFS5+dxbdw1ZL?= =?us-ascii?Q?KNTiUNsbaIXInmPxLQx0VoEEog5F25gAjxDfHfacoqS8W4VZFvnE/iAP5xr8?= =?us-ascii?Q?sPrsxoSSWjF5Dow4rUkz/JdAhWi4JdkwdeONh9Kjxsvd38vd6MLMdbZQ8bS1?= =?us-ascii?Q?RtmMuwZgwiFdkd6efHhB29o5PJsVQa+NH+/hIhBxtuH6gs9Ngn3oAvXGwr1+?= =?us-ascii?Q?AG3sgNsjX5YCaTRDL8yYnvF5edAFtOelPKo+OVmYLphyko2g+IeUCBwhU9ni?= =?us-ascii?Q?VLuceKR3668JoN/9YTXNY9TSKOappBzhR+h44j5pAyQkNWi6PGHJtbwkLhIh?= =?us-ascii?Q?hjqA3mYFXSYkxZJ5P0Ze7U2YF+t3TYTMSh2YXsdUajFsOl5xQmXFCHZM67fp?= =?us-ascii?Q?5P/je4I4kEJ8GKuUlTF3GwgFYwpQjoE9vTq8pa+XyK54KU3dGjMs4fHso+Kp?= =?us-ascii?Q?MyPNcCKOxtAoeyZMtT1akJ5h+KtfKos4QrOcMJXPpVlscypyHg9Pfi+hkVqe?= =?us-ascii?Q?oxjLTWeaYoNKd2Cl869CiahnB7fSHmWpuia9c+DEW7mBa9HJIIUN8WmfIDJc?= =?us-ascii?Q?r34ky2DbfHZ8qU4X+8CduvN/FYAsAsE41AUaJryOhl7H64R0hkcKuymIQYs2?= =?us-ascii?Q?/8y5/O2puo/MfUcs7cB/BNse8nMWMiG5llILCFDjMbL0RWzkrk0czMUxkBRr?= =?us-ascii?Q?sK/KgJPmwRwOcSYJgxqY48djo0emR2KrdWiGO48EQIcQlTvsVWOqW+dkWGnr?= =?us-ascii?Q?6s5/HIqUFRozCg99pA5HEcKxpp5kERmfFipgeyb3SQ4ETn9gPIEf36V5CAPQ?= =?us-ascii?Q?BTtAYzTQjPtEgNR+FThgHUjrY7RAm66lbj825Jl85FJDgon580+Tc16XQ81m?= =?us-ascii?Q?EIl4YVgw9axjp0tJK/VN6uI6PqpcxCxVcozWXxftz08DH8cZGg/QT2GZvZRf?= =?us-ascii?Q?86wJ7xYGRdjay4A1DVdaOhCAUWc3HiQRBD9BUmuQp9EFM+4mpojqjiwLeijQ?= =?us-ascii?Q?tfzR+gvCssbqXGofW2/WU8vuPBF3xUEjIYE4Fku/a96gv+Uc490FVMMEZCFe?= =?us-ascii?Q?12IWfZaaReDgpNaG9JYs4cspDKXXnRO0ghkZoPJOh0449jJlJd4EqjgbOv09?= =?us-ascii?Q?iio/o6NtYraM2i1GVakg4T+L3v1ThOJtf1M39/g0Xuwgwys31Cus8945Zg?= =?us-ascii?Q?=3D=3D?= X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:satlexmb08.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(36860700013)(7416014)(376014)(82310400026)(1800799024)(921020);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Sep 2025 10:54:45.7736 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: fff90245-ae9c-4a59-3290-08ddfceb1b12 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[satlexmb08.amd.com] X-MS-Exchange-CrossTenant-AuthSource: BL02EPF0002992A.namprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR12MB6239 Content-Type: text/plain; charset="utf-8" Add an I3C master driver and maintainers fragment for the AMD I3C bus controller. The driver currently supports the I3C bus operating in SDR i3c mode, with features including Dynamic Address Assignment, private data transfers, and CCC transfers in both broadcast and direct modes. It also supports operation in I2C mode. Signed-off-by: Manikanta Guntupalli --- Changes for V2: Updated commit description. Added mixed mode support with clock configuration. Converted smaller functions into inline functions. Used FIELD_GET() in xi3c_get_response(). Updated xi3c_master_rd_from_rx_fifo() to use cmd->rx_buf. Used parity8() for address parity calculation. Added guards for locks. Dropped num_targets and updated xi3c_master_do_daa(). Used __free(kfree) in xi3c_master_send_bdcast_ccc_cmd(). Dropped PM runtime support. Updated xi3c_master_read() and xi3c_master_write() with xi3c_is_resp_available() check. Created separate functions: xi3c_master_init() and xi3c_master_reinit(). Used xi3c_master_init() in bus initialization and xi3c_master_reinit() in error paths. Added DAA structure to xi3c_master structure. Changes for V3: Resolved merge conflicts. Changes for V4: Updated timeout macros. Removed type casting for xi3c_is_resp_available() macro. Used ioread32() and iowrite32() instead of readl() and writel() to keep consistency. Read XI3C_RESET_OFFSET reg before udelay(). Removed xi3c_master_free_xfer() and directly used kfree(). Skipped checking return value of i3c_master_add_i3c_dev_locked(). Used devm_mutex_init() instead of mutex_init(). Changes for V5: Used GENMASK_ULL for PID mask as it's 64bit mask. Changes for V6: Removed typecast for xi3c_getrevisionnumber(), xi3c_wrfifolevel(), and xi3c_rdfifolevel(). Replaced dynamic allocation with a static variable for pid_bcr_dcr. Fixed sparse warning in do_daa by typecasting the address parity value to u8. Fixed sparse warning in xi3c_master_bus_init by typecasting the pid value to u64 in info.pid calculation. Changes for V7: Updated timeout macro name. Updated xi3c_master_wr_to_tx_fifo() and xi3c_master_rd_from_rx_fifo() to use i3c_writel_fifo() and i3c_readl_fifo(). Changes for V8: Used time_left instead of timeout. Used __free(kfree) for xfer to simplify err path in multiple places. --- MAINTAINERS | 7 + drivers/i3c/master/Kconfig | 16 + drivers/i3c/master/Makefile | 1 + drivers/i3c/master/amd-i3c-master.c | 1002 +++++++++++++++++++++++++++ 4 files changed, 1026 insertions(+) create mode 100644 drivers/i3c/master/amd-i3c-master.c diff --git a/MAINTAINERS b/MAINTAINERS index 8886d66bd824..fe88efb41f5b 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -11782,6 +11782,13 @@ L: linux-i2c@vger.kernel.org S: Maintained F: drivers/i2c/i2c-stub.c =20 +I3C DRIVER FOR AMD AXI I3C MASTER +M: Manikanta Guntupalli +R: Michal Simek +S: Maintained +F: Documentation/devicetree/bindings/i3c/xlnx,axi-i3c.yaml +F: drivers/i3c/master/amd-i3c-master.c + I3C DRIVER FOR ASPEED AST2600 M: Jeremy Kerr S: Maintained diff --git a/drivers/i3c/master/Kconfig b/drivers/i3c/master/Kconfig index 13df2944f2ec..4b884a678893 100644 --- a/drivers/i3c/master/Kconfig +++ b/drivers/i3c/master/Kconfig @@ -1,4 +1,20 @@ # SPDX-License-Identifier: GPL-2.0-only + +config AMD_I3C_MASTER + tristate "AMD I3C Master driver" + depends on I3C + depends on HAS_IOMEM + help + Support for AMD I3C Master Controller. + + This driver allows communication with I3C devices using the AMD + I3C master, currently supporting Standard Data Rate (SDR) mode. + Features include Dynamic Address Assignment, private transfers, + and CCC transfers in both broadcast and direct modes. + + This driver can also be built as a module. If so, the module + will be called amd-i3c-master. + config CDNS_I3C_MASTER tristate "Cadence I3C master driver" depends on HAS_IOMEM diff --git a/drivers/i3c/master/Makefile b/drivers/i3c/master/Makefile index aac74f3e3851..109bd48cb159 100644 --- a/drivers/i3c/master/Makefile +++ b/drivers/i3c/master/Makefile @@ -1,4 +1,5 @@ # SPDX-License-Identifier: GPL-2.0-only +obj-$(CONFIG_AMD_I3C_MASTER) +=3D amd-i3c-master.o obj-$(CONFIG_CDNS_I3C_MASTER) +=3D i3c-master-cdns.o obj-$(CONFIG_DW_I3C_MASTER) +=3D dw-i3c-master.o obj-$(CONFIG_AST2600_I3C_MASTER) +=3D ast2600-i3c-master.o diff --git a/drivers/i3c/master/amd-i3c-master.c b/drivers/i3c/master/amd-i= 3c-master.c new file mode 100644 index 000000000000..e69c4ff89ebb --- /dev/null +++ b/drivers/i3c/master/amd-i3c-master.c @@ -0,0 +1,1002 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * I3C master driver for the AMD I3C controller. + * + * Copyright (C) 2025, Advanced Micro Devices, Inc. + */ + +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include + +#include "../internals.h" + +#define XI3C_VERSION_OFFSET 0x00 /* Version Register */ +#define XI3C_RESET_OFFSET 0x04 /* Soft Reset Register */ +#define XI3C_CR_OFFSET 0x08 /* Control Register */ +#define XI3C_ADDRESS_OFFSET 0x0C /* Target Address Register */ +#define XI3C_SR_OFFSET 0x10 /* Status Register */ +#define XI3C_CMD_FIFO_OFFSET 0x20 /* I3C Command FIFO Register */ +#define XI3C_WR_FIFO_OFFSET 0x24 /* I3C Write Data FIFO Register */ +#define XI3C_RD_FIFO_OFFSET 0x28 /* I3C Read Data FIFO Register */ +#define XI3C_RESP_STATUS_FIFO_OFFSET 0x2C /* I3C Response status FIFO Reg= ister */ +#define XI3C_FIFO_LVL_STATUS_OFFSET 0x30 /* I3C CMD & WR FIFO LVL Registe= r */ +#define XI3C_FIFO_LVL_STATUS_1_OFFSET 0x34 /* I3C RESP & RD FIFO LVL Regi= ster */ +#define XI3C_SCL_HIGH_TIME_OFFSET 0x38 /* I3C SCL HIGH Register */ +#define XI3C_SCL_LOW_TIME_OFFSET 0x3C /* I3C SCL LOW Register */ +#define XI3C_SDA_HOLD_TIME_OFFSET 0x40 /* I3C SDA HOLD Register */ +#define XI3C_TSU_START_OFFSET 0x48 /* I3C START SETUP Register */ +#define XI3C_THD_START_OFFSET 0x4C /* I3C START HOLD Register */ +#define XI3C_TSU_STOP_OFFSET 0x50 /* I3C STOP Setup Register */ +#define XI3C_OD_SCL_HIGH_TIME_OFFSET 0x54 /* I3C OD SCL HIGH Register */ +#define XI3C_OD_SCL_LOW_TIME_OFFSET 0x58 /* I3C OD SCL LOW Register */ +#define XI3C_PID0_OFFSET 0x6C /* LSB 4 bytes of the PID */ +#define XI3C_PID1_BCR_DCR 0x70 /* MSB 2 bytes of the PID, BCR and DCR */ + +#define XI3C_CR_EN_MASK BIT(0) /* Core Enable */ +#define XI3C_CR_RESUME_MASK BIT(2) /* Core Resume */ +#define XI3C_SR_RESP_NOT_EMPTY_MASK BIT(4) /* Resp Fifo not empty status = mask */ +#define XI3C_RD_FIFO_NOT_EMPTY_MASK BIT(15) /* Read Fifo not empty status= mask */ + +#define XI3C_BCR_MASK GENMASK(23, 16) +#define XI3C_DCR_MASK GENMASK(31, 24) +#define XI3C_PID_MASK GENMASK_ULL(63, 16) +#define XI3C_SCL_HIGH_TIME_MASK GENMASK(17, 0) +#define XI3C_SCL_LOW_TIME_MASK GENMASK(17, 0) +#define XI3C_SDA_HOLD_TIME_MASK GENMASK(17, 0) +#define XI3C_TSU_START_MASK GENMASK(17, 0) +#define XI3C_THD_START_MASK GENMASK(17, 0) +#define XI3C_TSU_STOP_MASK GENMASK(17, 0) +#define XI3C_REV_NUM_MASK GENMASK(15, 8) +#define XI3C_PID1_MASK GENMASK(15, 0) +#define XI3C_WR_FIFO_LEVEL_MASK GENMASK(15, 0) +#define XI3C_CMD_LEN_MASK GENMASK(11, 0) +#define XI3C_RESP_CODE_MASK GENMASK(8, 5) +#define XI3C_ADDR_MASK GENMASK(6, 0) +#define XI3C_CMD_TYPE_MASK GENMASK(3, 0) +#define XI3C_CMD_TID_MASK GENMASK(3, 0) +#define XI3C_FIFOS_RST_MASK GENMASK(4, 1) + +#define XI3C_OD_TLOW_NS 500000 +#define XI3C_OD_THIGH_NS 41000 +#define XI3C_I2C_TCASMIN_NS 600000 +#define XI3C_TCASMIN_NS 260000 +#define XI3C_MAXDATA_LENGTH 4095 +#define XI3C_MAX_DEVS 32 +#define XI3C_DAA_SLAVEINFO_READ_BYTECOUNT 8 + +#define XI3C_I2C_MODE 0 +#define XI3C_I2C_TID 0 +#define XI3C_SDR_MODE 1 +#define XI3C_SDR_TID 1 + +#define XI3C_WORD_LEN 4 + +/* timeout waiting for the controller finish transfers */ +#define XI3C_XFER_TIMEOUT_MS 100000 +#define XI3C_XFER_TIMEOUT_JIFFIES (msecs_to_jiffies(XI3C_XFER_TIMEOUT_MS)) + +#define xi3c_getrevisionnumber(master) \ + (FIELD_GET(XI3C_REV_NUM_MASK, \ + ioread32((master)->membase + XI3C_VERSION_OFFSET))) + +#define xi3c_wrfifolevel(master) \ + (ioread32((master)->membase + XI3C_FIFO_LVL_STATUS_OFFSET) & \ + XI3C_WR_FIFO_LEVEL_MASK) + +#define xi3c_rdfifolevel(master) \ + (ioread32((master)->membase + XI3C_FIFO_LVL_STATUS_1_OFFSET) & \ + XI3C_WR_FIFO_LEVEL_MASK) + +#define xi3c_is_resp_available(master) \ + (FIELD_GET(XI3C_SR_RESP_NOT_EMPTY_MASK, \ + ioread32((master)->membase + XI3C_SR_OFFSET))) + +struct xi3c_cmd { + void *tx_buf; + void *rx_buf; + u16 tx_len; + u16 rx_len; + u8 addr; + u8 type; + u8 tid; + bool rnw; + bool is_daa; + bool continued; +}; + +struct xi3c_xfer { + struct list_head node; + struct completion comp; + int ret; + unsigned int ncmds; + struct xi3c_cmd cmds[] __counted_by(ncmds); +}; + +/** + * struct xi3c_master - I3C Master structure + * @base: I3C master controller + * @dev: Pointer to device structure + * @xferqueue: Transfer queue structure + * @xferqueue.list: List member + * @xferqueue.cur: Current ongoing transfer + * @xferqueue.lock: Queue lock + * @membase: Memory base of the HW registers + * @pclk: Input clock + * @lock: Transfer lock + * @daa: daa structure + * @daa.addrs: Slave addresses array + * @daa.index: Slave device index + */ +struct xi3c_master { + struct i3c_master_controller base; + struct device *dev; + struct { + struct list_head list; + struct xi3c_xfer *cur; + /* Queue lock */ + spinlock_t lock; + } xferqueue; + void __iomem *membase; + struct clk *pclk; + /* Transfer lock */ + struct mutex lock; + struct { + u8 addrs[XI3C_MAX_DEVS]; + u8 index; + } daa; +}; + +static inline struct xi3c_master * +to_xi3c_master(struct i3c_master_controller *master) +{ + return container_of(master, struct xi3c_master, base); +} + +static int xi3c_get_response(struct xi3c_master *master) +{ + u32 resp_reg, response_data; + int ret; + + ret =3D readl_poll_timeout(master->membase + XI3C_SR_OFFSET, + resp_reg, + resp_reg & XI3C_SR_RESP_NOT_EMPTY_MASK, + 0, XI3C_XFER_TIMEOUT_MS); + if (ret) { + dev_err(master->dev, "XI3C response timeout\n"); + return ret; + } + + response_data =3D ioread32(master->membase + XI3C_RESP_STATUS_FIFO_OFFSET= ); + + /* Return response code */ + return FIELD_GET(XI3C_RESP_CODE_MASK, response_data); +} + +static void xi3c_master_write_to_cmdfifo(struct xi3c_master *master, + struct xi3c_cmd *cmd, u16 len) +{ + u32 transfer_cmd =3D 0; + u8 addr; + + addr =3D ((cmd->addr & XI3C_ADDR_MASK) << 1) | (cmd->rnw & BIT(0)); + + transfer_cmd =3D cmd->type & XI3C_CMD_TYPE_MASK; + transfer_cmd |=3D (u32)(!cmd->continued) << 4; + transfer_cmd |=3D (u32)(addr) << 8; + transfer_cmd |=3D (u32)(cmd->tid & XI3C_CMD_TID_MASK) << 28; + + /* + * For dynamic addressing, an additional 1-byte length must be added + * to the command FIFO to account for the address present in the TX FIFO + */ + if (cmd->is_daa) { + i3c_writel_fifo(master->membase + XI3C_WR_FIFO_OFFSET, + (u8 *)cmd->tx_buf, cmd->tx_len, I3C_FIFO_BIG_ENDIAN); + + len++; + master->daa.index++; + } + + transfer_cmd |=3D (u32)(len & XI3C_CMD_LEN_MASK) << 16; + iowrite32(transfer_cmd, master->membase + XI3C_CMD_FIFO_OFFSET); +} + +static inline void xi3c_master_enable(struct xi3c_master *master) +{ + iowrite32(ioread32(master->membase + XI3C_CR_OFFSET) | XI3C_CR_EN_MASK, + master->membase + XI3C_CR_OFFSET); +} + +static inline void xi3c_master_disable(struct xi3c_master *master) +{ + iowrite32(ioread32(master->membase + XI3C_CR_OFFSET) & (~XI3C_CR_EN_MASK), + master->membase + XI3C_CR_OFFSET); +} + +static inline void xi3c_master_resume(struct xi3c_master *master) +{ + iowrite32(ioread32(master->membase + XI3C_CR_OFFSET) | + XI3C_CR_RESUME_MASK, master->membase + XI3C_CR_OFFSET); +} + +static void xi3c_master_reset_fifos(struct xi3c_master *master) +{ + u32 data; + + /* Reset fifos */ + data =3D ioread32(master->membase + XI3C_RESET_OFFSET); + data |=3D XI3C_FIFOS_RST_MASK; + iowrite32(data, master->membase + XI3C_RESET_OFFSET); + ioread32(master->membase + XI3C_RESET_OFFSET); + udelay(10); + data &=3D ~XI3C_FIFOS_RST_MASK; + iowrite32(data, master->membase + XI3C_RESET_OFFSET); + ioread32(master->membase + XI3C_RESET_OFFSET); + udelay(10); +} + +static inline void xi3c_master_init(struct xi3c_master *master) +{ + /* Reset fifos */ + xi3c_master_reset_fifos(master); + + /* Enable controller */ + xi3c_master_enable(master); +} + +static inline void xi3c_master_reinit(struct xi3c_master *master) +{ + /* Reset fifos */ + xi3c_master_reset_fifos(master); + + /* Resume controller */ + xi3c_master_resume(master); +} + +static struct xi3c_xfer * +xi3c_master_alloc_xfer(struct xi3c_master *master, unsigned int ncmds) +{ + struct xi3c_xfer *xfer; + + xfer =3D kzalloc(struct_size(xfer, cmds, ncmds), GFP_KERNEL); + if (!xfer) + return NULL; + + INIT_LIST_HEAD(&xfer->node); + xfer->ncmds =3D ncmds; + xfer->ret =3D -ETIMEDOUT; + + return xfer; +} + +static void xi3c_master_rd_from_rx_fifo(struct xi3c_master *master, + struct xi3c_cmd *cmd) +{ + u16 rx_data_available; + u16 len; + + rx_data_available =3D xi3c_rdfifolevel(master); + len =3D rx_data_available * XI3C_WORD_LEN; + + if (len) { + i3c_readl_fifo(master->membase + XI3C_RD_FIFO_OFFSET, (u8 *)cmd->rx_buf, + len, I3C_FIFO_BIG_ENDIAN); + + cmd->rx_buf =3D (u8 *)cmd->rx_buf + len; + cmd->rx_len -=3D len; + } +} + +static int xi3c_master_read(struct xi3c_master *master, struct xi3c_cmd *c= md) +{ + unsigned long timeout; + u32 status_reg; + int ret; + + if (!cmd->rx_buf || cmd->rx_len > XI3C_MAXDATA_LENGTH) + return -EINVAL; + + /* Fill command fifo */ + xi3c_master_write_to_cmdfifo(master, cmd, cmd->rx_len); + + ret =3D readl_poll_timeout(master->membase + XI3C_SR_OFFSET, + status_reg, + status_reg & XI3C_RD_FIFO_NOT_EMPTY_MASK, + 0, XI3C_XFER_TIMEOUT_MS); + if (ret) { + if (cmd->is_daa) { + cmd->is_daa =3D false; + ret =3D I3C_ERROR_M2; + } else { + dev_err(master->dev, "XI3C read timeout\n"); + } + return ret; + } + + timeout =3D jiffies + XI3C_XFER_TIMEOUT_JIFFIES; + + /* Read data from rx fifo */ + while (cmd->rx_len > 0 && !xi3c_is_resp_available(master)) { + if (time_after(jiffies, timeout)) { + dev_err(master->dev, "XI3C read timeout\n"); + return -EIO; + } + xi3c_master_rd_from_rx_fifo(master, cmd); + } + + /* Read remaining data */ + xi3c_master_rd_from_rx_fifo(master, cmd); + + return 0; +} + +static void xi3c_master_wr_to_tx_fifo(struct xi3c_master *master, + struct xi3c_cmd *cmd) +{ + u16 wrfifo_space; + u16 len; + + wrfifo_space =3D xi3c_wrfifolevel(master); + if (cmd->tx_len > wrfifo_space * XI3C_WORD_LEN) + len =3D wrfifo_space * XI3C_WORD_LEN; + else + len =3D cmd->tx_len; + + if (len) { + i3c_writel_fifo(master->membase + XI3C_WR_FIFO_OFFSET, (u8 *)cmd->tx_buf, + len, I3C_FIFO_BIG_ENDIAN); + + cmd->tx_buf =3D (u8 *)cmd->tx_buf + len; + cmd->tx_len -=3D len; + } +} + +static int xi3c_master_write(struct xi3c_master *master, struct xi3c_cmd *= cmd) +{ + unsigned long timeout; + u16 cmd_len; + + cmd_len =3D cmd->tx_len; + if (!cmd->tx_buf || cmd->tx_len > XI3C_MAXDATA_LENGTH) + return -EINVAL; + + /* Fill Tx fifo */ + xi3c_master_wr_to_tx_fifo(master, cmd); + + /* Write to command fifo */ + xi3c_master_write_to_cmdfifo(master, cmd, cmd_len); + + timeout =3D jiffies + XI3C_XFER_TIMEOUT_JIFFIES; + /* Fill if any remaining data to tx fifo */ + while (cmd->tx_len > 0 && !xi3c_is_resp_available(master)) { + if (time_after(jiffies, timeout)) { + dev_err(master->dev, "XI3C write timeout\n"); + return -EIO; + } + + xi3c_master_wr_to_tx_fifo(master, cmd); + } + + return 0; +} + +static int xi3c_master_xfer(struct xi3c_master *master, struct xi3c_cmd *c= md) +{ + int ret; + + if (cmd->rnw) + ret =3D xi3c_master_read(master, cmd); + else + ret =3D xi3c_master_write(master, cmd); + + if (ret < 0) + goto err_xfer_out; + + ret =3D xi3c_get_response(master); + if (ret) + goto err_xfer_out; + + return 0; + +err_xfer_out: + xi3c_master_reinit(master); + return ret; +} + +static void xi3c_master_dequeue_xfer_locked(struct xi3c_master *master, + struct xi3c_xfer *xfer) +{ + if (master->xferqueue.cur =3D=3D xfer) + master->xferqueue.cur =3D NULL; + else + list_del_init(&xfer->node); +} + +static void xi3c_master_dequeue_xfer(struct xi3c_master *master, + struct xi3c_xfer *xfer) +{ + guard(spinlock_irqsave)(&master->xferqueue.lock); + + xi3c_master_dequeue_xfer_locked(master, xfer); +} + +static void xi3c_master_start_xfer_locked(struct xi3c_master *master) +{ + struct xi3c_xfer *xfer =3D master->xferqueue.cur; + int ret =3D 0, i; + + if (!xfer) + return; + + for (i =3D 0; i < xfer->ncmds; i++) { + struct xi3c_cmd *cmd =3D &xfer->cmds[i]; + + ret =3D xi3c_master_xfer(master, cmd); + if (ret) + break; + } + + xfer->ret =3D ret; + complete(&xfer->comp); + + xfer =3D list_first_entry_or_null(&master->xferqueue.list, + struct xi3c_xfer, + node); + if (xfer) + list_del_init(&xfer->node); + + master->xferqueue.cur =3D xfer; + xi3c_master_start_xfer_locked(master); +} + +static inline void xi3c_master_enqueue_xfer(struct xi3c_master *master, + struct xi3c_xfer *xfer) +{ + init_completion(&xfer->comp); + + guard(spinlock_irqsave)(&master->xferqueue.lock); + + if (master->xferqueue.cur) { + list_add_tail(&xfer->node, &master->xferqueue.list); + } else { + master->xferqueue.cur =3D xfer; + xi3c_master_start_xfer_locked(master); + } +} + +static inline int xi3c_master_common_xfer(struct xi3c_master *master, + struct xi3c_xfer *xfer) +{ + int ret, time_left; + + guard(mutex)(&master->lock); + + xi3c_master_enqueue_xfer(master, xfer); + time_left =3D wait_for_completion_timeout(&xfer->comp, + XI3C_XFER_TIMEOUT_JIFFIES); + if (!time_left) + ret =3D -ETIMEDOUT; + else + ret =3D xfer->ret; + + if (ret) + xi3c_master_dequeue_xfer(master, xfer); + + return ret; +} + +static int xi3c_master_do_daa(struct i3c_master_controller *m) +{ + struct xi3c_master *master =3D to_xi3c_master(m); + struct xi3c_cmd *daa_cmd; + struct xi3c_xfer *xfer __free(kfree); + u8 pid_bufs[XI3C_MAX_DEVS][8]; + u8 *pid_buf; + u8 data, last_addr =3D 0; + int addr, ret, i; + + xfer =3D xi3c_master_alloc_xfer(master, 1); + if (!xfer) { + ret =3D -ENOMEM; + goto err_daa; + } + + for (i =3D 0; i < XI3C_MAX_DEVS; i++) { + addr =3D i3c_master_get_free_addr(m, last_addr + 1); + if (addr < 0) { + ret =3D -ENOSPC; + goto err_daa; + } + master->daa.addrs[i] =3D (u8)addr; + last_addr =3D (u8)addr; + } + + /* Fill ENTDAA CCC */ + data =3D I3C_CCC_ENTDAA; + daa_cmd =3D &xfer->cmds[0]; + daa_cmd->addr =3D I3C_BROADCAST_ADDR; + daa_cmd->rnw =3D 0; + daa_cmd->tx_buf =3D &data; + daa_cmd->tx_len =3D 1; + daa_cmd->type =3D XI3C_SDR_MODE; + daa_cmd->tid =3D XI3C_SDR_TID; + daa_cmd->continued =3D true; + + ret =3D xi3c_master_common_xfer(master, xfer); + /* DAA always finishes with CE2_ERROR or NACK_RESP */ + if (ret && ret !=3D I3C_ERROR_M2) { + goto err_daa; + } else { + if (ret && ret =3D=3D I3C_ERROR_M2) { + ret =3D 0; + goto err_daa; + } + } + + master->daa.index =3D 0; + + while (true) { + struct xi3c_cmd *cmd =3D &xfer->cmds[0]; + + pid_buf =3D pid_bufs[master->daa.index]; + addr =3D (master->daa.addrs[master->daa.index] << 1) | + (u8)(!parity8(master->daa.addrs[master->daa.index])); + + cmd->tx_buf =3D (u8 *)&addr; + cmd->tx_len =3D 1; + cmd->addr =3D I3C_BROADCAST_ADDR; + cmd->rnw =3D 1; + cmd->rx_buf =3D pid_buf; + cmd->rx_len =3D XI3C_DAA_SLAVEINFO_READ_BYTECOUNT; + cmd->is_daa =3D true; + cmd->type =3D XI3C_SDR_MODE; + cmd->tid =3D XI3C_SDR_TID; + cmd->continued =3D true; + + ret =3D xi3c_master_common_xfer(master, xfer); + + /* DAA always finishes with CE2_ERROR or NACK_RESP */ + if (ret && ret !=3D I3C_ERROR_M2) { + goto err_daa; + } else { + if (ret && ret =3D=3D I3C_ERROR_M2) { + xi3c_master_resume(master); + master->daa.index--; + ret =3D 0; + break; + } + } + } + + for (i =3D 0; i < master->daa.index; i++) { + i3c_master_add_i3c_dev_locked(m, master->daa.addrs[i]); + + u64 data =3D FIELD_GET(XI3C_PID_MASK, get_unaligned_be64(pid_bufs[i])); + + dev_info(master->dev, "Client %d: PID: 0x%llx\n", i, data); + } + + return 0; + +err_daa: + xi3c_master_reinit(master); + return ret; +} + +static bool +xi3c_master_supports_ccc_cmd(struct i3c_master_controller *master, + const struct i3c_ccc_cmd *cmd) +{ + if (cmd->ndests > 1) + return false; + + switch (cmd->id) { + case I3C_CCC_ENEC(true): + case I3C_CCC_ENEC(false): + case I3C_CCC_DISEC(true): + case I3C_CCC_DISEC(false): + case I3C_CCC_ENTAS(0, true): + case I3C_CCC_ENTAS(0, false): + case I3C_CCC_RSTDAA(true): + case I3C_CCC_RSTDAA(false): + case I3C_CCC_ENTDAA: + case I3C_CCC_SETMWL(true): + case I3C_CCC_SETMWL(false): + case I3C_CCC_SETMRL(true): + case I3C_CCC_SETMRL(false): + case I3C_CCC_ENTHDR(0): + case I3C_CCC_SETDASA: + case I3C_CCC_SETNEWDA: + case I3C_CCC_GETMWL: + case I3C_CCC_GETMRL: + case I3C_CCC_GETPID: + case I3C_CCC_GETBCR: + case I3C_CCC_GETDCR: + case I3C_CCC_GETSTATUS: + case I3C_CCC_GETMXDS: + return true; + default: + return false; + } +} + +static int xi3c_master_send_bdcast_ccc_cmd(struct xi3c_master *master, + struct i3c_ccc_cmd *ccc) +{ + u16 xfer_len =3D ccc->dests[0].payload.len + 1; + struct xi3c_xfer *xfer __free(kfree); + struct xi3c_cmd *cmd; + int ret; + + xfer =3D xi3c_master_alloc_xfer(master, 1); + if (!xfer) + return -ENOMEM; + + u8 *buf __free(kfree) =3D kmalloc(xfer_len, GFP_KERNEL); + if (!buf) + return -ENOMEM; + + buf[0] =3D ccc->id; + memcpy(&buf[1], ccc->dests[0].payload.data, ccc->dests[0].payload.len); + + cmd =3D &xfer->cmds[0]; + cmd->addr =3D ccc->dests[0].addr; + cmd->rnw =3D ccc->rnw; + cmd->tx_buf =3D buf; + cmd->tx_len =3D xfer_len; + cmd->type =3D XI3C_SDR_MODE; + cmd->tid =3D XI3C_SDR_TID; + cmd->continued =3D false; + + ret =3D xi3c_master_common_xfer(master, xfer); + + return ret; +} + +static int xi3c_master_send_direct_ccc_cmd(struct xi3c_master *master, + struct i3c_ccc_cmd *ccc) +{ + struct xi3c_xfer *xfer __free(kfree); + struct xi3c_cmd *cmd; + int ret; + + xfer =3D xi3c_master_alloc_xfer(master, 2); + if (!xfer) + return -ENOMEM; + + /* Broadcasted message */ + cmd =3D &xfer->cmds[0]; + cmd->addr =3D I3C_BROADCAST_ADDR; + cmd->rnw =3D 0; + cmd->tx_buf =3D &ccc->id; + cmd->tx_len =3D 1; + cmd->type =3D XI3C_SDR_MODE; + cmd->tid =3D XI3C_SDR_TID; + cmd->continued =3D true; + + /* Directed message */ + cmd =3D &xfer->cmds[1]; + cmd->addr =3D ccc->dests[0].addr; + cmd->rnw =3D ccc->rnw; + if (cmd->rnw) { + cmd->rx_buf =3D ccc->dests[0].payload.data; + cmd->rx_len =3D ccc->dests[0].payload.len; + } else { + cmd->tx_buf =3D ccc->dests[0].payload.data; + cmd->tx_len =3D ccc->dests[0].payload.len; + } + cmd->type =3D XI3C_SDR_MODE; + cmd->tid =3D XI3C_SDR_TID; + cmd->continued =3D false; + + ret =3D xi3c_master_common_xfer(master, xfer); + return ret; +} + +static int xi3c_master_send_ccc_cmd(struct i3c_master_controller *m, + struct i3c_ccc_cmd *cmd) +{ + struct xi3c_master *master =3D to_xi3c_master(m); + bool broadcast =3D cmd->id < 0x80; + + if (broadcast) + return xi3c_master_send_bdcast_ccc_cmd(master, cmd); + + return xi3c_master_send_direct_ccc_cmd(master, cmd); +} + +static int xi3c_master_priv_xfers(struct i3c_dev_desc *dev, + struct i3c_priv_xfer *xfers, + int nxfers) +{ + struct i3c_master_controller *m =3D i3c_dev_get_master(dev); + struct xi3c_master *master =3D to_xi3c_master(m); + struct xi3c_xfer *xfer __free(kfree); + int i, ret; + + if (!nxfers) + return 0; + + xfer =3D xi3c_master_alloc_xfer(master, nxfers); + if (!xfer) + return -ENOMEM; + + for (i =3D 0; i < nxfers; i++) { + struct xi3c_cmd *cmd =3D &xfer->cmds[i]; + + cmd->addr =3D dev->info.dyn_addr; + cmd->rnw =3D xfers[i].rnw; + + if (cmd->rnw) { + cmd->rx_buf =3D xfers[i].data.in; + cmd->rx_len =3D xfers[i].len; + } else { + cmd->tx_buf =3D (void *)xfers[i].data.out; + cmd->tx_len =3D xfers[i].len; + } + + cmd->type =3D XI3C_SDR_MODE; + cmd->tid =3D XI3C_SDR_TID; + cmd->continued =3D (i + 1) < nxfers; + } + + ret =3D xi3c_master_common_xfer(master, xfer); + return ret; +} + +static int xi3c_master_i2c_xfers(struct i2c_dev_desc *dev, + struct i2c_msg *xfers, + int nxfers) +{ + struct i3c_master_controller *m =3D i2c_dev_get_master(dev); + struct xi3c_master *master =3D to_xi3c_master(m); + struct xi3c_xfer *xfer __free(kfree); + int i, ret; + + if (!nxfers) + return 0; + + xfer =3D xi3c_master_alloc_xfer(master, nxfers); + if (!xfer) + return -ENOMEM; + + for (i =3D 0; i < nxfers; i++) { + struct xi3c_cmd *cmd =3D &xfer->cmds[i]; + + cmd->addr =3D xfers[i].addr & XI3C_ADDR_MASK; + cmd->rnw =3D xfers[i].flags & I2C_M_RD; + + if (cmd->rnw) { + cmd->rx_buf =3D xfers[i].buf; + cmd->rx_len =3D xfers[i].len; + } else { + cmd->tx_buf =3D (void *)xfers[i].buf; + cmd->tx_len =3D xfers[i].len; + } + + cmd->type =3D XI3C_I2C_MODE; + cmd->tid =3D XI3C_I2C_TID; + cmd->continued =3D (i + 1) < nxfers; + } + + ret =3D xi3c_master_common_xfer(master, xfer); + return ret; +} + +static int xi3c_clk_cfg(struct xi3c_master *master, unsigned long sclhz, u= 8 mode) +{ + unsigned long core_rate, core_periodns; + u32 thigh, tlow, thold, odthigh, odtlow, tcasmin, tsustart, tsustop, thds= tart; + + core_rate =3D clk_get_rate(master->pclk); + if (!core_rate) + return -EINVAL; + + core_periodns =3D DIV_ROUND_UP(1000000000, core_rate); + + thigh =3D DIV_ROUND_UP(core_rate, sclhz) >> 1; + tlow =3D thigh; + + /* Hold time : 40% of tlow time */ + thold =3D (tlow * 4) / 10; + + /* + * For initial IP (revision number =3D=3D 0), minimum data hold time is 5. + * For updated IP (revision number > 0), minimum data hold time is 6. + * Updated IP supports achieving high data rate with low reference + * frequency. + */ + if (xi3c_getrevisionnumber(master) =3D=3D 0) + thold =3D (thold < 5) ? 5 : thold; + else + thold =3D (thold < 6) ? 6 : thold; + + iowrite32((thigh - 2) & XI3C_SCL_HIGH_TIME_MASK, + master->membase + XI3C_SCL_HIGH_TIME_OFFSET); + iowrite32((tlow - 2) & XI3C_SCL_LOW_TIME_MASK, + master->membase + XI3C_SCL_LOW_TIME_OFFSET); + iowrite32((thold - 2) & XI3C_SDA_HOLD_TIME_MASK, + master->membase + XI3C_SDA_HOLD_TIME_OFFSET); + + if (!mode) { + /* I2C */ + iowrite32((thigh - 2) & XI3C_SCL_HIGH_TIME_MASK, + master->membase + XI3C_OD_SCL_HIGH_TIME_OFFSET); + iowrite32((tlow - 2) & XI3C_SCL_LOW_TIME_MASK, + master->membase + XI3C_OD_SCL_LOW_TIME_OFFSET); + + tcasmin =3D DIV_ROUND_UP(XI3C_I2C_TCASMIN_NS, core_periodns); + } else { + /* I3C */ + odtlow =3D DIV_ROUND_UP(XI3C_OD_TLOW_NS, core_periodns); + odthigh =3D DIV_ROUND_UP(XI3C_OD_THIGH_NS, core_periodns); + + odtlow =3D (tlow < odtlow) ? odtlow : tlow; + odthigh =3D (thigh > odthigh) ? odthigh : thigh; + + iowrite32((odthigh - 2) & XI3C_SCL_HIGH_TIME_MASK, + master->membase + XI3C_OD_SCL_HIGH_TIME_OFFSET); + iowrite32((odtlow - 2) & XI3C_SCL_LOW_TIME_MASK, + master->membase + XI3C_OD_SCL_LOW_TIME_OFFSET); + + tcasmin =3D DIV_ROUND_UP(XI3C_TCASMIN_NS, core_periodns); + } + + thdstart =3D (thigh > tcasmin) ? thigh : tcasmin; + tsustart =3D (tlow > tcasmin) ? tlow : tcasmin; + tsustop =3D (tlow > tcasmin) ? tlow : tcasmin; + + iowrite32((tsustart - 2) & XI3C_TSU_START_MASK, + master->membase + XI3C_TSU_START_OFFSET); + iowrite32((thdstart - 2) & XI3C_THD_START_MASK, + master->membase + XI3C_THD_START_OFFSET); + iowrite32((tsustop - 2) & XI3C_TSU_STOP_MASK, + master->membase + XI3C_TSU_STOP_OFFSET); + + return 0; +} + +static int xi3c_master_bus_init(struct i3c_master_controller *m) +{ + struct xi3c_master *master =3D to_xi3c_master(m); + struct i3c_bus *bus =3D i3c_master_get_bus(m); + struct i3c_device_info info =3D { }; + unsigned long sclhz; + u64 pid1_bcr_dcr; + u8 mode; + int ret; + + switch (bus->mode) { + case I3C_BUS_MODE_MIXED_FAST: + case I3C_BUS_MODE_MIXED_LIMITED: + mode =3D XI3C_I2C_MODE; + sclhz =3D bus->scl_rate.i2c; + break; + case I3C_BUS_MODE_PURE: + mode =3D XI3C_SDR_MODE; + sclhz =3D bus->scl_rate.i3c; + break; + default: + return -EINVAL; + } + + ret =3D xi3c_clk_cfg(master, sclhz, mode); + if (ret) + return ret; + + /* Get an address for the master. */ + ret =3D i3c_master_get_free_addr(m, 0); + if (ret < 0) + return ret; + + info.dyn_addr =3D (u8)ret; + + /* Write the dynamic address value to the address register. */ + iowrite32(info.dyn_addr, master->membase + XI3C_ADDRESS_OFFSET); + + /* Read PID, BCR and DCR values, and assign to i3c device info. */ + pid1_bcr_dcr =3D ioread32(master->membase + XI3C_PID1_BCR_DCR); + info.pid =3D (((u64)(FIELD_GET(XI3C_PID1_MASK, pid1_bcr_dcr)) << 32) | + ioread32(master->membase + XI3C_PID0_OFFSET)); + info.bcr =3D (u8)FIELD_GET(XI3C_BCR_MASK, pid1_bcr_dcr); + info.dcr =3D (u8)FIELD_GET(XI3C_DCR_MASK, pid1_bcr_dcr); + + ret =3D i3c_master_set_info(&master->base, &info); + if (ret) + return ret; + + xi3c_master_init(master); + + return ret; +} + +static void xi3c_master_bus_cleanup(struct i3c_master_controller *m) +{ + struct xi3c_master *master =3D to_xi3c_master(m); + + xi3c_master_disable(master); +} + +static const struct i3c_master_controller_ops xi3c_master_ops =3D { + .bus_init =3D xi3c_master_bus_init, + .bus_cleanup =3D xi3c_master_bus_cleanup, + .do_daa =3D xi3c_master_do_daa, + .supports_ccc_cmd =3D xi3c_master_supports_ccc_cmd, + .send_ccc_cmd =3D xi3c_master_send_ccc_cmd, + .priv_xfers =3D xi3c_master_priv_xfers, + .i2c_xfers =3D xi3c_master_i2c_xfers, +}; + +static int xi3c_master_probe(struct platform_device *pdev) +{ + struct xi3c_master *master; + int ret; + + master =3D devm_kzalloc(&pdev->dev, sizeof(*master), GFP_KERNEL); + if (!master) + return -ENOMEM; + + master->membase =3D devm_platform_ioremap_resource(pdev, 0); + if (IS_ERR(master->membase)) + return PTR_ERR(master->membase); + + master->pclk =3D devm_clk_get_enabled(&pdev->dev, NULL); + if (IS_ERR(master->pclk)) + return dev_err_probe(&pdev->dev, PTR_ERR(master->pclk), + "Failed to get and enable clock\n"); + + master->dev =3D &pdev->dev; + + ret =3D devm_mutex_init(master->dev, &master->lock); + if (ret) + return ret; + + spin_lock_init(&master->xferqueue.lock); + INIT_LIST_HEAD(&master->xferqueue.list); + + platform_set_drvdata(pdev, master); + + return i3c_master_register(&master->base, &pdev->dev, + &xi3c_master_ops, false); +} + +static void xi3c_master_remove(struct platform_device *pdev) +{ + struct xi3c_master *master =3D platform_get_drvdata(pdev); + + i3c_master_unregister(&master->base); +} + +static const struct of_device_id xi3c_master_of_ids[] =3D { + { .compatible =3D "xlnx,axi-i3c-1.0" }, + { }, +}; + +static struct platform_driver xi3c_master_driver =3D { + .probe =3D xi3c_master_probe, + .remove =3D xi3c_master_remove, + .driver =3D { + .name =3D "axi-i3c-master", + .of_match_table =3D xi3c_master_of_ids, + }, +}; +module_platform_driver(xi3c_master_driver); + +MODULE_AUTHOR("Manikanta Guntupalli "); +MODULE_DESCRIPTION("AXI I3C master driver"); +MODULE_LICENSE("GPL"); --=20 2.34.1