configurations: catalina_pdb: Use current reading for cable tsense

The Catalina system previously used shunt voltage (`in0`) to calculate
the NCT thermistor value for the cable temperature sensor. H

configurations: catalina_pdb: Use current reading for cable tsense

The Catalina system previously used shunt voltage (`in0`) to calculate
the NCT thermistor value for the cable temperature sensor. However, the
hwmon driver represents voltage in millivolts (mV), while the shunt
voltage for the cable tsense is typically less than 1 mV.

Switch to using the current (`curr1`) reading to compute the NCT
thermistor value, providing more accurate and reliable temperature
measurements for the cable tsense sensors.

Signed-off-by: Potin Lai <potin.lai@quantatw.com>
Change-Id: I8a7db00a7dcb209a3b739bf6d53b219c06671f0c

show more ...

configurations: yosemite4: Revise MEDUSA voltage sensor

Revise MEDUSA voltage sensors: INA238, ISL28022, RTQ6056.
1. Change RTQ6056 to read shunt voltage.
2. Revise voltage scale and threshold.
3. A

configurations: yosemite4: Revise MEDUSA voltage sensor

Revise MEDUSA voltage sensors: INA238, ISL28022, RTQ6056.
1. Change RTQ6056 to read shunt voltage.
2. Revise voltage scale and threshold.
3. Add "in1_Min" and "in_voltage0_Min" in schemas/legacy.json.

Tested:
Check sensor reading of INA238, ISL28022, RTQ6056.
They all work fine.

Change-Id: I61ec1d6f98a1e5051486e2871b24e70f899e59ab
Signed-off-by: Yikai Tsai <yikai.tsai.wiwynn@gmail.com>

show more ...

configurations: catalina: add scale and offset for P12V & P48V sensors

Add scale and offset for calibration on below sensors
- PDB_P12V_NIC0_CURR_A
- PDB_P12V_NIC0_PWR_W
- PDB_P12V_NIC1_CURR_A
- PDB

configurations: catalina: add scale and offset for P12V & P48V sensors

Add scale and offset for calibration on below sensors
- PDB_P12V_NIC0_CURR_A
- PDB_P12V_NIC0_PWR_W
- PDB_P12V_NIC1_CURR_A
- PDB_P12V_NIC1_PWR_W
- PDB_P12V_SCM_CURR_A
- PDB_P12V_SCM_PWR_W
- PDB_P48V_HSC1_CURR_A
- PDB_P48V_HSC1_PWR_W
- PDB_P48V_HSC2_CURR_A
- PDB_P48V_HSC2_PWR_W

Signed-off-by: Potin Lai <potin.lai@quantatw.com>
Change-Id: I7d60ae27d043fb760b38bc198e39b217ad45f48e

show more ...

configuration: system1: Add GPIO monitoring type

Some GPIOs do not support event_read method to monitor for value
changes. Added a new MonitorType field to specify if the GPIO
needs to be manually p

configuration: system1: Add GPIO monitoring type

Some GPIOs do not support event_read method to monitor for value
changes. Added a new MonitorType field to specify if the GPIO
needs to be manually polled.

Tested:
- Verified no change when MonitorType not specified
- Verified functionality when Polling enum was specified

Change-Id: Ie5e8a2afe43d05192828292b629baade6b0c9c82
Signed-off-by: Chris Cain <cjcain@us.ibm.com>

show more ...

configurations: yosemite4: add power reading of Medusa 12V delta modules

Add power reading of Medusa 12V delta modules for sensor monitoring.

Tested:
Checked all the power reading of 12V delta modu

configurations: yosemite4: add power reading of Medusa 12V delta modules

Add power reading of Medusa 12V delta modules for sensor monitoring.

Tested:
Checked all the power reading of 12V delta modules are normal.

Change-Id: Idebc0c37ac5a66b7dfb01a00c3d24dc426f4a62c
Signed-off-by: Ricky CX Wu <ricky.cx.wu.wiwynn@gmail.com>

