Add support for multiple gpio banks

Currently, the code assumes that all the GPIOs are on one bank. Support
for systems that do not use the alphanumeric reference and have multiple
gpio banks is add

Add support for multiple gpio banks

Currently, the code assumes that all the GPIOs are on one bank. Support
for systems that do not use the alphanumeric reference and have multiple
gpio banks is added. Using the num/pin key distinction, the parsing
process is differentiated between an alphanumeric reference and integer
labeling.

NOTE: Since this change assumes /dev/gpiochip0 is present, systems using
it need to be using at least Linux kernel 4.8.

Change-Id: Ib11531bf326f6a9291fc03caaad323c0779b5a0d
Signed-off-by: Jonico Eustaquio <jonico.eustaquio@fii-na.com>

show more ...

meson: add basic meson

Since we don't have a CI for standard Make repositories anymore,
this repository has no CI. Begin a simple meson conversion to
enable some CI for the bare minimum aspects of

meson: add basic meson

Since we don't have a CI for standard Make repositories anymore,
this repository has no CI. Begin a simple meson conversion to
enable some CI for the bare minimum aspects of the repository still
being used in openbmc/openbmc.

- libopenbmc_intf
- op-pwrctl
- pgood_wait
- op-hostctl

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

show more ...

op-hostctl: fix various warnings

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

pgood_wait: fix various warnings

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

op-pwrctl: fix various warnings

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

libopenbmc_intf: fix various warnings

Trying to compile with stricter warnings results in numerous
errors, such as ignored results and unsigned/signed comparisons.
Fix these in libopenbmc_intf.

Sig

libopenbmc_intf: fix various warnings

Trying to compile with stricter warnings results in numerous
errors, such as ignored results and unsigned/signed comparisons.
Fix these in libopenbmc_intf.

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

show more ...

openbmc_intf: regenerate with latest gdbus-codegen

The openbmc_intf.[ch] have not been regenerated in over 6 years
and the existing ones do not compile with stricter compiler
settings. Regenerate t

openbmc_intf: regenerate with latest gdbus-codegen

The openbmc_intf.[ch] have not been regenerated in over 6 years
and the existing ones do not compile with stricter compiler
settings. Regenerate them.

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

show more ...

pgood_wait: fix bool include

We don't actually have a CI build type for plain Make anymore, so we
don't get any repository level CI done for this repository. The
Yocto recipe update failed with the

pgood_wait: fix bool include

We don't actually have a CI build type for plain Make anymore, so we
don't get any repository level CI done for this repository. The
Yocto recipe update failed with the following:

```
15:12:53 | pgood_wait.c:95:16: error: 'true' undeclared (first use in this function)
```

Make a simple fix to add `stdbool.h` in hopes that this solves it.

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

show more ...

Fix MD040 warnings

The following warnings are generated by using markdownlint analysis:
```
MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
```
Refer

Fix MD040 warnings

The following warnings are generated by using markdownlint analysis:
```
MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
```
Refer to markdown-lint [1] to fix MD040
[1]: https://github.com/updownpress/markdown-lint/blob/master/rules/040-fenced-code-language.md

Signed-off-by: George Liu <liuxiwei@ieisystem.com>
Change-Id: I7c5f191faacead8091a76d54230803d3bf6e6d4b

show more ...

Remove libmapper dependency

In the latest openbmc repo, there is no need to rely on libmapper.
This patch is to remove the dependence on libmapper, and then
libmapper will be deleted in the future a

Remove libmapper dependency

In the latest openbmc repo, there is no need to rely on libmapper.
This patch is to remove the dependence on libmapper, and then
libmapper will be deleted in the future and mapper calls will be used
directly.

Signed-off-by: George Liu <liuxiwei@ieisystem.com>
Change-Id: Ibe9c345e760e5a4215a2fe3c3c2ace3c54f8e179

show more ...

black: re-format

black and isort are enabled in the openbmc-build-scripts on Python files
to have a consistent formatting. Re-run the formatter on the whole
repository.

Change-Id: Ieb4226b16132640

black: re-format

black and isort are enabled in the openbmc-build-scripts on Python files
to have a consistent formatting. Re-run the formatter on the whole
repository.

