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.

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

Janga: config: added 10 tpm422 overtemp config in fan service file #356

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

zhongedward
Copy link
Contributor

Description
This PR added 10 tmp422 overtemp config for janga fan service.

Motivation

1.Added 10 tmp422 nodes to the fan service config. Threshold is 110 degrees and will run shoutdown cmd to shut down the J3A and J3B ASIC in case of over-temperature.
SMB_U68_TMP422_J3_A_TEMPDIODE_NIF1
SMB_U68_TMP422_J3_A_TEMPDIODE_HBM_PHY0
SMB_U150_TMP422_J3_A_TEMPDIODE_FAB1
SMB_U150_TMP422_J3_A_TEMPDIODE_HBM_PHY2
SMB_U351_TMP422_J3_B_TEMPDIODE_PADS
SMB_U351_TMP422_J3_B_TEMPDIODE_NIF0
SMB_U352_TMP422_J3_B_TEMPDIODE_NIF1
SMB_U352_TMP422_J3_B_TEMPDIODE_HBM_PHY0
SMB_U152_TMP422_J3_B_TEMPDIODE_FAB1
SMB_U152_TMP422_J3_B_TEMPDIODE_HBM_PHY2

image

2.If run the shutdown cmd i2cset -y -f "fbiob iob_i2c_master.1009 at 0xfb504900" 0x33 0x07 0x9f, Only shut down the J3A & J3B ASIC not the entire system.

Test Plan
1.The correctness of the format has been verified on this website https://jsonlint.com/
2.Used jq cmd to pretty the format.
3.Test log as follows:

sensor service read temp log:
image

fan service read temp log:
image

read reg value after shutdown:
image

The normal lspci:
14:05.0 PCI bridge: Intel Corporation Device 347d (rev 06)
15:00.0 Ethernet controller: Broadcom Inc. and subsidiaries Device 8890 (rev 02) ---->is J3A
16:00.0 Non-Volatile memory controller: Micron Technology Inc 7450 PRO NVMe SSD (rev 01)
17:00.0 Communication controller: Facebook, Inc. IO Bridge (rev 03)
18:00.0 Ethernet controller: Broadcom Inc. and subsidiaries Device 8890 (rev 02) ---->is J3B
fe:00.0 System peripheral: Intel Corporation Device 3450

After run the shutdown cmd:

image

sensors_110.txt
fan_test_110.txt
read_reg_110.txt

@zhongedward zhongedward changed the title janga: config: added 10 tpm422 overtemp config in fan service file Janga: config: added 10 tpm422 overtemp config in fan service file Mar 21, 2025
Copy link

@Abdullah4345 Abdullah4345 left a comment

Choose a reason for hiding this comment

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

Nice

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