Age | Commit message (Collapse) | Author | Files | Lines |
|
Requirements have arisen for OpenBMC firmware to run
certain services only during fresh power on operations.
To achieve that, break the obmc-host-start action target into
two targets. The existing obmc-host-start target will now
call the new obmc-host-startmin target. The startmin target
will be the minimum services required to start the host.
The obmc-host-start target is where services that should
only be called during a fresh power on can be placed.
The initial use case for this is the obmc-host-reboot target.
It will call this new obmc-host-startmin target to do the bare
minimum on a reboot operation.
This then allows the use case for OpenBMC to only reset the host
reboot count on a fresh power on operation. A new service will
be added to the obmc-host-start target to do this. If a reboot
operation is issued, or a host watchdog is triggered to cause a
reboot, this service would not be started, ensuring the reboot
count is properly decremented.
Tested: Verified on/off/reboot and ran CT regression suite.
https://gerrit.openbmc-project.xyz/#/c/9821/ is required
for reboots to work properly.
Change-Id: I86df03e6d671178c1525f852e196c4102bb73cb6
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
We have 236 commits, 127 files changed, 17766 insertions(+), 2128
deletions(-). Some of these are backports from upstream. This list
does not include patches from the 4.13 stable releases, but we do
include those in the dev-4.13 branch.
1 Alexey Khoroshilov
34 Andrew Jeffery
1 Arnd Bergmann
1 Benjamin Herrenschmidt
1 Bhumika Goyal
1 Brad Bishop
1 Brendan Higgins
11 Christopher Bostic
1 Cyril Bur
14 Cédric Le Goater
49 Edward A. James
3 Gavin Shan
1 Guenter Roeck
8 Ivan Mikhaylov
1 Jacek Anaszewski
1 James Feist
7 Jeremy Kerr
72 Joel Stanley
2 Julia Lawall
1 Ken Chen
6 Lei YU
3 Milton Miller
1 Mykola Kostenok
1 Patrick Venture
2 Philipp Zabel
1 Rick Altherr
11 Samuel Mendoza-Jonas
2 Wei Yongjun
1 Xo Wang
1 Yong Li
Note that the 4.13 branch is EOL'd by the Linux community, and as such
should not be used for any products beyond development.
React to removal of occ hwmon instances from device trees with a
new startup/shutdown mechanism for phosphor-hwmon.
To fix this, a helper script will be used to start the service that
will pass the service the device tree name if it is present, or the
udev device path if it isn't. This script will still run from the
udev rule as before, but it will stop and start the service itself
without using the SYSTEMD_WANTS attribute.
As the path to the hwmon environment file matches the service
template argument, the paths for the OCC .conf files need to change
to match the device path instead of the previous device tree path.
Note that the pure device path would have the hwmon instance number
in it, but since that can't be known ahead of time it is stripped
off by the script that starts the service.
In addition, the pure device path for the OCCs contain several
':'s, meaning the associated environment files would also need to.
However, Yocto/Bitbake cannot handle a ':' in a file path, so they
are converted to '--'s by the script that starts the service and
phosphor-hwmon will convert them back internally when it starts.
The service file also needed some changes now that the service
lifetime is no longer controlled by systemd via SYSTEMD_WANTS.
This script will be called by a udev rule to start and
stop phosphor-hwmon when the hwmon device driver is started
and stopped.
It is passed both the device path and the OF_FULLNAME device
tree attribute. If OF_FULLNAME is present, it will start the
service with that as its template argument, otherwise it will
use the device path. This is to handle devices that aren't in
the device tree so they won't have OF_FULLNAME.
If a '/hwmon/hwmonN' is in the path it is removed, as this path
is also used as a path to an environment file and so must be
known ahead of time, which the hwmon instance N is not.
If there is a ':' in the path name, it is converted to a '--'.
Yocto/Bitbake cannot handle a ':' in file paths.
Resolves openbmc/openbmc#2953
Change-Id: I815be4d6d9e1cbea8428bb1bb8c332776ee71ece
Signed-off-by: Joel Stanley <joel@jms.id.au>
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
The openpower layer supports a usage pattern of building non
openpower systems with the openpower layer included in the layer
configuration (introduced with 008b81bb).
As a part of 008b81bb a number of distro and machine features
were added. As with MRW and UBI before it, these features
are distro policy and not machine features. As such remove the
machine features.
Additionally converge the three feature variants down to a single
openpower feature since there aren't any in-tree systems that don't
enable all three.
Add a distro override and fix up combined feature checks to use
this override instead for improved readability.
Add a new distro configuration that enables these features and
point all Openpower sample local.conf files to this new distro
config.
Tested: Built and validated Witherspoon, Palmetto images
Change-Id: Ifa3df61fbe1803f00c38956c265604ba91d7d127
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
These _appends can potentially crowd their neighboring entries.
Change-Id: I1dae77f9a17744110175916935c01773708652b0
Tested: Built a witherspoon image
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
xyz.openbmc_project.Settings has replaced this function
Resolves openbmc/openbmc#2339
Change-Id: Ifec6a2fda088c4631e3835d3276a54e09b5d7147
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
The yaml config for pcie e2b path was pcie_e2b_present and it is
incorrect.
Fix it by pointing it to pcie_card_e2b
Change-Id: I02a50f8544f936f15a064b369908bb8e98df7470
Signed-off-by: Lei YU <mine260309@gmail.com>
|
|
This commit also adds the entityID and entityInstance property
for BMC FRUS config yaml.
These attributes are required to pass it through Get SDR command.
Currently we support two types of record format through Get SDR
1) FULL Record.
2) FRU Record.
Resolves openbmc/openbmc#2823
Change-Id: Ie7424fbc493dde5070a2c65c74ba83565154b9d4
Signed-off-by: Ratan Gupta <ratagupt@in.ibm.com>
|
|
The GPIO E7 is used to check if PCIE E2B is connected with a riser card.
Add a sensor for this card and provide IPMI response for the sensor.
Change-Id: Ie63a633c0c4eb5a88039a0087c2fd471e38f87af
Signed-off-by: Lei YU <mine260309@gmail.com>
|
|
Add OBMC_MACHINE_FEATURES for OpenPOWER support under openpower.inc.
Select OpenPOWER features in the POWER machines' DISTRO_FEATURES so that
the features are built.
The features are:
obmc-openpower for required base support
obmc-openpower-extra for non-essential tools
obmc-openpower-legacy for deprecated packages like org.openbmc and
skeleton packages
Part of openbmc/openbmc#2652.
Change-Id: Ib9f44a0c2e243eebb6594de6554cdc3fa0d076af
Signed-off-by: Xo Wang <xow@google.com>
|
|
Add override YAMLs for both phosphor and legacy time daemons to have
host to set BMC time.
Change-Id: I3a2b258f229ad4e110c456bd7e08c0264e85d0f6
Signed-off-by: Xo Wang <xow@google.com>
|
|
Change RDEPENDS for openpower-occ-control to override syntax predicated
on "zaius" (which is added by MACHINE_OVERRIDES).
Change-Id: Iaaae449bc55d129b7102052f2440a6f78d12bcd1
Signed-off-by: Xo Wang <xow@google.com>
|
|
Adding the missing PCI Resource initialization step to
boot progress stages.
Resolves openbmc/openbmc#2298
Change-Id: I5b3b42c15d5f0d43351921749a4e31aaa3f22b48
Signed-off-by: Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com>
|
|
For the checkstop GPIO monitoring configuration, use gpio-keys device
created by udev rule. Also port the DEVPATH fix for the $PATH env being
overridden.
Change-Id: I9ff522b20bdf92f1832dccb9f87664b6104bb5c1
Signed-off-by: Xo Wang <xow@google.com>
|
|
When marking a unit as functional both functional state
and presence need to be checked to avoid marking
non-present units as functional.
There was a change in yaml structure for accomodating
fix for marking non-present units as functional.
All these yamls are not sorted by sensor id, so it will
easier for a future script based change.
Change-Id: I122d652de32ba4f323ee7b31a59078d86e158246
Signed-off-by: Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com>
|
|
The settings manager now supports multiple interfaces per settings
object. For that reason, the settings yaml should contain a list of
interfaces under each settings object.
Change-Id: Ief5aff4c1305c3ee0629ecd1b7da5868b4b4c73e
Signed-off-by: Deepak Kodihalli <dkodihal@in.ibm.com>
|
|
Added boot count, boot progress and os status
Change-Id: I6cbc4e9698d34007f846eabb8bbb23d1ea79bf8d
Signed-off-by: Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com>
|
|
This commit enables patches from the /usr/local path by replacing
absolute paths to /usr/sbin or /usr/bin in every service file with
"/usr/bin/env ". Additionally, SyslogIdentifier is added to each
file to ensure that journal entries are still distinct.
In two cases, service files also had to change due to references
to ${{PATH}}, which doesn't work with /usr/bin/env. These
references - and the corresponding definitions - were changed to
${{DEVPATH}}.
Resolves openbmc/openbmc#2149
Change-Id: I12c5c6e5101eef6820b985a563313836250ff2c7
Signed-off-by: Michael Tritz <mtritz@us.ibm.com>
|
|
Change-Id: I32a9e0ee08731566c2958ce5703e12cc4afce5c1
Signed-off-by: Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com>
|
|
Change-Id: I8f6a00d33a2c398e423554a43118aaee823a8c98
Signed-off-by: Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com>
|
|
Zaius had a way to override the reboot policy setting. Make sure that
still works with the new settings application.
Change-Id: I51e32705b9aa4787fadab211f57b5fdcf75a3207
Signed-off-by: Deepak Kodihalli <dkodihal@in.ibm.com>
|
|
The CPU inventory from the host is what triggers the
OCC control object creation. To get OCC control support,
need to have the CPU inventory objects present and functional.
Change-Id: I5a88d55e5e0be5e3e61dd0ff8f519a7462588aa6
Signed-off-by: Andrew Geissler <andrewg@us.ibm.com>
|
|
The occ control application allows occ pass through support
as well as binding and unbinding the occ device at the
appropriate times. This binding and unbinding will enable
the hwmon sensors from the occ to appear.
Resolves openbmc/openbmc#1951
Change-Id: Ifa4d746d0af48522d877cf0238a03943a27245d8
Signed-off-by: Andrew Geissler <andrewg@us.ibm.com>
|
|
Change-Id: I32c71693b855208e4cb9579490e8f2d93fe113d8
Signed-off-by: Andrew Geissler <andrewg@us.ibm.com>
|
|
This reverts commit 6031bb6c798bdc86146836582034ab8ef8a3987c.
Signed-off-by: Xo Wang <xow@google.com>
Change-Id: Ia49ba69173098080daf6bf5ada4e3d02562f6f91
|
|
Override the checkstop monitor service environment file in order to add
the keyboard key number for the Zaius checkstop GPIO pin.
Signed-off-by: Xo Wang <xow@google.com>
Change-Id: I52a3f54d9f10c82c7bffcdc7012c178d97e8124d
|
|
Change-Id: I3061a488c129ed8e8451ab63ececeb13c16a81fb
Signed-off-by: Emily Shaffer <emilyshaffer@google.com>
|
|
Change-Id: Ia4741513fc5197ee7aa49f133ff99807da9b5327
Signed-off-by: Vishwanatha Subbanna <vishwa@linux.vnet.ibm.com>
|
|
The clock generator output fixup workaround temporarily disables the
clock output when run, crashing the host if it's up.
Fixes openbmc/openbmc#1274.
Signed-off-by: Xo Wang <xow@google.com>
Change-Id: Ib734ffc43eb29c7b7039419c8fdd42e4d47c3260
|
|
The i2cget/i2cset commands in this script can theoretically
interfere with the linux hwmon driver state so the bind/unbind
around running the i2c commands was added as a defensive measure.
But in reality the linux ucd9000 driver does not touch the GPIO registers
so there is no chance of any interference.
Remove the bind/unbind steps which have had issues in the past and
isntead use '-f' argument to i2cget/i2cset to allow the script to
run even when the hwmon driver is loaded.
Change-Id: I0b3748e263a0578b3da533d75fa2f3ccc3a68b09
Signed-off-by: Robert Lippert <rlippert@google.com>
|
|
This recipe is no longer needed on DD1 due to host firmware
changes that limit the PCIe speed to gen2.
Change-Id: I5bab73a7955b491d50666c3bc2564bfecb03e3fe
Signed-off-by: Robert Lippert <rlippert@google.com>
|
|
mboxd has been fixed so this workaround is no longer needed
Change-Id: I1eca65558c446e982174f1c4af2f484b8fe5c3b8
Signed-off-by: Andrew Geissler <andrewg@us.ibm.com>
|
|
Commit e1dcc23 mentions that the clock generator output
configuration fixup should be started before trying to
send any FSI transactions, so we need to re-order it to
prevent race condition.
Resolves openbmc/openbmc#1519
Change-Id: I11edb6ddf1982cbdda713707599fa62de51bcea4
Signed-off-by: John Wang <hsienchiang@gmail.com>
|
|
Add local-tty-baud option to Zaius BMC console configuration, to be
read by obmc-console-server when opening the tty for the local UART
mirroring.
Change-Id: Iae2322b383388d9475e04a6a54ca47563123c812
Signed-off-by: Xo Wang <xow@google.com>
|
|
The target change is:
obmc-chassis-stop -> obmc-host-stop
obmc-chassis-start -> obmc-host-start
Removed CHASSIS_FMT and CHASSIS_LINK_FMT
because they are no longer needed.
CHASSIS_POWER_FMT_2 did not change because
its used for obmc-chassis-reset.
Deleted files obmc-chassis-start@.target and
obmc-chassis-stop@.target because they are
no longer needed. I moved over their contents
to obmc-host-start and obmc-host-stop.
Updated phosphor-state-manager.bb and
skeleton-rev.bbclass to have correct commit
id.
Change-Id: I26cba4153038f39d60e8d450a05969a216848abe
Signed-off-by: Josh D. King <jdking@us.ibm.com>
|
|
The target change is:
obmc-power-chassis-on -> obmc-chassis-poweron
obmc-power-chassis-off -> obmc-chassis-poweroff
Also updated phosphor-state-manager.bb to now
have correct commit id.
Change-Id: I15aae803e5f0d44d4e6a137278f93e91f848b2c0
Signed-off-by: Josh D. King <jdking@us.ibm.com>
|
|
Add override for the default "powerpolicy" loaded from the settings
YAML to ALWAYS_POWER_ON.
Signed-off-by: Xo Wang <xow@google.com>
Change-Id: Ib753636c616c8f403f9656fa3aa3540e219b7e4a
|
|
Move vcs-on@.service, op-refclock@.service, and
avsbus-enable@0.service to run as part of
obmc-power-chassis-on.target instead of obmc-chassis-start.target
so that after starting just obmc-chassis-start@0.target cronus
will be able to run.
Resolves openbmc/openbmc#1396
Change-Id: Ic00614abc3ea27167041614e63eed392b5b622f9
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
When switching voltage regulator modules (VRM) on Zaius to AVSBus-
controlled operation, the initial output voltage should be copied from
the PMBus VOUT_COMMAND register. However, the Intersil VRM does not
perform the copy following the first switch from PMBus- to AVSBus-
controlled operation.
Add a workaround to the AVSBus enable step that writes the value of
VOUT_COMMAND to itself in order to initialize the otherwise BMC
inaccessible AVSBus voltage setpoint.
Signed-off-by: Xo Wang <xow@google.com>
Change-Id: I3335586d98e95eb52d44d6d31e688f9b7c292136
|
|
This is the first commit in a series that will ensure
we do not run the chassis power on services when the
chassis is already on.
Specifically this is to handle the situation where the
chassis is on and the BMC is rebooted.
Change-Id: I733ca076f044731a4ba327fe2f46302dce5f4edd
Signed-off-by: Andrew Geissler <andrewg@us.ibm.com>
|
|
Two fixes for FSI bugs that users were seeing often.
A revert of the Zaius reserved memory region device tree change due to a
kernel bug where u-boot will happily load data into the region that the
kernel subsequently marks as inaccessible. This is a temporary fix until
the ARM early boot code can be fixed.
Finally, a change to bring Romulus PNOR SPI settings to parity with the
other P9 platforms.
Christopher Bostic (2):
drivers/fsi: Change hub sysfs name
drivers/fsi: Don't scan after fsi-gpio_master unbound
Lei YU (1):
arm: aspeed: romulus: Set PNOR SPI address mapping
Patrick Williams (1):
Set Zaius back to old address range
Change-Id: I3887c7dbd3f6fd450fb096ba59e0189af87039d7
Signed-off-by: Joel Stanley <joel@jms.id.au>
|
|
A commit in the openbmc/linux tree moved the reserved memory for Zaius
to the end of the memory space. This seems to cause a conflict between
u-boot and the kernel resulting in an unbootable image.
Temporarily set the reserved memory address to match Witherspoon until
the issue openbmc/openbmc#1203 can be resolved.
Change-Id: I1bab9d29285d5b02fe03b2654c4458f1b22482be
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
|
|
Testing showed that the VDN max voltage setting is capping the VCS rail
setpoints when those setpoints are written by the host over AVSbus.
Raise VRMs' upper limit for voltage on the VDN rails to a value that
does not interfere with VCS rail voltages.
This is a hacky workaround for what is possibly a logic error with the
VRMs' firmware and will be rolled back when the root cause is discovered
and corrected.
Signed-off-by: Xo Wang <xow@google.com>
Change-Id: I85fd94f77509bc51aa7909dd7f522163666040fb
|
|
Capture return value of binding UCD90160 driver to power sequencer
rather than let it exit the entire script on first failure.
Only exit script if the all the driver bind attempts fail.
Signed-off-by: Xo Wang <xow@google.com>
Change-Id: I76303e1abb222daa7badb58b90d77bb49717236a
|
|
DD1.0 modules require a higher VIO for PCIE link stability.
Add service to set VIO voltage setpoint when BMC boots.
Add BitBake recipe to install service.
Add recipe to obmc-phosphor-image dependencies.
Signed-off-by: Xo Wang <xow@google.com>
Change-Id: Ia43d58683d3f26ac6ca684b5e662c45c38f47824
|
|
Stop applying the patch to disable it.
Change-Id: I4dc6c7acdce3bd3aa90969748aa9722ad8585d0d
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
Replace this virtual with two others:
-virtual/phosphor-ipmi-fru-hostfw-config
-virtual/phosphor-ipmi-fru-inventory.
Update ipmi-fru-parser machine recipes to provide
virtual/phosphor-ipmi-fru-inventory rather than
virtual/phosphor-ipmi-fru-config.
Remove machine recipes on machines that use MRW
since a single MRW recipe provides for any MRW
using system automatically.
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com?
Change-Id: Ia0d94b938eb2c7bfadd5a018ae5ea136f14dd874
|
|
A quick fix to ensure that mbox-reload.service has a chance to run
after the power off and before the power on during a reboot.
An upcoming refactoring will remove the need for this patch.
Change-Id: Ieee0573e9bd4dec518361da9244d95a27357abd3
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
For reasons yet fully debugged, PMBus communication with the UCD90160
power sequencer is flaky, with the symptom that some i2c-{get,set}
commands fail or the ucd9000 driver fails to probe.
Add retries in the script that performs these commands as a workaround
to their random failures.
Though this script exits at first error and returns its code, the
systemd services that use it have [Unit] Type=oneshot. So, systemd can
not be configured to restart the script during failures.
Signed-off-by: Xo Wang <xow@google.com>
Change-Id: Ia9eee013001c47ffc9706984b110f312ccdbc175
|
|
In bbappend files, use the override style syntax to modify variables and
specify appends and prepends with the machine name. This limits the
scope of their effects to only when MACHINE=zaius.
This allows meta-zaius to be present in BBLAYERS without causing side
effects except when building for Zaius BMC.
Signed-off-by: Xo Wang <xow@google.com>
Change-Id: Iaccc5cb1abac033dab54411eaa20855172ad3283
|
|
Add Zaius-specific override for obmc-console.conf containing the
'local-tty' configuration option.
This enables obmc-console-server to relay the LPC-controlled VUART to
another tty (in this case, ttyS0/UART1).
Signed-off-by: Xo Wang <xow@google.com>
Change-Id: I97b6982c163e22bab57004b62cc31dd8eae51680
|