Skip to content

Pull Request - lan743x driver commits for 26.04-linux-nvidia-bos#417

Closed
dthompso wants to merge 3 commits into
NVIDIA:26.04_linux-nvidia-bosfrom
dthompso:dthompso-lan743x-26.04-bos
Closed

Pull Request - lan743x driver commits for 26.04-linux-nvidia-bos#417
dthompso wants to merge 3 commits into
NVIDIA:26.04_linux-nvidia-bosfrom
dthompso:dthompso-lan743x-26.04-bos

Conversation

@dthompso
Copy link
Copy Markdown

These three upstream patches provide fixes to the Microchip lan743x driver,
plus add capability for using "ethtool -r" to restart autonegotiation on
the interface.

469faa5 net: lan743x: rename chip_rev to fpga_rev
e783e40 net: lan743x: fix SGMII detection on PCI1xxxx B0+ during warm reset
8636385 net: microchip: lan743x: add ethtool nway_reset support

These patches picked clean and were tested on BlueField-4 hardware,
which uses the board-level Microchip LAN743x device.

LP: https://bugs.launchpad.net/ubuntu/+source/linux-nvidia-6.17/+bug/2152064

nbuchwitz and others added 3 commits May 11, 2026 20:02
Wire phylink_ethtool_nway_reset() as the .nway_reset ethtool operation,
allowing userspace to restart PHY autonegotiation via 'ethtool -r'.

Signed-off-by: Nicolai Buchwitz <nb@tipi-net.de>
Reviewed-by: Russel King (Oracle) <rmk+kernel@armlinux.org.uk>
Link: https://patch.msgid.link/20260223085442.42852-1-nb@tipi-net.de
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
(cherry picked from commit 8636385)
Signed-off-by: David Thompson <davthompson@nvidia.com>
A warm reset on boards using an EEPROM-only strap configuration (where
no MAC address is set in the image) can cause the driver to incorrectly
revert to RGMII mode. This occurs because the ENET_CONFIG_LOAD_STARTED
bit may not persist or behave as expected.

Update pci11x1x_strap_get_status() to use revision-specific validation:

- For PCI11x1x A0: Continue using the legacy check (config load started
  or reset protection) to validate the SGMII strap.
- For PCI11x1x B0 and later: Use the newly available
  STRAP_READ_USE_SGMII_EN_ bit in the upper strap register to validate
  the lower SGMII_EN bit.

This ensures the SGMII interface is correctly identified even after a
warm reboot.

Signed-off-by: Thangaraj Samynathan <thangaraj.s@microchip.com>
Link: https://patch.msgid.link/20260318063228.17110-1-thangaraj.s@microchip.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
(cherry picked from commit e783e40)
Signed-off-by: David Thompson <davthompson@nvidia.com>
The variable chip_rev stores the value read from the FPGA_REV
register and represents the FPGA revision. Rename it to fpga_rev
to better reflect its meaning.

No functional change intended.

Signed-off-by: Thangaraj Samynathan <thangaraj.s@microchip.com>
Link: https://patch.msgid.link/20260410085710.9246-1-thangaraj.s@microchip.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
(cherry picked from commit 469faa5)
Signed-off-by: David Thompson <davthompson@nvidia.com>
@github-actions
Copy link
Copy Markdown
Contributor

PR Validation Report

Patchscan ✅ No Missing Fixes

All cherry-picked commits checked — no missing upstream fixes found.

PR Lint ⚠️ Warnings

Details
Checking 3 commits...

Cherry-pick digest:
┌──────────────┬──────────────────────────────────────────────────────────────────┬────────────┬─────────┬───────────────────────────┐
│ Local        │ Referenced upstream / Patch subject                              │ Patch-ID   │ Subject │ SoB chain                 │
├──────────────┼──────────────────────────────────────────────────────────────────┼────────────┼─────────┼───────────────────────────┤
│ 42d8db82d837 │ 469faa546e7a net: lan743x: rename chip_rev to fpga_rev           │ match      │ match   │ preserved + davthomp adde │
├──────────────┼──────────────────────────────────────────────────────────────────┼────────────┼─────────┼───────────────────────────┤
│ 9a4c7439d601 │ e783e40fb689 net: lan743x: fix SGMII detection on PCI1xxxx B0+ d │ match      │ match   │ preserved + davthomp adde │
├──────────────┼──────────────────────────────────────────────────────────────────┼────────────┼─────────┼───────────────────────────┤
│ ebd241aef1b0 │ 8636385b9f01 net: microchip: lan743x: add ethtool nway_reset sup │ match      │ match   │ preserved + davthomp adde │
└──────────────┴──────────────────────────────────────────────────────────────────┴────────────┴─────────┴───────────────────────────┘

Lint: all checks passed.

PR metadata:
W: PR title missing [<branch>] prefix: "Pull Request - lan743x driver commits for 26.04-linux-nvidia-bos"

@nvmochs
Copy link
Copy Markdown
Collaborator

nvmochs commented May 11, 2026

No issues with the backports.

Acked-by: Matthew R. Ochs <mochs@nvidia.com>

@clsotog
Copy link
Copy Markdown
Collaborator

clsotog commented May 11, 2026

Acked-by: Carol L Soto <csoto@nvidia.com>

@nvmochs
Copy link
Copy Markdown
Collaborator

nvmochs commented May 12, 2026

Merged, closing PR.

47ccce2c91e0 (nresolute/nvidia-bos-next) net: lan743x: rename chip_rev to fpga_rev
fe9b9a42d3ed net: lan743x: fix SGMII detection on PCI1xxxx B0+ during warm reset
53406eb4b45a net: microchip: lan743x: add ethtool nway_reset support

@nvmochs nvmochs closed this May 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants