Update MAINTAINERS list

Add Jayanth as maintainer.

Signed-off-by: Jayanth Othayoth <ojayanth@in.ibm.com>
Change-Id: Ib6a8dc1c723be91330e8ee8fc28ea3928c13d6b9

Update sysfs paths to use /sys/class/fsi-master

The OpenBMC kernel as of 5.3 supports class based paths for the FSI
devices. This means each master (hardware master, or hub master) appears
under /sy

Update sysfs paths to use /sys/class/fsi-master

The OpenBMC kernel as of 5.3 supports class based paths for the FSI
devices. This means each master (hardware master, or hub master) appears
under /sys/class/fsi-master, allowing a common path to be used no matter
the hub implementation.

Using this method allows us to the AST2600 hardware master and the
existing GPIO master in the same code base.

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

show more ...

targetting: Remove old sysfs path

This was required for old kernels (4.7 and earlier). It is unlikely
anyone is using modern OpenBMC on these kernels, so drop support.

Signed-off-by: Joel Stanley <

targetting: Remove old sysfs path

This was required for old kernels (4.7 and earlier). It is unlikely
anyone is using modern OpenBMC on these kernels, so drop support.

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

show more ...

Remove endian swapping abstraction

All supported kernels (and since 4.10) emit data in big endian. This
removes the run time abstraction and uses htobe32 unconditionally.

Signed-off-by: Joel Stanle

Remove endian swapping abstraction

All supported kernels (and since 4.10) emit data in big endian. This
removes the run time abstraction and uses htobe32 unconditionally.

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

show more ...

nmi: Use libpdbg directly

This avoids calling DBus to run a systemd unit that is effectively a
shell script when we already have libpdbg on the system anyway.

Signed-off-by: Andrew Jeffery <andrew@

nmi: Use libpdbg directly

This avoids calling DBus to run a systemd unit that is effectively a
shell script when we already have libpdbg on the system anyway.

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

show more ...

NMI (soft reset) control code enablement

Changes here include:
1) A service that triggers Open BMC App (openpower-proc-nmi)
2) An application that waits for a dbus event (NMI) to occur
3) A

NMI (soft reset) control code enablement

Changes here include:
1) A service that triggers Open BMC App (openpower-proc-nmi)
2) An application that waits for a dbus event (NMI) to occur
3) A reset logic that triggers a NMI/softreset dbus service which
will invoke pdbg call to trigger stop followed by sreset on all
threads
4) Necessary Makefile.am changes

Tested: Verified following
1) openpower-proc-nmi app is automatically started once host is started
2) After killing the app made sure app is restarted successfully
3) Verified NMI is detected on the Host side and crash dump being
collected in /var/crash/ and taking a watchdog triggered reboot
4) System coming back to original state once all #3 is completed
5) Powered off the system (obmcutil poweroff) service disabled and
enabled after host is powered on

Signed-off-by: Lakshminarayana R. Kammath <lkammath@in.ibm.com>
Change-Id: I16f3bb2a2ed0c0ffcea2a720a2ae39a2b303ef9e

show more ...

build: install into bin instead of sbin

Installs into bin instead of sbin per guidelines.

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

cleanupPCIE: Catch file_error::Open exception

The bus that is read may only show up at certain
times, such as during poweron. It is not
a fatal error condition if the bus is not available.
So we wil

cleanupPCIE: Catch file_error::Open exception

The bus that is read may only show up at certain
times, such as during poweron. It is not
a fatal error condition if the bus is not available.
So we will let it get logged to the journal, but not
let it get committed in this case.

Tested: It was an intermittent failure, but ran
poweron/poweroff a few times and didn't see
the error pop up again.

Resolves: openbmc/openbmc#3510

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

show more ...

build: pkg anti-pattern: use defaults

Use the defaults in the pkg check where the default error message is
sufficient to identify which package is missing.

Change-Id: Ib3c000db8dc8c4f9efedb66464c86

build: pkg anti-pattern: use defaults

Use the defaults in the pkg check where the default error message is
sufficient to identify which package is missing.

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

show more ...

Don't create errors in cleanupPCIE

This code always just runs in the power off path, and at times it
may be called when power is already off. In that case the FSI
access would fail since the proces

Don't create errors in cleanupPCIE

This code always just runs in the power off path, and at times it
may be called when power is already off. In that case the FSI
access would fail since the processor won't have power, so just
catch the error and continue on.

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

show more ...

