History log of /openbmc/phosphor-power/phosphor-regulators/src/vpd.cpp (Results 1 – 3 of 3)
Revision Date Author Comments
# 5e0346b3 30-Oct-2021 Shawn McCarney <shawnmm@us.ibm.com>

regulators: Treat missing VPD keyword as empty

Modify the VPD service to return an empty value if the specified VPD
interface or keyword does not exist on the specified object path.

This condition

regulators: Treat missing VPD keyword as empty

Modify the VPD service to return an empty value if the specified VPD
interface or keyword does not exist on the specified object path.

This condition can occur when the system is operating normally and
should not result in an error being logged.

Tested:
* Verified an empty value is returned if the VPD interface does not
exist on the object path.
* Verified an empty value is returned if the VPD keyword does not exist
on the object path.
* Verified correct value is returned if the VPD interface and keyword
exist
* See complete test plan at
https://gist.github.com/smccarney/47b167706da49c7d78c4df818040ae72

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

show more ...


# aacc2aaf 25-May-2021 Matt Spinler <spinler@us.ibm.com>

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 ...


# 4e0402cb 05-Feb-2021 Shawn McCarney <shawnmm@us.ibm.com>

regulators: Add VPD service

The regulators application needs to obtain VPD (Vital Product Data)
keyword values.

Sometimes regulator configuration and monitoring varies depending on
hardware type or

regulators: Add VPD service

The regulators application needs to obtain VPD (Vital Product Data)
keyword values.

Sometimes regulator configuration and monitoring varies depending on
hardware type or version. VPD keyword values can provide this
information about the hardware.

Add a new class to obtain hardware VPD from the D-Bus
xyz.openbmc_project.Inventory.Decorator.Asset interface.

Also define an abstract base class and a mock implementation to
enable use of gmock in test cases related to VPD.

Tested:
* Verified VPD values were successfully obtained from D-Bus.
* Verified VPD values were cached.
* Tested where object path was invalid.
* Tested where keyword was invalid.
* Verified cached VPD values were cleared when machine powered on.
* For the complete test plan, see
https://gist.github.com/smccarney/519a54353361e28b1d25f5783c15f471

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

show more ...