Skip to content

[PDDF] HLD Add new PSU Thermal fields #2022

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 13 additions & 0 deletions doc/platform/brcm_pdk_pddf.md
Original file line number Diff line number Diff line change
@@ -601,7 +601,20 @@ psu_i_out
psu_p_out
psu_fan1_speed_rpm
psu_temp1_input
psu_temp1_high_threshold
```
The following fields are optional and can be added via `pddf-device.json` to support up to 3 temperature sensors in the PSU.

```
psu_temp2_input
psu_temp2_high_threshold
psu_temp3_input
psu_temp3_high_threshold
```
Checks are added in the PSU driver to:
* Skip creating `psu_temp*_input` and `psu_temp*_high_threshold` SysFS paths if index exceeds the `num_psu_thermals` specified in the JSON file
* Only create `psu_temp*_high_threshold` SysFS paths if the attribute is explicitly specified in JSON file

##### 3.4.3.2 PSU JSON Design
PSU JSON is structured to include the access-data for all the supported SysFS attributes.
*attr_list* is an array object which stores the array of access-data for multiple attributes. If some of the field in the attribute object is not applicable to some particular attribute, it can be left and not filled.