Move TemporaryFile class to root directory

Source code that is shared across applications is located in the root
directory of the phosphor-power repository.

The TemporaryFile class was originally c

Move TemporaryFile class to root directory

Source code that is shared across applications is located in the root
directory of the phosphor-power repository.

The TemporaryFile class was originally created for the
phosphor-regulators application. However it is general purpose and is
now needed by the phosphor-power-sequencer application.

Move the TemporaryFile class to the root directory. Update C++ files
that use the TemporaryFile class since it is now in a different
namespace. Update affected meson.build files.

Change-Id: I6d18080d2436a5da87b7ab51ac003c503caa36a3
Signed-off-by: Shawn McCarney <shawnmm@us.ibm.com>

show more ...

regulators: Allow empty expected VPD value

Modify the compare_vpd action to support specifying an empty string or
empty byte vector as the expected keyword value.

The VPD service was modified in a

regulators: Allow empty expected VPD value

Modify the compare_vpd action to support specifying an empty string or
empty byte vector as the expected keyword value.

The VPD service was modified in a previous commit to return an empty
keyword value if the VPD interface or keyword does not exist on the
specified D-Bus object path. Thus, the actual keyword value may be
empty.

Signed-off-by: Shawn McCarney <shawnmm@us.ibm.com>
Change-Id: If48ab6423c40e5b24d6cff72264dc2efe750c85f

show more ...

regulators: Make inventory_path required property

Make "inventory_path" a required property of the "chassis" object in the
regulators JSON configuration file.

This property is used to create the ch

regulators: Make inventory_path required property

Make "inventory_path" a required property of the "chassis" object in the
regulators JSON configuration file.

This property is used to create the chassis <-> sensor associations
required by bmcweb for finding regulator sensors.

The property is already documented in the markdown file as being
required. However, that requirement was not being enforced in the JSON
schema nor in the C++ configuration file parser.

The existing configuration files did not have the property, and
requiring it would have caused errors.

Those configuration files have been updated to include the property, so
the requirement is now being enforced.

Signed-off-by: Shawn McCarney <shawnmm@us.ibm.com>
Change-Id: If9af7969f8b92068e66c750681179db70eae489e

show more ...

regulators: Add PhaseFaultDetection to Device

Add a PhaseFaultDetection data member to the Device class. This is the
first step in enabling phase fault detection for Device objects.

Create and mod

regulators: Add PhaseFaultDetection to Device

Add a PhaseFaultDetection data member to the Device class. This is the
first step in enabling phase fault detection for Device objects.

Create and modify gtest test cases to exercise new code.

Signed-off-by: Shawn McCarney <shawnmm@us.ibm.com>
Change-Id: Icc771eda5c79ace854acb1d7c395b82b34213996

show more ...

regulators: Add phase_fault_detection to parser

Enhance the JSON configuration file parser to support the new
phase_fault_detection object.

Add gtest test cases to exercise new code.

Signed-off-by

regulators: Add phase_fault_detection to parser

Enhance the JSON configuration file parser to support the new
phase_fault_detection object.

Add gtest test cases to exercise new code.

Signed-off-by: Shawn McCarney <shawnmm@us.ibm.com>
Change-Id: I5b987054ef35c8b27d7d984e5e9ec9b4488bf944

show more ...

regulators: Add log_phase_fault to parser

Enhance the JSON configuration file parser to support the new
log_phase_fault action.

Add gtest test cases to exercise new code.

Signed-off-by: Shawn McCa

regulators: Add log_phase_fault to parser

Enhance the JSON configuration file parser to support the new
log_phase_fault action.

Add gtest test cases to exercise new code.

Signed-off-by: Shawn McCarney <shawnmm@us.ibm.com>
Change-Id: I8ef9974df8ca8be07dd284552bd978802f502338

show more ...

regulators: Add phase fault type to parser

Enhance the JSON configuration file parser to support the new
phase fault type values.

Add gtest test cases to exercise new code.

Signed-off-by: Shawn Mc

regulators: Add phase fault type to parser

Enhance the JSON configuration file parser to support the new
phase fault type values.

Add gtest test cases to exercise new code.

Signed-off-by: Shawn McCarney <shawnmm@us.ibm.com>
Change-Id: If5ac011f740fdb6f24692c713a56d4947709d660

show more ...

regulators: Add i2c_capture_bytes to parser

Enhance the JSON configuration file parser to support the new
i2c_capture_bytes action.

Add gtest test cases to exercise new code.

Signed-off-by: Shawn

regulators: Add i2c_capture_bytes to parser

Enhance the JSON configuration file parser to support the new
i2c_capture_bytes action.

Add gtest test cases to exercise new code.

Signed-off-by: Shawn McCarney <shawnmm@us.ibm.com>
Change-Id: I92f50fe3df9b3469e16e03b04ae5d0df464eb3e2

show more ...

regulators: Support a string or vector for VPD

Add a 'byte_values' alternative to the 'value' entry in the compare VPD
action. This is to support VPD values that are not strings, such as
'HW', a ne

regulators: Support a string or vector for VPD

Add a 'byte_values' alternative to the 'value' entry in the compare VPD
action. This is to support VPD values that are not strings, such as
'HW', a new IBM keyword that describes the version of a piece of
hardware.

To support this, the VPD class now treats all VPD keyword values as
vectors of uint8_ts, including in its data cache. If a compare VPD
action in the JSON contains a string value, it will be converted to the
vector before the CompareVPDAction class is created.

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

show more ...

regulators: Add inventory path to Chassis class

A previous commit added the "inventory_path" property to the "chassis"
object in the JSON configuration file.

This commit adds that new property to t

regulators: Add inventory path to Chassis class

A previous commit added the "inventory_path" property to the "chassis"
object in the JSON configuration file.

This commit adds that new property to the C++ implementation:
* Chassis class and associated gtests
* JSON configuration file parser functions and associated gtests
* Other gtests affected by the change to the Chassis constructor

Signed-off-by: Shawn McCarney <shawnmm@us.ibm.com>
Change-Id: I7406f874263d01e6faa2b8c333cb1baf915cf2ac

show more ...

regulators: Convert to new SensorType enumeration

There are several types of voltage regulator sensors, such as vout,
iout, and temperature.

The sensor types were previously defined by the enumerat

regulators: Convert to new SensorType enumeration

There are several types of voltage regulator sensors, such as vout,
iout, and temperature.

The sensor types were previously defined by the enumeration
pmbus_utils::SensorValueType. This enumeration should not have been
placed in the pmbus_utils namespace because it is not specific to PMBus.

A new enumeration with the simpler name SensorType was created in
sensors.hpp in a previous commit.

Remove the old enumeration from pmbus_utils. Convert affected files to
use the new enumeration.

Signed-off-by: Shawn McCarney <shawnmm@us.ibm.com>
Change-Id: I4136b392b104edd3eee41ec29c779ad084c7c4de

show more ...

regulators: Enhance config file parser

Enhance JSON config file parser to convert relative inventory paths to
absolute form.

Tested:
Run local CI with -Dlong-tests=enabled to enable tests for
valid

regulators: Enhance config file parser

Enhance JSON config file parser to convert relative inventory paths to
absolute form.

Tested:
Run local CI with -Dlong-tests=enabled to enable tests for
validate-regulators-config.py.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I80237a673f9c5918898db15363847722141388e6

show more ...

regulators: Remove TmpFile class

The TmpFile class was created in the test directory for managing
temporary files used by testcases.

A new TemporaryFile class has replaced it. TemporaryFile has mo

regulators: Remove TmpFile class

The TmpFile class was created in the test directory for managing
temporary files used by testcases.

A new TemporaryFile class has replaced it. TemporaryFile has more
features and can be used by production (non-test) code as well.

This commit removes the TmpFile class and modifies testcases to use
TemporaryFile instead.

