Age | Commit message (Collapse) | Author | Files | Lines |
|
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
|
|
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I032031d25b0755611e870c779792844ef87d9e57
|
|
Commit the GBS console redirect setting for obmc-console
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: If4b39537bc71a00def2d788a24ab06bd1f40a6ad
|
|
We want to use packageconfig options instead of build system specific
ones.
Change-Id: Ib0b293c457ea418e98d294da9796c7b103209888
Signed-off-by: William A. Kennington III <wak@google.com>
|
|
Enable fallback watchdog since it is possible that
host fails to boot before enabling the watchdog
timer at all
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I4eeb48b618cd6aba3e48dd1337ea69e3ce8292c6
|
|
Lower CRITLO parameter to 1500.
Signed-off-by: Brandon Kim <brandonkim@google.com>
Change-Id: I34383068cf1a3469a4f946bac862a60398634f62
|
|
Use the PACKAGECONFIG option instead of setting the configure argument
directly.
Change-Id: I514138c63d58af1a0d83815241410a679a859d73
Signed-off-by: Benjamin Fair <benjaminfair@google.com>
|
|
Instead of appending virtual/obmc-gpio-presence to ${PN}-inventory,
simply move it to obmc-phosphor-image.bbappend
Signed-off-by: Brandon Kim <brandonkim@google.com>
Change-Id: Ie1ec9bdd3b29cc73acea272199c543b3da715bf1
|
|
The 0001-sensor-Implement-sensor-ASYNC_READ_TIMEOUT.patch was merged:
https://gerrit.openbmc-project.xyz/24337
And SRCREV bumped here:
https://gerrit.openbmc-project.xyz/43005
Remove the patch to fix build failures.
Change-Id: I38dcaa56a2d184ed9c65a8a653a4d34a80d8f83b
Signed-off-by: Brandon Kim <brandonkim@google.com>
|
|
ref: https://gerrit.openbmc-project.xyz/24337
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: Id02a797a515363f74fcff27fc6ce9fc88a4bd9cc
|
|
- update to use meson instead of cmake
- change http request body limit size to 40MB
to pass firmware update of Robot framework
test for GBS
- create /var/lib/bmcweb folder to satisfy the
path redfish eventservice required to save
config and subscriptions
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: Ic0de91547d11aad9cc6412a389d26c2da5c79644
|
|
Correct as the cable sensor type for cable sensors
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I5483e0cf103dccdf38a79ce781529e0f6b7b2d88
|
|
Add Max/Min value setting in json config, so sel-logger
could show correct sensor value
ref: https://gerrit.openbmc-project.xyz/42073
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: Ieebb60fec73a83a17babd4a918645dd1a387d8b4
|
|
This reverts commit b588235e34a679c45ca41848473da3f59613acd7.
Reason for revert: wrong sensor type number for cable sensors
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I865fb34b46295e727b652042619724148668183a
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I2a5eb57d100ced02043bf68f64e4596c59da6ebd
|
|
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: Idce541a84d476fa9c5af0fd1311eee9f9732679b
|
|
Use service files instead of setting D-bus properties for power
control and remove unused patches
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I6f80979906e87b77b36a6e333456d4e5c42249ce
|
|
- meson.build doesn't parse the led.yaml you provide,
so replace the yaml file of source before compiling
- install service-override.conf directly intead of
using SYSTEMD_OVERRIDE to avoid package name
changing
- remove unused json config
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I28c72dfabf32d22f917ca1e244f5b700ae941667
|
|
They should be the cable sensor type
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I5056a7bd51effdc34ca89966da8ab4f666bbc92a
|
|
Using mapper wait to ExecStartPre would quit immdiately
while the D-Bus path is not active yet, so change to
add to "After=" to make sure it would wait until the
D-Bus path work
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I86029f4eba6b34b5c313ec9cc1578ef173d5c2aa
|
|
limit the max number of IPMI SEL logs as 256 since the logs
would store in persistent path (/var/lib/phosphor-logging/errors)
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I120623a934bd9fa6e8dd5c9b372b110287ef85d5
|
|
remove u-boot-fw-utils/clear-once packages since we don't
use u-boot environment
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: Ideb8ece3529bd8fea658f9c160551950773d6d13
|
|
- Update IPMI Chassis Control command transition requests:
https://gerrit.openbmc-project.xyz/29051
- Add Chassis State Transition interface
https://gerrit.openbmc-project.xyz/29050
- Update Host State Transition function
https://gerrit.openbmc-project.xyz/29049
- Fix issues and support signed sensor values
https://gerrit.openbmc-project.xyz/26754
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: Ice305cb9a9e390099a828e872af451e832d59749
|
|
Remove ikvm and fru-device packages
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I3a6cb2b92ffcae11b775813bec12190399d27c88
|
|
1. Remake the patch to fix includes error:
- Implement sensor "ASYNC_READ_TIMEOUT"
https://gerrit.openbmc-project.xyz/24337
- Add power on monitor mechanism
https://gerrit.openbmc-project.xyz/28181
2. Set negative-errno-on-fail flag as false
explicitly, otherwise it would be true
conflicts with update-functional-on-fail
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: Id550bc5261e6fa7218e2ff2731e1af51487ee376
|
|
- change back to OpenBMC upstream for phosphor-host-ipmid
and phosphor-sel-logger
- add inventory sensors YAML for IPMI SEL
- enable SEL_LOGGER_SEND_TO_LOGGING_SERVICE and
SEL_LOGGER_MONITOR_THRESHOLD_ALARM_EVENTS for IPMI SEL
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I16c58444986aec6c927e175a2c69f9cca92a7bcb
|
|
The repository supports it, so switch it over in bitbake.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I8d96c79b395ee2614ddd869091569f245426c5c7
|
|
- add fans e-fuse PG GPIO config file
- add fans e-fuse PG GPIO as the sensor
- monitor fans e-fuse PG GPIO and when it assert low,
record the SEL log
Signed-off-by: George Hung <george.hung@quantatw.com>
Change-Id: I32298097b1e79036b67edea65b84ef4d78cbacfc
|