Create class for finding compatible system types

Create class for finding the list of compatible system types for the
current system.

Use the new D-Bus interface
xyz.openbmc_project.Inventory.Decor

Create class for finding compatible system types

Create class for finding the list of compatible system types for the
current system.

Use the new D-Bus interface
xyz.openbmc_project.Inventory.Decorator.Compatible to find the system
types.

Tested:
* See test plan at
https://gist.github.com/smccarney/7bfc20b258cd6ccfba908730102a32dd

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

show more ...

Support multiple D-Bus interfaces in getSubTree()

The utility function getSubTree() only allows a single D-Bus interface
to be specified.

This does not match the corresponding GetSubTree D-Bus meth

Support multiple D-Bus interfaces in getSubTree()

The utility function getSubTree() only allows a single D-Bus interface
to be specified.

This does not match the corresponding GetSubTree D-Bus method, which
allows multiple D-Bus interfaces to be specified.

Add a getSubTree() overload that accepts multiple D-Bus interfaces.

Tested:
* Existing getSubTree() function
* Test where it is successful
* Test where it fails due to a D-Bus error
* New getSubTree() function
* Test where zero interfaces are specified
* Test where one interface is specified
* Test where multiple interfaces are specified
* Test where it is successful
* Test where it fails due to a D-Bus error

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

show more ...

psu-ng: Fix chassis association call

The association name from a power supply to its chassis changed from
'chassis' to 'powering', so change that in the code.

Also update the code to find the assoc

psu-ng: Fix chassis association call

The association name from a power supply to its chassis changed from
'chassis' to 'powering', so change that in the code.

Also update the code to find the association using the new
GetAssociatedSubTreePaths mapper method which is like the existing
GetSubTreePaths but also ensures the results are an endpoint of the
passed in association path. This way if the 'powering' association is
used to show the power supply powers other things it will still work.

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

show more ...

sdbusplus: use shorter type aliases

The sdbusplus headers provide shortened aliases for many types.
Switch to using them to provide better code clarity and shorter
lines. Possible replacements are

sdbusplus: use shorter type aliases

The sdbusplus headers provide shortened aliases for many types.
Switch to using them to provide better code clarity and shorter
lines. Possible replacements are for:
* bus_t
* exception_t
* manager_t
* match_t
* message_t
* object_t
* slot_t

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

show more ...

psu-ng: Get the power config full load property

The supported configuration schema now includes the PowerConfigFullLoad
property to indicate the value that the power-config-full-load GPIO
should be

psu-ng: Get the power config full load property

The supported configuration schema now includes the PowerConfigFullLoad
property to indicate the value that the power-config-full-load GPIO
should be set to for systems that support it.

Reference:
https://github.com/openbmc/docs/blob/master/designs/device-tree-gpio-naming.md#power-config-full-load

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

show more ...

psu-ng: Update supported configuration properties

The Supported Configuration schema has been updated to have a single
model instead of array of models, and to have an array of input voltages
instea

psu-ng: Update supported configuration properties

The Supported Configuration schema has been updated to have a single
model instead of array of models, and to have an array of input voltages
instead of a single voltage.
In addition, update the name of the supported number of power supplies
from max to count, so that it better reflects that a specific number of
power supplies are expected, and not an 'up to' max number.

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

show more ...

utility: Add Get All Properties

Add a utility function to call the D-Bus GetAll method. This will be
used to get the values of all the properties from the Supported
Configuration interface from Enti

utility: Add Get All Properties

Add a utility function to call the D-Bus GetAll method. This will be
used to get the values of all the properties from the Supported
Configuration interface from Entity Manager. Start with the variant
values for the properties in the Supported Configuration interface which
are uint64_t (count), string (type), and array of strings (models).

Make the service parameter optional since the caller may already have
the service name from another D-Bus call like getSubTree, or the caller
may decide to call getService() before the getAll function if they are
going to call getAll multiple times and the service name can be reused.
If the service name is not passed, the new getAll will call getService().

Tested: Verified that calling the new function worked as expected
calling it with and without a service name.

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

show more ...

Add a phosphor::power::util getSubTree helper

Adding a helper function to do the GetSubTree method call on the object
mapper.

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

Add a phosphor::power::util getSubTree helper

Adding a helper function to do the GetSubTree method call on the object
mapper.

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

show more ...

sdbusplus: replace message::variant with std::variant

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

sdbusplus: remove deprecated variant_ns

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

util: Logging error on getService optional

