Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   wei.fang@nxp.com
To:     robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
        shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de,
        festevam@gmail.com, linux-imx@nxp.com, Frank.Li@nxp.com,
        shenwei.wang@nxp.com, xiaoning.wang@nxp.com
Cc:     devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-kernel@vger.kernel.org
Subject: [PATCH v2] arm64: dts: imx8dxl: Disable hibernation mode of AR8031 for EQOS
Date:   Fri, 16 Dec 2022 09:14:34 +0800
Message-Id: <20221216011434.967813-1-wei.fang@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?+cnUSIOpYTja33tk96c24Wt2IHdF+J1ANiHP5+vmrzwv+v4DfJI/EFpdGC2q?=
 =?us-ascii?Q?itkAwwDfdufwFuU/FLU6q6+dzbSYTKFzTeRp7c02fOWry7j23fRjo0RIcTJ0?=
 =?us-ascii?Q?30/7PBC/+wza+ZJsv+To3+gIMgkyyl1q3cj5YQqpMPGUgkIJHO661psr+C4K?=
 =?us-ascii?Q?kXs9Lw39r3mwJF2/DlkGi3lHAH8G8ED8LkANlyXiJ+iZhkNzK00UAN3II0r0?=
 =?us-ascii?Q?tQJEaiVnCIsRcX0pR24tPHfURauAEbQI3ZbI5O/eQ37PR1jvB22x8qYc0+bd?=
 =?us-ascii?Q?fsARpFoQdQuJkK9M/ANVjxME0NC9Cj0B852EnePLiOcQOW/8SJyg8hfY7jZN?=
 =?us-ascii?Q?ehbwN2qQVz8ErKFE92gECrS+xvs4ZwOokz8l89YhjjzRLNsGdj+eLjMQW77T?=
 =?us-ascii?Q?Znu+MunnZfkxHLe/D+guu2m8o6lu9z0lsUOkj0mXxZjEQbnUnozrxRMNJZm/?=
 =?us-ascii?Q?hJ57oEHORhRbNo7yXo7uYV2Axti97UTIoyJkR6GZmHmi61yDMutF3FbIXVLW?=
 =?us-ascii?Q?s+BLtPue1Bj2xqa1RDI/ltE/EQbkOnTxjjO7WvFcLH/rnszQ7luG6ts/2fu8?=
 =?us-ascii?Q?cyHZNxbodiIevEvszcLDVYcXwnWvorzwDLPAkdcX6H8JTMmKHLXcczZHq94N?=
 =?us-ascii?Q?La86/y67EpjGVd4wLfxF/8sTcOwpR31UJhHYtYQhjXWmZ6A7S943VVsIDR3D?=
 =?us-ascii?Q?yn0ZX3G/ZHWSAI3PIS0F7TrON+M5tty/R46nn6Nxc0dASxENsf3N08hjoeGZ?=
 =?us-ascii?Q?1hvIyBhTiqSqcQPBceaWzz3khFupUMzULXb2OUPUDCDaDRB08Cao6LAiZVYh?=
 =?us-ascii?Q?Fy6BG5ds1BlrvFwf7HKxcPgkNgBPagZhAsV5RTpHhyeACKliVWJNC+qZuWAn?=
 =?us-ascii?Q?sRwijWu58Fu5XN9W4TPUVKhK9hFFQny+lZwY+9UZ4VEUMg8n/xDJRWcr5fCe?=
 =?us-ascii?Q?ocv1fIHmEyyeOcw5MDtiDqYpyUkb4sdw16cxhctS0HVNRd7ODSVm8gYGi2D3?=
 =?us-ascii?Q?uPBTafKWfeOqJFwSl2EhoQaufAPWyiljxqTyUMfXlJR9xGLnYp/q36YU1OYC?=
 =?us-ascii?Q?DdSpJ2mGkcnq/8n5Y+WjduDY+L2Dzsxwg7JUs/GdzpMWcXDCJ1gcJu0pg/0o?=
 =?us-ascii?Q?vBcXZc/SNy90vhGscyyij6muRClnkkusmi6F66e6zpKgXzjatA6QM6z4Z+Tu?=
 =?us-ascii?Q?TMnK0cRTbgy8Cmfmoe/WYJPCMQZ1qfzjxs1TI5RuoPSYywHoCvP1C4N8/taT?=
 =?us-ascii?Q?eSmRDkXB04zaFl4eKrBKWuK+AArgpypJGOf1xMr1Hz1rdxMk3x57ZHNiivry?=
 =?us-ascii?Q?HamA0YuanoGY80ebX3OuW/HteFVcwbCOOsQxlNTjfikemcclFuCsTdMUWOfP?=
 =?us-ascii?Q?DrkUO+9dsQWhgyk3nl4+9w2ntpRWyj6k9jAChCRwVq8Wp0vjlGfHPHaaoXej?=
 =?us-ascii?Q?gqVCbWoXZUvs6VgVOwBxkXSfwKCNoqxQYOQG6hSCKlsYd1VYuntMkKm9Bb8f?=
 =?us-ascii?Q?CSiXJPg53dBMzLxBZI7ix6Fb76QwIp31nWnDHKnyQtQ4errgMsUud3sb1nuo?=
 =?us-ascii?Q?zCiROJsVuIcht8OC5JDSv7Ybg8s7qNZtXoMRgedr?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 42f7bf23-1da6-451d-5a66-08dadf035ad8
