PHAL: Helper function to set PDATA_INFODB

systemd runntime global cofiguration keeps a copy of all
the varibles in the application space. Since the PDATA_INFODB
config variable is consumed by limite

PHAL: Helper function to set PDATA_INFODB

systemd runntime global cofiguration keeps a copy of all
the varibles in the application space. Since the PDATA_INFODB
config variable is consumed by limited applications, removed
systemd global settings related to PDATA_INFODB environment.
So application level environment setting required. Added this
helper function to set PDATA_INFODB environment variable for
the attributes tool.

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

show more ...

phal: libipl enum name change IPL_NO_ERR to IPL_ERR_OK

Enable support related to new enum name change in libipl.

Tested: Build is success

Signed-off-by: Jayanth Othayoth <ojayanth@in.ibm.com>
Chan

phal: libipl enum name change IPL_NO_ERR to IPL_ERR_OK

Enable support related to new enum name change in libipl.

Tested: Build is success

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

show more ...

phal: ipl_error_callback_func prototype related changes

libipl ipl_error_callback_func prototype changed to support additional
error handling support. This commits includes the changes related to
th

phal: ipl_error_callback_func prototype related changes

libipl ipl_error_callback_func prototype changed to support additional
error handling support. This commits includes the changes related to
this function prototype function.

Also made changes in internal error handling to make direct
call to processBootError instead of callback function.

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

show more ...

op-reset-host-check: add logging as service dep

When the CFAM registers indicate the host is running, this service logs
an error. Ensure the phosphor logging service is running before starting
this

op-reset-host-check: add logging as service dep

When the CFAM registers indicate the host is running, this service logs
an error. Ensure the phosphor logging service is running before starting
this service.

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

show more ...

op-reset-host-check: update for recent state mgr changes

The following commit refactored the way the host state is discovered
when the BMC is reset and the host is running:
https://github.com/openb

op-reset-host-check: update for recent state mgr changes

The following commit refactored the way the host state is discovered
when the BMC is reset and the host is running:
https://github.com/openbmc/phosphor-state-manager/commit/0d1c3f1f9329c853677f0581287afef83eeea0f0

All of the discovery now happens in the
xyz.openbmc_project.State.Host.service so set the appropriate dependency
on that.

Tested:
- Did a chassis power on, set the CFAM register to indicate host is
running, and rebooted the BMC. Verified that service ran after the
host had been probed over IPMI and PLDM and appropriate error was
logged.

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

show more ...

exception: switch to public sdbus exception

SdBusError was intended to be a private error type inside sdbusplus.
Switch all catch locations to use the general sdbusplus::exception type.

Signed-off-

exception: switch to public sdbus exception

SdBusError was intended to be a private error type inside sdbusplus.
Switch all catch locations to use the general sdbusplus::exception type.

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

show more ...

cleanup sdbus CAMELCASE define

The transition from old camelcase to new camelcase is long completed,
so clean up the old camelcase defines.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Chang

cleanup sdbus CAMELCASE define

The transition from old camelcase to new camelcase is long completed,
so clean up the old camelcase defines.

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

show more ...

PHAL: add devtree data export service

Service to export r/w devtree attribute data and
save to a known location.

This service will start after Chassis power is on

Tested: Verified service starts o

PHAL: add devtree data export service

Service to export r/w devtree attribute data and
save to a known location.

This service will start after Chassis power is on

Tested: Verified service starts on every boot after
chassis powered-on.

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

show more ...

PHAL: add devtree import service

Service to restore r/w devtree attribute data in the
reboot path. This is required only in concurrent software
update reboot path. The attribute data files creates o

PHAL: add devtree import service

Service to restore r/w devtree attribute data in the
reboot path. This is required only in concurrent software
update reboot path. The attribute data files creates only
in the concurrent code update path. so no action in the
regular reboot path.

This service will only run in the following condition:
- Chassis power is on when the BMC comes out of reset

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

show more ...

PHAL: remove devtree import data file in genesis boot

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

PHAL: import devtree tool support

Added tool to import devtree attributes from BMC predefined
location to current phal devtree in the system. This tool
is used to override that r/w attribute data af

PHAL: import devtree tool support

Added tool to import devtree attributes from BMC predefined
location to current phal devtree in the system. This tool
is used to override that r/w attribute data after concurrent
code update BMC reboot path.

./openpower-proc-control importDevtree

Tested : regular reboot path ( with attribute data file)

~# systemctl status phal-import-devtree@0.service
○ phal-import-devtree@0.service - Import Host0 PHAL devtree attributes
Loaded: loaded (/lib/systemd/system/phal-import-devtree@.service; static)
Active: inactive (dead) since Mon 2021-08-02 12:03:09 UTC; 2min 55s ago
Process: 493 ExecStart=/usr/bin/openpower-proc-control importDevtree (code=exited, status=0/SUCCESS)
Main PID: 493 (code=exited, status=0/SUCCESS)

Aug 02 12:03:02 rain71bmc systemd[1]: Starting Import Host0 PHAL devtree attributes...
Aug 02 12:03:03 rain71bmc systemd[1]: Finished Import Host0 PHAL devtree attributes.
Aug 02 12:03:09 rain71bmc systemd[1]: phal-import-devtree@0.service: Deactivated successfully.
Aug 02 12:03:09 rain71bmc systemd[1]: Stopped Import Host0 PHAL devtree attributes.

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

