Age | Commit message (Collapse) | Author | Files | Lines |
|
This takes the default critical services list from meta-phosphor and
appends on extra services IBM is interested in.
Ideally, there would be an easy way to just append an extra file to the
command line but this is going to need some more thought. For now, just
override the entire file.
Tested:
- Booted witherspoon up in QEMU and verified new json file was loaded
and that stopping one of the new critical services resulted in a
dump, error log, and BMC Quiesced state.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Icf5db5331072cbcbd0359e0b789b664e31423124
|
|
This commit adds p10bmc specific configuration files to display
CPU and DIMM FRU information. Without these configuration files
the default phosphor recipe configuration files are used, and we
do not get FRU info for all configured CPUs and DIMMs.
The changes are meta-ibm specific, the image was built and tested
against our simulation model.
Signed-off-by: Shantappa Teekappanavar <sbteeks@yahoo.com>
Change-Id: I60dd8ce05941105dfaf6b5e67fa8c7b0bb06534a
|
|
* Deprecate N-1 release (hardknott).
* Enable N+1 release (kirkstone).
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I39e027e02dab64b4390b46ffbd9c299c858f403e
|
|
The power_mode and power_ips dbus objects are being consolidated and
moved under the org.open_power.OCC.Control service.
Their default values will now be set via entity-manager.
Verified on Everest hardware.
Change-Id: Ibfa1e55126862db3521855ad0743a1423e555dac
Signed-off-by: Chris Cain <cjcain@us.ibm.com>
|
|
Providing `opkg` in the image allows us to easily install packages
after-the-fact, usually to debug systems in the lab.
This is not a mechanism for providing firmware updates.
No specific configuration is supplied to enable its use, and hosting of
package archives is left as an exercise to the user. That said, bitbake
generates its own package archive in the course of creating the output
image. It also provides a `package-index` target, which after execution,
allows you to expose the archive with no more work than:
```
$ python3 -m http.server --directory tmp/deploy/ipk 8080`
```
The addition of opkg with the chosen options increases the uncompressed
filesystem size by 3855360 bytes (~3.68MiB).
Change-Id: I7731a4ebe08249e2216e8ca1984f8e8d87558687
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
The MAINTAINERS files are no longer used and are out of date (references
to IRC still exist in most of them). Remove them and rely on the OWNERS
exclusively.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I0f5d2719ad97e218ce03c4503efe1b1c92ac455e
|
|
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I7a4e86c61417f006e5ba218e3e5862dce3cd9e9a
|
|
e.g.
WARNING: /home/peter/openbmc/meta-ibm/meta-romulus/recipes-phosphor/occ/romulus-occ-control-config-native.bb: QA Issue: romulus-occ-control-config-native: native/nativesdk class is not inherited last, this can result in unexpected behaviour. Classes inherited after native/nativesdk: openpower-occ-control.bbclass [native-last]
WARNING: /home/peter/openbmc/meta-ibm/meta-romulus/recipes-phosphor/logging/romulus-phosphor-logging-callouts-native.bb: QA Issue: romulus-phosphor-logging-callouts-native: native/nativesdk class is not inherited last, this can result in unexpected behaviour. Classes inherited after native/nativesdk: phosphor-logging.bbclass [native-last]
Signed-off-by: Peter Foley <pefoley@google.com>
Change-Id: I31be11e264e1701215d2eadd28dc2249153a5d61
|
|
For PowerVM, Mutual TLS has no current use case. Disabling mTLS auth
by setting meson build option to disabled.
Note, bmcweb can still use mTLS auth to connect with LDAP server;
however for this use case the bmcweb user must auth in another way.
Signed-off-by: Ali Ahmed <ama213000@gmail.com>
Change-Id: I35adaeb1b6318d5f1de619509e05ae49970d8486
|
|
Match on everything under meta-ibm/recipes-kernel.
Change-Id: I56a2b578d1cb05ab7a2f3be9bb434a695e57b4f3
Signed-off-by: Joel Stanley <joel@jms.id.au>
|
|
It's been a pleasure working with the community on these bitbake layers
over the years. The other maintainers have been doing all the work for
awhile now so it's time to make it offical.
Change-Id: I0f8390417294cb82575aba87b5c0e7bd7528d4c1
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
As mentioned by I970e06ca6f9d0a9792af122eb25da1bb9a06f058, append and
remove directives with '+=' are no longer valid Yocto syntax and raise a
warning.
See https://git.yoctoproject.org/poky/tree/documentation/migration-guides/migration-3.5.rst#n40
One interesting caveat has to do with the difference between "append"
and "+=".
foo:append = "a"
foo:append = "b"
Results in `foo = "ab"`, but
foo += "a"
foo += "b"
Results in `foo = "a b"`. When `:append +=` is used it has behavior
like the `+=` operator. Therefore, in some cases we need to insert
additional whitespace after the `:append = "` to ensure concatenation
behaves as expected. I've manually reviewed the results to ensure there
is no ill side-effects from an additional space potentially being added.
Fix up the entire tree with the following one-liner (and similar for
`remove`):
git ls-files -- ':!:poky/**' ':!:meta-security/**' \
':!:meta-raspberrypi/**' ':!:meta-openembedded/**' \
| grep "\.bb" | xargs grep -l ":append.*+=" \
| xargs sed -i 's/:append\(.*\)+=\([^"]*\)" */:append\1=\2" /'
or ...
| xargs grep -l ":remove.*+=" \
| xargs sed -i "s/:remove\(.*\)+=/:remove\1=/"
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Iea6235e7c76e252f4d473652957c2925f6b8845a
|
|
In order to disambiguate from other uses of the FMT variable name,
rename to OBMC_CONTROL_POWER_FMT in obmc-op-control-power bit bake
files. Change was suggested in the following review:
https://gerrit.openbmc-project.xyz/c/openbmc/openbmc/+/50765
Signed-off-by: Jim Wright <jlwright@us.ibm.com>
Change-Id: If74dc281b8dfa9c06b86012f010d5b5a40d1a0f6
|
|
Some addition development details must be managed in this bbappend due
to co-development of the AST2600 and IBM p10bmc designs. IBM did bringup
of secure-boot on pre-production AST2600 silicon and this shaped how the
platform's OTP was configured.
The PEM files represent the public portion of the IBM signing key-pairs
for p10bmc systems. These are included to provide a canonical location
for the production OTP image artifact.
Change-Id: I7caa6cfd5848b1d671ef95f8031b76088673900a
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
Don't support the hardware root-of-trust on Tacoma boards.
Change-Id: I144d3bf445063f6341ae1958611873f79c9c00c7
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
Make the power supply sensors visible via redfish, chassis, all_sensors.
Input voltage, input power, output voltage, output current.
Signed-off-by: Brandon Wyman <bjwyman@gmail.com>
Change-Id: I79c9a07c849ad8015fb9fa42242aeab932b755c9
|
|
Add the power-supply configuration files for the ibm-cffps power
supplies used in the Rainier and Everest systems in order to get
telemetry data added to D-Bus via phosphor-hwmon.
Rainier power supplies on bus 3 at 0x68, 0x69, 0x6a, and 0x6b.
Everest power supplies on bus 3 at 0x68, 0x69, 0x6d, and 0x6b. Older
hardware has power supply at 0x6a instead of 0x6d.
Input voltage, input power, output voltage, and output current.
No critical or warning low values, to avoid logging errors when AC is
pulled, or other fault conditions handled by the PSU monitor. AC loss,
brownout, over-voltage, over-current, and other conditions should be
handled by the PSU monitor application.
No critical/warning high values, for similar doubled up fault logging.
Change-Id: I9f39e41e2dd0a7b77910207322506d8376f98772
Signed-off-by: Brandon Wyman <bjwyman@gmail.com>
|
|
Add bb files to support phosphor-power-control application in p10bmc
image. Remove skeleton power control application and phosphor-power
power sequencer application from image.
Signed-off-by: Jim Wright <jlwright@us.ibm.com>
Change-Id: Ib1c3cb5ba6880ea73265332107503b61def1a2ef
|
|
Add the associations from the sensors on the panel and the sensors on
the motherboard to their parent FRUs in the inventory.
The chassis association isn't needed here because hwmontempsensor
already creates it.
The association will be used for sensor callouts.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I4292646b0b101608f2bcd4c2d0f65e13f16d3d87
|
|
Tested on Everest HW
Change-Id: I667434f8701b0a635a7d3ca8077b47e4ae297f45
Signed-off-by: Chris Cain <cjcain@us.ibm.com>
|
|
The PL2303 is a serial-to-USB converter chip, needed to communicate
with a UPS. These are commonly attached via USB1.1, meaning that a
UHCI driver is required to communicate with them, so enable that too.
Signed-off-by: Eddie James <eajames@linux.ibm.com>
Change-Id: I3a7550a51d084af498e7a5aab5c6b6e75002a4e3
|
|
This fixes power supply addresses.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I36cfa1097590d01ea26953c3d32f4aec8d3aff7d
|
|
Per [1], Yocto is going to start requiring both branch names and
`protocol=https` in GitHub SRC_URI variables. Ran the upstream tool to
automatically fix our URIs.
1. https://git.yoctoproject.org/poky/tree/documentation/migration-guides/migration-3.5.rst#n10
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I5f9619f7ff707050b469ae74c8aa16846568c5e4
|
|
This package does not currently support openssl v3:
| TpmToOsslMath.h:79:5: error: #error Untested OpenSSL version
| 79 | # error Untested OpenSSL version
| | ^~~~~
For now, remove this package so we can make forward progress with
openssl v3 in openbmc.
The following issue appears to be tracking support:
https://sourceforge.net/p/ibmswtpm2/tickets/18/
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I2249c323b8219525dcef856d8ceab63de417e299
|
|
To support the watchdog application being able to restart after expiring
the watchdog (by being killed) multiple times per BMC reboot, a manual
magic close must be issued by writing a 'V' into the watchdog device
before starting the watchdog. Without that, the underlying hardware
watchdog wouldn't actually start running when the watchdog app was
started again, such as on the next boot.
Most watchdogs simply reboot the system upon expiration, but in this
case it's being used to trip a GPIO that the fan hardware latches on to
go to full speed so must support being restarted multiple times.
The busybox watchdog application used to support this, but it was
removed since it apparently caused some undesireable print statements
from the kernel on other platforms.
An alternative may be to fix this in the ASPEED watchdog driver, though
I'm not sure what all that would entail or if it even makes sense.
Tested: Expire and start the watchdog multiple times and the fans now go
to full speed each time the watchdog expires, unlike before.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I770c4fd7cca15949abd7de9dc07975562080f032
|
|
This enables the TPM driver in the kernel configuration.
Signed-off-by: Joel Stanley <joel@jms.id.au>
Change-Id: I0278543d753c7526d14eb971ac678ff364719d49
|
|
This sets u-boot to the package containing the v2016.07-aspeed-openbmc
branch. This ensures there will be no change if the default changes in
the future.
Change-Id: Iea12e1691dbdda34a3f95d6d0862add0b15bcf37
Signed-off-by: Joel Stanley <joel@jms.id.au>
|
|
This sets u-boot to the package containing the v2016.07-aspeed-openbmc
branch. This ensures there will be no change if the default changes in
the future.
Signed-off-by: Joel Stanley <joel@jms.id.au>
Change-Id: Iaf709343b966c39ca707a10ef57fa0010bf6eb6f
|
|
strace(1) is a highly effective utility that allows for inspection of
the execution of a program through its interactions with the kernel.
strace can be used to extract syscall, data and timing information and
does not require any change to the running applications or firmware
image.
https://www.man7.org/linux/man-pages/man1/strace.1.html
Change-Id: I431f5ea6246a47fba259cc5af0cdc932b38be111
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
gdbserver can invoke or attach to running processes to expose a network
interface for a remote gdb connection. From there the instructions for
remote debugging in the Poky Reference Manual can be used to inspect the
state of processes on the BMC at runtime:
https://www.yoctoproject.org/docs/1.0/poky-ref-manual/poky-ref-manual.html#platdev-gdb-remotedebug-launch-gdb-inferiorbins
Change-Id: I8a54d642955da8f2491b77553c81e8b63452901c
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
The kdump effort has been set aside in favour of the lighter-weight and
better supported use of ramoops to capture debug data.
Change-Id: I4439a159a9783a2ff0e7335dd7efead3d82e0bc7
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
Ben Pai (1):
Modify OWNERS file
Change-Id: I323eba87ed91fec6dba524f2a71528e24e907d68
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Witherspoon is running tight on space, and as our primary hardware CI
machine, we need to keep it going.
croserver can still be loaded into /tmp and run when needed
Tested:
- Reduced image size by ~100KB
- Will rely on CI to regression test
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I49b375d32462f0564b8310b430394077d0e887ed
|
|
Change-Id: Ifca5f828e58b6f86f4c7fe11544b50ff5f754702
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
Tested By:
Was able to sucessfully build post-code-manager using meson
& was able to see that the daemon is working as before on a
BMC.
Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>
Change-Id: I4cf90fc8bf0a3350b944d5ef94fa131cb6f1c35f
|
|
Witherspoon is running tight on space, and as our primary hardware CI
machine, we need to keep it going.
The GUI has already been pulled out so the KVM package doesn't make much
sense anyway.
Tested:
- Reduced image size by ~200KB
- Will rely on CI to regression test
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I3e55702da764e4e1ca1673e158c7acf4f74c9bda
|
|
phosphor-led-manager already has platform and partition system attention
indicator D-Bus objects implementing xyz.openbmc_project.Led.Group. but,
was not added to association json which this commit is doing.
Signed-off-by: Lakshminarayana R. Kammath <lkammath@in.ibm.com>
Change-Id: I3803cbb2d07626e3af6474ea8e74d987f4a963b0
|
|
Make altitude sensor visible via redfish, chassis, all-sensors.
Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: I7e032e74ad8889e7e80e197dccc7b65a26482c0a
|
|
`BBLAYERS_NON_REMOVABLE` is obsolete and no longer required.
As it said by Yocto documentation it can be used by `Hob`
https://www.yoctoproject.org/docs/1.5.2/ref-manual/ref-manual.html#var-BBLAYERS_NON_REMOVABLE
that already removed since Yocto-2.1
https://www.yoctoproject.org/tools-resources/projects/hob
Change-Id: Ibc2d8268a9d837a81e9cf6b0131dba8d0a030a3f
Signed-off-by: Alexander Filippov <a.filippov@yadro.com>
|
|
Earlier enclosure_identify and enclosure_fault associations were missing in the inventory manager.
With this checkin we are adding those associations.
busctl get-property xyz.openbmc_project.Inventory.Manager /xyz/openbmc_project/inventory/system/chassis xyz.openbmc_project.Association.Definitions Associations
a(sss) 24 "fault_led_group" "fault_inventory_object" "/xyz/openbmc_project/led/groups/enclosure_fault" "identify_led_group" "identify_inventory_object"
"/xyz/openbmc_project/led/groups/enclosure_identify" "chassis" "inventory" "/xyz/openbmc_project/inventory/system/chassis" ....
After closely looking into all the LED's it was identified that
enclosure_fault and enclosure_identify is missing in the association
and this commit is to enable same
Signed-off-by: Lakshminarayana R. Kammath <lkammath@in.ibm.com>
Change-Id: I42728d994e9110c56c607aeea98623fe2613af92
|
|
Dumps are disabled by default. This commit enables
the dump feature for ibm based systems.
Tested By:
* Dump flag enabled:
GET https://${bmc}/redfish/v1/Managers/bmc/LogServices
{
"@odata.id": "/redfish/v1/Managers/bmc/LogServices",
"@odata.type": "#LogServiceCollection.LogServiceCollection",
"Description": "Collection of LogServices for this Manager",
"Members": [
{
"@odata.id": "/redfish/v1/Managers/bmc/LogServices/Dump"
}
],
"Members@odata.count": 1,
"Name": "Open BMC Log Services Collection"
}
* Dump flag disabled:
GET https://${bmc}/redfish/v1/Managers/bmc/LogServices
{
"@odata.id": "/redfish/v1/Managers/bmc/LogServices",
"@odata.type": "#LogServiceCollection.LogServiceCollection",
"Description": "Collection of LogServices for this Manager",
"Members": [],
"Members@odata.count": 0,
"Name": "Open BMC Log Services Collection"
}
Signed-off-by: Asmitha Karunanithi <asmitk01@in.ibm.com>
Change-Id: Iad5a364ac46b5bf62bc2186eb29786454f650802
|
|
Added "ipl" dependency to enable libphal library usage
in phosphor logging.
libphal library provides C++ wrapper functions for phal based
libraries provided API.
Signed-off-by: Jayanth Othayoth <ojayanth@in.ibm.com>
Change-Id: I754cc549af7af56e7026aaf0bea079ea562b84c2
|
|
`eslint` reported an issue with the supplied JSON and it appeared to
be missing a simple comma to separate two values in an array.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I2dd167143ea31c730c26254b3ddc2ad25109e34f
|
|
Manojkiran Eda (1):
Add OWNERS file
Change-Id: Iadf9a75728ff8321a6745d843abe649adbb6cf19
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
This recipe adds SBE UserData and SRC parsers in BMC
images for use by peltool.
Signed-off-by: Jayanth Othayoth <ojayanth@in.ibm.com>
Change-Id: Ia6d3e2296fa1b874341eeade3e518de1576e0e98
|
|
Manojkiran Eda (1):
Add OWNERS file
Change-Id: Ic4bc0b1c8e3913db8406a0ec75cb28327a41230b
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Manojkiran Eda (1):
Add OWNERS file
Change-Id: Icb83eac96a7ab1a9e57439768ef3b5dc685ef43e
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Modify the machine meta-layer configs to remove the use of the
OBMC_MACHINE_FEATURES indirection and favor the Yocto MACHINE_FEATURES
variable instead.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ifafb79e4e4c010e9476b9547cd4982f5b645060e
|
|
OBMC_MACHINE_FEATURES is a variable that probably shouldn't even exist
as all it does is provide a level of indirection to the real
MACHINE_FEATURES (in phosphor-base.inc, OBMC_ is added to the base).
There are a few recipes that have developed checks against
OBMC_MACHINE_FEATURES for enabling recipe-level features when they
should be checking the standard MACHINE_FEATURES variable. Clean up
this recipe-level indirection.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Id437b93aa6c1de9a27726eea9b7e2c30e71ab3bf
|
|
openpower-pnor-code-mgmt: srcrev bump 9c2ef681e3..6fc7fcd6b1
Adriana Kobylak (2):
Add a new pldm feature option
Add support for the host PLDM transport that takes place of hiomap
(mboxd). Add this new feature to p10bmc.
Tested: Verified that tacoma (that doesn't support pldm) and p10bmc
(supports pldm) reached runtime.
Change-Id: I31e8beba5ff8a413b7b7fa7476f5f2403d3781dd
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|