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

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

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

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

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

skeleton: Use ioctl gpio handling

Now that chassiskill is a standalone app and is
using ioctl gpio handling (through the gpioplus library),
the gpio libraries here need to use ioctl as well for
chas

skeleton: Use ioctl gpio handling

Now that chassiskill is a standalone app and is
using ioctl gpio handling (through the gpioplus library),
the gpio libraries here need to use ioctl as well for
chassiskill to be compatible.

Tested: Verified chassiskill works as intended and checked
that nothing else broke.

Change-Id: I67c431027d31e87c332c3e1771fc2d3423f56652
Signed-off-by: Anthony Wilson <wilsonan@us.ibm.com>

show more ...

Remove unnecessary args from GPIO functions

Now that gpio_init() and read_gpios() doesn't use
D-Bus, they don't need the D-Bus connection parameter.

Change-Id: Id7f3ee6547bed7c0c0eed5d75cba2d6ff239

Remove unnecessary args from GPIO functions

Now that gpio_init() and read_gpios() doesn't use
D-Bus, they don't need the D-Bus connection parameter.

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

show more ...

Remove host control GPIOs from GpioConfigs

These are not used anymore.

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

Fill in the function that finds the GPIO base

Finds the GPIO base value to use in the GPIO number calculation.

This is most likely specific to ASPEED BMCs, though as with
the calling code additiona

Fill in the function that finds the GPIO base

Finds the GPIO base value to use in the GPIO number calculation.

This is most likely specific to ASPEED BMCs, though as with
the calling code additional support can be added in the
future if required.

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

show more ...

Fill in function to convert GPIO pin to a number

Convert the pin, like "A7" to the number that identifies
that GPIO in /sys/class/gpio/.

This most likely only supports ASPEED BMCs, though of course

Fill in function to convert GPIO pin to a number

Convert the pin, like "A7" to the number that identifies
that GPIO in /sys/class/gpio/.

This most likely only supports ASPEED BMCs, though of course
support can be added for others if the need arises and this
code is still in use.

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

show more ...

Use cJSON to get the GPIO definitions

In gpio_init, read the GPIO definitions out of the JSON
instead of D-Bus.

As many applications configure several GPIOs at once, the
cJSON structure for the GPI

Use cJSON to get the GPIO definitions

In gpio_init, read the GPIO definitions out of the JSON
instead of D-Bus.

As many applications configure several GPIOs at once, the
cJSON structure for the GPIOs stays loaded in memory until
explicitly freed by calling gpio_inits_done().

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

show more ...

Use cJSON to read the GPIO configuration

Change the read_gpios function to get the power GPIO
configuration out of JSON instead of D-Bus.

Note that the Host Control GPIOs are no longer used and wil

Use cJSON to read the GPIO configuration

Change the read_gpios function to get the power GPIO
configuration out of JSON instead of D-Bus.

Note that the Host Control GPIOs are no longer used and will
be removed from the GpioConfigs structure in a future commit.

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

show more ...

Add documentation on GPIO JSON

Document the JSON format for the GPIO configuration.

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

Add function to load in the GPIO data from JSON.

Use the cJSON module to load in the GPIO definitions used by
the other skeleton code into a cJSON structure. Its memory
should be released by callin

Add function to load in the GPIO data from JSON.

Use the cJSON module to load in the GPIO definitions used by
the other skeleton code into a cJSON structure. Its memory
should be released by calling cJSON_Delete on it when complete.

This is the first commit in a series that will move the GPIO
configuration data from D-Bus into a JSON file, which is both
more appropriate for read only data and the first step in
getting rid of the system_manager.py file that currently hosts it.

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

show more ...

hostctl: Move to kernel FSI interface

The Power8 systems now use the in-kernel FSI driver, so the GPIOs used
for bitbanging are not available. This modifies the FSI SBE startup
code to use the kerne

hostctl: Move to kernel FSI interface

The Power8 systems now use the in-kernel FSI driver, so the GPIOs used
for bitbanging are not available. This modifies the FSI SBE startup
code to use the kernel FSI infrastructure.

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

show more ...

Remove use of legacy system state

The OpenBMC state function has been moved to the phosphor-state-manager
repository. The chassis power state is tracked via systemd targets
so notification from this

Remove use of legacy system state

The OpenBMC state function has been moved to the phosphor-state-manager
repository. The chassis power state is tracked via systemd targets
so notification from this power application is no longer needed.

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

show more ...

Add lseek to support GPIO read/write

GPIO api does not support repeat read(read multi times after one open).
A workaround is close and reopen the device before the second read.
The root cause is tha

Add lseek to support GPIO read/write

GPIO api does not support repeat read(read multi times after one open).
A workaround is close and reopen the device before the second read.
The root cause is that the file descriptor is not moved to beginning of the file, this patch can fix this issue

Change-Id: I40e5602669c9ab6f0dddf5aa77040466cbfa3738
Signed-off-by: Yong Li <yong.b.li@linux.intel.com>

show more ...

Spelling fixes

Change-Id: Ia4ded0c195204543c3df671ea471610237f5f44b
Signed-off-by: Gunnar Mills <gmills@us.ibm.com>

Rename power_gpio to gpio_configs.

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

GPIO abstraction for power and host control

In libopenbmc_intf/power_gpio.c there are GPIO abstractions for power
related GPIOs.
Host control related GPIOs need to do the same abstraction.

The chan

GPIO abstraction for power and host control

In libopenbmc_intf/power_gpio.c there are GPIO abstractions for power
related GPIOs.
Host control related GPIOs need to do the same abstraction.

The changes include:
1. Add host control related GPIOs in gpio_configs and update
system_manager.py
2. Update control_host_obj.c to use the abstracted GPIOs
3. Update GPIO_CONFIGS for all machines and add host control related
GPIOs

power_gpio will be renamed to gpio_configs in next commit;

Resolves openbmc/openbmc#814

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

show more ...

Initialize GError* to NULL before calling g_dbus_proxy_new_syn

This fixes the crash issue of power_control when manually build and
replace executable or library.
The reason why it does not crash on

Initialize GError* to NULL before calling g_dbus_proxy_new_syn

This fixes the crash issue of power_control when manually build and
replace executable or library.
The reason why it does not crash on a full OpenBMC build may be due to
the prelink clear the stack (not sure).

Fixes openbmc/openbmc#818.

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

show more ...

op-pwrctl: Hold PCI resets until BootProgress passes BASE_INITIALIZATION state

When powering on host, hold PCI resets until hostboot passes stage
"BASE_INITIALIZATION"(0x14).

This fix applies to Fi

op-pwrctl: Hold PCI resets until BootProgress passes BASE_INITIALIZATION state

When powering on host, hold PCI resets until hostboot passes stage
"BASE_INITIALIZATION"(0x14).

This fix applies to Firestone and Garrison.

Partially resolves openbmc/openbmc#315

Change-Id: Ic71c81406ac188b34df89569e2264ea0b94406f3
Signed-off-by: Yi Li <adamliyi@msn.com>

show more ...

Add IPMI 2.0 ColdReset command support

* Implement reset by reboot command issued in a shell
* Work with related changes in phosphor-host-ipmid repo

Resolves openbmc/openbmc#437

Change-Id: I8143b1

Add IPMI 2.0 ColdReset command support

* Implement reset by reboot command issued in a shell
* Work with related changes in phosphor-host-ipmid repo

Resolves openbmc/openbmc#437

Change-Id: I8143b1ee9c0d547ee47e5bd3659a5126b6e5af98
Signed-off-by: Nan Li <william.bjlinan@hotmail.com>

show more ...