Skip to content

Integrate HW-MGMT Version 7.0060.1047#584

Open
DannyIsa wants to merge 2 commits into
sonic-net:masterfrom
DannyIsa:master_d58f058_integrate_7.0060.1047_2026-05-31
Open

Integrate HW-MGMT Version 7.0060.1047#584
DannyIsa wants to merge 2 commits into
sonic-net:masterfrom
DannyIsa:master_d58f058_integrate_7.0060.1047_2026-05-31

Conversation

@DannyIsa
Copy link
Copy Markdown

@DannyIsa DannyIsa commented May 31, 2026

Why I did it

Integrate HW-MGMT 7.0060.1047 Changes

Patch List

  • 0060-leds-mlxreg-Provide-conversion-for-hardware-LED-colo.patch :
  • 0060-platform-mellanox-nvsw-bmc-Downstream-Add-protection.patch :
  • 0061-platform-mellanox-mlxreg-io-Increase-max-supported-a.patch :
  • 0062-Add-support-for-NXP-s-PCF85053A-RTC-chip.patch :
  • 0063-reset-phy-using-polling-function-not-register-write.patch :

BMC Patch List

  • 0001-mctp-driver-net-Extend-MCTP-support.patch :
  • 0002-mctp-aspeed-IRoT-Add-initial-support.patch :
  • 0003-mctp-driver-net-Add-global-APIs-for-MCTP.patch :
  • 0004-mctp-driver-net-Fix-MCTP-over-VDM-PCIe.patch :
  • 0005-mctp-driver-net-Fix-MCTP-over-USB.patch :
  • 0006-jtag-Add-JTAG-core-headers-GPIO-mux-and-locking-support.patch :
  • 0007-JTAG-Aspeed-Fix-kernel-configuration.patch :

Signed-off-by: Daniel Isakov disakov@nvidia.com

How I did it

Run make integrate-mlnx-hw-mgmt

@DannyIsa DannyIsa marked this pull request as ready for review May 31, 2026 08:37
@DannyIsa DannyIsa requested a review from a team as a code owner May 31, 2026 08:37
@mssonicbld
Copy link
Copy Markdown

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Contributor

@paulmenzel paulmenzel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the merge/pull requests. Some notes:

Bots cannot sign off anything. Please avoid this:

Signed-off-by: sw-r2d2-bot sw-r2d2-bot@nvidia.com

Additionally, it’d be great if the upstream status of the commits could be more clear. I assume they have been sent for review. Could you maybe add the URLs of lore.kernel.org?

Comment thread config.local/arm64/config.sonic-aspeed

From 9f11c71d770ea5a66ef3a4e6bea134aaaf62adaa Mon Sep 17 00:00:00 2001
From: Aaron Komisar <akomisar@nvidia.com>
Date: Thu, 28 Aug 2025 20:57:42 +0300
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the upstream status really pending?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Those patches are not up-streamed and are not sent for review, so marked as pending.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Strange over half a year later.

@DannyIsa DannyIsa force-pushed the master_d58f058_integrate_7.0060.1047_2026-05-31 branch from de20d0d to e9de66b Compare June 1, 2026 17:42
@mssonicbld
Copy link
Copy Markdown

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

DannyIsa added 2 commits June 2, 2026 14:25
These patches are not yet upstreamed and have not been sent for review,
hence marked Upstream-Status: Pending.

Signed-off-by: Daniel Isakov <disakov@nvidia.com>
Signed-off-by: Daniel Isakov <disakov@nvidia.com>
@DannyIsa DannyIsa force-pushed the master_d58f058_integrate_7.0060.1047_2026-05-31 branch from e9de66b to aa30808 Compare June 2, 2026 11:26
@mssonicbld
Copy link
Copy Markdown

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@DannyIsa DannyIsa requested a review from paulmenzel June 2, 2026 11:28
@DannyIsa
Copy link
Copy Markdown
Author

DannyIsa commented Jun 3, 2026

@paulmenzel

Could you maybe add the URLs of lore.kernel.org?

All these patches were not sent for review yet.

Copy link
Copy Markdown
Contributor

@paulmenzel paulmenzel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some more comments. Feel free to mark them as resolved.

Date: Thu, 23 Apr 2026 21:40:40 +0300
Subject: [PATCH 6.12 1/1] JTAG: Aspeed: Fix kernel configuration

Change Kconfig and Makefile.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The reasoning/motivation is missing.

Comment thread config.local/arm64/config.sonic-aspeed
Date: Sun, 10 May 2026 14:37:56 +0300
Subject: [PATCH 6.12 4/5] mctp: driver: net: Fix MCTP over VDM/PCIe

Add fixes for MCTP over VDM/PCIe (option MCTP_TRANSPORT_PCIE_VDM).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is also too short and does state the problems fixed/motivation.

From d380e09439f8576e4a89dd1f5c32850ce1c14d62 Mon Sep 17 00:00:00 2001
From: Vadim Pasternak <vadimp@nvidia.com>
Date: Thu, 2 Apr 2026 12:24:00 +0300
Subject: [PATCH BMC Realtek PHY 1/1] From
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With the lines below this looks strangely formatted.


replace the code to reset the phy with a call to a
function that resets the phy and waits for the reset to be done,
preventing race conditions.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which exactly?

return 0;
}

+#ifdef CONFIG_RTL_RGMII_SGMII_3809
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use if in C and not preprocessor?

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.

4 participants