From: Birger Koblitz <mail@birger-koblitz.de>
To: Emir Beganovic <beganovic.emir@gmail.com>
Cc: Andrew Lunn <andrew+netdev@lunn.ch>,
"David S. Miller" <davem@davemloft.net>,
Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
Russell King <linux@armlinux.org.uk>,
Andrew Lunn <andrew@lunn.ch>,
Heiner Kallweit <hkallweit1@gmail.com>,
Huacai Chen <chenhuacai@kernel.org>,
WANG Xuerui <kernel@xen0n.name>,
Eric Dumazet <edumazet@kernel.org>,
Jianhui Xu <neuromoments@gmail.com>,
Nicolai Buchwitz <nb@tipi-net.de>,
linux-usb@vger.kernel.org, netdev@vger.kernel.org,
linux-kernel@vger.kernel.org, loongarch@lists.linux.dev
Subject: Re: [PATCH net-next v19 00/13] ax88179_178a: Add support for AX88179A-based chips
Date: Fri, 30 Oct 2026 16:42:44 +0100 [thread overview]
Message-ID: <2a66e94c-0fd0-4c90-bc1c-9e5699f80737@birger-koblitz.de> (raw)
In-Reply-To: <20261029005809.19405-1-beganovic.emir@gmail.com>
Thanks so much Emir for testing!
On 29/10/2026 1:58 am, Emir Beganovic wrote:
> Hi Birger,
>
> I tested v19 on a Dell DA305 USB-C adapter. It has an AX88179A
> (0b95:1790, bcdDevice 0x0200, FW 1.0.7.131) behind a VIA VL822 hub,
> plugged into a Dell XPS 16 (Panther Lake). The peer is an Intel I225-V
> behind a switch.
>
> I built ax88179, libphy, phylink and ax88796b as modules for a 7.2.8
> kernel. v19 needed two things on that base: a4e22e9253e7 and a backport
> of genphy_{read,write}_mmd_c45() from the adin1140 series.
>
> With the current driver in configuration 1 this adapter never gets a
> link ("Failed to read reg index 0x0040: -32"). With v19:
>
> - 1000/full link with flow control
> - download 940-941 Mbit/s in nine runs. Upload was 937-941 in five of
> nine and lower in the other four, and I couldn't find why. Eight
> later runs with tracing on all averaged about 932.
I have seen similar behaviour with iperf3 for an AX88279 and a 2.5GBit
link: in the first round, some 4-10 packets get dropped by the
RTL8372-switch as a link partner and then no further frame drops happen.
Looking into the switch's error counts, the drops are due to Flow
Control. I was not able to investigate further, but I have seen the same
behaviour with RTL8156/7/9-adapters using the r8152 driver and that
switch. So that seems to be "normal" and the speed of the adapter in
iperf3 is very close (about 2.35Mbit/s) to the theoretical limit.
> - 1-byte TCP request/response around 190-200 us, against about 1.46 ms
> with cdc_ncm on the same port (its default 400 us TX timer)
> - switching itself to configuration 1 when the device comes up in the
> CDC configuration
> - 100M and 10M links
> - VLAN with rx-vlan-offload on and off, and rx-vlan-filter dropping a
> VID that isn't configured
> - multicast filtering and allmulti
> - suspend/resume (s2idle) and wake-on-LAN with a magic packet
>
> Two small things I noticed:
>
> - "Switching to vendor mode" is printed twice, once per CDC interface,
> and before the netdev is registered, so it shows up as
> "(unnamed net_device) (uninitialized)".
The device can be registered only once it has been switched from CDC to
the vendor mode, that is why you see the message for the unregistered
device. That you see it twice can be due to the fact that it has 2
active interfaces, see the explanation given in commit c67cc43 for Anker
adapters.
The devices I have print it only once and that also disappears once the
device has been switched and stays plugged even if the module is
loaded/unloaded.
> - After a WoL wake the link goes down and up twice within about 2.5 s,
> where a normal resume shows a single link up. It comes back fine
> either way.
I see this, too. It is not triggered in any way by the host or the link
partner, I checked. It appears that this is due to the firmware on the
device trying to optimize the link settings further, the out-of tree
ASIX driver displays the same behaviour, although it works very differently.
>
> I only have this one adapter, so nothing on the other chips.
>
> Tested-by: Emir Beganovic <beganovic.emir@gmail.com>
Will add in upcoming v20.
Birger
next prev parent reply other threads:[~2026-10-30 15:42 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-10 4:54 Birger Koblitz
2026-10-10 4:54 ` [PATCH net-next v19 01/13] phylink: Add phylink_mac_interrupt Birger Koblitz
[not found] ` <sashiko-outbox-167008@kernel.org>
2026-10-11 5:32 ` Birger Koblitz
2026-10-20 0:18 ` Jakub Kicinski
2026-10-20 3:50 ` Birger Koblitz
2026-10-10 4:54 ` [PATCH net-next v19 02/13] phylib: Add support for PHYs with broken forced mode Birger Koblitz
2026-10-10 4:54 ` [PATCH net-next v19 03/13] ax88179_178a: Split driver into library and device specific code Birger Koblitz
2026-10-10 4:54 ` [PATCH net-next v19 04/13] ax88179_178a: Add HW support for AX179A-based chips Birger Koblitz
2026-10-10 4:54 ` [PATCH net-next v19 05/13] ax88179_178a: Add EEE configuration support for AX88179A MACs Birger Koblitz
2026-10-10 4:54 ` [PATCH net-next v19 06/13] ax88179_178a: Add EEE configuration support for AX88179A PHYs Birger Koblitz
2026-10-10 4:54 ` [PATCH net-next v19 07/13] ax88179_178a: Add VLAN offload support for AX88179A Birger Koblitz
2026-10-10 4:54 ` [PATCH net-next v19 08/13] ax88179_178a: Add AX179A/AX279 multicast configuration Birger Koblitz
2026-10-10 4:54 ` [PATCH net-next v19 09/13] ax88179_178a: Add Suspend/resume support for AX88179A/772D/279 Birger Koblitz
2026-10-10 4:54 ` [PATCH net-next v19 10/13] ax88179_178a: Add ethtool get_drvinfo Birger Koblitz
2026-10-10 4:54 ` [PATCH net-next v19 11/13] ax88179_178a: Update driver name and information Birger Koblitz
2026-10-10 4:54 ` [PATCH net-next v19 12/13] ax88179_178a: Add support for AX88179A/772D/279 EEPROM access Birger Koblitz
2026-10-10 4:54 ` [PATCH net-next v19 13/13] ax88796b: Add support for AX88772D, AX88179A and AX88279 Birger Koblitz
2026-10-10 6:20 ` [PATCH v19 0/13] ax88179_178a: Add support for AX88179A-based chips Jianhui Xu
2026-10-10 7:42 ` Birger Koblitz
2026-10-29 0:58 ` [PATCH net-next v19 00/13] " Emir Beganovic
2026-10-30 15:42 ` Birger Koblitz [this message]
2026-10-30 16:47 ` Andrew Lunn
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=2a66e94c-0fd0-4c90-bc1c-9e5699f80737@birger-koblitz.de \
--to=mail@birger-koblitz.de \
--cc=andrew+netdev@lunn.ch \
--cc=andrew@lunn.ch \
--cc=beganovic.emir@gmail.com \
--cc=chenhuacai@kernel.org \
--cc=davem@davemloft.net \
--cc=edumazet@kernel.org \
--cc=hkallweit1@gmail.com \
--cc=kernel@xen0n.name \
--cc=kuba@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=loongarch@lists.linux.dev \
--cc=nb@tipi-net.de \
--cc=netdev@vger.kernel.org \
--cc=neuromoments@gmail.com \
--cc=pabeni@redhat.com \
/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
all inboxes | Powered by JetHome®