From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id EA204C4450A for ; Thu, 16 Jul 2026 07:30:59 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=GCrqmH9b1H/YVE63PUByvlMfO6u776WKzGDBctjFpRU=; b=4iyKDnlRklO8IUertnjn7MtbEV Io0s7HFYib1Y/jORyz/fb4ol0ZDmLS6TXQ/B8yoJ7Bt/qYU/vqa3xfAG5CpDdMO6REn7fg9V1x4ub ydgOD18b1gZq2M21yGJGaV/5Y1Qylq5eJ/CCsbyat87WUgEPXch6u2G0qFy1Fy+c88sA44obmcL1C NykuUNniSy43WoTZd9gHiXVAWJUHaz/8XCnW17vj2uTxQvV8AtPsSlM0WQWq+uRUXZZgtQX9CD0FA I4Sqcic1n+sYgWoyaAPf//y0win19P8PdX1k1xMrPJt4s+op350U8B5qHPUazL/DPn1SL2yMselLG 7Y+PLZKg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wkGYV-0000000Geth-3xaQ; Thu, 16 Jul 2026 07:30:51 +0000 Received: from smtpout-03.galae.net ([185.246.85.4]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1wkGYT-0000000Gese-0iIO for linux-arm-kernel@lists.infradead.org; Thu, 16 Jul 2026 07:30:50 +0000 Received: from smtpout-01.galae.net (smtpout-01.galae.net [212.83.139.233]) by smtpout-03.galae.net (Postfix) with ESMTPS id 1588E4E40DFF; Thu, 16 Jul 2026 07:30:45 +0000 (UTC) Received: from mail.galae.net (mail.galae.net [212.83.136.155]) by smtpout-01.galae.net (Postfix) with ESMTPS id CDAA2602B8; Thu, 16 Jul 2026 07:30:44 +0000 (UTC) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 08DB311BD057C; Thu, 16 Jul 2026 09:30:34 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=dkim; t=1784187044; h=from:subject:date:message-id:to:cc:mime-version:content-type: content-transfer-encoding:content-language:in-reply-to:references; bh=GCrqmH9b1H/YVE63PUByvlMfO6u776WKzGDBctjFpRU=; b=pcyisqzjUShEU8xVfu/cSx/J1/B5EedBEBPC6x2l3O0Agw/mDY3UrHEkRQy2aCxE0zFqMn f1cybtCjaxz+DEUvEhn7AFNdLCztSNagzHWalHFlsZViRWhpCrNZsG4IWlf8/DTCjGEBp6 43uLpyhowiDx6ouf/1/Q0b9bSs2PKw7Hl3n9zJ4OVfWPT0UwrN3mLVc5qDtKeOKeJdIjCn jUTro2i0AfG4bAgYMm4p+bicBjQxgKdihNmGgMQb+wQKyx31kCj8w33KEs5/iEie0IPsDW mWJnxJR9nXTb9ag1gZYCLEWp1949HjUBDp796k17o381cP2HTj33u4+JxBt46A== Message-ID: Date: Thu, 16 Jul 2026 09:30:33 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH net-next] net: stmmac: dwmac-rk: enable the reference clock for output mode To: Jiaxing Hu , andrew+netdev@lunn.ch, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, heiko@sntech.de, mcoquelin.stm32@gmail.com, alexandre.torgue@foss.st.com Cc: netdev@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, linux-kernel@vger.kernel.org References: <02ef3409-4078-4c3a-8c17-cb1a7f1f07e4@smtp-relay.sendinblue.com> From: Maxime Chevallier Content-Language: en-US In-Reply-To: <02ef3409-4078-4c3a-8c17-cb1a7f1f07e4@smtp-relay.sendinblue.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Last-TLS-Session-Version: TLSv1.3 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260716_003049_345923_768DCD42 X-CRM114-Status: UNSURE ( 8.33 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi, On 7/16/26 08:56, Jiaxing Hu wrote: > > rk_gmac_clk_init() only requests the refout clock group for RMII. The > ArmSoM CM5 has an on-module YT8531 RGMII PHY with no crystal that needs > the SoC 25 MHz reference (clk_mac_refout), so in RGMII the clock was > never enabled and the PHY did not respond on MDIO. > > Request the group whenever the SoC drives the clock (clock_in_out = > "output"), not just for RMII. The clocks are optional, so other boards > are unaffected. DT binding for rockchip-dwmac says : clock_in_out: description: For RGMII, it must be "input", means main clock(125MHz) is not sourced from SoC's PLL, but input from PHY. For RMII, "input" means PHY provides the reference clock(50MHz), "output" means GMAC provides the reference clock. So the statement "For RGMII, it must be input [...]" is now incorrect : - it can be output for RGMII - it will be 25Hz if set to output, from what I get from the commit log. Can you update it ? Maxime