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 <evgeniy.di...@synopsys.com> > Cc: Eugeniy Paltsev <eugeniy.palt...@synopsys.com> > Cc: Alexey Brodkin <abrod...@synopsys.com>
@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