show more ...

schemas: legacy: Extend PSUSensor properties

PSUSensor now supports a new sensor type named smpro_hwmon and the
sensor can report more sensors inside its sysfs. This commit extends the
properties s

schemas: legacy: Extend PSUSensor properties

PSUSensor now supports a new sensor type named smpro_hwmon and the
sensor can report more sensors inside its sysfs. This commit extends the
properties so the sensor can be configured in Entity-Manager with full
support for the component sensors.

Tested: on Mt.Jade platform with config "Type": "smpro_hwmon" and
necessary _Min, _Max, _Name, _Scale configs
=> The CI build passes and the sensor can be created successfully on
D-Bus.

Signed-off-by: Chau Ly <chaul@amperecomputing.com>
Change-Id: Ica45569eb349169efa3032267b4bfd4368e7aadd

show more ...

configurations: minerva_cmm: add calibration for HSC sensor

Add the calibration formula for the HSC sensor according to the power
team's measurement result.

Change-Id: I79ec9c74dc4d25204748b3883ef3

configurations: minerva_cmm: add calibration for HSC sensor

Add the calibration formula for the HSC sensor according to the power
team's measurement result.

Change-Id: I79ec9c74dc4d25204748b3883ef377d1606db201
Signed-off-by: Yang Chen <yang.chen@quantatw.com>

show more ...

configurations: catalina_pdb: add max/min value for vdrop sensor

We noticed that the vdrop sensor values ​​were not updating correctly
and we discovered this was due to incorrect MaxValue and MinVal

configurations: catalina_pdb: add max/min value for vdrop sensor

We noticed that the vdrop sensor values ​​were not updating correctly
and we discovered this was due to incorrect MaxValue and MinValue
settings.
The value of the vdrop sensor is usually between 1 and -1 volts. Add
in0_Max and in0_Mix definitions for the vdrop sensor.

Tested results:
```
root@bmc:~# busctl introspect xyz.openbmc_project.PSUSensor /xyz/openbmc_project/sensors/voltage/PDB_POSITIVE_VDROP_VOLT_V xyz.openbmc_project.Sensor.Value
NAME TYPE SIGNATURE RESULT/VALUE FLAGS
.MaxValue property d 1 emits-change
.MinValue property d -1 emits-change
.Unit property s "xyz.openbmc_project.Sensor.Value.Unit.… emits-change
.Value property d 0.104 emits-change writable
root@bmc:~# busctl introspect xyz.openbmc_project.PSUSensor /xyz/openbmc_project/sensors/voltage/PDB_RETURN_VDROP_VOLT_V xyz.openbmc_project.Sensor.Value
NAME TYPE SIGNATURE RESULT/VALUE FLAGS
.MaxValue property d 1 emits-change
.MinValue property d -1 emits-change
.Unit property s "xyz.openbmc_project.Sensor.Value.Unit.… emits-change
.Value property d 0.107 emits-change writable
```

Signed-off-by: Potin Lai <potin.lai@quantatw.com>
Change-Id: I37125808a772d0e12fcf93a56a161b573c09135a

show more ...

json: adjust definitions alias

JSON Schema suggests[1] using "$defs" rather than "definitions" as
the location for reusable definitions in a schema. While, this
isn't strictly required, there is so

json: adjust definitions alias

JSON Schema suggests[1] using "$defs" rather than "definitions" as
the location for reusable definitions in a schema. While, this
isn't strictly required, there is some implication that using it
lends to better schema validation error reporting[2].

Adjust all the schema files to use "$defs" rather than "definitions".

[1]: https://json-schema.org/understanding-json-schema/structuring#defs
[2]: https://github.com/orgs/json-schema-org/discussions/151

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I4b23890950bd2ebb7a7de99fd911fb3c9e0a4dbd

show more ...

configurations: yosemite4: Add PowerState to Medusa Board

