prettier: re-format

Prettier is enabled in openbmc-build-scripts on Markdown, JSON, and YAML
files to have consistent formatting for these file types. Re-run the
formatter on the whole repository.

prettier: re-format

Prettier is enabled in openbmc-build-scripts on Markdown, JSON, and YAML
files to have consistent formatting for these file types. Re-run the
formatter on the whole repository.

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

show more ...

psu-ng: Get max PSU number from entity manager

Read the max number of PSUs supported from entity manager instead of
JSON since entity manager would have the value based on system. If the

psu-ng: Get max PSU number from entity manager

Read the max number of PSUs supported from entity manager instead of
JSON since entity manager would have the value based on system. If the
entity manager interface does not exist by the time the psu manager
starts, subscribe to interfaces added. The psu manager can take action
once the max count property is available on D-Bus.

Tested: On Rainier 2U with debug traces:
1. Restarted phosphor-psu-monitor and verified maxPowerSupplies was 2.
2. Stopped xyz.openbmc_project.Inventory.Manager. Restarted
phosphor-psu-monitor and verified maxPowerSupplies was 0.
3. Started xyz.openbmc_project.Inventory.Manager and verified the
psu callback was triggered setting maxPowerSupplies to 2.
4. Stopped and started xyz.openbmc_project.Inventory.Manager and
verified the psu callback was not called again.

Change-Id: Ieff796ca5c79bfddc1fc6e15e339a0b15eb422d7
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>

show more ...

psu-ng: Remove Min Power Supplies and cleanup

The psu manager would not prevent a power on based on the number of
power supplies, it's only concerned about the max number so that if the

psu-ng: Remove Min Power Supplies and cleanup

The psu manager would not prevent a power on based on the number of
power supplies, it's only concerned about the max number so that if the
number of plugged power supplies is less than the specified max, then it
can take action such as log an error.

Therefore there's not a current use case for the min setting, so
removing it.

Also cleanup the function that parses that information from the JSON
config file, like removing passing parameters that are class members,
and adding a doxygen description. In addition, have the structure of
system properties as part of the class instead of individual variables,
since it's duplicate work to set the structure values then set the
individual variables.

Tested: With debug traces verified that maxPowerSupplies was set to 2
which is the value in the JSON config file.

Change-Id: Id98900d213fcbef38dc3879b425161a8af172716
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>

show more ...

Update psu poll rate for phosphor-power-supply.

Use a hard-coded poll rate rather than the configuration value from
psu_config.json

Tested: Ran existing unit tests for phosphor-

Update psu poll rate for phosphor-power-supply.

Use a hard-coded poll rate rather than the configuration value from
psu_config.json

Tested: Ran existing unit tests for phosphor-power with the build using
meson -Doe-sdk=enabled -Dtests=enabled x86build
Tested: Using a psu_config.json file that did not contain a poll rate,
demonstrating that pollInterval is no longer needed.
Tested: By adding temporary logging messages to verify successful
startup without existence of pollInterval field.

Signed-off-by: Jay Meyer <jaymeyer@us.ibm.com>
Change-Id: I969017c10474860f8b0a5f578a44a704c2f6a2e5

show more ...

Parse I2C bus and address out of JSON config file

The I2C bus and address need to be passed down to the PowerSupply
constructor, so get those out of the JSON configuration file now.

Parse I2C bus and address out of JSON config file

The I2C bus and address need to be passed down to the PowerSupply
constructor, so get those out of the JSON configuration file now.

Change I2C address format in JSON to be a string, without the
hexadecimal format since it needs to be appended to a string, allowing
to skip conversion from string to number and then from number back to
string.

Signed-off-by: Brandon Wyman <bjwyman@gmail.com>
Change-Id: I2707799bb60d1f4a9783a0403bc3e4920461b7a1

show more ...

psu: Updates for power supply presence

Update the JSON file to indicate the minimum and maximum number of power
supplies this system can have.

Update the JSON file to have an ar

psu: Updates for power supply presence

Update the JSON file to indicate the minimum and maximum number of power
supplies this system can have.

Update the JSON file to have an array of power supply objects, each with
their various own properties. Each power supply will have:
1) An associated D-Bus path for the inventory. This will have the
Present property.
2) An associated I2C bus number.
3) An associated I2C bus adddress.

Update PowerSupply class to have a present member variable, an
isPresent() accessor function, and two functions for updating the
presence. One function will be the call back function for either
inventory change events or interface added events, which will use
associated match member variables. The second function being one that
attempts to read the inventory Present property on startup. If reading
the Present property on startup fails, the property changed or interface
added events should work to get the value when that property is
available.

Signed-off-by: Brandon Wyman <bjwyman@gmail.com>
Change-Id: Ib34510a6da66ba1b8f1e927093b3d10b09beb410

show more ...

psu: Add code to get pollInterval from JSON file

Add in a helper function to open JSON file and parse out the configuration
properties. Use the helper function to get the pollInterval in

psu: Add code to get pollInterval from JSON file

Add in a helper function to open JSON file and parse out the configuration
properties. Use the helper function to get the pollInterval in the constructor.

Signed-off-by: Brandon Wyman <bjwyman@gmail.com>
Change-Id: Ia8cf1c148a1685e2e09703c02e469a0bff034af5

show more ...