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

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

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

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

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

gpio: Avoid glitching 'out'-direction GPIOs

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

gpio: Fix sprintf to uninitialized pointer

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

gpio: Clean trailing whitespace

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

Merge pull request #115 from bradbishop/refactoring

More skeleton refactoring

libopenbmc_intf include path update

Remove the interfaces prefix from applications including openbmc_intf.h.

Use brackets rather than quotes in prep for a libopenbmc_intf SDK
package.

Signed-off-b

libopenbmc_intf include path update

Remove the interfaces prefix from applications including openbmc_intf.h.

Use brackets rather than quotes in prep for a libopenbmc_intf SDK
package.

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

show more ...

Merge pull request #101 from bradbishop/refactoring

skeleton refactoring

Reorganize directory structure

Moving to directory per-application layout. This facilitates
building single applications which is useful in the Yocto build
environment since different applications

Reorganize directory structure

Moving to directory per-application layout. This facilitates
building single applications which is useful in the Yocto build
environment since different applications satisfy different OpenBMC
build requirements.

A number of issues are also addressed:
- All applications were pulling in libsystemd and the gdbus libs
irrespective of whether or not they were needed.
- gpio.o duplicated in every application - moved to libopenbmc_intf
- Added install target

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

show more ...