mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Stefan Kerkmann <s.kerkmann@pengutronix.de>
To: Sascha Hauer <s.hauer@pengutronix.de>,
	 BAREBOX <barebox@lists.infradead.org>
Cc: Stefan Kerkmann <s.kerkmann@pengutronix.de>
Subject: [PATCH 0/2] usb: net: ax88179_178a: sync reset and rx with upstream
Date: Wed, 13 Aug 2025 09:52:19 +0200	[thread overview]
Message-ID: <20250813-fix-ax88179_178a-dropped-packets-v1-0-2ce605407d0b@pengutronix.de> (raw)

These patches are the result of investigating dropped packages while
working with an AX88179A based UGREEN USB ethernet adapter. The main
improvements are bounds and consistency checking in the RX path. What I
couldn't fix at the moment is that the adapter sometimes stops receiving
packets after a "warm reset" of the device which doesn't power cycle the
device e.g. chain loading barebox via tftp without a power cycle of the
whole USB bus and devices.

I tried the following U-Boot commits, but they didn't fix the issue.

- 9713c15d2e6 ("eth: asix88179: Fix ASIX AX88179A PHY hang")
- 1bd136715a4 ("eth: asix88179: reset during probe")

What helps in the situation is physically re-plugging the ethernet
cable, which triggers a link reset in software. This link reset also
happens while opening the device so I'm suspecting a bug in the
controller itself (or there are still some missing reset instructions).
The Realtek RTL8153 chipsets seem handle this situation better, so it is
adviced to use them in a "warm reset" scenario until this is fixed.

Signed-off-by: Stefan Kerkmann <s.kerkmann@pengutronix.de>
---
Stefan Kerkmann (2):
      net: usb: ax88179_178a: align rx_fixup with upstream
      net: usb: ax88179_178a: align reset with upstream

 drivers/net/usb/ax88179_178a.c | 79 ++++++++++++++++++++++++++++++------------
 1 file changed, 57 insertions(+), 22 deletions(-)
---
base-commit: ed59f724912d0b4a86f88fa318836260c88271d1
change-id: 20250813-fix-ax88179_178a-dropped-packets-39cbcd9f2c87

Best regards,
-- 
Stefan Kerkmann <s.kerkmann@pengutronix.de>




             reply	other threads:[~2025-08-13  8:05 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-08-13  7:52 Stefan Kerkmann [this message]
2025-08-13  7:52 ` [PATCH 1/2] net: usb: ax88179_178a: align rx_fixup " Stefan Kerkmann
2025-08-13  7:52 ` [PATCH 2/2] net: usb: ax88179_178a: align reset " Stefan Kerkmann
2025-08-14 11:00 ` [PATCH 0/2] usb: net: ax88179_178a: sync reset and rx " Sascha Hauer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250813-fix-ax88179_178a-dropped-packets-v1-0-2ce605407d0b@pengutronix.de \
    --to=s.kerkmann@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    --cc=s.hauer@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox