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 janga SMB & COME card pvt sensor config #313

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

Conversation

zhongedward
Copy link
Contributor

@zhongedward zhongedward commented Dec 23, 2024

Description

This PR is for janga SMB & COME card pvt sensor config.

Motivation

1.According to the factory they have added SMB card '3 5 20' about productionstate, productversion,productsubversion.
COME card '4 1 1','4 1 2' about productionstate, productversion,productsubversion.

image

2.So i have added the following configs.

image

3.Note: I still put
SMB_U68_TMP422
SMB_U150_TMP422
SMB_U351_TMP422
SMB_U352_TMP422
SMB_U152_TMP422
them in SMB_FRU_SLOT because they can be recognized in SMB eeprom can not Recognized in JANGA eeprom.I have manually change the SMB eeprom to '3 5 20' it tested ok, if you put them in JANGA slot it tested fail because they belongs to SMB board.

image image

In the hardware table, they belongs to SMB board
https://docs.google.com/spreadsheets/d/1r0Vbv1cx3KelYLEM4vl0VNyIeEpFwurG5JJZ6BckMCI/edit?gid=1519693354#gid=1519693354
image

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:
janga_main_source_platform_test_log_11_28.txt
janga_main_source_sensor_test_log_11_28.txt

@zhongedward zhongedward changed the title added janga pvt sensor config janga: config: added janga SMB & COME card pvt sensor config Dec 23, 2024
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.

2 participants