[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <d8ad476c-d0c7-4e97-9e76-540a539ffb52@lunn.ch>
Date: Mon, 6 Oct 2025 17:51:15 +0200
From: Andrew Lunn <andrew@...n.ch>
To: Liangbin Lian <jjm2473@...il.com>
Cc: robh@...nel.org, krzk+dt@...nel.org, conor+dt@...nel.org,
heiko@...ech.de, quentin.schulz@...rry.de,
kever.yang@...k-chips.com, naoki@...xa.com, honyuenkwun@...il.com,
inindev@...il.com, ivan8215145640@...il.com,
neil.armstrong@...aro.org, mani@...nel.org, dsimic@...jaro.org,
pbrobinson@...il.com, alchark@...il.com, didi.debian@...ow.org,
jbx6244@...il.com, devicetree@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org,
linux-rockchip@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 3/3] arm64: dts: rockchip: add LinkEase EasePi R1
> +&gmac0 {
> + phy-mode = "rgmii";
Did i really miss this patch series in its earlier version, or did you
ignore me?
https://elixir.bootlin.com/linux/v6.15/source/Documentation/devicetree/bindings/net/ethernet-controller.yaml#L287
> + tx_delay = <0x3c>;
> + rx_delay = <0x2f>;
Please change it to rgmii-id, and smaller tx/rx_delay values. Or show
us the schematics which clearly show extra long clock lines.
> +/* Micro SD card slot is not mounted */
> +&sdmmc0 {
> + max-frequency = <150000000>;
> + no-sdio;
> + no-mmc;
> + bus-width = <4>;
> + cap-mmc-highspeed;
> + cap-sd-highspeed;
> + disable-wp;
> + pinctrl-names = "default";
> + pinctrl-0 = <&sdmmc0_bus4 &sdmmc0_clk &sdmmc0_cmd &sdmmc0_det>;
> + vmmc-supply = <&vcc3v3_sd>;
> + vqmmc-supply = <&vccio_sd>;
> + status = "disabled";
> +};
> +
> +/* Wifi module is not mounted */
> +&sdmmc2 {
What do you mean by "not mounted"?
Often you would say "not populated", to indicate the PCB has all the
tracks in place, but the chip has simply not been soldered in place.
Or is there a connector here, and nothing plugged into the connector?
Andrew
Powered by blists - more mailing lists