To let psusensor get real value of MEDUSA_MBX_ sensors while
corresponding chassis power state is ON.
Add "SlotId" in schemas/legacy.json to

configurations: yosemite4: Add PowerState to Medusa Board

To let psusensor get real value of MEDUSA_MBX_ sensors while
corresponding chassis power state is ON.
Add "SlotId" in schemas/legacy.json to indicate corresponding chassis
slot number.

Test plan:
1. Modify property "CurrentPowerState" to
xyz.openbmc_project.State.Chassis.PowerState.Off, the sensor value of
corresponding slot should be nan.
2. Change property "CurrentPowerState" to
xyz.openbmc_project.State.Chassis.PowerState.Off, the sensor value of
corresponding slot should be real value.

Change-Id: I2d3a5f33a6091b69cb6a78357692e6066a72dcbc
Signed-off-by: Bright Cheng <bright.cheng.wiwynn@gmail.com>

show more ...

configurations: yosemite4: Support second sources Spider board

To support second sources spider board
Test Case: sensor config and sensors can be loaded and read.

Change-Id: Ic28f2fd4a9942ef2b1aa69

configurations: yosemite4: Support second sources Spider board

To support second sources spider board
Test Case: sensor config and sensors can be loaded and read.

Change-Id: Ic28f2fd4a9942ef2b1aa698c6f862b0149584eaf
Signed-off-by: Ian Chien <ianchien.wiwynn@gmail.com>

show more ...

configurations: yosemite4: Set fan current sensors max value

Set fan current sensors max value to an appropriate value.
Otherwise, fan current sensors value might remain unchanged when the fan
maint

configurations: yosemite4: Set fan current sensors max value

Set fan current sensors max value to an appropriate value.
Otherwise, fan current sensors value might remain unchanged when the fan
maintains a constant speed.

Test Case: Maxvalue is set and can be read from PSU sensor service.

Change-Id: I1b63eca5b8a16bc1babfb068ab764722f15f421d
Signed-off-by: Ian Chien <ianchien.wiwynn@gmail.com>

show more ...

schema: Add physical topology fields

Add ConnectsToType and PowerPort to the schema. Code support was added
awhile ago with
https://gerrit.openbmc.org/c/openbmc/entity-manager/+/68030 and
https://g

schema: Add physical topology fields

Add ConnectsToType and PowerPort to the schema. Code support was added
awhile ago with
https://gerrit.openbmc.org/c/openbmc/entity-manager/+/68030 and
https://gerrit.openbmc.org/c/openbmc/entity-manager/+/57133.

Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I579d593a95ebbd3b9e6a2be8865390b77a2126e0

show more ...

configurations: yosemite4: Support second sources fan board

Support second sources fan board config.
Test case: Fan board config can be loaded and fan sensors can be read.

Change-Id: I7c67f782f45bd

configurations: yosemite4: Support second sources fan board

Support second sources fan board config.
Test case: Fan board config can be loaded and fan sensors can be read.

Change-Id: I7c67f782f45bddbfa034e35ecd1b01dba259c686
Signed-off-by: Ian Chien <ianchien.wiwynn@gmail.com>

show more ...

configurations: harma_mb: Add Power State

The MB_P1V8_CPU_RT_VOLT_V is DC sensor, so
add power status to avoid touch threshold.

Change-Id: Idfa1def185e1886f436ff7c9268845cb95963d35
Signed-off-by: P

configurations: harma_mb: Add Power State

The MB_P1V8_CPU_RT_VOLT_V is DC sensor, so
add power status to avoid touch threshold.

Change-Id: Idfa1def185e1886f436ff7c9268845cb95963d35
Signed-off-by: Peter Yin <peter.yin@quantatw.com>

show more ...

Revert "Add mctp eids configuration for Yosemite 4"

This reverts commit 6fa0602db8250905808991e5f7206151dd28b346.

MCTP support in the backend is nowhere near functional, or complete, as
discussions

