x86-power-control: add passthrough option

Add a power and reset button passthrough option
for emulating button behavior by directly controlling
the control output pin.

Change-Id: I50df3e604401b4afd

x86-power-control: add passthrough option

Add a power and reset button passthrough option
for emulating button behavior by directly controlling
the control output pin.

Change-Id: I50df3e604401b4afd43fa89a0709466bbf01638b
Signed-off-by: Renze Nicolai <renze@rnplus.nl>

show more ...

Update PostComplete polarity

After recent changes adding support for GPIO polarity in the config file
and using only active-high polarity for D-Bus, the PostComplete polarity
was left out of sync in

Update PostComplete polarity

After recent changes adding support for GPIO polarity in the config file
and using only active-high polarity for D-Bus, the PostComplete polarity
was left out of sync in a couple places.

This fixes PostComplete, so the correct polarity is used in all cases.

Tested:
Confirmed in D-Bus mode that PostComplete correctly initializes the OS
State and correctly updates the OS State on PropertiesChanged events.

Change-Id: Iae626de79b5f13292372081f14ca5ca75420a1c2
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>

show more ...

OWNERS: Remove Zev

I'm changing jobs and am no longer going to be working on OpenBMC.

Signed-off-by: Zev Weiss <zev@bewilderbeest.net>
Change-Id: I81b42ed92325feea1c1965f199ec0567ca900bc8

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: I474dc261d5698b702bab7b9305c1d6958f43d40f
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

Filter more precisely in dbusGPIOMatcher()

Previously we'd been matching only on the interface name. It is
possible, however, that there might exist multiple providers of the same
interface while w

Filter more precisely in dbusGPIOMatcher()

Previously we'd been matching only on the interface name. It is
possible, however, that there might exist multiple providers of the same
interface while we only want to match on a specific object from one
provider. Since we have the service name and object path available in
the config, let's add them to the match filter rule to specify only
exactly the signals we're interested in.

Signed-off-by: Zev Weiss <zev@bewilderbeest.net>
Change-Id: Ib29c5d021e519c3501b177f6e26eda664cc29964

show more ...

Ensure all dbus inputs are treated as active-high

Previously the polarity member of the ConfigData struct wasn't getting
set in loadConfigValues(), so it retained its default value
(active-low). Ac

Ensure all dbus inputs are treated as active-high

Previously the polarity member of the ConfigData struct wasn't getting
set in loadConfigValues(), so it retained its default value
(active-low). Active-low inputs via dbus don't make much sense, so
let's normalize them all to be active-high.

Signed-off-by: Zev Weiss <zev@bewilderbeest.net>
Change-Id: I529fd6362db9a49b859f97a231c09d23a782c23d

show more ...

Extend dbus config with MatchRegex support

Previously only boolean properties were supported as dbus inputs. For
some signals other types may be useful however, one common case being
strings for en

Extend dbus config with MatchRegex support

Previously only boolean properties were supported as dbus inputs. For
some signals other types may be useful however, one common case being
strings for enum values. We can now specify a MatchRegex key in a dbus
input's config block to specify a regular expression that determines
whether a given string value is converted to true (asserted) or false.

Signed-off-by: Zev Weiss <zev@bewilderbeest.net>
Change-Id: Ib048151cb20d4c6a9370587d2ee583cb92f617f6

show more ...

Respect polarity config on input signals

Previously the polarity setting was (in all but one case) only being
used in setting outputs driven by the daemon. With this change we also
take it into acc

Respect polarity config on input signals

Previously the polarity setting was (in all but one case) only being
used in setting outputs driven by the daemon. With this change we also
take it into account in determining whether input signals are asserted
or deasserted.

Signed-off-by: Zev Weiss <zev@bewilderbeest.net>
Change-Id: Ifd326e9c64de144dc45d217be5212329a5f3d242

show more ...

meson: Provide wrap file for boost

Currently local build on the system without boost can fail with
the following message:
"""
meson.build: ERROR: Dependency "boost" not found, tried system
"""
Provi

meson: Provide wrap file for boost

Currently local build on the system without boost can fail with
the following message:
"""
meson.build: ERROR: Dependency "boost" not found, tried system
"""
Provide "boost.wrap" file to fix the issue.

Change-Id: Id63f70c02369923959d42c1cd621110f6b42af33
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>

show more ...

build: Add boost to the dependencies

Since the project uses boost library add it to meson dependencies.

