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 528A6C7EE30 for ; Tue, 1 Jul 2025 15:34:30 +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:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=GwNQqB8pBJVwu+Xy+ljesQ25T/+91BIQflo1XREJg40=; b=fBImfwi/UlozVhqLshgwVg62Ow weBTzn5yWtRQpSqrkWrUyXp2HQ2uPeYUW1hATeGUvzSUD95e2yOaHP4boQKh4oLaT+5SN36AjalEQ fZ12JR2tzKJyeqxG1AxOJheJrfk9/aYfCknzfJcy2cEZho4nwy6N1TjdHpFt+7M8I3Z759psC+WnC MmdfQl//AGi8IQsf3zd5HLtg3XmSWEvLLe8W4NCOcMa4mmr6EYSVQ8HEi7lvHpwJb/pf1fpfngfjw NM+n8RY6Gi/Vcil7i0yNV018zIto3Q6TdWis9T86umg8TFwPCBdAILqoWZnxwpdBdRGAvks4ItlcM fC6x13lg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1uWd04-00000005u6K-0cnm; Tue, 01 Jul 2025 15:34:24 +0000 Received: from mail-m16.yeah.net ([1.95.21.17]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1uWbS1-00000005bNp-3Ndm for linux-arm-kernel@lists.infradead.org; Tue, 01 Jul 2025 13:55:11 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=yeah.net; s=s110527; h=Date:From:To:Subject:Message-ID:MIME-Version: Content-Type; bh=GwNQqB8pBJVwu+Xy+ljesQ25T/+91BIQflo1XREJg40=; b=mdMixECcLagBJQzyNSbxLKjgirGpbSIPT6x2HT/8O2EpMR17+Byz+sqe1JLTY/ lAyKizyHyoNHO6ZomxZOwgR0m7sC3MaPg6bzzo5wsuHifiunl0uJc0qhK6mQbKVj FNT0NPsUI02xkLVZyWysdE1D8qNIn5MN7Rv9Ej2+tE2qo= Received: from dragon (unknown []) by gzsmtp1 (Coremail) with SMTP id Mc8vCgDnL6eS6GNopYgBAA--.3609S3; Tue, 01 Jul 2025 21:54:28 +0800 (CST) Date: Tue, 1 Jul 2025 21:54:26 +0800 From: Shawn Guo To: Stefano Radaelli Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, othacehe@gnu.org, andrew@lunn.ch, Rob Herring , Krzysztof Kozlowski , Conor Dooley , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v3] arm64: dts: freescale: imx93-var-som: update eqos support for MaxLinear PHY Message-ID: References: <20250605085904.12199-1-stefano.radaelli21@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20250605085904.12199-1-stefano.radaelli21@gmail.com> X-CM-TRANSID: Mc8vCgDnL6eS6GNopYgBAA--.3609S3 X-Coremail-Antispam: 1Uf129KBjvJXoWxZFW3tryfuF4DJF4ruw4Uurg_yoWrKF48pa yfurWIqrZ3t34fK3yxWa13CF1jya15Ar9rur15trW093WxCF9rtr1Skws8Xr4DCrW8Aw13 Xryq9F13C3Z293DanT9S1TB71UUUUU7qnTZGkaVYY2UrUUUUjbIjqfuFe4nvWSU5nxnvy2 9KBjDUYxBIdaVFxhVjvjDU0xZFpf9x07jYyIUUUUUU= X-Originating-IP: [114.216.146.217] X-CM-SenderInfo: pvkd40hjxrjqh1hdxhhqhw/1tbiNBdSt2hj6JeipQAA31 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250701_065510_175777_E375640D X-CRM114-Status: GOOD ( 24.83 ) 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 On Thu, Jun 05, 2025 at 10:59:04AM +0200, Stefano Radaelli wrote: > Variscite has updated the Ethernet PHY on the VAR-SOM-MX93 from the > ADIN1300BCPZ to the MaxLinear MXL86110, as documented in the > August 2023 revision changelog. > Link: https://variwiki.com/index.php?title=VAR-SOM-MX93_rev_changelog > > Update the device tree accordingly: > - Drop the regulator node used to power the previously PHY. > - Add support for the reset line using GPIO1_IO07 with proper timings. > - Configure the PHY LEDs via the LED subsystem under /sys/class/leds/, > leveraging the support implemented in the mxl86110 PHY driver > (drivers/net/phy/mxl-86110.c). > Two LEDs are defined to match the LED configuration on the Variscite > VAR-SOM Carrier Boards: > * LED@0: Yellow, netdev trigger. > * LED@1: Green, netdev trigger. > - Adjust the RGMII clock pad control settings to match the updated PHY > requirements. > > These changes ensure proper PHY initialization and LED status indication > for the new MaxLinear MXL86110, improving board compatibility with the > latest hardware revision. > > Signed-off-by: Stefano Radaelli > --- > v3: > - Add "PATCH" to subject line. > - Fix wrong reference to previous PHY in commit message. > > v2: https://lore.kernel.org/imx/20250604153510.55689-1-stefano.radaelli21@gmail.com/ > - Clarified the use of 'rgmii' mode by adding a comment in the DT, > explaining that hardware delays are already implemented on the SOM PCB. > > v1: https://lore.kernel.org/imx/20250603221416.74523-1-stefano.radaelli21@gmail.com/ > > .../boot/dts/freescale/imx93-var-som.dtsi | 45 ++++++++++++------- > 1 file changed, 30 insertions(+), 15 deletions(-) > > diff --git a/arch/arm64/boot/dts/freescale/imx93-var-som.dtsi b/arch/arm64/boot/dts/freescale/imx93-var-som.dtsi > index 783938245e4f..cea8d792328c 100644 > --- a/arch/arm64/boot/dts/freescale/imx93-var-som.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx93-var-som.dtsi > @@ -19,26 +19,19 @@ mmc_pwrseq: mmc-pwrseq { > reset-gpios = <&gpio4 14 GPIO_ACTIVE_LOW>, /* WIFI_RESET */ > <&gpio3 7 GPIO_ACTIVE_LOW>; /* WIFI_PWR_EN */ > }; > - > - reg_eqos_phy: regulator-eqos-phy { > - compatible = "regulator-fixed"; > - pinctrl-names = "default"; > - pinctrl-0 = <&pinctrl_reg_eqos_phy>; > - regulator-name = "eth_phy_pwr"; > - regulator-min-microvolt = <3300000>; > - regulator-max-microvolt = <3300000>; > - gpio = <&gpio1 7 GPIO_ACTIVE_HIGH>; > - enable-active-high; > - startup-delay-us = <100000>; > - regulator-always-on; > - }; > }; > > &eqos { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_eqos>; > + /* > + * The required RGMII TX and RX 2ns delays are implemented directly > + * in hardware via passive delay elements on the SOM PCB. > + * No delay configuration is needed in software via PHY driver. > + */ > phy-mode = "rgmii"; > phy-handle = <ðphy0>; > + snps,clk-csr = <5>; > status = "okay"; > > mdio { > @@ -51,6 +44,27 @@ ethphy0: ethernet-phy@0 { > compatible = "ethernet-phy-ieee802.3-c22"; > reg = <0>; > eee-broken-1000t; > + reset-gpios = <&gpio1 7 GPIO_ACTIVE_LOW>; > + reset-assert-us = <10000>; > + reset-deassert-us = <100000>; Missing a newline between property list and child node. I fixed it up and applied the patch. Shawn > + leds { > + #address-cells = <1>; > + #size-cells = <0>; > + > + led@0 { > + reg = <0>; > + color = ; > + function = LED_FUNCTION_LAN; > + linux,default-trigger = "netdev"; > + }; > + > + led@1 { > + reg = <1>; > + color = ; > + function = LED_FUNCTION_LAN; > + linux,default-trigger = "netdev"; > + }; > + }; > }; > }; > }; > @@ -75,14 +89,15 @@ MX93_PAD_ENET1_RD0__ENET_QOS_RGMII_RD0 0x57e > MX93_PAD_ENET1_RD1__ENET_QOS_RGMII_RD1 0x57e > MX93_PAD_ENET1_RD2__ENET_QOS_RGMII_RD2 0x57e > MX93_PAD_ENET1_RD3__ENET_QOS_RGMII_RD3 0x57e > - MX93_PAD_ENET1_RXC__CCM_ENET_QOS_CLOCK_GENERATE_RX_CLK 0x5fe > + MX93_PAD_ENET1_RXC__CCM_ENET_QOS_CLOCK_GENERATE_RX_CLK 0x58e > MX93_PAD_ENET1_RX_CTL__ENET_QOS_RGMII_RX_CTL 0x57e > MX93_PAD_ENET1_TD0__ENET_QOS_RGMII_TD0 0x57e > MX93_PAD_ENET1_TD1__ENET_QOS_RGMII_TD1 0x57e > MX93_PAD_ENET1_TD2__ENET_QOS_RGMII_TD2 0x57e > MX93_PAD_ENET1_TD3__ENET_QOS_RGMII_TD3 0x57e > - MX93_PAD_ENET1_TXC__CCM_ENET_QOS_CLOCK_GENERATE_TX_CLK 0x5fe > + MX93_PAD_ENET1_TXC__CCM_ENET_QOS_CLOCK_GENERATE_TX_CLK 0x58e > MX93_PAD_ENET1_TX_CTL__ENET_QOS_RGMII_TX_CTL 0x57e > + MX93_PAD_UART2_TXD__GPIO1_IO07 0x51e > >; > }; > > > base-commit: a9dfb7db96f7bc1f30feae673aab7fdbfbc94e9c > prerequisite-patch-id: 2335ebcc90360b008c840e7edf7e34a595880edf > -- > 2.43.0 >