clang-format: re-format for clang-18

clang-format-18 isn't compatible with the clang-format-17 output, so we
need to reformat the code with the latest version. The way clang-18
handles lambda forma

clang-format: re-format for clang-18

clang-format-18 isn't compatible with the clang-format-17 output, so we
need to reformat the code with the latest version. The way clang-18
handles lambda formatting also changed, so we have made changes to the
organization default style format to better handle lambda formatting.

See I5e08687e696dd240402a2780158664b7113def0e for updated style.
See Iea0776aaa7edd483fa395e23de25ebf5a6288f71 for clang-18 enablement.

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

show more ...

build: add cli11 dependency as necessary

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

power-sequencer: Replace Argument class with CLI11

Signed-off-by: George Liu <liuxiwei@inspur.com>
Change-Id: I8f48a1fb18ed2c190c8292317e883b05754a5558

meson: add necessary subprojects

Add subprojects for all the direct dependencies of the project such
that the project can be built outside of a Yocto SDK on a typical
Linux development system.

-

meson: add necessary subprojects

Add subprojects for all the direct dependencies of the project such
that the project can be built outside of a Yocto SDK on a typical
Linux development system.

- Update the .gitignore to ignore the subproject source directories.
- Update the .gitignore to ignore alternative build subdirectory
names as is typical with most projects in the openbmc organization.
- Add wrap files for direct dependencies, copied from other
repositories that already have the same dependencies.
- Modify the meson.build dependency for libgpiod so that the C++
bindings are included.
- Add many missing dependencies throughout.
- Sort all dependencies alphabetically for consistency.

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

show more ...

clang-format: copy latest and re-format

clang-format-16 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest

clang-format: copy latest and re-format

clang-format-16 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest .clang-format from the docs repository and reformat the
repository.

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

show more ...

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

python: fix flake8 warnings and format with black

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

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

catch exceptions as const

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

clang updates

The openbmc CI was recently updated to clang-12 which requires some
formatting updates to the code:
https://github.com/openbmc/openbmc-build-scripts/commit/4569bf49daaebff06be048242690

clang updates

The openbmc CI was recently updated to clang-12 which requires some
formatting updates to the code:
https://github.com/openbmc/openbmc-build-scripts/commit/4569bf49daaebff06be0482426908bb5497fad08

Make those updates and refresh the clang-format file with the one from
docs:
https://github.com/openbmc/docs/blob/master/style/cpp/.clang-format

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

show more ...

Fix the issue of clearRegister function

Modify the clearCPLDregister() mechanism to avoid clearing the
register record when the power error event still exists.
Before clearing the record, add condit

Fix the issue of clearRegister function

Modify the clearCPLDregister() mechanism to avoid clearing the
register record when the power error event still exists.
Before clearing the record, add conditions to determine whether
the psu pgood status is normal. (mihawk have two psus)

Change-Id: I28d353c13329145128b2633f134b01b7b3515038
Signed-off-by: Andy YF Wang <Andy_YF_Wang@wistron.com>

show more ...

Add CPLD class for Mihawk platform

If PGOOD signal is abnormal when chassis power_on, read
Mihawk's CPLD-register via I2C to confirm the error.

First, confirm whether the power_on_error signal is 1

Add CPLD class for Mihawk platform

If PGOOD signal is abnormal when chassis power_on, read
Mihawk's CPLD-register via I2C to confirm the error.

First, confirm whether the power_on_error signal is 1
when chassis power_on(1 means abnormal).
If the signal is 1, read the error-code-register to
analysis reason.

Second, runtime to confirm whether the power_ready_error
signal is 1 after chassis power_on(1 means abnormal).
If the signal is 1, read the error-code-register to
analysis reason and shutdown the chassis.

Tested:
Use command "obmcutil chassiskill" to trigger PGOOD error
action analysis during chassis power on.

Signed-off-by: Andy YF Wang <Andy_YF_Wang@wistron.com>
Change-Id: I5f9c0d508627324a6c784ded125c28f0437bf52d
Signed-off-by: Alvin Wang <alvinwang@msn.com>

show more ...

Use python3

python2 is EOL, specify the python scripts to use python3.

Tested: Verify it builds fine in local CI. With bitbake recipe changes,
verify it builds OK with python3native.

Signe

Use python3

python2 is EOL, specify the python scripts to use python3.

Tested: Verify it builds fine in local CI. With bitbake recipe changes,
verify it builds OK with python3native.

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

show more ...

Update .clang-format to latest version from docs

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

Update example power sequencer YAML

Update the example power sequencer YAML. Sync to Witherspoon YAML,
including demonstrating prefered device paths.