Make logging an error on service name lookup for a given path/interface
optional for the case where a service may not be available yet. This is
to handle t

util: Logging error on getService optional

Make logging an error on service name lookup for a given path/interface
optional for the case where a service may not be available yet. This is
to handle the case where a service dependent on a property from another
may be started in parallel by systemd where the property would need to
be retrieved by other means (i.e. interfacesAdded signal).

Tested:
No change to current users of getService util function
No error logged when service not found

Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
Change-Id: Ib570b83fe2feae07031fa477f468327a7b0554b0

show more ...

Update .clang-format to latest version from docs

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

power-utils: Implement isReadyToUpdate

Check other PSUs' status word and status vout, return false when the
status has input/output fault.

Tested: Verify on Witherspoon that the update is not start

power-utils: Implement isReadyToUpdate

Check other PSUs' status word and status vout, return false when the
status has input/output fault.

Tested: Verify on Witherspoon that the update is not started and log
Unable to update PSU when other PSU has input/ouput fault
when the other PSU has input/output error;
And the update continues when the other PSU has no errors.

Signed-off-by: Lei YU <mine260309@gmail.com>
Change-Id: Ia2a4a23a43c18a417b8a85fbd5339f487984e689

show more ...

Refactor: Move isPoweredOn to utility

The check of power on will be used by other components, so move it to
utility.
There are several constexpr definitions are used by multiple components,
move the

Refactor: Move isPoweredOn to utility

The check of power on will be used by other components, so move it to
utility.
There are several constexpr definitions are used by multiple components,
move them into types.hpp so that we do not have duplicated constexprs.

Signed-off-by: Lei YU <mine260309@gmail.com>
Change-Id: I42ad142d6e7ae8da9c0cf6d8f5cb21dee229eba2

show more ...

Refactor: move get pmbus access type to utility

The function to get the pmbus access type is common, move it to utility.

Signed-off-by: Lei YU <mine260309@gmail.com>
Change-Id: I26e7da044c88c1efba1

Refactor: move get pmbus access type to utility

The function to get the pmbus access type is common, move it to utility.

Signed-off-by: Lei YU <mine260309@gmail.com>
Change-Id: I26e7da044c88c1efba141349ee4c18b24cdef81f

show more ...

Refactor: rename namespace witherspoon to phosphor

This repo now becomes phosphor-power, rename namespace witherspoon to
phosphor to make it generic.

Signed-off-by: Lei YU <mine260309@gmail.com>
Ch

Refactor: rename namespace witherspoon to phosphor

This repo now becomes phosphor-power, rename namespace witherspoon to
phosphor to make it generic.

Signed-off-by: Lei YU <mine260309@gmail.com>
Change-Id: Icb9f8a7c33065add99f2bf42ed55745a28fa0424

show more ...

Add loadJSONFromFile() in utility

It provides a util function to load json from a file.

Tested: Manually verify the function works.

Signed-off-by: Lei YU <mine260309@gmail.com>
Change-Id: I4dc9466

Add loadJSONFromFile() in utility

It provides a util function to load json from a file.

Tested: Manually verify the function works.

Signed-off-by: Lei YU <mine260309@gmail.com>
Change-Id: I4dc946663e0ce1bf511de90b27b7aa425a338543

show more ...

Remove unused is_method_error calls

According to
https://lists.ozlabs.org/pipermail/openbmc/2018-October/013696.html,
message::is_method_error() will never fail and it doesn't need to
be called anym

Remove unused is_method_error calls

According to
https://lists.ozlabs.org/pipermail/openbmc/2018-October/013696.html,
message::is_method_error() will never fail and it doesn't need to
be called anymore.

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

show more ...

Start using .clang-format

Used the one from docs/style/cpp.

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

Add a utility function for setting a property

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

Pass error to log into shutdown function

Changing the power off utility function to take a
template parameter that specifies the error to log
before the shutdown instead of hardcoding it in
the func

Pass error to log into shutdown function

Changing the power off utility function to take a
template parameter that specifies the error to log
before the shutdown instead of hardcoding it in
the function.

Also change the 2 callers of this function to pass
in the error type.

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

show more ...

Add powerOff utility function

Logs an error so the shutdown reason is known and then
does a hard power off.

Change-Id: I52e2d07668a777c0aacf5c269f2632fbd6f15246
Signed-off-by: Matt Spinler <spinler

Add powerOff utility function

Logs an error so the shutdown reason is known and then
does a hard power off.

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

show more ...

Add getProperty and getService utility functions

Upcoming code will use them.

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