Signed-off-by: Shawn McCarney <shawnmm@us.ibm.com>
Change-Id: Id06d333416e43d5685c377ee0780b475d20cba8f

show more ...

regulators: Implements support for pmbus_read_sensor action

Enhance the configuration file parser to support the pmbus_read_sensor
action element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Cha

regulators: Implements support for pmbus_read_sensor action

Enhance the configuration file parser to support the pmbus_read_sensor
action element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I7f87a7551469bb494fe17b6d6d79ebb1ca517e86

show more ...

regulators: Implements support for compare_vpd action

Enhance the configuration file parser to support the compare_vpd
action element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: Ia87

regulators: Implements support for compare_vpd action

Enhance the configuration file parser to support the compare_vpd
action element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: Ia8766a0d2f893898c5ff2655a757d60f7cdcc7df

show more ...

regulators: Implements support for compare_presence action

Enhance the configuration file parser to support the
compare_presence action element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Chang

regulators: Implements support for compare_presence action

Enhance the configuration file parser to support the
compare_presence action element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I9bfa3d260e0a6f7d8f41bed473b436148141ea50

show more ...

regulators: Implements support for set_device action

Enhance the configuration file parser to support the set_device
action element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: If9d1e

regulators: Implements support for set_device action

Enhance the configuration file parser to support the set_device
action element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: If9d1e25dc35370536fab9ab4ef7f2bf269e46d20

show more ...

regulators: Implements support for presence_detection

Enhance the configuration file parser to support the
presence_detection element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I40e

regulators: Implements support for presence_detection

Enhance the configuration file parser to support the
presence_detection element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I40ed2e849b6baaebdfd1899000fc389429d46543

show more ...

regulators: Implements support for if action

Enhance the configuration file parser to support the if action element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: Iadb6835dd28151a1dd278

regulators: Implements support for if action

Enhance the configuration file parser to support the if action element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: Iadb6835dd28151a1dd278f6b2a19568a3072ad5e

show more ...

regulators: Implements support for or action

Enhance the configuration file parser to support the or action element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I6e6dee26c23a2ea414e96

regulators: Implements support for or action

Enhance the configuration file parser to support the or action element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I6e6dee26c23a2ea414e96a0cda3525b91b806509

show more ...

regulators: Implements support for not action

Enhance the configuration file parser to support the not action element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: Ia88eee705a9fae28a81

regulators: Implements support for not action

Enhance the configuration file parser to support the not action element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: Ia88eee705a9fae28a816aeb6dff9e0b6f0deba39

show more ...

regulators: Implements support for and action

Enhance the configuration file parser to support the and action element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I9099bf06606fb52ea7e

regulators: Implements support for and action

Enhance the configuration file parser to support the and action element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I9099bf06606fb52ea7eaa374e2a2b1f924904d4f

show more ...

regulators: Implements support for i2c_compare actions

Enhance the configuration file parser to support the i2c_compare_bit,
i2c_compare_byte, and i2c_compare_bytes actions.

Signed-off-by: Bob King

regulators: Implements support for i2c_compare actions

Enhance the configuration file parser to support the i2c_compare_bit,
i2c_compare_byte, and i2c_compare_bytes actions.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I752afde05f16d82da22d404bf24a8c1ab3b7785c

show more ...

regulators: Implements support for rail

Enhance the configuration file parser to support the rail element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: Id10809cac4271295c5eb3bd318be775

regulators: Implements support for rail

Enhance the configuration file parser to support the rail element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: Id10809cac4271295c5eb3bd318be775e1d470286

show more ...

regulators: Implements support for configuration

Enhance the configuration file parser to support the configuration
element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I6862c9784a3ad

regulators: Implements support for configuration

Enhance the configuration file parser to support the configuration
element.

Signed-off-by: Bob King <Bob_King@wistron.com>
Change-Id: I6862c9784a3ade6378eb7a1d19a174f1515fc845

show more ...