Skip to content

Conversation

@dprital
Copy link
Collaborator

@dprital dprital commented Oct 12, 2025

Why I did it

Integrate HW-MGMT 7.0040.4034 Changes

Patch List

  • 8013-hwmon-pmbus-mp2975-Clear-interrupts-at-probe.patch :
  • 8014-platform-mellanox-mlxreg-dpu-Debug-trace-for-dpu-pro.patch :

How I did it

Run make integrate-mlnx-hw-mgmt

 ## Patch List
* 8013-hwmon-pmbus-mp2975-Clear-interrupts-at-probe.patch :
* 8014-platform-mellanox-mlxreg-dpu-Debug-trace-for-dpu-pro.patch :
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dprital dprital changed the title Integrate HW-MGMT Version 7.0040.4034 [202505][Mellanox] Integrate HW-MGMT Version 7.0040.4034 Oct 12, 2025
@dprital dprital closed this Oct 15, 2025
@dprital dprital reopened this Oct 15, 2025
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dprital dprital marked this pull request as ready for review October 15, 2025 15:02
@dprital dprital requested a review from a team as a code owner October 15, 2025 15:02
@dprital dprital requested a review from keboliu October 16, 2025 11:17
@lguohan lguohan requested a review from Copilot October 19, 2025 00:55
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Integrates two new Mellanox HW management patches: one adds debug trace messages to mlxreg-dpu probe path, and another clears mp2975 PMBus faults during probe to avoid spurious ALARM conditions.

  • Adds 8013 patch to clear mp2975 faults (labeled as interrupts) at probe.
  • Adds 8014 patch to introduce probe-time logging for mlxreg-dpu.
  • Updates patch/series to include the new patch entries.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
patch/series Registers the two new Mellanox HW-MGMT patches in build sequence.
patch/8014-platform-mellanox-mlxreg-dpu-Debug-trace-for-dpu-pro.patch Adds probe-time informational logs and deferred probe trace in mlxreg-dpu.c.
patch/8013-hwmon-pmbus-mp2975-Clear-interrupts-at-probe.patch Introduces function to clear PMBus fault registers during mp2975 device probe.
Comments suppressed due to low confidence (1)

patch/8014-platform-mellanox-mlxreg-dpu-Debug-trace-for-dpu-pro.patch:1

  • [nitpick] Using dev_info for routine probe and deferred retry messages can produce noisy kernel logs; since the patch subject calls this a "Debug trace", dev_dbg would be more appropriate (e.g. dev_dbg(&pdev->dev, ...)) to allow conditional visibility under dynamic debug.
From e7838edd2acc793f3b43ab04b3693a340273c330 Mon Sep 17 00:00:00 2001

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

From 8d8f4d52f5655747df20317ceaf628e29070475e Mon Sep 17 00:00:00 2001
From: Ciju Rajan K <[email protected]>
Date: Tue, 9 Sep 2025 08:48:47 +0300
Subject: hwmon: pmbus: mp2975: Clear interrupts at probe
Copy link

Copilot AI Oct 19, 2025

Choose a reason for hiding this comment

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

Both the patch subject and the in-code comment refer to "interrupts" while the implementation clears fault registers (CLEAR_FAULTS and CLEAR_CAT_FAULTS); adjust wording to "Clear faults at probe" to accurately reflect behavior.

Copilot uses AI. Check for mistakes.
if (ret)
return ret;

+ /* Clear interrupts. */
Copy link

Copilot AI Oct 19, 2025

Choose a reason for hiding this comment

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

Both the patch subject and the in-code comment refer to "interrupts" while the implementation clears fault registers (CLEAR_FAULTS and CLEAR_CAT_FAULTS); adjust wording to "Clear faults at probe" to accurately reflect behavior.

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants