[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <7c01712f-776d-4073-9829-f18a51df6d4c@lunn.ch>
Date: Fri, 2 May 2025 23:51:59 +0200
From: Andrew Lunn <andrew@...n.ch>
To: John Clark <inindev@...il.com>
Cc: heiko@...ech.de, robh@...nel.org, krzk+dt@...nel.org,
conor+dt@...nel.org, nicolas.frattaroli@...labora.com,
detlev.casanova@...labora.com, linux-arm-kernel@...ts.infradead.org,
linux-rockchip@...ts.infradead.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1 3/3] arm64: dts: rockchip: Add Luckfox Omni3576
Carrier Board with Core3576 Module
> +&gmac1 {
> + clock_in_out = "output";
> + phy-handle = <&rgmii_phy1>;
> + phy-mode = "rgmii-rxid";
This is unlikely to be correct. It probably works, but that does not
make it correct.
> + pinctrl-names = "default";
> + pinctrl-0 = <ð1m0_miim
> + ð1m0_tx_bus2
> + ð1m0_rx_bus2
> + ð1m0_rgmii_clk
> + ð1m0_rgmii_bus
> + ðm0_clk1_25m_out>;
> + snps,reset-gpio = <&gpio2 RK_PB4 GPIO_ACTIVE_LOW>;
> + snps,reset-active-low;
> + snps,reset-delays-us = <0 20000 100000>;
> + tx_delay = <0x20>;
> + status = "disabled";
> +};
Please see the discussion about rgmii delays with other rockchip
boards from the last week, and
https://patchwork.kernel.org/project/netdevbpf/patch/20250430-v6-15-rc3-net-rgmii-delays-v2-1-099ae651d5e5@lunn.ch/
Andrew
Powered by blists - more mailing lists