Update to get GPIConfigs from parsing YAML

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

Update to get RailNames from parsing YAML

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

Update script and template to parse index and path

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

Fill in example YAML file based on Witherspoon

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

Use YAML for UCD definitions

Use a Python script to generate the UCD power sequencer information
needed for the pseq-monitor application.

Change-Id: I51f4b61bb4f22705584b73ba599b517780fa9adc
Signed

Use YAML for UCD definitions

Use a Python script to generate the UCD power sequencer information
needed for the pseq-monitor application.

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

show more ...

Cleanup: Use filesystem instead of experimental

The GCC version bump in Yocto 2.6 allows for use of std::filesystem
instead of std::experimental::filesystem.

Change-Id: Ibfd6c285bcbd6695e5cd4e3090f

Cleanup: Use filesystem instead of experimental

The GCC version bump in Yocto 2.6 allows for use of std::filesystem
instead of std::experimental::filesystem.

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

show more ...

Remove witherspoon prefix from power sequencer app

The Swift system is a follow-on system very similar to the Witherspoon.
The plan is to re-use this application for Swift, while continuing to
suppo

Remove witherspoon prefix from power sequencer app

The Swift system is a follow-on system very similar to the Witherspoon.
The plan is to re-use this application for Swift, while continuing to
support Witherspoon, without the confusion of having a witherspoon
executable/app on a Swift system.

Tested: Built and flashed image to Witherspoon system. Verified process
running, powered on, and verified faults logged by executing 'obmcutil
chassiskill'.

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

show more ...

build: install into bin instead of sbin

Installs into bin instead of sbin per guidelines.

Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: Id782714cdf6f8345330ba1cf241c615be8256bbf

Fix I2C dev path for the 5.0 kernel

In the 5.0 Linux kernel, an I2C device path segment changed:
i2c@1e78a000 -> bus@1e78a000

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

Fix I2C dev path for the 5.0 kernel

In the 5.0 Linux kernel, an I2C device path segment changed:
i2c@1e78a000 -> bus@1e78a000

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

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>

timer: Remove in favor of sdeventplus/timer

This change removes the ad-hoc timer implementation and uses the common
one that comes with sdeventplus.

Tested: Compiled

Change-Id: Id3b7e464a472a74217

timer: Remove in favor of sdeventplus/timer

This change removes the ad-hoc timer implementation and uses the common
one that comes with sdeventplus.

Tested: Compiled

Change-Id: Id3b7e464a472a7421785601634af58f681ebd3a6
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

event: Remove in favor of sdeventplus

Use the new library provided to all openbmc projects instead of rolling
our own managed pointer.

Tested: Compiled

Change-Id: I4993d4340e0e0aa5898e73ef815baa81

event: Remove in favor of sdeventplus

Use the new library provided to all openbmc projects instead of rolling
our own managed pointer.

Tested: Compiled

Change-Id: I4993d4340e0e0aa5898e73ef815baa81b0e8a2bc
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

Handle new UCD90160 MFR_STATUS sysfs location

The 4.13 UCD90160 device driver moved the sysfs location
of the mfr_status register from
/sys/kernel/debug/ucd9000.0/mfr_status
to
/sys/kernel/debug

Handle new UCD90160 MFR_STATUS sysfs location

The 4.13 UCD90160 device driver moved the sysfs location
of the mfr_status register from
/sys/kernel/debug/ucd9000.0/mfr_status
to
/sys/kernel/debug/pmbus/hwmonN/ucd90160/mfr_status

The PMBus class can already handle this, just need to
change the enum to tell it where to look.

MFR_STATUS is only read when collecting metadata
on fails.

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

show more ...

Still report errors when metadata capture fails

In the very unlikely scenario that a PMBus command
fails while trying to collect error log metadata,
don't let the thrown exception break the code out

Still report errors when metadata capture fails

In the very unlikely scenario that a PMBus command
fails while trying to collect error log metadata,
don't let the thrown exception break the code out
of creating the error log.

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

show more ...

Call out GPUs using their inventory path

The existing code was calling out GPUs in the metadata
with 'GPU=GPUn' because at the time the GPUs were not
in the inventory.

Now that the GPUs are in the

Call out GPUs using their inventory path

The existing code was calling out GPUs in the metadata
with 'GPU=GPUn' because at the time the GPUs were not
in the inventory.