X-MS-Exchange-CrossTenant-AuthSource: DB9PR04MB8106.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Dec 2022 01:18:26.0487
 (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: VBljxpuhhCT9rfxQS1MSdLEfzzQtMgwp3du+oavVAadpd7zZECn/esdkNNMZMkS6eOsUDP4x50CGGP1vyiCvLw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS1PR04MB9358
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 autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

From: Wei Fang <wei.fang@nxp.com>

The hibernation mode of AR8031 PHY defaults to be enabled after hardware
reset. When the cable is unplugged, the PHY will enter hibernation mode
after about 10 senconds and the PHY clocks will be stopped to save
power. However, due to the design of EQOS, the mac needs the RX_CLK of
PHY for software reset to complete. Otherwise the software reset of EQOS
will be failed and do not work correctly. The only way is to disable
hibernation mode of AR8031 PHY for EQOS, the "qca,disable-hibernation-mode"
property is used for this purpose and has already been submitted to the
upstream, for more details please refer to the below link:
https://lore.kernel.org/netdev/20220818030054.1010660-2-wei.fang@nxp.com/

This issue is easy to reproduce, just unplug the cable and "ifconfig eth0
down", after about 10 senconds, then "ifconfig eth0 up", you will see
failure log on the serial port. The log is shown as following:
root@imx8dxlevk:~#
[34.941970] imx-dwmac 5b050000.ethernet eth0: Link is Down
root@imx8dxlevk:~# ifconfig eth0 down
[35.437814] imx-dwmac 5b050000.ethernet eth0: FPE workqueue stop
[35.507913] imx-dwmac 5b050000.ethernet eth0: PHY [stmmac-1:00] driver [Qualcomm Atheros AR8031/AR8033] (irq=POLL)
[35.518613] imx-dwmac 5b050000.ethernet eth0: configuring for phy/rgmii-id link mode
root@imx8dxlevk:~# ifconfig eth0 up
[71.143044] imx-dwmac 5b050000.ethernet eth0: Register MEM_TYPE_PAGE_POOL RxQ-0
[71.215855] imx-dwmac 5b050000.ethernet eth0: PHY [stmmac-1:00] driver [Qualcomm Atheros AR8031/AR8033] (irq=POLL)
[72.230417] imx-dwmac 5b050000.ethernet: Failed to reset the dma
[72.236512] imx-dwmac 5b050000.ethernet eth0: stmmac_hw_setup: DMA engine initialization failed
[72.245258] imx-dwmac 5b050000.ethernet eth0: __stmmac_open: Hw setup failed
SIOCSIFFLAGS: Connection timed out

After applying this patch, the software reset of EQOS will be
successful. And the log is shown as below.
root@imx8dxlevk:~# ifconfig eth0 up
[96.114344] imx-dwmac 5b050000.ethernet eth0: Register MEM_TYPE_PAGE_POOL RxQ-0
[96.171466] imx-dwmac 5b050000.ethernet eth0: PHY [stmmac-1:00] driver [Qualcomm Atheros AR8031/AR8033] (irq=POLL)
[96.188883] imx-dwmac 5b050000.ethernet eth0: No Safety Features support found
[96.196221] imx-dwmac 5b050000.ethernet eth0: IEEE 1588-2008 Advanced Timestamp supported
[96.204846] imx-dwmac 5b050000.ethernet eth0: registered PTP clock
[96.225558] imx-dwmac 5b050000.ethernet eth0: FPE workqueue start
[96.236858] imx-dwmac 5b050000.ethernet eth0: configuring for phy/rgmii-id link mode
[96.249358] 8021q: adding VLAN 0 to HW filter on device eth0

Signed-off-by: Wei Fang <wei.fang@nxp.com>
Reviewed-by: Clark Wang <xiaoning.wang@nxp.com>
---
V2 change:
Remove "net" from the subject.
---
 arch/arm64/boot/dts/freescale/imx8dxl-evk.dts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8dxl-evk.dts b/arch/arm64/boot/dts/freescale/imx8dxl-evk.dts
index 280a9c9d8bd9..1bcf228a22b8 100644
--- a/arch/arm64/boot/dts/freescale/imx8dxl-evk.dts
+++ b/arch/arm64/boot/dts/freescale/imx8dxl-evk.dts
@@ -135,6 +135,7 @@ ethphy0: ethernet-phy@0 {
 			reg = <0>;
 			eee-broken-1000t;
 			qca,disable-smarteee;
+			qca,disable-hibernation-mode;
 			vddio-supply = <&vddio0>;
 
 			vddio0: vddio-regulator {
-- 
2.25.1