Remove is_method_error method

Remove the usage of is_method_error()[1], and add try-catch to handle
D-Bus exceptions around mapper call.

[1]https://github.com/openbmc/sdbusplus/commit/079fb85a398d9

Remove is_method_error method

Remove the usage of is_method_error()[1], and add try-catch to handle
D-Bus exceptions around mapper call.

[1]https://github.com/openbmc/sdbusplus/commit/079fb85a398d90800935e3985bb1266a7530a26e#diff-945669e8bd9cab4ecc83a574a732921281b2c79eb8bba65efff11736ad18f92bR237-R240

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

show more ...

gpio-util: Remove Makefile.am

The Makefile.am file is no longer used in meson.

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

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: I04ae932ba0948b66f31cc

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

show more ...

meson: add subproject support

Enable building the repository outside of an OpenBMC Docker container or
SDK by leveraging meson subproject support. Copy wrap files from
elsewhere in the organization

meson: add subproject support

Enable building the repository outside of an OpenBMC Docker container or
SDK by leveraging meson subproject support. Copy wrap files from
elsewhere in the organization and update the meson dependencies for some
missing dependencies.

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

show more ...

presence: optimize emplace_back

The emplace_back call is created using temporary strings obtained from a
substr call which are not used past that point. Use std::move to avoid
string copy during th

presence: optimize emplace_back

The emplace_back call is created using temporary strings obtained from a
substr call which are not used past that point. Use std::move to avoid
string copy during the emplacement.

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

show more ...

meson: remove cppfs dependency and experimental::filesystem

The dependency on the c++fs library is something very old from the C++14
era and is no longer necessary with a modern compiler. Remove th

meson: remove cppfs dependency and experimental::filesystem

The dependency on the c++fs library is something very old from the C++14
era and is no longer necessary with a modern compiler. Remove the
explicit dependency from the meson setup.

Remove all use of `std::experimental::filesystem` and replace with the
real `std::filesystem`.

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

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

show more ...

Support event multi targets

Description:
- Support event multi targets.

Design:
- The origin multi-gpio-monitor doesn't detect the comming events are
rising or falling and it can only start one s

Support event multi targets

Description:
- Support event multi targets.

Design:
- The origin multi-gpio-monitor doesn't detect the comming events are
rising or falling and it can only start one service after events
triggered.

- We need to do corresponding actions when gpio pin rising or falling.
So we modify multi-gpio-monitor config json and read the rising
actions and falling actions in it.

- When service monitored one gpio status is changed,
service will detect that it is a rising or falling event and then
call systemd startUnit to start services that set in config json.

- For example with config json below:
When PowerGood is falling, "PowerGoodFalling.service"
and "PowerOff.service" will start.
[
{
"Name": "PowerGood",
"ChipId": "0",
"GpioNum": 14,
"EventMon": "BOTH",
"Targets": {
"FALLING": ["PowerGoodFalling.service", "PowerOff.service"],
"RISING": ["PowerGoodRising.service", "PowerOn.service"]
},
"Continue": true
}
]

Test Case:
Check that corresponding targets start or not - pass

Signed-off-by: Delphine CC Chiu <Delphine_CC_Chiu@wiwynn.com>
Change-Id: I043d4385b91a04d360a4d50048320db15e63ac74

show more ...

Change io_service to io_context

This was renamed a while back in boost to be compliant with the
std::executors proposal.

Change-Id: I221f120fc16d6b9e57eee17417cf7f5793c9629f
Signed-off-by: Ed Tanou

Change io_service to io_context

This was renamed a while back in boost to be compliant with the
std::executors proposal.

Change-Id: I221f120fc16d6b9e57eee17417cf7f5793c9629f
Signed-off-by: Ed Tanous <edtanous@google.com>

show more ...

reformat and fix markdownlint issues

Fix the prettier / markdownlint format issues in the repository.

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

reformat and fix markdownlint issues

Fix the prettier / markdownlint format issues in the repository.

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

show more ...

OWNERS: add Matt and Patrick