Change-Id: Ieb4226b161326404ccd8c14abfef5ea5bc442fba
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: I82877be37f35cb7373294a1d3c53e45097d1a745
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

linter: disable python2 scripts

There are numerous python2 scripts in this repository that do not
conform to pycodestyle/flake8. Disable linters on them so that we
can enable these globally.

Signe

linter: disable python2 scripts

There are numerous python2 scripts in this repository that do not
conform to pycodestyle/flake8. Disable linters on them so that we
can enable these globally.

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

show more ...

OWNERS: update

Add Adriana as maintainer. She was an early contributor to the project,
and even though this project is used in a limited capacity and may
eventually be deprecated, she can maintain a

OWNERS: update

Add Adriana as maintainer. She was an early contributor to the project,
and even though this project is used in a limited capacity and may
eventually be deprecated, she can maintain any changes for now.

Fixes: openbmc/skeleton#138

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

show more ...

OWNERS: update

It's been quite some time since I've helped maintain skeleton 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 he

OWNERS: update

It's been quite some time since I've helped maintain skeleton 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: I06e151f746bd0f50434bb5e770048cdf49bdd1e0

show more ...

MAINTAINERS: remove file

The MAINTAINERS file is deprecated in favor of OWNERS.

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

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

show more ...

python: reformat using black

Code cannot currently be submitted to this repo because it fails
in the unit-test phase. Attempt to reformat the code using `black`
to hopefully make pycodestyle happie

python: reformat using black

Code cannot currently be submitted to this repo because it fails
in the unit-test phase. Attempt to reformat the code using `black`
to hopefully make pycodestyle happier.

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

show more ...

Add OWNERS file

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

Fix build time error

The new security flags require that calls to system check their return
code, this one doesn't.

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

Fix build time error

The new security flags require that calls to system check their return
code, this one doesn't.

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

show more ...

Fix -Wformat-security issue

An extra set of parens is likely not what this line intended to do.
This was caught by -Wformat-security flags.

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

Fix -Wformat-security issue

An extra set of parens is likely not what this line intended to do.
This was caught by -Wformat-security flags.

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

show more ...

fix sizeof bug

This code path is most likely not utilized anymore but upstream yocto is
calling out a compile error on this:

| control_host_obj.c: In function 'on_boot':
| control_host_obj.c:131:8:

fix sizeof bug

This code path is most likely not utilized anymore but upstream yocto is
calling out a compile error on this:

| control_host_obj.c: In function 'on_boot':
| control_host_obj.c:131:8: error: 'write' reading 4 bytes from a region of size 2 [-Werror=stringop-overflow=]
| 131 | rc = write(fd, one, sizeof(one));
| | ^~~~~~~~~~~~~~~~~~~~~~~~~~~

Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Iabb5bcc0b888a4e831c8fe8c1006e9b99097402a

show more ...

Add Joel to maintainers

Signed-off-by: Joel Stanley <joel@jms.id.au>
Change-Id: I9332ba81bf5ad2dd3257927160b1e1569372b9a2

control-host: Use fsi-master class paths

As of dev-5.3, and upstream v5.4, the FSI core creates fsi master class
based sysfs paths. These paths do not change depending on the FSI
master implantation

control-host: Use fsi-master class paths

As of dev-5.3, and upstream v5.4, the FSI core creates fsi master class
based sysfs paths. These paths do not change depending on the FSI
master implantation or device tree layout, and therefore should be more
stable.

Signed-off-by: Joel Stanley <joel@jms.id.au>
Change-Id: I279024ca73e3f3d87c3456cf4fcd8d54615cd285

show more ...

Revert "Revert "skeleton: Remove obmcutil""

This reverts commit c0da84767c00657c0b3b1afff7d9c5c2f1ed5fdb.

We have switched to the bash implementation of obmcutil, so this is no
longer required.

Ch

Revert "Revert "skeleton: Remove obmcutil""

This reverts commit c0da84767c00657c0b3b1afff7d9c5c2f1ed5fdb.

We have switched to the bash implementation of obmcutil, so this is no
longer required.

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

show more ...