Revert "Add mctp eids configuration for Yosemite 4"

This reverts commit 6fa0602db8250905808991e5f7206151dd28b346.

MCTP support in the backend is nowhere near functional, or complete, as
discussions on discord and gerrit[1] make clear.

The tested statement on this patch wasn't really acceptable, and while
normally it doesn't matter what configurations someone sets, they should
at least have code backing them that works.

Therefore, reverting this patch until here are actually MCTP capable
backends, and we've resolved the discussion around MCTP static EID
assignment.

[1] https://gerrit.openbmc.org/c/openbmc/entity-manager/+/69600

Change-Id: Idc0e3c31f7683fd5ac968ca18d1b7239e8bb0c77

show more ...

yosemite4: fine tune medusa HSC power and current

ADM1272: current value*4 and power value*4.
ADM1281: current value*2 and power value*2.

Testing:
1. Unit tests passed.
2. Get sensor reading passed

yosemite4: fine tune medusa HSC power and current

ADM1272: current value*4 and power value*4.
ADM1281: current value*2 and power value*2.

Testing:
1. Unit tests passed.
2. Get sensor reading passed.

Change-Id: Ie55b8f3b55da93ffc28b82e2316ddaa9ac2d1def
Signed-off-by: Delphine CC Chiu <Delphine_CC_Chiu@wiwynn.com>

show more ...

schemas: Descriptions must be strings

```
$ python3 -c 'import json; import jsonschema; schema = json.load(open("schemas/global.json")); cls = jsonschema.validators.validator_for(schema); cls.check_

schemas: Descriptions must be strings

```
$ python3 -c 'import json; import jsonschema; schema = json.load(open("schemas/global.json")); cls = jsonschema.validators.validator_for(schema); cls.check_schema(schema)'
Traceback (most recent call last):
File "<string>", line 1, in <module>
File "/usr/lib/python3/dist-packages/jsonschema/validators.py", line 204, in check_schema
raise exceptions.SchemaError.create_from(error)
jsonschema.exceptions.SchemaError: ['The schema for an entity manager configuration file. An entity ', 'mananger configuration file can consist of a single object, or an ', 'array of objects. Objects must be of type EMConfig.'] is not of type 'string'

Failed validating 'type' in metaschema['properties']['description']:
{'type': 'string'}