show more ...

PHAL: export devtree attribute support

Added function to export devtree data based on the
filter file using attributes tool based infrastructure.
Data is stored in pre defined location in BMC to res

PHAL: export devtree attribute support

Added function to export devtree data based on the
filter file using attributes tool based infrastructure.
Data is stored in pre defined location in BMC to restore
in the reboot path.

Tested: verified devtree data exported manually.

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

show more ...

PHAL: add phal-export-devtree app

d-bus signal based watch app for software path interface add.

This implements methods for watching for software path
interface add signal and call appropriate func

PHAL: add phal-export-devtree app

d-bus signal based watch app for software path interface add.

This implements methods for watching for software path
interface add signal and call appropriate function to
initiate phal devtree attribute data collection and save
to preserve partition.
Rules:
- Watch for interfaces added for the software path
- If interface added is “Activation”
- if Activation property value is “Ready”
- Then software update is going to start
- Collect phal devtree required attribute list and
save to pre-defined location

This commit enables the d-bus app only.

Tested : verified manually the signals by forcing
code update.
Jul 30 14:27:35 xxx phal-export-devtree[2624]: Software path
interface add signal received

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

show more ...

PHAL: refactoring common_utils

Moved pdbg specific utils function into new file.
This will help to avoid adding additional phal library
dependency for apps , which are not really consumed
phal repos

PHAL: refactoring common_utils

Moved pdbg specific utils function into new file.
This will help to avoid adding additional phal library
dependency for apps , which are not really consumed
phal repos.

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

show more ...

PHAL: added createPEL wrapper function

Added wrapper function to create PEL based on event type.

Why: concurrent code update feature requires to create to
PEL, based on event types in various use

PHAL: added createPEL wrapper function

Added wrapper function to create PEL based on event type.

Why: concurrent code update feature requires to create to
PEL, based on event types in various use cases. This will
help to reduce duplicate code.

Tested: verified PEL log

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

show more ...

PHAL: Helper function to set PDBG_DTB

Helper function to set PDBG_DTB environment variable for the
CEC device tree path.

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

PHAL: Helper function to set PDBG_DTB

Helper function to set PDBG_DTB environment variable for the
CEC device tree path.

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

show more ...

phal: move common_utils.cpp to extensions/phal

tested: verified build

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

phal: Added extensions/phal folder

Added phal specific folder to keep all phal feature
based function in common place

Moved phal error specific source files to extensions/phal

Tested: build verif

phal: Added extensions/phal folder

Added phal specific folder to keep all phal feature
based function in common place

Moved phal error specific source files to extensions/phal

Tested: build verified.

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

show more ...

test: removed unused makefile

repository already completed meson migration.

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

MAINTAINERS: Added new reviewers

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

bmc-reset: call new clear host running status service

A simple service to ensure the CFAM register used by the host to
communicate when it is up and running, is cleared when the host is
stopped.

Te

bmc-reset: call new clear host running status service

A simple service to ensure the CFAM register used by the host to
communicate when it is up and running, is cleared when the host is
stopped.

Tested:
- Verified service is run on a host stop and the CFAM register was
cleared

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

show more ...

bmc-reset: add support to clear host running register

A simple procedure to ensure the CFAM register used by the host to
communicate when it is up and running, is cleared when the host is
stopped.

bmc-reset: add support to clear host running register

A simple procedure to ensure the CFAM register used by the host to
communicate when it is up and running, is cleared when the host is
stopped.

The host is also responsible for clearing this but given there are
scenarios where the host crashes, this is the backup plan to ensure it
is cleared.

Its a best effort because there's not much to be done if it cannot be
cleared. We want the default to lean towards not powering down the host
if there is any chance the host is running.

Tested:
- Verified master processor CFAM register was set to 0 when this
procedure was called

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

show more ...

bmc-reset: add support to write a cfam

A future commit requires this support so refactor the getCFAM a bit to
move some shared code to a common function and add putCFAM

Tested:
- Verified the putCF

bmc-reset: add support to write a cfam

A future commit requires this support so refactor the getCFAM a bit to
move some shared code to a common function and add putCFAM

Tested:
- Verified the putCFAM worked as expected when called on p10bmc hardware

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

show more ...

bmc-reset: call new host reset check service

This service will only run in the following condition:
- Chassis power is on when the BMC comes out of reset
- Host state management is unable to communi

bmc-reset: call new host reset check service

This service will only run in the following condition:
- Chassis power is on when the BMC comes out of reset
- Host state management is unable to communicate with the host over IPMI
or PLDM channels

Tested:
- Verified service does not run if chassis power is not on when coming
to BMC Ready (i..e /run/openbmc/chassis@0-on present)
- Verified service does not run if host has responded over PLDM or IPMI
(i.e. /run/openbmc/host@0-on present)
- Verified if we get past the above checks, the hw procedure is called
as expected

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

show more ...

bmc-reset: ensure error is created if host up

A separate commit will be submitted to phosphor-logging to get this new
error into the PEL registry.

Tested:
- Verified error is generated when CFAM re

bmc-reset: ensure error is created if host up

A separate commit will be submitted to phosphor-logging to get this new
error into the PEL registry.

Tested:
- Verified error is generated when CFAM register is A5000001

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

show more ...