On Wed, Sep 19, 2018 at 02:32:00PM +0200, Andrew Lunn wrote:
> > > And here also.
> >
> > Thanks for raising this, I noticed it too.
> >
> > > Looking at the code, i see:
> > >
> > > /* E-MAC init function */
> > > static void ravb_emac_init(struct net_device *ndev)
> > > {
> > > struct ravb_private *priv = netdev_priv(ndev);
> > >
> > > /* Receive frame limit set register */
> > > ravb_write(ndev, ndev->mtu + ETH_HLEN + VLAN_HLEN + ETH_FCS_LEN, RFLR);
> > >
> > > /* EMAC Mode: PAUSE prohibition; Duplex; RX Checksum; TX; RX */
> > > ravb_write(ndev, ECMR_ZPF | (priv->duplex ? ECMR_DM : 0) |
> > > (ndev->features & NETIF_F_RXCSUM ? ECMR_RCSC : 0) |
> > > ECMR_TE | ECMR_RE, ECMR);
> > >
> > > Does this mean Pause is not supported in the hardware?
> >
> > According to my reading of the documentation Pause is supported by the
> > hardware and the above code seems to conflict with the comment (possibly
> > both the code and comment predate the current documentation). My reading of
> > the documentation is that the above unconditionally _enables_ receiving and
> > sending Pause frames with time parameter value 0.
>
> Hi Simon
>
> We should first prove that this additional Pause is causing the
> issue. After that, we can decide if we want to add Pause support to
> the driver. Please could you test this patch.
Hi Andrew,
thanks for your patch. I agree with the approach you have taken here,
however, unfortunately this patch does not seem to resolve the problem that
I have observed.
With this patch on top of net-next ([1] & [2]) I see:
[1] net-next as of two days ago, the version used when reporting
results earlier in this thread
cf7d97e1e54d ("net: mdio: remove duplicated include from mdio_bus.c")
# mii-tool -vv eth0
Using SIOCGMIIPHY=0x8947
eth0: no link
registers for MII PHY 0:
1140 7949 0022 1622 0981 c1e1 000d 0000
0000 0300 0000 0000 0000 0000 0000 3000
0000 0000 0000 0000 7002 0000 0000 0200
0000 0000 0000 0500 0000 0000 0000 0000
product info: vendor 00:08:85, model 34 rev 2
basic mode: autonegotiation enabled
basic status: no link
capabilities: 1000baseT-HD 1000baseT-FD 100baseTx-FD 100baseTx-HD 10baseT-FD 10baseT-HD
advertising: 100baseTx-FD 100baseTx-HD
link partner: 100baseTx-FD 100baseTx-HD 10baseT-FD 10baseT-HD
[2] net-next as of this morning
faa08325b429 ("isdn/hisax: Remove unnecessary parenthesis")
# mii-tool -vv eth0
Using SIOCGMIIPHY=0x8947
eth0: no link
registers for MII PHY 0:
1140 7949 0022 1622 0981 c1e1 000d 0000
0000 0300 0000 0000 0000 0000 0000 3000
0000 0000 0000 0000 6002 0000 0000 0200
0000 0000 0000 0500 0000 0000 0000 0000
product info: vendor 00:08:85, model 34 rev 2
basic mode: autonegotiation enabled
basic status: no link
capabilities: 1000baseT-HD 1000baseT-FD 100baseTx-FD 100baseTx-HD 10baseT-FD 10baseT-HD
advertising: 100baseTx-FD 100baseTx-HD
link partner: 100baseTx-FD 100baseTx-HD 10baseT-FD 10baseT-HD
I note a difference in the 3rd line of hex output: 7002 vs 6002
but I am unsure if that is relevant.
