phosphor-gpio-monitor: ignore extra args

The phosphor-gpio-monitor@.service template has an EXTRA_ARGS which
in some configs can be empty. systemd will end up launching the
process with an extra, e

phosphor-gpio-monitor: ignore extra args

The phosphor-gpio-monitor@.service template has an EXTRA_ARGS which
in some configs can be empty. systemd will end up launching the
process with an extra, empty argument. CLI11 fails on this unless
we give an explicit `allow_extra` call.

Observed in hardware testing:
```
Aug 09 19:28:46 witherspoon-Y230UF71K03T phosphor-gpio-monitor[632]: The following argument was not expected:
Aug 09 19:28:46 witherspoon-Y230UF71K03T phosphor-gpio-monitor[632]: Run with --help for more information.
```

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

show more ...

phosphor-gpio-monitor: fix continue flag

"--continue" is intended to be a flag and not a boolean option[1].

Change the CLI11 call so that we do not need a boolean value for it.

[1]: https://github

phosphor-gpio-monitor: fix continue flag

"--continue" is intended to be a flag and not a boolean option[1].

Change the CLI11 call so that we do not need a boolean value for it.

[1]: https://github.com/openbmc/openbmc/blob/master//meta-ibm/meta-romulus/recipes-phosphor/gpio/id-button/obmc/gpio/id_button#L5

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

show more ...

phosphor-gpio-monitor: Replace Argument class with CLI11

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

logging: switch to lg2

After switching to C++20, it is recommended to use `phosphor::lg2`
to format log, and the correct `CODE_LINE` and `CODE_FUNC` values
can be used in log tracking.

Signed-off-b

logging: switch to lg2

After switching to C++20, it is recommended to use `phosphor::lg2`
to format log, and the correct `CODE_LINE` and `CODE_FUNC` values
can be used in log tracking.

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

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

Add clang-format to repo

Add clang-format to repo.

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

Add --continue argument

The monitor will exit when key is pressed.
Add --continue argument and if it is set to "true", the monitor will
continue after key is pressed.

This i

Add --continue argument

The monitor will exit when key is pressed.
Add --continue argument and if it is set to "true", the monitor will
continue after key is pressed.

This is useful in cases to monitor a button presses.
E.g. It can be used to monitor ID button and start a system unit
everytime when the button is pressed.

This argument is optional so it does not break the current code.

Tested: With "--continue=true", the monitor does not exit when the key
is pressed.
Without the argument of if it is set to other strings, the
monitor exits after the key is pressed.

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

show more ...

Call defined target when GPIO line is asserted

Change-Id: Ia5445a8d6585acfec69783ba158c8d866d526e97
Signed-off-by: Vishwanatha Subbanna <vishwa@linux.vnet.ibm.com>

Add sd_event loop and callback handler

Change-Id: I032d1156be2b8082fdf347a60ec883a9bc0038ae
Signed-off-by: Vishwanatha Subbanna <vishwa@linux.vnet.ibm.com>

Populate the file descriptor for the device

On a GPIO chip, there could be many gpio lines and for each line,
there would be a corresponding input device event file.
To know the asse

Populate the file descriptor for the device

On a GPIO chip, there could be many gpio lines and for each line,
there would be a corresponding input device event file.
To know the assertion state of a GPIO line, a descriptor is needed.
This descriptor will later be plugged into sd_event so that the
GPIO state changes can be caught and handled.

Change-Id: Idc8c2b429688fea2a5124b96677085b1be48128b
Signed-off-by: Vishwanatha Subbanna <vishwa@linux.vnet.ibm.com>

show more ...

Add code to accept command line arguments

Accepts command line arguments for path, interested GPIO transition
and an optional systemd unit file to be called into when the GPIO
transi

Add code to accept command line arguments

Accepts command line arguments for path, interested GPIO transition
and an optional systemd unit file to be called into when the GPIO
transitions per expectation.

Change-Id: I98d967d36cfbb768bc9d0dd04517575fa7e408fe
Signed-off-by: Vishwanatha Subbanna <vishwa@linux.vnet.ibm.com>

show more ...