From: "Jernej Škrabec" <jernej.skrabec@gmail.com> To: "Maxime Ripard" <mripard@kernel.org>, "Chen-Yu Tsai" <wens@csie.org>, "Clément Bœsch" <u@pkh.me> Cc: "Willy Liu" <willy.liu@realtek.com>, "Rob Herring" <robh+dt@kernel.org>, "David S. Miller" <davem@davemloft.net>, "Andrew Lunn" <andrew@lunn.ch>, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org, "Clément Bœsch" <u@pkh.me> Subject: Re: [PATCH] arm64: dts: sun50i: h5: NanoPI Neo 2: phy-mode rgmii-id Date: Mon, 30 Aug 2021 22:49:37 +0200 [thread overview] Message-ID: <116454729.UZi3dMzWh7@jernej-laptop> (raw) In-Reply-To: <20210830151645.18018-1-u@pkh.me> Hi! Dne ponedeljek, 30. avgust 2021 ob 17:16:45 CEST je Clément Bœsch napisal(a): > Since commit bbc4d71d6354 ("net: phy: realtek: fix rtl8211e rx/tx delay > config") network is broken on the NanoPi Neo 2. > > This patch changes the phy-mode to use internal delays both for RX and > TX as has been done for other boards affected by the same commit. > > Fixes: bbc4d71d6354 ("net: phy: realtek: fix rtl8211e rx/tx delay config") This commit fixes DT issue, so "fixes" tag should be: Fixes: 44a94c7ef989 ("arm64: dts: allwinner: H5: Restore EMAC changes") Here, a node with wrong phy-mode property was added to NanoPi Neo 2 board DT. Other than that, this patch is fine and once fixes tag is fixed, you can add: Reviewed-by: Jernej Skrabec <jernej.skrabec@gmail.com> For next version, you should: - change fixed tag - add my review-by tag right above your signed-off-by tag - mark patch as v2 (add "-v2" parameter to git format-patch) - describe change right under "---" line Note, if you borked something when sending, you should mark patch or patch series as "RESEND", so recipients don't look for changes in two subsequent e- mails (--subject-prefix="RESEND PATCH"). Thanks for the fix! Best regards, Jernej > Signed-off-by: Clément Bœsch <u@pkh.me> > --- > arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts > b/arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts index > 02f8e72f0cad..05486cccee1c 100644 > --- a/arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts > +++ b/arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts > @@ -75,7 +75,7 @@ &emac { > pinctrl-0 = <&emac_rgmii_pins>; > phy-supply = <®_gmac_3v3>; > phy-handle = <&ext_rgmii_phy>; > - phy-mode = "rgmii"; > + phy-mode = "rgmii-id"; > status = "okay"; > };
WARNING: multiple messages have this Message-ID (diff)
From: "Jernej Škrabec" <jernej.skrabec@gmail.com> To: "Maxime Ripard" <mripard@kernel.org>, "Chen-Yu Tsai" <wens@csie.org>, "Clément Bœsch" <u@pkh.me> Cc: "Willy Liu" <willy.liu@realtek.com>, "Rob Herring" <robh+dt@kernel.org>, "David S. Miller" <davem@davemloft.net>, "Andrew Lunn" <andrew@lunn.ch>, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org, "Clément Bœsch" <u@pkh.me> Subject: Re: [PATCH] arm64: dts: sun50i: h5: NanoPI Neo 2: phy-mode rgmii-id Date: Mon, 30 Aug 2021 22:49:37 +0200 [thread overview] Message-ID: <116454729.UZi3dMzWh7@jernej-laptop> (raw) In-Reply-To: <20210830151645.18018-1-u@pkh.me> Hi! Dne ponedeljek, 30. avgust 2021 ob 17:16:45 CEST je Clément Bœsch napisal(a): > Since commit bbc4d71d6354 ("net: phy: realtek: fix rtl8211e rx/tx delay > config") network is broken on the NanoPi Neo 2. > > This patch changes the phy-mode to use internal delays both for RX and > TX as has been done for other boards affected by the same commit. > > Fixes: bbc4d71d6354 ("net: phy: realtek: fix rtl8211e rx/tx delay config") This commit fixes DT issue, so "fixes" tag should be: Fixes: 44a94c7ef989 ("arm64: dts: allwinner: H5: Restore EMAC changes") Here, a node with wrong phy-mode property was added to NanoPi Neo 2 board DT. Other than that, this patch is fine and once fixes tag is fixed, you can add: Reviewed-by: Jernej Skrabec <jernej.skrabec@gmail.com> For next version, you should: - change fixed tag - add my review-by tag right above your signed-off-by tag - mark patch as v2 (add "-v2" parameter to git format-patch) - describe change right under "---" line Note, if you borked something when sending, you should mark patch or patch series as "RESEND", so recipients don't look for changes in two subsequent e- mails (--subject-prefix="RESEND PATCH"). Thanks for the fix! Best regards, Jernej > Signed-off-by: Clément Bœsch <u@pkh.me> > --- > arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts > b/arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts index > 02f8e72f0cad..05486cccee1c 100644 > --- a/arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts > +++ b/arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts > @@ -75,7 +75,7 @@ &emac { > pinctrl-0 = <&emac_rgmii_pins>; > phy-supply = <®_gmac_3v3>; > phy-handle = <&ext_rgmii_phy>; > - phy-mode = "rgmii"; > + phy-mode = "rgmii-id"; > status = "okay"; > }; _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2021-08-30 20:49 UTC|newest] Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-08-30 15:16 [PATCH] arm64: dts: sun50i: h5: NanoPI Neo 2: phy-mode rgmii-id Clément Bœsch 2021-08-30 15:16 ` Clément Bœsch 2021-08-30 20:49 ` Jernej Škrabec [this message] 2021-08-30 20:49 ` Jernej Škrabec 2021-08-30 21:25 ` Clément Bœsch 2021-08-30 21:25 ` Clément Bœsch 2021-08-31 20:17 ` Jernej Škrabec 2021-08-31 20:17 ` Jernej Škrabec 2021-08-30 21:50 ` Andrew Lunn 2021-08-30 21:50 ` Andrew Lunn 2021-08-31 19:53 ` Jernej Škrabec 2021-08-31 19:53 ` Jernej Škrabec 2021-08-31 20:53 ` Andrew Lunn 2021-08-31 20:53 ` Andrew Lunn 2021-09-01 15:31 ` Jernej Škrabec 2021-09-01 15:31 ` Jernej Škrabec 2021-09-03 9:21 ` Maxime Ripard 2021-09-03 9:21 ` Maxime Ripard 2021-09-05 0:20 ` [PATCH v2] arm64: dts: allwinner: h5: NanoPI Neo 2: Fix ethernet node Clément Bœsch 2021-09-05 0:20 ` Clément Bœsch 2021-09-05 0:42 ` Andrew Lunn 2021-09-05 0:42 ` Andrew Lunn 2021-09-06 8:03 ` Maxime Ripard 2021-09-06 8:03 ` Maxime Ripard 2021-08-30 21:52 ` [PATCH] arm64: dts: sun50i: h5: NanoPI Neo 2: phy-mode rgmii-id Andrew Lunn 2021-08-30 21:52 ` Andrew Lunn -- strict thread matches above, loose matches on Subject: below -- 2021-08-30 15:15 Clément Bœsch
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=116454729.UZi3dMzWh7@jernej-laptop \ --to=jernej.skrabec@gmail.com \ --cc=andrew@lunn.ch \ --cc=davem@davemloft.net \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-sunxi@lists.linux.dev \ --cc=mripard@kernel.org \ --cc=robh+dt@kernel.org \ --cc=u@pkh.me \ --cc=wens@csie.org \ --cc=willy.liu@realtek.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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.