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

Call gpio_inits_done() from GPIO consumers

This frees the cJSON structures that were created in
the gpio_init() calls. Doing it this way allows the
JSON to be parsed only once across multiple gpio_

Call gpio_inits_done() from GPIO consumers

This frees the cJSON structures that were created in
the gpio_init() calls. Doing it this way allows the
JSON to be parsed only once across multiple gpio_init()
calls.

Change-Id: Ia2c37c941b458fe271287b441c30b00dcb67d742
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 ...

Add MAINTAINERS file

Change-Id: I0980c573087588bbdac7f224b36bdb9c48370d66
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

Spelling fixes

Spelling errors found using github.com/lucasdemarchi/codespell
A tool to fix common misspellings.
This tool is licensed under GNU General Public License, version 2.

Change-Id: Ic21c2

Spelling fixes

Spelling errors found using github.com/lucasdemarchi/codespell
A tool to fix common misspellings.
This tool is licensed under GNU General Public License, version 2.

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

show more ...

Add error handing to recovery from gpio config failure

Sometimes these chassis related services will fail with below error:
Power GPIO: call to getGpioConfiguration failed:
GDBus.Error:org.freedeskt

Add error handing to recovery from gpio config failure

Sometimes these chassis related services will fail with below error:
Power GPIO: call to getGpioConfiguration failed:
GDBus.Error:org.freedesktop.DBus.Error.Service
Unknown: The name org.openbmc.managers.System was not provided by any
.service files
These gpio pins cannot be configured properly, and cannot power on/off
host.
Add error handing to terminate the service in such error, systemd will
restart them to try again

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

show more ...

bmc_update.py: Use BMC state manager to reboot BMC

It was using BMC control service to call warmReset() to reboot BMC.
But now legacy BMC control service is removed so the call will fail.

Use BMC s

bmc_update.py: Use BMC state manager to reboot BMC

It was using BMC control service to call warmReset() to reboot BMC.
But now legacy BMC control service is removed so the call will fail.

Use BMC state manager instead to trigger BMC reboot.

Partially resolves openbmc/openbmc#3064.

Tested: Run legacy code update and verify PrepareForupdate() does reboot
the BMC.

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

show more ...

Remove inventory_items.py from installation

The old inventory manager is no longer needed with the
final integration of phosphor-inventory-manager complete

https://gerrit.openbmc-project.xyz/#/c/87

Remove inventory_items.py from installation

The old inventory manager is no longer needed with the
final integration of phosphor-inventory-manager complete

https://gerrit.openbmc-project.xyz/#/c/8793/ must be
merged before this commit.

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

show more ...

Add getObjectFrom* interfaces back in

Got a bit zealous with the removal of "legacy" interfaces
from system_manager.py in https://gerrit.openbmc-project.xyz/#/c/8874/
and removed some still required

Add getObjectFrom* interfaces back in

Got a bit zealous with the removal of "legacy" interfaces
from system_manager.py in https://gerrit.openbmc-project.xyz/#/c/8874/
and removed some still required by phosphor-host-ipmid code for
some of the legacy sensors.

Only adding the minimum required back in with this commit.

Tested: Flashed image on witherspoon and verified host errors
are no longer reported.

Resolves openbmc/openbmc#2975

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

show more ...

Port to python 3

Port code to python 3 yet remain backward compatible to python 2.
This port is in preparation for yocto 2.4 upgrade.

Partially resolves openbmc/openbmc#2920

Change-Id: If174ce9444

Port to python 3

Port code to python 3 yet remain backward compatible to python 2.
This port is in preparation for yocto 2.4 upgrade.

Partially resolves openbmc/openbmc#2920

Change-Id: If174ce9444bde4e21fb086764f4def017e423201
Signed-off-by: CamVan Nguyen <ctnguyen@us.ibm.com>

show more ...

pytools: Add completion script for obmcutil

Change-Id: Ia248f971e2c8a1ff80f1e6947ef0af283d4ec9ba
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

Remove all legacy function from system manager

This commit removes all of the un-used interfaces in
order to get it down to its bare function and to
make eventual removal of this legacy application

Remove all legacy function from system manager

This commit removes all of the un-used interfaces in
order to get it down to its bare function and to
make eventual removal of this legacy application
easier

Tested: Built image and verified witherspoon system
still powers on as expected and no new errors or issues
show up in basic regression.

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

show more ...

Remove use of settings in chassis_control.py

This application is slowly being deprecated. This change
is to allow us to pull the legacy settings object out
of the image.

Tested: I built an image wi

Remove use of settings in chassis_control.py

This application is slowly being deprecated. This change
is to allow us to pull the legacy settings object out
of the image.

Tested: I built an image with these changes plus the removal of
settings and verified our CT regression bucket still passed.

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

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

pep8 fixes

Also ignore the configs/ directory since it only contains
system configuration files named <system>.py.

Change-Id: I0416239c4030ce5bc39473e65e560016cd4ffc75
Signed-off-by: Adriana Kobyla

pep8 fixes

Also ignore the configs/ directory since it only contains
system configuration files named <system>.py.

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

show more ...

obmcutil: alias status to state

Save users who assume the machine status is available with 'obmcutil
status' by aliasing status to state.

Change-Id: Icdfa6f62ff6f719386edd892ec80fe29c32087f0
Signed

obmcutil: alias status to state

Save users who assume the machine status is available with 'obmcutil
status' by aliasing status to state.

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

show more ...

Allow PGOOD polling interval of 500ms

The code has an explicit check for a minimum allowed
PGOOD polling interval. Decrease that from 1s to 500ms
as there are new requirements on a specific system

Allow PGOOD polling interval of 500ms

The code has an explicit check for a minimum allowed
PGOOD polling interval. Decrease that from 1s to 500ms
as there are new requirements on a specific system
to poll at the 500ms.

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

show more ...

Rename board configuration file

The device tree file is s2600wf.dts, update to match.

Change-Id: Iefcfefd8b6329aa9afb4b831882e2861a0f6a30a
Signed-off-by: James Feist <james.feist@linux.intel.com>