On schema['description']:
['The schema for an entity manager configuration file. An entity ',
'mananger configuration file can consist of a single object, or an ',
'array of objects. Objects must be of type EMConfig.']
```

Change-Id: I2b8102e7d6047c8a7e624e182465065268fb8ad9
Signed-off-by: Andrew Jeffery <andrew@codeconstruct.com.au>

show more ...

schemas: legacy: Add Rearm property

This commit adds `Rearm` property to `Aggregate` as a property for
Chassis Intrusion Sensor. This is of type enum [Automatic, Manual] and
is the method of how the

schemas: legacy: Add Rearm property

This commit adds `Rearm` property to `Aggregate` as a property for
Chassis Intrusion Sensor. This is of type enum [Automatic, Manual] and
is the method of how the intrusion sensor is rearmed to normal state.

This property was defined in phosphor-dbus-interface [1].

Example:
{
"Class": "Aspeed2600_Hwmon",
"Name": "Chassis_Intrusion_Status",
"Rearm": "Manual",
"Type": "ChassisIntrusionSensor"
}

[1]: https://github.com/openbmc/phosphor-dbus-interfaces/blob/1abc0c57692b679753955b7273c1393369ef4987/yaml/xyz/openbmc_project/Chassis/Intrusion.interface.yaml#L12

Signed-off-by: Chau Ly <chaul@amperecomputing.com>
Change-Id: I7815b1e095832769c823177f96eae399eb379f90

show more ...

Use common Polarity in schema

Point the definition of the Polarity property in the Presence entry at
the common Polarity definition, and also make that an enum since that is
how all code uses it.

R

Use common Polarity in schema

Point the definition of the Polarity property in the Presence entry at
the common Polarity definition, and also make that an enum since that is
how all code uses it.

Recently, CI started failing without this change, though why it started
failing is a mystery to me, and the error messages aren't that helpful.

Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I966974b894a715c563bed3823a6b47b92a7ee94a

show more ...

configurations: Add Minerva Harma system

Introduce initial configuration files for Minerva Harma system

- harma_bsm: BMC storage module
- harma_fanboard: Fan board
- harma_mb: Motherboard
- harma_n

configurations: Add Minerva Harma system

Introduce initial configuration files for Minerva Harma system

- harma_bsm: BMC storage module
- harma_fanboard: Fan board
- harma_mb: Motherboard
- harma_nic: NIC Mezz
- harma_scm: SCM board
- minerva_pdb: PDB board
- minerva_pttv: Power and thermal test vehicle
- minerva_sitv: Signal integrity test vehicle

Signed-off-by: Potin Lai <potin.lai@quantatw.com>
Change-Id: I7ff14af2d07c21c58b540499b598547c979339f6

show more ...

Add mctp eids configuration for Yosemite 4

The MCTP specification is defined in DSP0236. As describe in section
8.17.2, we need a configuration for static EIDs. The `mctpd` from
CodeConstuct/mctp wi

Add mctp eids configuration for Yosemite 4

The MCTP specification is defined in DSP0236. As describe in section
8.17.2, we need a configuration for static EIDs. The `mctpd` from
CodeConstuct/mctp will be able to know where those static EIDs located.
The other services will be able to know what those EIDs stand for by the
"Name" as well.

schemas: add `EndpointId` with type `string` and `number`.

Tested: check D-Bus path for configuration.
Change-Id: I587f5a69051f783753f5855d2f51242aa1cd5bc4
Signed-off-by: Delphine CC Chiu <Delphine_CC_Chiu@wiwynn.com>

show more ...

Add mgnt/spider/medusa board sensors for Yosemite 4

- Configure the following type sensors:
- adm1272
- adm1281
- adc128d818
- aspeed adc
- ina233
- pmbus
- tmp75
- tmp421
- Revise condition

Add mgnt/spider/medusa board sensors for Yosemite 4

- Configure the following type sensors:
- adm1272
- adm1281
- adc128d818
- aspeed adc
- ina233
- pmbus
- tmp75
- tmp421
- Revise condition to probe json config for entity-manager

Tested:
- Read sensors from entity-manager through dbus

Change-Id: I7d4d8e94aab5f1cbca281e32deab958b593a6930
Signed-off-by: Delphine CC Chiu <Delphine_CC_Chiu@wiwynn.com>

show more ...

schemas: Expand set of supported hwmon labels

Specifically, iin{1..3}, iout3, pin{1..3}, and temp{2..4}.

Tested: In combination with a configuration for the ASRock E3C256D4I
(which uses all of thes

schemas: Expand set of supported hwmon labels

Specifically, iin{1..3}, iout3, pin{1..3}, and temp{2..4}.

Tested: In combination with a configuration for the ASRock E3C256D4I
(which uses all of these for an ISL69269 device),
scripts/validate_configs.py successfully validates the schema.

Signed-off-by: Zev Weiss <zev@bewilderbeest.net>
Change-Id: I742b453cde4815f788a95f2f9c5c475f8ef3c0f7

show more ...

schemas: Add more names for labels

The MAX5970 has two separate outputs, so add the ability to name
both channels. This will be used in the following commit that adds
MAX5970 support on SBP1.

All a

schemas: Add more names for labels

The MAX5970 has two separate outputs, so add the ability to name
both channels. This will be used in the following commit that adds
MAX5970 support on SBP1.

All add to name input current.

Tested: The configuration passes validation.

Change-Id: I28c1c5b6e70fe7b8489e6946cb1a2e2abda12ae8
Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com>

show more ...