Hi Mattijs,
вт, 8 окт. 2024 г. в 15:58, Mattijs Korpershoek <mkorpersh...@baylibre.com>:
>
> Hi Maksim,
>
> Thank you for the patch and sorry for the review delay.
>
> On jeu., sept. 19, 2024 at 16:58, biguncle...@gmail.com wrote:
>
> > From: Maksim Kiselev <biguncle...@gmail.com>
> >
> > Some USB-ETH adapters based on the AX88179 (ex. D-link DUB-2312)
> > has link issue if ADVERTISE_NPAGE flag is set.
>
> Do you see the issue only if ADVERTISE_NPAGE is set? What about
> ADVERTISE_LPACK ?
Removing ADVERTISE_NPAGE fixed link issue for me.
But I decided to remove ADVERTISE_LPACK too.
Because I couldn't find any mention of this flag in the Linux driver.
> Could you detail a bit what you mean by link issue?
>
> >
> > Also Linux drivers for ASIX chips doesn't use these flags.
>
> I've double checked this looking on master,
> commit 87d6aab2389e ("Merge tag 'for_linus' of
> git://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost")
>
> I inspected asix_devices.c and ax88179_178a.c and I could indeed not
> find anything related to ADVERTISE_LPACK and ADVERTISE_NPAGE.
>
> I'm not super knowledgeable on network so I'd like someone else to test
> this as well, if possible.
>
> >
> > Signed-off-by: Maksim Kiselev <biguncle...@gmail.com>
> > ---
> > drivers/usb/eth/asix88179.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/usb/eth/asix88179.c b/drivers/usb/eth/asix88179.c
> > index 4bd3b9d10d..1f469a1a7d 100644
> > --- a/drivers/usb/eth/asix88179.c
> > +++ b/drivers/usb/eth/asix88179.c
> > @@ -345,8 +345,8 @@ static int asix_basic_reset(struct ueth_data *dev,
> > asix_write_cmd(dev, AX_ACCESS_MAC, AX_MEDIUM_STATUS_MODE, 2, 2,
> > tmp16);
> >
> > u16 adv = 0;
> > - adv = ADVERTISE_ALL | ADVERTISE_CSMA | ADVERTISE_LPACK |
> > - ADVERTISE_NPAGE | ADVERTISE_PAUSE_ASYM | ADVERTISE_PAUSE_CAP;
> > + adv = ADVERTISE_ALL | ADVERTISE_CSMA | ADVERTISE_PAUSE_ASYM |
> > + ADVERTISE_PAUSE_CAP;
> > asix_write_cmd(dev, AX_ACCESS_PHY, 0x03, MII_ADVERTISE, 2, &adv);
> >
> > adv = ADVERTISE_1000FULL;
> > --
> > 2.45.2
