From patchwork Wed Jul 26 11:24:57 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Bough Chen X-Patchwork-Id: 13327857 Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net [23.128.96.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 667FDCA66 for ; Wed, 26 Jul 2023 11:21:44 +0000 (UTC) Received: from EUR02-VI1-obe.outbound.protection.outlook.com (mail-vi1eur02on2080.outbound.protection.outlook.com [40.107.241.80]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9D39110D4; Wed, 26 Jul 2023 04:21:34 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Q6YVy6M1VItomtlEfxzaHu9r62t8Qy3pXJpaGeWdy+n323GK8cDfoyf6yZXBNWzOlOJWzH9Mx8HUEp3YFqq6Xo+FdQoNAYQOzdPwkoj4BjKfodpFabC6ddVkaRFlT+nl4vMUOEdSKyKYvKfDTW9xR/a6swsQoKHfAipSH8nBHhIp34x0hlty3KmagrtqmxykRB1/gAh/i2lS959lcm7YkSDKVDmg3mkO1zziWhGZsv8YqRxDW9FrvzAyXtan5JjyuFHrjNEnzXBNvt6Lb+WNIzqtJtnDq3gjEB4L7rXiUKYbXg1uOiTl6TgHtj4dhoF6A+i2AoDfBrK8NC5ed2jHNA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; 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=9IOQxIa8FKC0yUlr0xsHuBIdtG+zywvS5eDQEPXrJo4=; b=g8E5en/jMQtCXwkUVmxTNqmYJwIC6xacGzKJe4LR3Ny492p6r/zdMak/z8x5LOHthrtw3oIgPXcCYwrS5YZPXxolXSZw3le35rp0gAXIgls+oH62iCqoJ4mMhEQjAqR3+owELgusGsDoJhKl5kDQeQKIESv9GVymgQ3cvdg0K04NVQNMUm+ciEkrYL6KFjlt5Z4+uFsREwcQyfu2YB7iZ7/keYYOehTWZSo/ugpg67fG+EqGxGSu1tOaYR1QLCFUPQ/O94KZMc1I/t/VnDV9UJ8yyFu5o/EZmlP2DbdRmpmNFbPGSts42m8sR/wKI8inBt3OcRJTJO1x3sCJZZjyWA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=9IOQxIa8FKC0yUlr0xsHuBIdtG+zywvS5eDQEPXrJo4=; b=Y5zQgd5oZVxgZ6cUkfOaPAfM83KJkdQKm1o3fSH2w59GHXIwBIQhteTEVuKw0LH6CIwbFU6gqgI91mm/qO1kkNDM4dkpZKH00iPK7YMy1ktHBXjHLlCrKH7AAlunI0wNLZKzXIHB7UnYXhAn4woOQdIHezTz/rPSrgkRLdVn2R0= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from DB7PR04MB4010.eurprd04.prod.outlook.com (2603:10a6:5:21::30) by DB8PR04MB6892.eurprd04.prod.outlook.com (2603:10a6:10:113::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6631.29; Wed, 26 Jul 2023 11:21:31 +0000 Received: from DB7PR04MB4010.eurprd04.prod.outlook.com ([fe80::d73c:e747:3911:dcc]) by DB7PR04MB4010.eurprd04.prod.outlook.com ([fe80::d73c:e747:3911:dcc%5]) with mapi id 15.20.6631.026; Wed, 26 Jul 2023 11:21:31 +0000 From: haibo.chen@nxp.com To: robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de, wg@grandegger.com, mkl@pengutronix.de Cc: kernel@pengutronix.de, linux-imx@nxp.com, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, haibo.chen@nxp.com, devicetree@vger.kernel.org, linux-can@vger.kernel.org, netdev@vger.kernel.org Subject: [PATCH v3 1/2] arm64: dts: imx93: add the Flex-CAN stop mode by GPR Date: Wed, 26 Jul 2023 19:24:57 +0800 Message-Id: <20230726112458.3524165-1-haibo.chen@nxp.com> X-Mailer: git-send-email 2.34.1 X-ClientProxiedBy: SG2P153CA0049.APCP153.PROD.OUTLOOK.COM (2603:1096:4:c6::18) To DB7PR04MB4010.eurprd04.prod.outlook.com (2603:10a6:5:21::30) Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DB7PR04MB4010:EE_|DB8PR04MB6892:EE_ X-MS-Office365-Filtering-Correlation-Id: db47d5ad-573b-4b15-5c9c-08db8dca767c X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: nyO6F1nn1XhuY1ex9HGqpiV3XtayohqgQpwhf3LwhpWOeuRwkFqm6ODv/twM70igBUxJDQLVeuJx+pBurtBzuQgFyW5p313Yh3Yz5iVeLfkNTu+t52z3TVnjHclcCc0sSTxjLzd/M0MFMaarc2HSa9fj91EzyyfUAgIBN4vZLH4UD+rQ0wciZauu43geeyYrEkZcXjfVsZv/h9J5SIxjLDTPMhggBtEVz5lrXFmbz4hL4ahtz+wMy04scOMUErC50qTyVBB6FZoQr2x9I5AMqSUi9QP8keHpXZg2a8q9ehtk1mfXzY3wg5ln8NA0sk0sCO/v5atqggk1hsSteA9QGvLUB6TJNer1xqirobn1qPX5mV4jdrti0uQezR0X37AhalF1eCpgFlw6JMpfN7oUP1pEf3z/u9hcRmEurer7f65zlTu8RJdsOuUp/IxZwSZOD2NpxnA6m24Pduu3mtDq+XfQDMThmM1rEzW1xa6uoWJE2iQw+MT5GbaZkPq17pE/J5ATNRVlEjeQ/KSy3TIy6BwamHkB1gAzNUEaGl+M0Nhb54HDYk4m0FDoiMYKhg0vXPq4mN18Qkg0qCgpAvwnZ6WLhxQ7t8rmViotCJgOuHKyxdukWsqyvGZKke79y1Lw X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DB7PR04MB4010.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(4636009)(39860400002)(396003)(346002)(376002)(366004)(136003)(451199021)(2906002)(52116002)(6486002)(6666004)(2616005)(38350700002)(38100700002)(36756003)(186003)(7416002)(5660300002)(86362001)(83380400001)(41300700001)(8676002)(316002)(6506007)(26005)(8936002)(1076003)(66556008)(478600001)(6512007)(4326008)(66946007)(9686003)(66476007);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?8p0GrCTY8MEgm3vvSnFBXmHBz0dq?= =?utf-8?q?HzH5N28Ioz8VvCzvi1gbLijHAGk3kJqQWwSF7wbLAPTFa1P2pntSoq5hPWc0uRQx8?= =?utf-8?q?23QLTaUTzaxVK3etGHsoQwJB6IDqEH/KeFuOiuSWz9U9uhf6RW1ix6vTBOltBuLGE?= =?utf-8?q?pgRn8d07np4GAR/dULdFw93DdNJHF7LmO2ErVJAX8wvJl41toeXPViNEx2/04QmSH?= =?utf-8?q?JKIA5/ZiscvXLoUzVKpgeWck1deZ5oIpQYRH7vo82Q9ROZ1osMtHBdgSiEmJ7uux6?= =?utf-8?q?oFmLgZzwgdwQzOF1lHaGHjgkSPmXk8zdm7ppGcC3M9b+enuBfg3qajcNTHWASkrW8?= =?utf-8?q?1coiDTA77hSieZV0dxTdP5MHhsNDEiPW1yftqQpLxmMEFcQjhElZtsQLPRHdWz4Gq?= =?utf-8?q?U0p+ligr6hngcVLyiQDfda4gE5mstf+TMUTm/uwq2g45AQHgt7UeUBL8gdOWq7vDF?= =?utf-8?q?lcZCZjpGzE77nk/iXGl496tU5eMq+FMOxkyTO4QJQmiVA8XO28L/tIGqrNI1lx3+F?= =?utf-8?q?IGrG4gkb7NqMJ3jrGzRRQpnlcebZ0aHxNh9MdFEiMxfGvH5EwpJdxxNGcLDP08Qge?= =?utf-8?q?L6J2KhpNaaROlw9fRZwhnUW+i119QeD0sAsl4IyegXA7xL/7B8Etk2ZbavxBHZ3nl?= =?utf-8?q?kRdf20mYl+qHEzFs3+RydZng8bXApMu0WkZ1P9n5OAcDZr7TppIrxYpgxbJsFNkl8?= =?utf-8?q?LGpms4NtSxzZi3oxkQtXrhJUCTVXZhuQCgeCVjRAU13xw/O5xn1jYaMkdBYyL8pDK?= =?utf-8?q?FUOifsU8JeMtSjIqmCYC9XCiqbF5hwnllVzCn0KaC+Ed9KZzLvzj6bXs992vSXQoi?= =?utf-8?q?TG/2PJvt6lU2tNA3JYklvmfEZmwkQ6yMokRZFXooV7vjWwQoJLW0FQDjCjHVFK9M+?= =?utf-8?q?3WnKQCtUmcsT+l5ltq0odzcDPVJJcP4pSPSxQsEPiX6RZIF/sxKzuZr/j41lXr/h/?= =?utf-8?q?rICNIcBMBncXqhuJteHZ31ZG71OAKjEb0FApAWtQVTszw9PGe+gvHQWa4ejykwHdq?= =?utf-8?q?SfqkR7HuAdjDEFtjTsn4qljpNBx+XzGCpMbDnqRGoRmyEXihaeI9bg7NZCD0hrQ5j?= =?utf-8?q?i0DYamS/uJAJzJQerpVuAmCf4i3SI1N9lKX5A4gUmj7OhDj5j7I6lPLokPhMT62oY?= =?utf-8?q?DNWuvrR79e5pISs6NY+Vpmqk6VVniROtmprly1AYRZrGzyqwevHnQ3Os6VPdrdx3g?= =?utf-8?q?nwEd+jae9GX6iKjSAW8RpRD/wEeTMjjbFXudit0xuTwi/lrdadFMKDc5Rt5F11YSb?= =?utf-8?q?ikvC6aoPRbQHBMQB6QmDunckKOB8mxvIP8EP/8zAfGZYyZOxhYuDOMfz+1Jaqf9Gm?= =?utf-8?q?enRO7/FiEAdDleyQWMe4fpTwOn7uVJ2e+HghGU8vNmnEefIooliZTeLO36zNdVv4/?= =?utf-8?q?ABReLZ82/3A1LCKPGUHlCt9gpgU+t1Uam0Eih2nyB4b9XdNwS4ddhYCMoQtCGb+Ov?= =?utf-8?q?wWBi9qfjG9siczn2R4N0QdFs0P9kBy64fioSAFb2b+s5LHcOR7tJSTji5jCx1pAbb?= =?utf-8?q?PwYqY/XgbwAX?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: db47d5ad-573b-4b15-5c9c-08db8dca767c X-MS-Exchange-CrossTenant-AuthSource: DB7PR04MB4010.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Jul 2023 11:21:31.6428 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 7zp4guQRAJ6nTFdbEUhM7H6kTyNkM7Hp4wGse/erOq5sBB5yzs70C35pLtf4RyPeBqNLsoRNR5pL28x44G6m7w== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB8PR04MB6892 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net From: Haibo Chen imx93 A0 chip use the internal q-channel handshake signal in LPCG and CCM to automatically handle the Flex-CAN stop mode. But this method meet issue when do the system PM stress test. IC can't fix it easily. So in the new imx93 A1 chip, IC drop this method, and involve back the old way，use the GPR method to trigger the Flex-CAN stop mode signal. Now NXP claim to drop imx93 A0, and only support imx93 A1. So here add the stop mode through GPR. This patch also fix a typo for aonmix_ns_gpr. Signed-off-by: Haibo Chen --- arch/arm64/boot/dts/freescale/imx93.dtsi | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/arch/arm64/boot/dts/freescale/imx93.dtsi b/arch/arm64/boot/dts/freescale/imx93.dtsi index 4ec9df78f205..e0282c4ba11d 100644 --- a/arch/arm64/boot/dts/freescale/imx93.dtsi +++ b/arch/arm64/boot/dts/freescale/imx93.dtsi @@ -185,7 +185,7 @@ aips1: bus@44000000 { #size-cells = <1>; ranges; - anomix_ns_gpr: syscon@44210000 { + aonmix_ns_gpr: syscon@44210000 { compatible = "fsl,imx93-aonmix-ns-syscfg", "syscon"; reg = <0x44210000 0x1000>; }; @@ -319,6 +319,7 @@ flexcan1: can@443a0000 { assigned-clock-parents = <&clk IMX93_CLK_SYS_PLL_PFD1_DIV2>; assigned-clock-rates = <40000000>; fsl,clk-source = /bits/ 8 <0>; + fsl,stop-mode = <&aonmix_ns_gpr 0x14 0>; status = "disabled"; }; @@ -591,6 +592,7 @@ flexcan2: can@425b0000 { assigned-clock-parents = <&clk IMX93_CLK_SYS_PLL_PFD1_DIV2>; assigned-clock-rates = <40000000>; fsl,clk-source = /bits/ 8 <0>; + fsl,stop-mode = <&wakeupmix_gpr 0x0c 2>; status = "disabled"; }; From patchwork Wed Jul 26 11:24:58 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bough Chen X-Patchwork-Id: 13327858 X-Patchwork-Delegate: kuba@kernel.org Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net [23.128.96.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 A58159475 for ; Wed, 26 Jul 2023 11:21:46 +0000 (UTC) Received: from EUR02-VI1-obe.outbound.protection.outlook.com (mail-vi1eur02on2080.outbound.protection.outlook.com [40.107.241.80]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0F8D1FD; Wed, 26 Jul 2023 04:21:44 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=LN8ylth24eTMiLMqm+A+aq3KwzVzL1mWpXIL5lhk1zXyeXP6w+16R/dzfHIb13gaZPR55Fm84x81BBTdttOXDvbw/hR+tSwfaJlbRAlqBxZvSNvAJYSnp409HNT29H6gbvl/SFGBcNDX7vXOBzVU3nImT0DF0lnuKiga4zJtj7x9yQazuMgaHXPQkaskmr3agyxwFjrfwZHorwWtAidvbyA9Wkno1Hol8oTtpwD8YCgb8CBEhyebfQpXBB07bz9bVID9IEWNXVi5jlKhee703p3cDWXNtY1qFezY1HIyc6sNYBA5Llb+iCT32/zbBaP1QnF3ti5KpoGAU1xZ3rXkNQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; 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=ElRugvWN47yKwMJ5g5V6v/TdVSudwimQn3gWZdJLl4Q=; b=mm/R9/Y+0wD7yzQIWhberVBXxcUZU4HsbsaKBAWvf/7Nx3o3+wfe9hq2xBzL0cWB8iIoFg7wAuLecacg/kqxIjlzGYrOBvee4bdbE2t8sBvYVzH1lHGIU5H7zaH4yu+Q5amJwJmuclQquBhHM/gmc7mkouTLnAJYOQm+K5H9WiBvf4+mtxzwfZfsze/Qr8b8I/OXikIx76zwhCEekL/4uuXFM28Fc+kth3pWcjCcKzfRsSpxgo7wwE3s0v/FRZulWNHGlmQt0WSU/lEx+huwCKvOIEpFdYWtoLOeSviyMxAZFbNG6+4X8Jtu/3CDFp2439fkzQj4vy7lqleEtgdHbA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=ElRugvWN47yKwMJ5g5V6v/TdVSudwimQn3gWZdJLl4Q=; b=LFEHxdHjDNJmG6iyc4BLQfbQ7dqMS8TLsf5gHBOs6/STbvFAtB6p1zxJ3dPw+alA4DRe0KeDA3lLkXrKOipkcpyNgMhItwBF7DGo+pB9++swvXKWOUNT7FgKbxh8kVXb9A3RDkLTo0Z+sAtLFyrfcpU6yt2zAX9cqARU9aA5BlU= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from DB7PR04MB4010.eurprd04.prod.outlook.com (2603:10a6:5:21::30) by DB8PR04MB6892.eurprd04.prod.outlook.com (2603:10a6:10:113::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6631.29; Wed, 26 Jul 2023 11:21:36 +0000 Received: from DB7PR04MB4010.eurprd04.prod.outlook.com ([fe80::d73c:e747:3911:dcc]) by DB7PR04MB4010.eurprd04.prod.outlook.com ([fe80::d73c:e747:3911:dcc%5]) with mapi id 15.20.6631.026; Wed, 26 Jul 2023 11:21:36 +0000 From: haibo.chen@nxp.com To: robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de, wg@grandegger.com, mkl@pengutronix.de Cc: kernel@pengutronix.de, linux-imx@nxp.com, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, haibo.chen@nxp.com, devicetree@vger.kernel.org, linux-can@vger.kernel.org, netdev@vger.kernel.org Subject: [PATCH v3 2/2] can: flexcan: remove the auto stop mode for IMX93 Date: Wed, 26 Jul 2023 19:24:58 +0800 Message-Id: <20230726112458.3524165-2-haibo.chen@nxp.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20230726112458.3524165-1-haibo.chen@nxp.com> References: <20230726112458.3524165-1-haibo.chen@nxp.com> X-ClientProxiedBy: SG2P153CA0049.APCP153.PROD.OUTLOOK.COM (2603:1096:4:c6::18) To DB7PR04MB4010.eurprd04.prod.outlook.com (2603:10a6:5:21::30) Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DB7PR04MB4010:EE_|DB8PR04MB6892:EE_ X-MS-Office365-Filtering-Correlation-Id: f02a20f3-a5f0-4210-a13b-08db8dca7979 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: eKnf84boF3F2D9x2ioNJqcvT43lbCLZlRn74TVKEcFS2yuYZY4QJhbfUdhr5A+rkajg00X8xQRSB8qPrYt0BxpAuDstQEpq/CrRiD7xA9npTJHeb+vWHsKmRIpvjzlEzKJEt3ImCaRs2zzekL//4GXWylw6m1ptnwVjvcAdGCD0cgXoxaHqyqGKjEDmYKHgMOHZuxypUv73GJ3wf8CyvAwzMvkfDl37b76AF9PBvjI4y8Dzc/MYI0KxDjocV5FALUY2Np4SHrvBQ38lO3m0F8e4v8mFtgHFtMDpWJR+TD2S0iOtuNYkhJrcZqXslpWDdz2dfjhqsTSqE1p9/oB2+gTU/RtqtTcCR/enNmVYa3AfaxLuU1RUHXnPkaj9Ehn0VxUCa0BeTwqiAKlppR3zxn1dEhxSoR1QUH4grpu0Ym3zKeNAwsDGnkekyuQ0t5tkX2qK77ZdpMUZJazMdQn2PDHGNh8BuW/rONkjAvTwxNR+uMewXZSSEITk8KMM6Og3V5F/xRuEQndOoKZ9wGTJbn845FhV0fpRUckmGJKA+7GvE7bUk401DrYH7f4iqAHhmQHoJmcr/k+sKdtaABZm9lEy8K5/5o3Ov37FhcJAVhn61O2sFfCKC4TbhdEdZzFHh X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DB7PR04MB4010.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(4636009)(39860400002)(396003)(346002)(376002)(366004)(136003)(451199021)(2906002)(52116002)(6486002)(6666004)(2616005)(38350700002)(38100700002)(36756003)(186003)(7416002)(5660300002)(86362001)(83380400001)(41300700001)(8676002)(316002)(6506007)(26005)(8936002)(1076003)(66556008)(478600001)(6512007)(4326008)(66946007)(9686003)(66476007);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: 5PvQl5XkUj8yu3b/BTpzMkUCGr6ka3u7h36FJ/Ok6rDc0mdAQeTMmmeLPdSqcyZRNsbV6iRcAPNO/z3Y2gWJTP/FfxstEJ54J79vbY8wKl/bzLGntmIdFBX/errNsQon7KNAjpQUntndEgxqNQJY7Qa6YtP1J6eN023Fg5gyboiiseJLn/m4YdktutY5+jP5tjLnxsBbeakj8cTsiXD8dp1pqD2kKMs8WOpiXBz01jzdIIvrjG+1MzYQX67GjTDL2NMv7kpT75lp6XFWgfDoMOvNUOE7Htk9nNsIVhzya34e21FxHckNPZj5NSzI2aCRjEBTAwpjl65L1xwssdnQ9q1v6KDRsbuo3M4sD3mVXQofCdkezyNpyECqiECECXuNUKwkVMQHMluFNoc3E+p5/XVmC48rm1xwUsJeobeF+fLjvP/2OO1XBl8qyWrhjI4syD048Y7Z8e7PSswzSfwPy5SZEr/8PtZa9bF39GNXiBYai/VMwp/Oi7jmz+rpJtHrcUv53mUAx4lDvnRQsvpPOl8QQeTstI9bQZ1HD/QzDOcnxKyoph1kWToxODoz/BYiAk/UW3hc3Te1XIbo3Bj64xa0MwCerCjuwOM+8Y/Z7wDmIXMSlnQcoeys6uhuxbhKRos3QYnO+0YdyDCjsXCAkO8isa9Ow30FJlbaKCC4I1xXXceHvNI3uEJlsLl1r0AaC32OsKFoJz0SfDh7Avsp2laqIUu7sl4t071JmucrkFhitQUQNUF7muHIwT1wO3NhXGroTZg3Z+wg5R9SSB3JdF7f7wmo+0+Ayj52VjJ3F+kyIRgdpOpDlGNTEr5Ukran2hjSnonsj6QWckL/bG7li2OCBmPJpWPL8hWpc6ZDQZ0YWxKqa4KWIVAgms8z0WbAdMPw0smTeBhJIWmvnhLwLboEOcfwwniBWjHixD602RH7PpWSKHXfalHVT+XZrWcGeOdnfiXSYyA63pM0lrfxnFZ6177HdlzsbxC3rnhzCn3LVW3x4pZRdP31DWatvXsbj6/2UrpI1MA8si1C9MezXaEK2ls10HoIWfSOdIaJJ+p6KlABVhbN6kFx8Kz8N+o0oP3Mb5k0PNhogycuUCg1W7LJtYIwTLtArZS7vRQmiQhWVUjhBGDYIwb9erVQLqU3tCs4v/gmYZ17lezXDWDCbm7eA/vtkuAlBmUDFJX3RM0SRXWuEYhu9zVPIK7XqxqPH8Lp/N6OC2BSnytAiYP4U6PHi6tgQQPVEc7c2QidniLutwYNEk2H4bD73mgNHApVacytXa4sW7Wc0i5yu61tkq8D0eJefIHtELHq7pt6zU6gxWaS6HXh6lyCEoin9T4sAdp64m+aiC3JYjdwGNrcH5HB0Nwy/hoalan0aqHJcnE60HT4kihuNL3aWd/MEbvhdDG3W6qHjL1ah1t64ekFVh791DmKI9u9lWZ5EBip4FbyljDOgrgARoe/X5Cc1C/6wKYv67Oi1J0dovR7Bh0mgC/Yl5w90v62XfQmtAoJqnH7w/fRYvKxlrbeleioLgwHXhgqldvbJmATfgu3l9y4GKKgTntB4onefY90Ala9ikfPy9lfMpY1y+RjqbcrAcXG X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: f02a20f3-a5f0-4210-a13b-08db8dca7979 X-MS-Exchange-CrossTenant-AuthSource: DB7PR04MB4010.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Jul 2023 11:21:36.6684 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: Y3xxxGIq7NkbnmTRiXWlKbJ42OHw0eSRyAG1Pqo7nnRkylRLGVXgNveXp0/Z4yk49jSNbw0hiGMGkbz1XTKk4A== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB8PR04MB6892 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net X-Patchwork-Delegate: kuba@kernel.org From: Haibo Chen IMX93 A0 chip involve the internal q-channel handshake in LPCG and CCM to automatically handle the Flex-CAN IPG STOP signal. Only after FLEX-CAN enter stop mode then can support the self-wakeup feature. But meet issue when do the continue system PM stress test. When config the CAN as wakeup source, the first time after system suspend, any data on CAN bus can wakeup the system, this is as expect. But the second time when system suspend, data on CAN bus can't wakeup the system. If continue this test, we find in odd time system enter suspend, CAN can wakeup the system, but in even number system enter suspend, CAN can't wakeup the system. IC find a bug in the auto stop mode logic, and can't fix it easily. So for the new imx93 A1, IC drop the auto stop mode and involve the GPR to support stop mode (used before). IC define a bit in GPR which can trigger the IPG STOP signal to Flex-CAN, let it go into stop mode. And NXP claim to drop IMX93 A0, and only support IMX93 A1. So this patch remove the auto stop mode, and add flag FLEXCAN_QUIRK_SETUP_STOP_MODE_GPR to imx93. Signed-off-by: Haibo Chen --- drivers/net/can/flexcan/flexcan-core.c | 46 ++++++++------------------ drivers/net/can/flexcan/flexcan.h | 2 -- 2 files changed, 13 insertions(+), 35 deletions(-) diff --git a/drivers/net/can/flexcan/flexcan-core.c b/drivers/net/can/flexcan/flexcan-core.c index ff0fc18baf13..d8be69f4a0c3 100644 --- a/drivers/net/can/flexcan/flexcan-core.c +++ b/drivers/net/can/flexcan/flexcan-core.c @@ -348,7 +348,7 @@ static struct flexcan_devtype_data fsl_imx8mp_devtype_data = { static struct flexcan_devtype_data fsl_imx93_devtype_data = { .quirks = FLEXCAN_QUIRK_DISABLE_RXFG | FLEXCAN_QUIRK_ENABLE_EACEN_RRS | FLEXCAN_QUIRK_DISABLE_MECR | FLEXCAN_QUIRK_USE_RX_MAILBOX | - FLEXCAN_QUIRK_BROKEN_PERR_STATE | FLEXCAN_QUIRK_AUTO_STOP_MODE | + FLEXCAN_QUIRK_BROKEN_PERR_STATE | FLEXCAN_QUIRK_SETUP_STOP_MODE_GPR | FLEXCAN_QUIRK_SUPPORT_FD | FLEXCAN_QUIRK_SUPPORT_ECC | FLEXCAN_QUIRK_SUPPORT_RX_MAILBOX | FLEXCAN_QUIRK_SUPPORT_RX_MAILBOX_RTR, @@ -544,11 +544,6 @@ static inline int flexcan_enter_stop_mode(struct flexcan_priv *priv) } else if (priv->devtype_data.quirks & FLEXCAN_QUIRK_SETUP_STOP_MODE_GPR) { regmap_update_bits(priv->stm.gpr, priv->stm.req_gpr, 1 << priv->stm.req_bit, 1 << priv->stm.req_bit); - } else if (priv->devtype_data.quirks & FLEXCAN_QUIRK_AUTO_STOP_MODE) { - /* For the auto stop mode, software do nothing, hardware will cover - * all the operation automatically after system go into low power mode. - */ - return 0; } return flexcan_low_power_enter_ack(priv); @@ -574,12 +569,6 @@ static inline int flexcan_exit_stop_mode(struct flexcan_priv *priv) reg_mcr &= ~FLEXCAN_MCR_SLF_WAK; priv->write(reg_mcr, ®s->mcr); - /* For the auto stop mode, hardware will exist stop mode - * automatically after system go out of low power mode. - */ - if (priv->devtype_data.quirks & FLEXCAN_QUIRK_AUTO_STOP_MODE) - return 0; - return flexcan_low_power_exit_ack(priv); } @@ -1994,13 +1983,18 @@ static int flexcan_setup_stop_mode(struct platform_device *pdev) ret = flexcan_setup_stop_mode_scfw(pdev); else if (priv->devtype_data.quirks & FLEXCAN_QUIRK_SETUP_STOP_MODE_GPR) ret = flexcan_setup_stop_mode_gpr(pdev); - else if (priv->devtype_data.quirks & FLEXCAN_QUIRK_AUTO_STOP_MODE) - ret = 0; else /* return 0 directly if doesn't support stop mode feature */ return 0; - if (ret) + /* If ret is -EINVAL, this means SoC claim to support stop mode, but + * dts file lack the stop mode property definition. For this case, + * directly return 0, this will skip the wakeup capable setting and + * will not block the driver probe. + */ + if (ret == -EINVAL) + return 0; + else if (ret) return ret; device_set_wakeup_capable(&pdev->dev, true); @@ -2320,16 +2314,8 @@ static int __maybe_unused flexcan_noirq_suspend(struct device *device) if (netif_running(dev)) { int err; - if (device_may_wakeup(device)) { + if (device_may_wakeup(device)) flexcan_enable_wakeup_irq(priv, true); - /* For auto stop mode, need to keep the clock on before - * system go into low power mode. After system go into - * low power mode, hardware will config the flexcan into - * stop mode, and gate off the clock automatically. - */ - if (priv->devtype_data.quirks & FLEXCAN_QUIRK_AUTO_STOP_MODE) - return 0; - } err = pm_runtime_force_suspend(device); if (err) @@ -2347,15 +2333,9 @@ static int __maybe_unused flexcan_noirq_resume(struct device *device) if (netif_running(dev)) { int err; - /* For the wakeup in auto stop mode, no need to gate on the - * clock here, hardware will do this automatically. - */ - if (!(device_may_wakeup(device) && - priv->devtype_data.quirks & FLEXCAN_QUIRK_AUTO_STOP_MODE)) { - err = pm_runtime_force_resume(device); - if (err) - return err; - } + err = pm_runtime_force_resume(device); + if (err) + return err; if (device_may_wakeup(device)) flexcan_enable_wakeup_irq(priv, false); diff --git a/drivers/net/can/flexcan/flexcan.h b/drivers/net/can/flexcan/flexcan.h index 91402977780b..025c3417031f 100644 --- a/drivers/net/can/flexcan/flexcan.h +++ b/drivers/net/can/flexcan/flexcan.h @@ -68,8 +68,6 @@ #define FLEXCAN_QUIRK_SUPPORT_RX_MAILBOX_RTR BIT(15) /* Device supports RX via FIFO */ #define FLEXCAN_QUIRK_SUPPORT_RX_FIFO BIT(16) -/* auto enter stop mode to support wakeup */ -#define FLEXCAN_QUIRK_AUTO_STOP_MODE BIT(17) struct flexcan_devtype_data { u32 quirks; /* quirks needed for different IP cores */