Age | Commit message (Collapse) | Author | Files | Lines |
|
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
|
|
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
|