From: Biao Huang <biao.huang@mediatek.com>
To: Matthias Brugger <matthias.bgg@gmail.com>
Cc: <davem@davemloft.net>, Jakub Kicinski <kuba@kernel.org>,
Rob Herring <robh+dt@kernel.org>,
<angelogioacchino.delregno@collabora.com>,
"Giuseppe Cavallaro" <peppe.cavallaro@st.com>,
Alexandre Torgue <alexandre.torgue@foss.st.com>,
Jose Abreu <joabreu@synopsys.com>,
"Maxime Coquelin" <mcoquelin.stm32@gmail.com>,
<netdev@vger.kernel.org>, <devicetree@vger.kernel.org>,
<linux-kernel@vger.kernel.org>,
<linux-arm-kernel@lists.infradead.org>,
<linux-mediatek@lists.infradead.org>,
<linux-stm32@st-md-mailman.stormreply.com>,
<srv_heupstream@mediatek.com>, <macpaul.lin@mediatek.com>,
<dkirjanov@suse.de>
Subject: Re: [PATCH net-next v12 4/7] arm64: dts: mt2712: update ethernet device node
Date: Wed, 19 Jan 2022 09:24:28 +0800 [thread overview]
Message-ID: <6d9edd18e64a973265b92d98395f8ce7470e59e6.camel@mediatek.com> (raw)
In-Reply-To: <20220117070706.17853-5-biao.huang@mediatek.com>
Dear Matthias,
Any comments about this dts patch?
Thanks in advance.
Regards!
Biao
On Mon, 2022-01-17 at 15:07 +0800, Biao Huang wrote:
> Since there are some changes in ethernet driver:
> update ethernet device node in dts to accommodate to it.
>
> 1. stmmac_probe_config_dt() in stmmac_platform.c will initialize
> specified
> parameters according to compatible string "snps,dwmac-4.20a",
> then,
> dwmac-mediatek.c can skip the initialization if add compatible
> string
> "snps,dwmac-4.20a" in eth device node.
> 2. commit 882007ed7832 ("net-next: dt-binding: dwmac-mediatek: add
> more
> description for RMII") added rmii internal support, we should add
> corresponding clocks/clocks-names in eth device node.
> 3. add "snps,reset-delays-us = <0 10000 10000>;" to ensure reset
> delay
> can meet PHY requirement.
>
> Signed-off-by: Biao Huang <biao.huang@mediatek.com>
> ---
> arch/arm64/boot/dts/mediatek/mt2712-evb.dts | 1 +
> arch/arm64/boot/dts/mediatek/mt2712e.dtsi | 14 +++++++++-----
> 2 files changed, 10 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt2712-evb.dts
> b/arch/arm64/boot/dts/mediatek/mt2712-evb.dts
> index 7d369fdd3117..11aa135aa0f3 100644
> --- a/arch/arm64/boot/dts/mediatek/mt2712-evb.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt2712-evb.dts
> @@ -110,6 +110,7 @@ ð {
> phy-handle = <ðernet_phy0>;
> mediatek,tx-delay-ps = <1530>;
> snps,reset-gpio = <&pio 87 GPIO_ACTIVE_LOW>;
> + snps,reset-delays-us = <0 10000 10000>;
> pinctrl-names = "default", "sleep";
> pinctrl-0 = <ð_default>;
> pinctrl-1 = <ð_sleep>;
> diff --git a/arch/arm64/boot/dts/mediatek/mt2712e.dtsi
> b/arch/arm64/boot/dts/mediatek/mt2712e.dtsi
> index a9cca9c146fd..9e850e04fffb 100644
> --- a/arch/arm64/boot/dts/mediatek/mt2712e.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt2712e.dtsi
> @@ -726,7 +726,7 @@ queue2 {
> };
>
> eth: ethernet@1101c000 {
> - compatible = "mediatek,mt2712-gmac";
> + compatible = "mediatek,mt2712-gmac", "snps,dwmac-
> 4.20a";
> reg = <0 0x1101c000 0 0x1300>;
> interrupts = <GIC_SPI 237 IRQ_TYPE_LEVEL_LOW>;
> interrupt-names = "macirq";
> @@ -734,15 +734,19 @@ eth: ethernet@1101c000 {
> clock-names = "axi",
> "apb",
> "mac_main",
> - "ptp_ref";
> + "ptp_ref",
> + "rmii_internal";
> clocks = <&pericfg CLK_PERI_GMAC>,
> <&pericfg CLK_PERI_GMAC_PCLK>,
> <&topckgen CLK_TOP_ETHER_125M_SEL>,
> - <&topckgen CLK_TOP_ETHER_50M_SEL>;
> + <&topckgen CLK_TOP_ETHER_50M_SEL>,
> + <&topckgen CLK_TOP_ETHER_50M_RMII_SEL>;
> assigned-clocks = <&topckgen CLK_TOP_ETHER_125M_SEL>,
> - <&topckgen CLK_TOP_ETHER_50M_SEL>;
> + <&topckgen CLK_TOP_ETHER_50M_SEL>,
> + <&topckgen
> CLK_TOP_ETHER_50M_RMII_SEL>;
> assigned-clock-parents = <&topckgen
> CLK_TOP_ETHERPLL_125M>,
> - <&topckgen CLK_TOP_APLL1_D3>;
> + <&topckgen CLK_TOP_APLL1_D3>,
> + <&topckgen
> CLK_TOP_ETHERPLL_50M>;
> power-domains = <&scpsys MT2712_POWER_DOMAIN_AUDIO>;
> mediatek,pericfg = <&pericfg>;
> snps,axi-config = <&stmmac_axi_setup>;
next prev parent reply other threads:[~2022-01-19 1:24 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-01-17 7:06 [PATCH net-next v12 0/7] MediaTek Ethernet Patches on MT8195 Biao Huang
2022-01-17 7:07 ` [PATCH net-next v12 1/7] stmmac: dwmac-mediatek: add platform level clocks management Biao Huang
2022-01-17 7:07 ` [PATCH net-next v12 2/7] stmmac: dwmac-mediatek: Reuse more common features Biao Huang
2022-01-17 7:07 ` [PATCH net-next v12 3/7] stmmac: dwmac-mediatek: re-arrange clock setting Biao Huang
2022-01-17 10:38 ` AngeloGioacchino Del Regno
2022-01-18 1:51 ` Biao Huang
2022-01-17 7:07 ` [PATCH net-next v12 4/7] arm64: dts: mt2712: update ethernet device node Biao Huang
2022-01-19 1:24 ` Biao Huang [this message]
2022-01-17 7:07 ` [PATCH net-next v12 5/7] net: dt-bindings: dwmac: Convert mediatek-dwmac to DT schema Biao Huang
2022-01-17 7:07 ` [PATCH net-next v12 6/7] stmmac: dwmac-mediatek: add support for mt8195 Biao Huang
2022-01-17 7:07 ` [PATCH net-next v12 7/7] net: dt-bindings: dwmac: " Biao Huang
2022-01-17 15:35 ` Rob Herring
2022-01-19 1:21 ` Biao Huang
2022-02-01 23:48 ` Rob Herring
2022-03-14 7:13 ` Biao Huang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=6d9edd18e64a973265b92d98395f8ce7470e59e6.camel@mediatek.com \
--to=biao.huang@mediatek.com \
--cc=alexandre.torgue@foss.st.com \
--cc=angelogioacchino.delregno@collabora.com \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=dkirjanov@suse.de \
--cc=joabreu@synopsys.com \
--cc=kuba@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-stm32@st-md-mailman.stormreply.com \
--cc=macpaul.lin@mediatek.com \
--cc=matthias.bgg@gmail.com \
--cc=mcoquelin.stm32@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=peppe.cavallaro@st.com \
--cc=robh+dt@kernel.org \
--cc=srv_heupstream@mediatek.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).