In startHost, don't set bit 2 in scratch reg 8

This was keeping hostboot from using some default scratch
register settings.

Tested: Tested by HW team

Change-Id: I1290acbabfa785e67a6038a3aadb7ececd

In startHost, don't set bit 2 in scratch reg 8

This was keeping hostboot from using some default scratch
register settings.

Tested: Tested by HW team

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

show more ...

p9: pcie-workaround: Add service file to repo

systemd unit file distribution is typically done
by the application build process.

Testing: Verified service runs after obmc-power-stop-pre
an

p9: pcie-workaround: Add service file to repo

systemd unit file distribution is typically done
by the application build process.

Testing: Verified service runs after obmc-power-stop-pre
and before op-power-stop.

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

show more ...

p9: cleanupPcie: Run procedure on all CPUs

Procedure should be run on all of the master and
slave CPUs.

Change-Id: I323519b059cfbe14ccd6145e22b1bdae4fd3d476
Signed-off-by: Anthony Wilson <wilsonan@

p9: cleanupPcie: Run procedure on all CPUs

Procedure should be run on all of the master and
slave CPUs.

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

show more ...

p9: cleanupPcie: Correct line endings

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

Fix std::variant usage

Converts usage of mapbox specific .get() to std::get() interface.

Change-Id: I7a07bbad9bc7d3d79079325aed11353deaf9aa63
Signed-off-by: William A. Kennington III <wak@google.co

Fix std::variant usage

Converts usage of mapbox specific .get() to std::get() interface.

Change-Id: I7a07bbad9bc7d3d79079325aed11353deaf9aa63
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

add .clang-format

Add .clang-format for automatic style.

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

s/copyright char/(C)/g to make compiler happy

Compiler complaining about the copyright characters.

Change-Id: I6730a50a668899a33e9dfcc4accb67ee207a3f08
Signed-off-by: Patrick Venture <venture@googl

s/copyright char/(C)/g to make compiler happy

Compiler complaining about the copyright characters.

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

show more ...

Add cleanupPcie to openpower-proc-control

This disables the drivers and receiver in the
PCIE root control 1 register as follows:
BIT NAME VALUE
19 TP_RI_DC_B 0b0
2

Add cleanupPcie to openpower-proc-control

This disables the drivers and receiver in the
PCIE root control 1 register as follows:
BIT NAME VALUE
19 TP_RI_DC_B 0b0
20 TP_DI1_DC_B 0b0
21 TP_DI2_DC_B 0b0

This should be run on a power off.

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

show more ...

openpower-proc-control: use c++17

Update configure.ac to choose the c++17 standard

Change-Id: Ic782f5ca1a8435ecce99a92c5261d98eeac23647
Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>

Add MAINTAINERS file

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

README: Reword cleaning the repo

Reworded cleaning the repo. The old wording did not make sense.

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

Log istep info on watchdog timeout

1. SBE istep info (cfam 2809)
2. HB istep info (cfam 283C)

Change-Id: I67632c58fe1148b980791dc017e5c099107caae5
Signed-off-by: Shakeeb Pasha <shakeebbk@in.ibm.com>

Add FSI scan procedure

This procedure will perform both the FSI master
and FSI hub scans.

It replaces doing the scans by writing to sysfs
using the echo command so that real errors can
be created o

Add FSI scan procedure

This procedure will perform both the FSI master
and FSI hub scans.

It replaces doing the scans by writing to sysfs
using the echo command so that real errors can
be created on failures.

Currently the only way to tell if the master scan
fails is by checking for missing sysfs files, and
this code will do that by checking for the hub
scan file.

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

show more ...

Fix makefile generator script when multiple types

The script takes the arguments:
base-directory type1 type2..typeN

Where the types are the directory names
to find procedures in.

However, when m

Fix makefile generator script when multiple types

The script takes the arguments:
base-directory type1 type2..typeN

Where the types are the directory names
to find procedures in.

However, when multiple types are passed in
during the build, they actually show up as
1 parameter:
base-directory "type1 type2..typeN"

This change looks for all of the types in
the second parameter.

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

show more ...

Use all phosphor-dbus-interfaces errors

Remove the local error definitions and use the
ones in phosphor-dbus-interfaces instead as the ones
there suit our purposes and it simplifies this repository

Use all phosphor-dbus-interfaces errors

Remove the local error definitions and use the
ones in phosphor-dbus-interfaces instead as the ones
there suit our purposes and it simplifies this repository
to do so.

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

show more ...