Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Mellanox] Upgrade HW-MGMT to 7.0030.2008 and update platform-api #17134

Merged
merged 11 commits into from
Nov 17, 2023
Merged

[Mellanox] Upgrade HW-MGMT to 7.0030.2008 and update platform-api #17134

merged 11 commits into from
Nov 17, 2023

Conversation

vivekrnv
Copy link
Contributor

Why I did it

Add platform support for Debian 12 (Bookworm) on Mellanox Platform

How I did it

  1. Update hw-management to v7.0030.2008
  2. Deprecate the sfp_count == module_count approach in favour of asic init completion
    Ref: Mellanox/hw-mgmt@bf4f593
  3. Add xxd package to base image which is required by hw-management scripts
  4. Add the non-upstream flag into linux kernel cache options
  5. Update the thermalctl logic based on new sysfs attributes
  6. Fix the integrate-mlnx-hw-mgmt script to not populate the arm64 Kconfig

How to verify it

Build kernel and run platform tests

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

vivekrnv and others added 10 commits November 10, 2023 01:51
@vivekrnv
Account non-upstream patch flags in cache hashing
f4a0e64
@vivekrnv
Fix the cache problem in non upstream patches
4feb6e5 
Signed-off-by: Vivek Reddy <vkarri@nvidia.com>
@vivekrnv
include the xxd package
9b299a4 
Signed-off-by: Vivek Reddy <vkarri@nvidia.com>
@Junchao-Mellanox @vivekrnv
[Mellanox] Remove thermal zone related code and replace with new one
bd67ba7
@Junchao-Mellanox @vivekrnv
Revert "Revert "[Mellanox] Align PSU temperature sysfs node name with…
b35c354 
… hw-management change (sonic-net#16820)" (sonic-net#16956)"

This reverts commit c2edc6f.
@vivekrnv
[pmon] Update the platform_wait script to wait on asic init
0feaeb3 
Deprecate the sfp_count == module_count approach in favour of asic init completion
Ref: Mellanox/hw-mgmt@bf4f593

Signed-off-by: Vivek Reddy <vkarri@nvidia.com>
@vivekrnv
Intgerate HW-MGMT 7.0030.2008 Changes (#83)
d0afa87 
## Patch List
* 0042-hwmon-mlxreg-fan-Add-support-for-new-flavour-of-capa.patch :
* 0044-leds-mlxreg-Add-support-for-new-flavour-of-capabilit.patch :
* 0045-leds-mlxreg-Remove-code-for-amber-LED-colour.patch :
* 0046-Extend-driver-to-support-Infineon-Digital-Multi-phas.patch :
* 0047-dt-bindings-trivial-devices-Add-infineon-xdpe1a2g7.patch :
* 0048-hwmon-pmbus-Add-support-for-MPS-Multi-phase-mp2891-c.patch :
* 0049-dt-bindings-trivial-devices-Add-mps-mp2891.patch :
* 0053-platform-mellanox-Add-support-for-dynamic-I2C-channe.patch :
* 0054-platform-mellanox-Introduce-support-for-switches-equ.patch :
* 0055-mellanox-Relocate-mlx-platform-driver.patch :
* 0066-UBUNTU-SAUCE-sdhci-of-dwcmshc-Enable-host-V4-support.patch : torvalds/linux@95921151e043
* 0067-UBUNTU-SAUCE-mmc-sdhci-of-dwcmshc-add-the-missing-de.patch : torvalds/linux@cfd4ea4815d1
* 0068-UBUNTU-SAUCE-mmc-sdhci-of-dwcmshc-Add-runtime-PM-ope.patch :
* 0069-UBUNTU-SAUCE-mlxbf_gige-add-ethtool-mlxbf_gige_set_r.patch :
* 0070-UBUNTU-SAUCE-Fix-OOB-handling-RX-packets-in-heavy-tr.patch :
* 0071-UBUNTU-SAUCE-mlxbf-gige-Fix-intermittent-no-ip-issue.patch :
* 0072-mlxbf_gige-add-MDIO-support-for-BlueField-3.patch : torvalds/linux@2321d69f92aa
* 0073-mlxbf_gige-support-10M-100M-1G-speeds-on-BlueField-3.patch : torvalds/linux@20d03d4d9437
* 0074-mlxbf_gige-add-set_link_ksettings-ethtool-callback.patch :
* 0075-UBUNTU-SAUCE-mlxbf-gige-Fix-kernel-panic-at-shutdown.patch :
* 0076-UBUNTU-SAUCE-platform-mellanox-Updates-to-mlxbf-boot.patch :
* 0077-UBUNTU-SAUCE-mlx-bootctl-support-icm-carveout-eeprom.patch :
* 0078-UBUNTU-SAUCE-mlxbf-bootctl-support-SMC-call-for-sett.patch :
* 0079-UBUNTU-SAUCE-mlxbf-ptm-power-and-thermal-management-.patch :
* 0080-UBUNTU-SAUCE-mlxbf-ptm-update-license.patch :
* 0081-UBUNTU-SAUCE-mlxbf-ptm-use-0444-instead-of-S_IRUGO.patch :
* 0082-UBUNTU-SAUCE-mlxbf-ptm-add-atx-debugfs-nodes.patch :
* 0083-UBUNTU-SAUCE-mlxbf-ptm-update-module-version.patch :
* 0084-UBUNTU-SAUCE-mlxbf-bootctl-Fix-kernel-panic-due-to-b.patch :
* 8000-mlxsw-Use-weak-reverse-dependencies-for-firmware-fla.patch :
* 8004-mlxsw-minimal-Downstream-Ignore-error-reading-SPAD-r.patch :
@vivekrnv
Intgerate HW-MGMT 7.0030.2008 Changes
089b131
@vivekrnv
Intgerate HW-MGMT 7.0030.2008 Changes
9c56c46
@vivekrnv
[Mellanox] Don't populate arm64 Kconfig when integrating hw-mgmt
45227b2 
Signed-off-by: Vivek Reddy <vkarri@nvidia.com>
@vivekrnv vivekrnv changed the title [Mellanox] Upgrade HW-MGMT to 7.0030.2008 and update platform-api accordingly [mellanox] Upgrade HW-MGMT to 7.0030.2008 and update platform-api Nov 10, 2023
This was referenced Nov 14, 2023
Closed
Closed
@liat-grozovik liat-grozovik changed the title [mellanox] Upgrade HW-MGMT to 7.0030.2008 and update platform-api Mellanox] Upgrade HW-MGMT to 7.0030.2008 and update platform-api Nov 14, 2023
@liat-grozovik liat-grozovik changed the title Mellanox] Upgrade HW-MGMT to 7.0030.2008 and update platform-api [Mellanox] Upgrade HW-MGMT to 7.0030.2008 and update platform-api Nov 14, 2023
saiarcot895
saiarcot895 reviewed Nov 15, 2023
View reviewed changes
rules/linux-kernel.dep Outdated
DEP_FLAGS := $(SONIC_COMMON_FLAGS_LIST) \
$(KERNEL_PROCURE_METHOD) $(KERNEL_CACHE_PATH) $(SECURE_UPGRADE_MODE) $(SECURE_UPGRADE_SIGNING_CERT)
DEP_FLAGS := $(SONIC_COMMON_FLAGS_LIST) $(INCLUDE_EXTERNAL_PATCHES) \
$(KERNEL_PROCURE_METHOD) $(KERNEL_CACHE_PATH) $(SECURE_UPGRADE_MODE) $(SECURE_UPGRADE_SIGNING_CERT) \
Copy link
Contributor

