Message ID | 20220624075139.3139300-2-o.rempel@pengutronix.de (mailing list archive) |
---|---|
State | Accepted |
Commit | ce95ab775f8d8e89a038c0e5611a7381a2ef8e43 |
Delegated to: | Netdev Maintainers |
Headers | show |
Series | [net,v1,1/2] net: asix: fix "can't send until first packet is send" issue | expand |
On Fri, Jun 24, 2022 at 09:51:39AM +0200, Oleksij Rempel wrote: > We should respect link partner capabilities and not force flow control > support on every link. Even more, in current state the MAC driver do not > advertises pause support so we should not keep flow control enabled at > all. > > Fixes: e532a096be0e ("net: usb: asix: ax88772: add phylib support") > Reported-by: Anton Lundin <glance@acc.umu.se> > Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de> In different mail thread Anton reported as tested. Tested-by: Anton Lundin <glance@acc.umu.se> > --- > drivers/net/usb/asix.h | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/net/usb/asix.h b/drivers/net/usb/asix.h > index 2c81236c6c7c..45d3cc5cc355 100644 > --- a/drivers/net/usb/asix.h > +++ b/drivers/net/usb/asix.h > @@ -126,8 +126,7 @@ > AX_MEDIUM_RE) > > #define AX88772_MEDIUM_DEFAULT \ > - (AX_MEDIUM_FD | AX_MEDIUM_RFC | \ > - AX_MEDIUM_TFC | AX_MEDIUM_PS | \ > + (AX_MEDIUM_FD | AX_MEDIUM_PS | \ > AX_MEDIUM_AC | AX_MEDIUM_RE) > > /* AX88772 & AX88178 RX_CTL values */ > -- > 2.30.2 > >
diff --git a/drivers/net/usb/asix.h b/drivers/net/usb/asix.h index 2c81236c6c7c..45d3cc5cc355 100644 --- a/drivers/net/usb/asix.h +++ b/drivers/net/usb/asix.h @@ -126,8 +126,7 @@ AX_MEDIUM_RE) #define AX88772_MEDIUM_DEFAULT \ - (AX_MEDIUM_FD | AX_MEDIUM_RFC | \ - AX_MEDIUM_TFC | AX_MEDIUM_PS | \ + (AX_MEDIUM_FD | AX_MEDIUM_PS | \ AX_MEDIUM_AC | AX_MEDIUM_RE) /* AX88772 & AX88178 RX_CTL values */
We should respect link partner capabilities and not force flow control support on every link. Even more, in current state the MAC driver do not advertises pause support so we should not keep flow control enabled at all. Fixes: e532a096be0e ("net: usb: asix: ax88772: add phylib support") Reported-by: Anton Lundin <glance@acc.umu.se> Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de> --- drivers/net/usb/asix.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)