Change-Id: I0c075ff0deaa1feb0be1a8a56d9c21eab305345c
Signed-off-by: Konstantin Aladyshev <ala

build: Add boost to the dependencies

Since the project uses boost library add it to meson dependencies.

Change-Id: I0c075ff0deaa1feb0be1a8a56d9c21eab305345c
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>

show more ...

Move OS state interface to state/hostX

Move OS state interface from state/os to state/hostX to make it
consistent with phosphor-state-manager.

Tested result:
```
root@harma:~# busctl introspect xyz

Move OS state interface to state/hostX

Move OS state interface from state/os to state/hostX to make it
consistent with phosphor-state-manager.

Tested result:
```
root@harma:~# busctl introspect xyz.openbmc_project.State.Host0 /xyz/openbmc_project/state/host0 xyz.openbmc_project.State.OperatingSystem.Status
NAME TYPE SIGNATURE RESULT/VALUE FLAGS
.OperatingSystemState property s "xyz.openbmc_project.State.OperatingSys… emits-change
```

Signed-off-by: Potin Lai <potin.lai@quantatw.com>
Change-Id: Idd744285ce3c43add37d17ddd2de4eb6c250853d

show more ...

Correct naming of NMI button in BMCSourceSignal

NMISource related properties already moved from "intel-dbus-interface"
to "phospher-dbus-interfaces".

However, the name "FpBtn" is using for "com.int

Correct naming of NMI button in BMCSourceSignal

NMISource related properties already moved from "intel-dbus-interface"
to "phospher-dbus-interfaces".

However, the name "FpBtn" is using for "com.intel", not for
"xyz.openbmc_project" that is using "FrontPanelButton" according to
"xyz/openbmc_project/Chassis/Control/NMISource.interface.yaml".

Thus, need to correct this naming to match current design.

Signed-off-by: Tim Lee <timlee660101@gmail.com>
Change-Id: I9629950fbb077ebf59cd3ef14e382fd40a739895

show more ...

Add HpmStbyEn signal

HpmStbyEn signal is a special signal that can be found on AMD boards.
It indicates that all DC-SCM STBY power rails are enabled and good.
The signal enables STBY power rails on

Add HpmStbyEn signal

HpmStbyEn signal is a special signal that can be found on AMD boards.
It indicates that all DC-SCM STBY power rails are enabled and good.
The signal enables STBY power rails on the HPM.
It is a level GPIO, and if it is not activated, onboard CPLD wouldn't
relinquish control of power control GPIOs to BMC making it impossible
to power on the host.
Therefore if this GPIO is present in the JSON configuration activate it
in the main function.

Tested on the AMD EthanolX/Onyx CRB
Onyx verification: power on/off, application restart at power on/off
and bmc reboot.

Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>
Change-Id: I3a21c2e5214dfcf05cdae8efc2ebbb21c4b4e080

show more ...

OWNERS: Update Jason's email address

This updates the OWNERS file with Jason's Gerrit email address so
approvals will work correctly.

Change-Id: I9541b5c1003f9e17c976cd972908b9b15b94cb77
Signed-off

OWNERS: Update Jason's email address

This updates the OWNERS file with Jason's Gerrit email address so
approvals will work correctly.

Change-Id: I9541b5c1003f9e17c976cd972908b9b15b94cb77
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>

show more ...

event: reset button pressed event updates

Existing code doesn't handle "resetButtonPressed" correctly.
This commits adds the "resetButtonPressed" event handling
during power state on.

Tested: Verif

event: reset button pressed event updates

Existing code doesn't handle "resetButtonPressed" correctly.
This commits adds the "resetButtonPressed" event handling
during power state on.

Tested: Verified power sequencing on AMD CRB Onyx.

Change-Id: I65e5b2236032f7b04b4345da752b467981409612
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

meson: use include_type for nlohmann-json dependency

Using the include_type field aligns with usages in other repositories
and prevents the library from getting flagged by clang-tidy and similar
too

meson: use include_type for nlohmann-json dependency

Using the include_type field aligns with usages in other repositories
and prevents the library from getting flagged by clang-tidy and similar
tools.

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

show more ...

build: use allowed over enabled

Meson feature options are typically in a tri-state of enabled, disabled,
or auto. The enabled and disabled functions on an option (from
`get_option`) no longer retur

build: use allowed over enabled

Meson feature options are typically in a tri-state of enabled, disabled,
or auto. The enabled and disabled functions on an option (from
`get_option`) no longer return true for auto features. Instead, the
expectation is to use `allowed()` which is true for both enabled and auto.

Switch all uses of `enabled` to `allowed`.

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

show more ...

clang-format: copy latest and re-format

clang-format-17 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-17 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: Iff253307fce5ebb3dff66b87e2742919cc5b30b8
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

meson_options.txt: Support for reading options from meson.options

Support has been added for reading options from meson.options instead
of meson_options.txt[1]. These are equivalent, but not using t

meson_options.txt: Support for reading options from meson.options

Support has been added for reading options from meson.options instead
of meson_options.txt[1]. These are equivalent, but not using the .txt
extension for a build file has a few advantages, chief among them
many tools and text editors expect a file with the .txt extension to
be plain text files, not build scripts.

[1] https://mesonbuild.com/Release-notes-for-1-1-0.html#support-for-reading-options-from-mesonoptions

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

show more ...

build: upgrade to C++23

Meson 1.1.1 and GCC-13 both support C++23 and a sufficient portion of
the standard has been implemented. Upgrade the build to leverage it.

Change-Id: I9c02f330a8f0cc2924d45

build: upgrade to C++23

Meson 1.1.1 and GCC-13 both support C++23 and a sufficient portion of
the standard has been implemented. Upgrade the build to leverage it.

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

show more ...

Handle possible SdBus exceptions

Right now service allows to access several signals using DBus
interface, instead of GPIO pins. In case of that option selected in
config, service will try to read gi

Handle possible SdBus exceptions

Right now service allows to access several signals using DBus
interface, instead of GPIO pins. In case of that option selected in
config, service will try to read given DBus property using function
getProperty. Unfortunately that attempt might fail, for example when
that specific object doesn't exist yet, which will cause with
unhandled exception and evantually service restart.

This change adds handler for that sort of exception. If read property
fails, another one will be scheduled in the next second. That process
shall be repeated until value is successfully read. As there are
several possible properties to read, each one shall utilize its own
timer, so that every failure can be handled independently from each
other.

Tested:
Change has been tested on Intel platform, which showed symptoms
described above: service tried to get property which has not yet been
created, which caused its crash and restart. With this fix applied,
issue stopped reproducing.

Change-Id: Ib9b89e07a348868d2f15ffda31b3dc9a47340873
Signed-off-by: Michal Orzel <michalx.orzel@intel.com>

show more ...

Fix json exception when processing event_configs

power-control is crashing while processing the JSON config:

terminate called after throwing an instance of 'nlohmann::json_abi_v3_11_2::detail::type

Fix json exception when processing event_configs

power-control is crashing while processing the JSON config:

terminate called after throwing an instance of 'nlohmann::json_abi_v3_11_2::detail::type_error'
what(): [json.exception.type_error.302] type must be boolean, but is object

due to casting the `events` object to a `bool` when the underlying JSON
type is not a boolean.

Tested: Added debug print to show the value of `nmiWhenPoweredOff` and
confirmed that it worked for three cases: true, false, and not specified
in config (defaults to true).

Change-Id: I0182771a4125234df01ec39374531e5c51dec55a
Signed-off-by: Jonathan Doman <jonathan.doman@intel.com>

show more ...

Add option to only log nmiButtonPress when host is running

While NMI button presses are of little interest when the host is off,
some systems (e.g. romed8hm3) generates spurious events on shutdown,

Add option to only log nmiButtonPress when host is running

While NMI button presses are of little interest when the host is off,
some systems (e.g. romed8hm3) generates spurious events on shutdown,
probably due to some lack of electronic stability.
This patch adds an option to disable handling these events when host is off.

Change-Id: Idc74b2dcb7131ddf6d99d1b8c49d947be98de78e
Signed-off-by: Olivier FAURAX <olivier.faurax@eu.equinix.com>

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: I844971bd190187ba9ef2e8cf16809e2d16317ec4
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

meson: remove deprecated get_pkgconfig_variable

Since meson 0.56, the `get_pkgconfig_variable` has been deprecated. In
meson 0.58 the `get_variable` was enhanced to no longer require the
`pkgconfig

meson: remove deprecated get_pkgconfig_variable

Since meson 0.56, the `get_pkgconfig_variable` has been deprecated. In
meson 0.58 the `get_variable` was enhanced to no longer require the
`pkgconfig` keyword argument. Ensure meson 0.58 is required and update
the usage of all `get_pkgconfig_variable` and `get_variable` to be the
modern variant.

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

show more ...