The last OWNER was removed from this repo. Add Matt and Patrick
as OWNERS based on the most recent activity in the last 3 years
to ensure this repository continues to h

OWNERS: add Matt and Patrick

The last OWNER was removed from this repo. Add Matt and Patrick
as OWNERS based on the most recent activity in the last 3 years
to ensure this repository continues to have coverage.

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

show more ...

OWNERS: update

It's been quite some time since I've helped maintain gpio-monitor in any
meaningful way, and I don't intend to going forward, so it is time to
drop my name from the list.

I'm happy t

OWNERS: update

It's been quite some time since I've helped maintain gpio-monitor in any
meaningful way, and I don't intend to going forward, so it is time to
drop my name from the list.

I'm happy to help debug, provide historical insight or help out in some
other way; however, and will try to make the time if anyone reaches out.

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

show more ...

MAINTAINERS: remove file

The MAINTAINERS file is deprecated in favor of OWNERS.

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

OWNERS: switch 'matches' to 'matchers'

The original OWNERS template had a mistake which used 'matches' instead
of the field supported by the Gerrit plugin 'matchers'. Update the
OWNERS file to have

OWNERS: switch 'matches' to 'matchers'

The original OWNERS template had a mistake which used 'matches' instead
of the field supported by the Gerrit plugin 'matchers'. Update the
OWNERS file to have the correct field.

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

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: Id8497e7fed0c6a278e29528d07486f5159687696

show more ...

Add OWNERS file

Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>
Change-Id: I9e43579b596826d6f7d814c3ca439656d95205ee

build: switch to C++20

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

catch exceptions as const

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

clang-format: apply clang-12 changes

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

presence: Get DRIVER_BIND_DELAY_MS value from env

Add a delay variable to the Presence class, defaulted to 0. Attempt to
set this value from the DRIVER_BIND_DELAY_MS environment variable. This
delay

presence: Get DRIVER_BIND_DELAY_MS value from env

Add a delay variable to the Presence class, defaulted to 0. Attempt to
set this value from the DRIVER_BIND_DELAY_MS environment variable. This
delay is then used between seeing the power supply present and binding
the device driver.

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

show more ...

Fix SDK build issue due to updated arm-sdk

After a recent change to enable security flags:
d47f9a099a4ef992351237b0f2f562b5dc843397
compilation of the code in this repository with the arm-sdk result

Fix SDK build issue due to updated arm-sdk

After a recent change to enable security flags:
d47f9a099a4ef992351237b0f2f562b5dc843397
compilation of the code in this repository with the arm-sdk results in
error messages from FORTIFY_SOURCE, which only compiles with
optimization levels 1 and 2.

Change the default build type to debugoptimized.

Reference changes:
I910256c3aeffcbab0cbc33ced541b90576ce4151
Ic1a91e58253fed283f67c44989fbc9126e18b7fe

Tested:
SDK build with arm-sdk passes.
Bitbake build passes.

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

show more ...

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

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

Enable restart for gpio monitor

1. Additional restart service setting to avoid
GPIO service stop as it is crashed.
- Restart=no
+ Restart=always
+ RestartSec=5

Tested:
It has been teste

Enable restart for gpio monitor

1. Additional restart service setting to avoid
GPIO service stop as it is crashed.
- Restart=no
+ Restart=always
+ RestartSec=5

Tested:
It has been tested on system with openbmc

Signed-off-by: Payne Yang <pyang4@lenovo.com>
Change-Id: Idd332dda4b2c8391fd9c2ba1844a1c69b55b8e76

show more ...

Fix "Add device dependency for services"

Not all systems use gpio-keys-polled, so adding a hard requirement on
that in the service files will cause the services to fail on those
systems.

Signed-off

Fix "Add device dependency for services"

Not all systems use gpio-keys-polled, so adding a hard requirement on
that in the service files will cause the services to fail on those
systems.

Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
Change-Id: I62ab4ebc496bc1fb3ed7f2fc258930f2e509a189
Fixes: 58dc277a9df27cc79565b20f48d42c95a877d3c2.

show more ...