Age | Commit message (Collapse) | Author | Files | Lines |
|
In phosphor-dbus-interfaces the terminology used for RestrictionMode is getting updated. This change is to keep in sync.
Change-Id: Idd41d8bfa838aadb81c11f9cee9fe74e3c28673c
Signed-off-by: Matt Simmering <matthew.simmering@intel.com>
|
|
The silkscreen labels were incorrect.
Tested: Verified in the lab physically by changing the PWM and see that
the reported RPMs are correct and matches what the silkscreen report.
Signed-off-by: Brandon Kim <brandonkim@google.com>
Change-Id: I6170f2ebbdbb3d61b3cbc9935e2af02e7abb1a4b
|
|
bmcweb is not used in gbs anymore and can be removed to avoid building
the recipe.
Change-Id: I23dd4f0d656bb4472d97b8cc2d0053fa31e6d5dc
Signed-off-by: Willy Tu <wltu@google.com>
|
|
The meta-phosphor recipe now includes a variable for setting the cap
on info logs. Leverage that rather than an explicit EXTRA_OEMESON
extension.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ib0bc935691f4e1acb8fb667dffd60df8ce4d8118
|
|
Patrick Williams (6):
clang-format: copy latest and re-format
writefrudata: remove vla
strgfnhandler: suppress unused parameter warnings
frup: suppress stringop-truncation warning
scripts: accept absolute paths to YAML
build: enable meson builds
build: install strgfnhandler into ipmi-providers dir
Vernon Mauery (1):
Set new maintainer as Patrick Williams
Willy Tu (1):
fru-parser: Remove the use of mktime
Switch to Meson.
Change-Id: Iec9c41cc17a72f84248b8cd75f70353e053f17e2
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I189baf142cc86d44ebbf615edd3bb0ec1785be8d
|
|
Mapper ships with systemd unit and dbus configuration files now, so
remove unnecessary bitbake classes.
Mapper is dbus-activated, so obmc-mapper.target dependencies are
redundant and no longer necessary.
obmc-mapper.target is the last target in obmc-targets, so remove that
recipe and any dependencies on it.
Change-Id: I59ff8dcd480b4449d0ef59422ba80c12848dade9
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
The nic-ethtool option was removed from phosphor-network, so we also
need to remove the PACKAGECONFIG option to build the phosphor-network
bump.
Tested:
Confirmed that the latest phosphor-network builds successfully.
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>
Change-Id: Id1e12f3f2af2450500ec6c0b2acef284cb69d0a7
|
|
This enables timesync support for the gbmc distro.
Change-Id: Idb653e6b98a347b5d8c3a0b3a16c6a4abbe268ac
Signed-off-by: William A. Kennington III <wak@google.com>
|
|
All of these abstractions have exactly one implementation. Remove the
indirection to improve at a glance comprehension.
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
Change-Id: I5d701aff6d0876fa3b2d16c841cbdcb0433b221f
|
|
Harvey.Wu (2):
dbus-sdr: fix SEL record Generator ID to fit SPEC
dbus-sdr: fix the vector size in sdrWriteTable and sdrStatsTable
Jian Zhang (1):
Add data types to Value to extend setDbusProperty
Vernon Mauery (1):
fix header install
Willy Tu (4):
cleanup: Remove all warning errors for the Meson build support
build: Add meson build for phosphor-host-ipmi
build: move all shared_library/module to library with version suffix
build: Remove dynamicsensor library if it is not enabled
Change-Id: I92503c6625d1ec2bf88c0a28afac9b4eeb8fbd75
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
Signed-off-by: Willy Tu <wltu@google.com>
|
|
Noticed that the " were right next to = sign with no spaces. Added
a space between =" in the bbappend file.
Signed-off-by: Brandon Kim <brandonkim@google.com>
Change-Id: I957610d65d1465f268b3b93b4ff565d36020f09f
|
|
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
|
|
Set the max size to 512 since 0x300 is used for something else.
Change-Id: Ic7772247cb3cbcff4a27a12a4f85be97fd6b0b4d
Signed-off-by: Willy Tu <wltu@google.com>
|
|
This change is a part of the privilege seperation work
which is tracked in:
https://github.com/openbmc/openbmc/issues/3383
This change should be merged after individual repo change:
https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-certificate-manager/+/41166
Signed-off-by: Anton D. Kachalov <gmouse@google.com>
Change-Id: I72e4842e7aa6de2ae4bcbdbf00953b7a79a0f414
|
|
Upstream bbclasses changed to typically use the `:${PN}` override
syntax, including the SYSTEMD_ variables. Change our systemd.bbclass
to do the same for consistency and perform a tree-wide variable
replacement.
Spot checked by building bletchley and witherspoon and checking
some of the SYSTEMD_LINK directives on installed packages under qemu.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I20a9dd809bff8af8759488734f80486c7228c6eb
|
|
Some services need obmc-mapper.target to start in correct
order, such as phosphor-virtual-sensors.
ref:
https://gerrit.openbmc-project.xyz/43952
https://gerrit.openbmc-project.xyz/43535
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: Ia463c9f189d715336626ce9e46cad0a370e9b8dd
|
|
This reverts commit 97466ae307a67c30d592c9d97a1e9d6e458e0e5b.
Reason for revert: x86-power-control supports DBUS service name
numbering for single and multi hosts now.
ref: https://gerrit.openbmc-project.xyz/45004
Change-Id: Id1fb9e61a10a457158c52990fe6c196d6fc44708
Signed-off-by: George Hung <george.hung@quantatw.com>
|
|
'SYSTEMD_ENVIRONMENT_FILE', 'SYSTEMD_LINK', 'SYSTEMD_OVERRIDE'
should stay underscore
Signed-off-by: David Wang <davidwang@quantatw.com>
Change-Id: Ia30f306857d1552b19ff5ed06664c8bd5360cb4c
|
|
This is dependent on https://gerrit.openbmc-project.xyz/45865
Signed-off-by: Brandon Kim <brandonkim@google.com>
Change-Id: Ia78c0a9f4bf4b2f97685e187acfeed90d1e0d126
|
|
This reverts commit cf7bc62babb57ea27e7a00cd6271ca53579281fe.
Reason for revert:
Enable fallback watchdog since it is possible that host fails to boot
before enabling the watchdog timer at all.
It was thought that DC cycle may be needed for fallback watchdog but it
is not required.
Signed-off-by: Brandon Kim <brandonkim@google.com>
Change-Id: I8b0f2677f078a091b626536a7ca474afa6ce77d9
|
|
- Cleanup subtree meta-f0b, meta-olympus-nuvoton, meta-q71l with
convert-overrides.py meta-quanta/meta-f0b
convert-overrides.py meta-quanta/meta-olympus-nuvoton
convert-overrides.py meta-quanta/meta-q71l
git grep "_[a-z0-9_/-]*[ :]" -- meta-quanta/meta-f0b
git grep "_[a-z0-9_/-]*[ :]" -- meta-quanta/meta-olympus-nuvoton
git grep "_[a-z0-9_/-]*[ :]" -- meta-quanta/meta-q71l
git grep -l _f0b -- meta-quanta/meta-f0b | xargs sed -i 's/_f0b/:f0b/'
git grep -l _olympus-nuvoton -- meta-quanta/meta-olympus-nuvoton | xargs sed -i 's/_olympus-nuvoton/:olympus-nuvoton/'
git grep -l _quanta-q71l -- meta-quanta/meta-q71l | xargs sed -i 's/_quanta-q71l/:quanta-q71l/'
- Fix up a few by hand
- Revert 'BBFILE_PATTERN' and 'LAYERSERIES_COMPAT'
- Add honister to 'LAYERSERIES_COMPAT'
- Update 'CONF_VERSION'
- Fix up 'SYSTEMD_ENVIRONMENT_FILE', 'SYSTEMD_LINK' and 'SYSTEMD_OVERRIDE' in all subtree
Signed-off-by: David Wang <davidwang@quantatw.com>
Change-Id: Ic431624d09dc5899b1d84439559bea9efca08ad8
|
|
Cleanup with
```
convert-overrides.py meta-quanta/meta-gbs
git grep "_[a-z0-9_/-]*[ :]" -- meta-quanta/meta-gbs
git grep -l _gbs -- meta-quanta/meta-gbs \
| xargs sed -i 's/_gbs/:gbs/'
```
After cleanup
```
$ git grep "_[a-z0-9_/-]*[ :]" -- meta-quanta/meta-gbs/ | grep '.bb'
meta-quanta/meta-gbs/recipes-bsp/images/npcm7xx-igps-native_%.bbappend:addtask do_gbs_prepare_xmls after do_patch before do_install
meta-quanta/meta-gbs/recipes-gbs/gbs-sysinit/gbs-sysinit.bb:do_install () {
meta-quanta/meta-gbs/recipes-google/acpi-power-state/acpi-power-state-daemon_%.bbappend:do_install:append:gbs() {
meta-quanta/meta-gbs/recipes-phosphor/fans/phosphor-pid-control_%.bbappend:do_install:append:gbs() {
meta-quanta/meta-gbs/recipes-phosphor/gpio/phosphor-gpio-monitor_%.bbappend:do_install:append:gbs() {
meta-quanta/meta-gbs/recipes-phosphor/interfaces/bmcweb_%.bbappend:do_install:append:gbs(){
meta-quanta/meta-gbs/recipes-phosphor/inventory/phosphor-inventory-manager_%.bbappend:do_install:append:gbs() {
meta-quanta/meta-gbs/recipes-phosphor/ipmi/phosphor-ipmi-blobs-binarystore_%.bbappend:do_install:append:gbs() {
meta-quanta/meta-gbs/recipes-phosphor/leds/phosphor-led-manager_%.bbappend:do_install:append:gbs() {
meta-quanta/meta-gbs/recipes-phosphor/sensors/phosphor-nvme_%.bbappend:do_install:append:gbs() {
meta-quanta/meta-gbs/recipes-phosphor/sensors/phosphor-virtual-sensor_%.bbappend:do_install:append:gbs() {
meta-quanta/meta-gbs/recipes-quanta/network/usb-network.bbappend:do_install:append:gbs() {
meta-quanta/meta-gbs/recipes-x86/chassis/x86-power-control_%.bbappend:do_install:append:gbs() {
```
Signed-off-by: Brandon Kim <brandonkim@google.com>
Change-Id: If640ddd3c6f6084bb0a1d82e2d18fa2640d9d83e
|
|
Modify i2cool_1 sensor UCT to 75C according to thermal
requirement
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I48fb5cada8e893688e450207985fe8466119bb3b
|
|
This is to avoid conflicts with inplace-gbmc-update that's defualt on
gBMC overrides.
Signed-off-by: Brandon Kim <brandonkim@google.com>
Change-Id: I8ccab5760dd8d4b0b5eb8de5cd270e756d5fa2aa
|
|
This makes it possible for us to change out build specific settings.
Change-Id: Ib9ba56a832a035b7ded1bf1aed838db1d91df6bf
Signed-off-by: William A. Kennington III <wak@google.com>
|
|
- monitor pwrgd_p12v_all_slots GPIO signal
- add pwrgd_p12v_all_slots as IPMI sensor with pwrgd_p12v_slots
sensor name
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I70ce89ad2a065cdc9ee59ace73085fea7036da0d
|
|
Since x86-power-control add multi-node support, the all related
states D-bus service name add the node instance additionally
Thus, update the Chassis/Host/RestartCause D-bus service name
for all related scripts/services
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: If6ba080ff8757e01e230ee7d636e293922cdc127
|
|
The led.yaml not be passed issue is fixed, so
remove the manual replace action
ref: https://gerrit.openbmc-project.xyz/43927
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: Iaaf891f4d88527806b3f52694b17815e3b7fca17
|
|
Change Ie1a7c389edb6b7a048836a49283ceb62de51bba5 will be transitioning
the 'Type' property in Inventory.Item.Chassis to an enumeration.
In order to avoid crashing in PIM we need these default values in the
starting YAML to be fully-qualified enumeration names that will match
the values which will start in Ie1a7c.
This code is safe to merge as is without any PDI or PIM changes. Prior
to the PDI change, we must make a change to PIM that allows conversion
automatic from string->enum, leveraging library interfaces available in
sdbusplus. These will be submitted independently.
I checked the codebase for usage of this string. It appears that the
value is currently, effectively, write-only. There is code in bmcweb
that fills in the equivalent Redfish value but currently just hard-codes
the string 'RackMount'.
Tested: Booted Witherspoon in a QEMU model with this change and proposed
changes to sdbusplus + PIM. PIM no longer coredumps with the PDI change
and yields an expected persistence file:
```
$ pwd
/var/lib/phosphor-inventory-manager/xyz/openbmc_project/inventory/system/chassis
$ cat xyz.openbmc_project.Inventory.Item.Chassis
{
"value0": {
"cereal_class_version": 2,
"Type": 3
}
}
```
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Icaf9447f31ccdd945cdf74b3e017682e4aed686f
|
|
Revert back to install phosphor-ipm-kcs to image
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: Iad151c27c1f67fb8c6b8e422dde6a870ff32867f
|
|
Also need to update the UCT and UNCT to 72 degree as default
for nvme rewrite script
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I7c744a1b21b4cfe6eb7204464ff77e3c3abb7ff5
|
|
This reverts commit 3f2570e74ef929c7449911df18c1a8b9a59f02f1.
Change-Id: Ie9ccec722df88d857bcdc3b2661adb55490fc13a
Signed-off-by: William A. Kennington III <wak@google.com>
|
|
Enable phosphor-health-monitor for CPU/Memory/Storage_RW
utilization
ref: https://gerrit.openbmc-project.xyz/43387
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: If76a2d60ebe3db1bd6c1486d039677f35e7e8c5d
|
|
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I23757a6df671ed98f67d1e4280abfcc4ae22d233
|
|
update the UCT and UNCT to 72 degree as default
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I1baad8f3b621c7a704d5e4872d634653ea75c197
|
|
ipmitool:
- debug/manufacturing purpose
phosphor-user-manager:
- basic authenication for Refish
phosphor-nslcd-authority-cert-config:
- be able to install your owned CA certificate
for Redfish
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I8697fe674d58268dbe348c9374f7de32037041c1
|
|
remove packagegroup-obmc-apps-extras/packagegroup-gbs-apps
and add all the needed binaries to obmc-phosphor-image.bbappend
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: Ic56cb146987881e5b6312057ca412427f1d1ba29
|
|
remove patches:
- Update IPMI Chassis Control command transition requests
- Add Chassis State Transition interface
- Update Host State Transition function
- Fix issues and support signed sensor values
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I8f9f7d576ef7f7672bedf803ee73f331f96768ff
|
|
remove packages and recipe files:
- mac-address
- gbs-nvme-pwr-ctrl
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: Ic9d6a48d16ec058d1f444badc2f3e69628f77859
|
|
remove patch:
- Add the ability to dynamically set the failSafe percent
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I3fdb60d288fb2fcf83928ba712a546f823849652
|
|
Since the phosphor-logging still has other event
types other than IPMI SEL and also put them in the
"/var/lib/phosphor-logging/errors" directory. It
would make "ipmitool sel list" result in the
unspecified error
Thus, map these logs to the system event sensor as
"Undetermined system hardware failure"
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I00d557fba529d9b762746f8e582df336a16e630f
|
|
remove patch:
- Add power on monitor mechanism patch
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I932828d69948c3e0daa8fd428d620a75325e0ac9
|
|
- Add always restart to the service file
ref: https://gerrit.openbmc-project.xyz/42451
- Add retry 10 times when the service failed
- remove unused script
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: Ia3b027077d8cc9fae65013d38bf1ea730dbbbe42
|
|
- Remove read-margin-temp files and codes
- Modified sensor type in pid json to temp
setpoint set to target temp
Kp set to Kp * scalar
Ki set to Ki * scalar
- Pre-detect nvme present status in fan-table-init.sh,
and delete settings if nvme present is false.
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: Iba4ac164a7a39ca1be109301402911d432294997
|
|
Use mapper get-service to check if each LED D-bus
is ready and also make LED group service restart
not too fast when the serive start failed
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: Idd002e4f42d2664bc2f573a8dcc1afcfa593178a
|
|
Since the nvmem sysfs path would use nvme config id
in kernel 5.10, replace nvmem sysfs path by eeprom
to compatible with all kernel version
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I861f9bf8e26bf5b96e0727d4dfb3e2a7e4032116
|
|
This reverts commit f5f78f62d720ef57a54660927ba9a45515243190.
Reason for revert: due to x86-power-control not allowing DC cycle when host is down for fallback watchdog action
Change-Id: Idcc6e5d585bfcd9664755b32277d441118beef15
Signed-off-by: George Hung <george.hung@quantatw.com>
|
|
The patch added an implicit boost dependency
Change-Id: Ic990f59d46101684d4576bfbba89ea0f4c9015e1
Signed-off-by: William A. Kennington III <wak@google.com>
|
|
Since the VR may report negative current to BMC with
light loading, fine tune p12v_fan input/output current
lower threshold to -1
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I90c133d8ca5b9e399337fbe82472e5ed78e457a6
|