Choose a reason for hiding this comment

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

Should the trailing \ be here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will remove

@vivekrnv
Remove trailing slash
993c4e4 
Signed-off-by: Vivek Reddy <vkarri@nvidia.com>
@vivekrnv
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vivekrnv
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yxieca yxieca merged commit 3fc48d5 into sonic-net:bookworm Nov 17, 2023
18 checks passed
saiarcot895 pushed a commit to saiarcot895/sonic-buildimage that referenced this pull request Nov 21, 2023
@vivekrnv @Junchao-Mellanox @saiarcot895
[Mellanox] Upgrade HW-MGMT to 7.0030.2008 and update platform-api (so…
7c5cbd0 
…nic-net#17134)

Why I did it
Add platform support for Debian 12 (Bookworm) on Mellanox Platform

How I did it
Update hw-management to v7.0030.2008
Deprecate the sfp_count == module_count approach in favour of asic init completion
Ref: Mellanox/hw-mgmt@bf4f593
Add xxd package to base image which is required by hw-management scripts
Add the non-upstream flag into linux kernel cache options
Update the thermalctl logic based on new sysfs attributes
Fix the integrate-mlnx-hw-mgmt script to not populate the arm64 Kconfig
How to verify it
Build kernel and run platform tests

Signed-off-by: Vivek Reddy <vkarri@nvidia.com>
Co-authored-by: Junchao-Mellanox <junchao@nvidia.com>
Co-authored-by: Junchao-Mellanox <57339448+Junchao-Mellanox@users.noreply.github.com>
yxieca pushed a commit that referenced this pull request Nov 22, 2023
@vivekrnv @Junchao-Mellanox @yxieca
[Mellanox] Upgrade HW-MGMT to 7.0030.2008 and update platform-api (#1…
787dd72 
…7134)

Why I did it
Add platform support for Debian 12 (Bookworm) on Mellanox Platform

How I did it
Update hw-management to v7.0030.2008
Deprecate the sfp_count == module_count approach in favour of asic init completion
Ref: Mellanox/hw-mgmt@bf4f593
Add xxd package to base image which is required by hw-management scripts
Add the non-upstream flag into linux kernel cache options
Update the thermalctl logic based on new sysfs attributes
Fix the integrate-mlnx-hw-mgmt script to not populate the arm64 Kconfig
How to verify it
Build kernel and run platform tests

Signed-off-by: Vivek Reddy <vkarri@nvidia.com>
Co-authored-by: Junchao-Mellanox <junchao@nvidia.com>
Co-authored-by: Junchao-Mellanox <57339448+Junchao-Mellanox@users.noreply.github.com>
@vivekrnv vivekrnv mentioned this pull request Apr 15, 2024
Merged
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@saiarcot895 saiarcot895 saiarcot895 approved these changes

@dgsudharsan dgsudharsan dgsudharsan approved these changes

@Junchao-Mellanox Junchao-Mellanox Junchao-Mellanox approved these changes

@qiluo-msft qiluo-msft Awaiting requested review from qiluo-msft qiluo-msft is a code owner

@xumia xumia Awaiting requested review from xumia xumia is a code owner

@lguohan lguohan Awaiting requested review from lguohan lguohan is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@vivekrnv @mssonicbld @saiarcot895 @dgsudharsan @Junchao-Mellanox @yxieca