Now that the GPUs are in the inventory, they can be called
out using their inventory objects which is the correct
way to do it.

Resolves openbmc/openbmc#2869

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

show more ...

types: Add missing functional include

std::function is not found with a GCC 7.2 runtime.

Change-Id: I7e054bde6249b5913634cc067582a6f9d4c20f48
Tested: Built repository with GCC 7.2
Signed-off-by: Br

types: Add missing functional include

std::function is not found with a GCC 7.2 runtime.

Change-Id: I7e054bde6249b5913634cc067582a6f9d4c20f48
Tested: Built repository with GCC 7.2
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

show more ...

Move GPIO class to common library

It will soon by used by the power supply code as
well as the power sequencer code, so moving it to
a common location.

Change-Id: Ia3e2bcf27239c6861fcb6a9d45ae08a4e

Move GPIO class to common library

It will soon by used by the power supply code as
well as the power sequencer code, so moving it to
a common location.

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

show more ...

Change interval units to milliseconds

Allow the monitoring intervals to be specified in
milliseconds instead of seconds in order to handle
polling intervals less than a second.

Change-Id: I3ef078df

Change interval units to milliseconds

Allow the monitoring intervals to be specified in
milliseconds instead of seconds in order to handle
polling intervals less than a second.

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

show more ...

Stop polling the GPU PGOOD fault GPIO

The UCD90160 has been recently reconfigured to tie its
GPU PGOOD GPIO into its master PGOOD. This means we no
longer have to poll it every second since now if

Stop polling the GPU PGOOD fault GPIO

The UCD90160 has been recently reconfigured to tie its
GPU PGOOD GPIO into its master PGOOD. This means we no
longer have to poll it every second since now if there is a
GPU PGOOD error it will drop the master PGOOD signal SYS_PWR_OK
and the power fault analysis code will be called to perform
a full check and the isolation will be done there.

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

show more ...

Don't log errors for voltage warnings

Some of the bits in the STATUS_VOUT command response
are just warnings, as defined by the PMBUS spec.

Previously the code would create voltage fault errors
if

Don't log errors for voltage warnings

Some of the bits in the STATUS_VOUT command response
are just warnings, as defined by the PMBUS spec.

Previously the code would create voltage fault errors
if any bits were on in that response, and this change
is to check for the actual fault (non-warning) bits only
before creating error logs.

Also, the code will throw the response value into the journal
now if any bits are on at all because warnings are still
useful when getting a full picture of fault conditions.

Resolves openbmc/openbmc#2714

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

show more ...

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

Use Common.Device.Error for device failures

The ReadFailure and WriteFailure errors are now
available in Common.Device, and code will now use
those instead of the ones in Sensor.Device and
Control.D

Use Common.Device.Error for device failures

The ReadFailure and WriteFailure errors are now
available in Common.Device, and code will now use
those instead of the ones in Sensor.Device and
Control.Device, which were meant for sensor errors.

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

show more ...

Move power errors to org.open_power.witherspoon

Move xyz.openbmc_project.Power to org.open_power.Witherspoon.
This repo is Witherspoon specific, so moving to specific tree to have
its own namespace.

Move power errors to org.open_power.witherspoon

Move xyz.openbmc_project.Power to org.open_power.Witherspoon.
This repo is Witherspoon specific, so moving to specific tree to have
its own namespace. Update generated file, and corresponding namespace
statements in the source files.

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

show more ...

Shutdown system on GPU over-temps

Resolves openbmc/openbmc#1726

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

Find and call out faulted GPUs

Isolate down to the GPU that caused the GPU PGOOD or
overtemp summary fault bit to turn on. On Witherspoon
this involves reading GPIOs on a pca9552 device to find
the

Find and call out faulted GPUs

Isolate down to the GPU that caused the GPU PGOOD or
overtemp summary fault bit to turn on. On Witherspoon
this involves reading GPIOs on a pca9552 device to find
the GPU signaling the fault.

GPUs are not currently in the inventory, so the code
isn't doing the standard callout by adding a certain
metadata field. The GPU number that failed will just
be added to the error log metadata, and work will be done
with support to make sure that is documented. Also, the
other power fault callouts don't use the standard inventory
callouts either as they are more complicated than just a single
FRU, so this method is consistent with that.

Note that these faults do not cause the system to
power off automatically like other power faults, though
a future commit will power off the system on a GPU overtemp.

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

show more ...