Signed-off-by: Jim Wright <jlwright@us.ibm.com

Update example power sequencer YAML

Update the example power sequencer YAML. Sync to Witherspoon YAML,
including demonstrating prefered device paths.

Signed-off-by: Jim Wright <jlwright@us.ibm.com>
Change-Id: I2ef4f31a545cac7b39976c919b6358a943e37ee1

show more ...

Make GPIOAnalysis section optional in power sequencer YAML

Per https://gerrit.openbmc-project.xyz/c/openbmc/meta-ibm/+/26103,
implement Matt's better fix and make 'GPIOAnalysis' section optional.
Ra

Make GPIOAnalysis section optional in power sequencer YAML

Per https://gerrit.openbmc-project.xyz/c/openbmc/meta-ibm/+/26103,
implement Matt's better fix and make 'GPIOAnalysis' section optional.
Rainier system does not need this section. Update to remove dummy
entry from Rainier power sequencer YAML to follow.

Signed-off-by: Jim Wright <jlwright@us.ibm.com>
Change-Id: I792976472cdb10c05746aa75dabf22b2b179481f

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

build: drop autotools

With previously added support for building with meson/ninja autotools
support is no longer required.

Change-Id: I80654ada9d099a57447ded73da71f39293cd2f0a
Signed-off-by: Brad B

build: drop autotools

With previously added support for building with meson/ninja autotools
support is no longer required.

Change-Id: I80654ada9d099a57447ded73da71f39293cd2f0a
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

show more ...

build: add support for building with meson

Follow the OpenBMC herd and support a more modern, more comprehensible,
more performant build framework.

To build using meson:
meson build
ninja -C bu

build: add support for building with meson

Follow the OpenBMC herd and support a more modern, more comprehensible,
more performant build framework.

To build using meson:
meson build
ninja -C build
ninja -C build install

Maintained support for:
Configurable dbus parameters:
INPUT_HISTORY_BUSNAME_ROOT= -> -Dinput-history-busname-root=
INPUT_HISTORY_SENSOR_ROOT = -> -Dinput-history-sensor-root=
UCD90160 support:
UCD90160_DEF_YAML_FILE= -> -Ducd90160-yaml=
--enable-turn-off-ucd90160-access = -> -Ducd90160-access=

Added -Dtests to match de-facto OpenBMC meson usage conventions.

Dropped UCD90160_DEF_OUTPUT_DIR. Much like the destination of
intermediate artifacts like object files are not configurable, the
location of the generated UCD definition file should not be configurable
either.

Dropped support for --enable-oe-sdk rpath munging. This was a
workaround for broken oe sdks that don't figure out the correct rpath
when running make check or ninja test.

Tested by building with both meson and autotools and then compared build
artifacts from each build system:
generated source code is the same
generated binaries have the same symbols

Ran existing unit tests (which passed) using ninja test.

Change-Id: Iac80ba1fe5c2d01abe0e0a95e0bec03b8b828ef5
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

show more ...

build: fix some warnings

Fix a couple of warnings:

-Werror=unused-parameter
-Werror=non-virtual-dtor

so we can use meson's warning_level=3 and -Werror without build
failures.

Change-Id: Id3c15c2f

build: fix some warnings

Fix a couple of warnings:

-Werror=unused-parameter
-Werror=non-virtual-dtor

so we can use meson's warning_level=3 and -Werror without build
failures.

Change-Id: Id3c15c2fee70799bee98a8d2890c33ab5ebfd563
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

show more ...

Don't try to read GPIOs when no device path

For certain PGOOD faults, the code may try to read a GPIO off of an IO
expander to further isolate the error. If there was a problem with the
device driv

Don't try to read GPIOs when no device path

For certain PGOOD faults, the code may try to read a GPIO off of an IO
expander to further isolate the error. If there was a problem with the
device driver detecting that IO expander, then the code was running down
a path where it would still try to read that IO expander, but using an
empty device path which would just generate extra error traces in the
journal. Avoid this by checking for this empty path and then not
reading those GPIOs.

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

show more ...

Add YAML and error function for MEM_GOODx faults

Add a section to the error and metadata YAML files for the MEM_GOODx
faults. Use that to update/generate a new elog-errors.hpp include file.

Update

Add YAML and error function for MEM_GOODx faults

Add a section to the error and metadata YAML files for the MEM_GOODx
faults. Use that to update/generate a new elog-errors.hpp include file.

Update the power sequencer source to have a memGoodFault function for
handling callouts for MEM_GOODx type errors.

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

show more ...

Update to get GPIOAnalysis from parsing YAML

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

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>