Re: [PATCH] ARC: [plat-hsdk]: Switch ethernet phy-mode to rgmii-id
Hi Vineet, Evgeniy! > From: Vineet Gupta > Sent: Tuesday, September 1, 2020 9:41 PM > To: Evgeniy Didin ; linux-snps-arc@lists.infradead.org > > Cc: Alexey Brodkin ; Eugeniy Paltsev > > Subject: Re: [PATCH] ARC: [plat-hsdk]: Switch ethernet phy-mode to rgmii-id > > On 7/7/20 8:38 AM, Evgeniy Didin wrote: > > HSDK board has Micrel KSZ9031, recent commit > > bcf3440c6dd ("net: phy: micrel: add phy-mode support for the KSZ9031 PHY") > > caused a breakdown of Ethernet. > > Using 'phy-mode = "rgmii"' is not correct because accodring RGMII > > specification it is necessary to have delay on RX (PHY to MAX) > > which is not generated in case of "rgmii". > > Using "rgmii-id" adds necessary delay and solves the issue. > > > > Also adding name of PHY placed on HSDK board. > > > > Signed-off-by: Evgeniy Didin > > Cc: Eugeniy Paltsev > > Cc: Alexey Brodkin > > @Alexey - u ok with this change ? Sure, Acked-by: Alexey Brodkin > > -Vineet > > > --- > > arch/arc/boot/dts/hsdk.dts | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/arch/arc/boot/dts/hsdk.dts b/arch/arc/boot/dts/hsdk.dts > > index 9acbeba832c0..d6427d47e5a1 100644 > > --- a/arch/arc/boot/dts/hsdk.dts > > +++ b/arch/arc/boot/dts/hsdk.dts > > @@ -208,7 +208,7 @@ > > reg = <0x8000 0x2000>; > > interrupts = <10>; > > interrupt-names = "macirq"; > > - phy-mode = "rgmii"; > > + phy-mode = "rgmii-id"; > > snps,pbl = <32>; > > snps,multicast-filter-bins = <256>; > > clocks = <&gmacclk>; > > @@ -226,7 +226,7 @@ > > #address-cells = <1>; > > #size-cells = <0>; > > compatible = "snps,dwmac-mdio"; > > - phy0: ethernet-phy@0 { > > + phy0: ethernet-phy@0 { /* Micrel KSZ9031 */ > > reg = <0>; > > }; > > }; > ___ linux-snps-arc mailing list linux-snps-arc@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-snps-arc
Re: [PATCH] ARC: [plat-hsdk]: Switch ethernet phy-mode to rgmii-id
On 7/7/20 8:38 AM, Evgeniy Didin wrote: > HSDK board has Micrel KSZ9031, recent commit > bcf3440c6dd ("net: phy: micrel: add phy-mode support for the KSZ9031 PHY") > caused a breakdown of Ethernet. > Using 'phy-mode = "rgmii"' is not correct because accodring RGMII > specification it is necessary to have delay on RX (PHY to MAX) > which is not generated in case of "rgmii". > Using "rgmii-id" adds necessary delay and solves the issue. > > Also adding name of PHY placed on HSDK board. > > Signed-off-by: Evgeniy Didin > Cc: Eugeniy Paltsev > Cc: Alexey Brodkin @Alexey - u ok with this change ? -Vineet > --- > arch/arc/boot/dts/hsdk.dts | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/arc/boot/dts/hsdk.dts b/arch/arc/boot/dts/hsdk.dts > index 9acbeba832c0..d6427d47e5a1 100644 > --- a/arch/arc/boot/dts/hsdk.dts > +++ b/arch/arc/boot/dts/hsdk.dts > @@ -208,7 +208,7 @@ > reg = <0x8000 0x2000>; > interrupts = <10>; > interrupt-names = "macirq"; > - phy-mode = "rgmii"; > + phy-mode = "rgmii-id"; > snps,pbl = <32>; > snps,multicast-filter-bins = <256>; > clocks = <&gmacclk>; > @@ -226,7 +226,7 @@ > #address-cells = <1>; > #size-cells = <0>; > compatible = "snps,dwmac-mdio"; > - phy0: ethernet-phy@0 { > + phy0: ethernet-phy@0 { /* Micrel KSZ9031 */ > reg = <0>; > }; > }; ___ linux-snps-arc mailing list linux-snps-arc@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-snps-arc
[PATCH] ARC: [plat-hsdk]: Switch ethernet phy-mode to rgmii-id
HSDK board has Micrel KSZ9031, recent commit bcf3440c6dd ("net: phy: micrel: add phy-mode support for the KSZ9031 PHY") caused a breakdown of Ethernet. Using 'phy-mode = "rgmii"' is not correct because accodring RGMII specification it is necessary to have delay on RX (PHY to MAX) which is not generated in case of "rgmii". Using "rgmii-id" adds necessary delay and solves the issue. Also adding name of PHY placed on HSDK board. Signed-off-by: Evgeniy Didin Cc: Eugeniy Paltsev Cc: Alexey Brodkin --- arch/arc/boot/dts/hsdk.dts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arc/boot/dts/hsdk.dts b/arch/arc/boot/dts/hsdk.dts index 9acbeba832c0..d6427d47e5a1 100644 --- a/arch/arc/boot/dts/hsdk.dts +++ b/arch/arc/boot/dts/hsdk.dts @@ -208,7 +208,7 @@ reg = <0x8000 0x2000>; interrupts = <10>; interrupt-names = "macirq"; - phy-mode = "rgmii"; + phy-mode = "rgmii-id"; snps,pbl = <32>; snps,multicast-filter-bins = <256>; clocks = <&gmacclk>; @@ -226,7 +226,7 @@ #address-cells = <1>; #size-cells = <0>; compatible = "snps,dwmac-mdio"; - phy0: ethernet-phy@0 { + phy0: ethernet-phy@0 { /* Micrel KSZ9031 */ reg = <0>; }; }; -- 2.16.2 ___ linux-snps-arc mailing list linux-snps-arc@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-snps-arc