| Age | Commit message (Collapse) | Author | Files | Lines |
|
Change-Id: I1d53510329749ff122c8a65157a72c8ff88bd306
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
|
|
The newest yocto version flags any assignment without a space around it
as a warning. The apply the recommended script[1] to fix it.
[1] https://github.com/shr-project/meta-webosose/blob/jansa/master/meta-webos/scripts/add-whitespace-around-assignment.sh
Change-Id: I93c715c38d3387fbef976452aefc3f0a7b50cab9
Signed-off-by: Ed Tanous <etanous@nvidia.com>
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I6291eeaf0ca7d4bd674bf0717f1234bfd757f6fe
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I2665e9e7e37242d546eafdb4d1c65dd0369ca635
|
|
GBS fails in Jenkins with the following error:
```
inplace-gbmc-update.bb: QA Issue: RPROVIDES is set to virtual/bmc-update but the substring 'virtual/' holds no meaning in this context. It only works for build time dependencies, not runtime ones. It is suggested to use 'VIRTUAL-RUNTIME_' variables instead. [virtual-slash]
```
Fix all usages of `RPROVIDES = "virtual/.*"` in meta-google and related
systems.
There are some remaining `PREFERRED_PROVIDER.* = "virtual/.*"` config
settings that probably aren't doing what the author(s) expected but
since those aren't being complained about I'll leave them be for now.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ie74c8190c9380c10ffc362fb4d8d728df3c33fba
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I8f74fcbb7bf66bb24fde1571ef09fac1095c7e59
|
|
The latest yocto release(styhead,5.1) has made a significant change[1]
to the use of the WORKDIR parameter.
This is additional fixes added to fix some of machines.
[1]: https://docs.yoctoproject.org/next/migration-guides/migration-5.1.html#workdir-changes
Change-Id: Ic43be2f9036377166e3e064f5570dddf2418af45
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>
|
|
The latest yocto release(styhead,5.1) has made a significant change[1]
to the use of the WORKDIR parameter.
This is not the complete change required but gets us going in the right
direction.
[1]: https://docs.yoctoproject.org/next/migration-guides/migration-5.1.html#workdir-changes
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I4f184b361be69e7704a0657a4a00cdad981aff8a
|
|
There is currently a lot of complexity of setting up an image feature
which is only used to enable the fru-device feature of entity-manager.
Simplify this by using RRECOMMENDS, in entity-manager, and eliminating
the IMAGE_FEATURE.
Tested:
Ran `bitbake entity-manager -ne` and observed the following on two
affected platforms:
In witherspoon:
```
# $PACKAGECONFIG [5 operations]
# :remove[pn-entity-manager] openbmc/openbmc/meta-ibm/conf/machine/witherspoon.conf:41
# "ipmi-fru"
# set openbmc/openbmc/meta/conf/documentation.conf:321
# [doc] "This variable provides a means of enabling or disabling features of a recipe on a per-recipe basis."
# set openbmc/openbmc/meta-phosphor/recipes-phosphor/configuration/entity-manager_git.bb:14
# [_defaultval] "ipmi-fru"
# set openbmc/openbmc/meta-phosphor/recipes-phosphor/configuration/entity-manager_git.bb:15
# [ipmi-fru] "-Dfru-device=true, -Dfru-device=false, i2c-tools,"
# set openbmc/openbmc/meta-phosphor/recipes-phosphor/configuration/entity-manager_git.bb:18
# [validate-json] "-Dvalidate-json=true, -Dvalidate-json=false, ${PYTHON_PN}-jsonschema-native"
# pre-expansion value:
# "ipmi-fru"
PACKAGECONFIG=""
```
In p10bmc:
```
# $PACKAGECONFIG [7 operations]
# :append[pn-debug-trigger] openbmc/openbmc/meta-ibm/conf/machine/p10bmc.conf:72
# " triggers"
# :remove[pn-entity-manager] openbmc/openbmc/meta-ibm/conf/machine/p10bmc.conf:75
# "ipmi-fru"
# set openbmc/openbmc/meta/conf/documentation.conf:321
# [doc] "This variable provides a means of enabling or disabling features of a recipe on a per-recipe basis."
# set openbmc/openbmc/meta-phosphor/recipes-phosphor/configuration/entity-manager_git.bb:14
# [_defaultval] "ipmi-fru"
# set openbmc/openbmc/meta-phosphor/recipes-phosphor/configuration/entity-manager_git.bb:15
# [ipmi-fru] "-Dfru-device=true, -Dfru-device=false, i2c-tools,"
# set openbmc/openbmc/meta-phosphor/recipes-phosphor/configuration/entity-manager_git.bb:18
# [validate-json] "-Dvalidate-json=true, -Dvalidate-json=false, ${PYTHON_PN}-jsonschema-native"
# override[pn-opkg]:set openbmc/openbmc/meta-ibm/conf/machine/p10bmc.conf:74
# "curl openssl ssl-curl"
# pre-expansion value:
# "ipmi-fru"
PACKAGECONFIG=""
```
Built an image for yosemite4 and confirmed that fru-device is still
installed:
```
root@yosemite4:~# systemctl | grep -i Fru
xyz.openbmc_project.FruDevice.service loaded active running Fru Device
root@yosemite4:~# systemctl status xyz.openbmc_project.FruDevice.service
* xyz.openbmc_project.FruDevice.service - Fru Device
Loaded: loaded (/usr/lib/systemd/system/xyz.openbmc_project.FruDevice.service; enabled; preset: enabled)
Active: active (running) since Tue 2024-02-27 09:26:26 PST; 8 months 1 day ago
Main PID: 368 (fru-device)
CPU: 757ms
CGroup: /system.slice/xyz.openbmc_project.FruDevice.service
`-368 /usr/libexec/entity-manager/fru-device
```
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Iaa3d9a8c2a2cf521816b42e36e15de21cf2a0bc2
|
|
The notable change is to the phosphor-hwmon configuration paths for all
Aspeed-based platforms. The changes were generated with the following
incantations:
```
git ls-files -- :/:*/ahb/apb/bus@1e78* | while read -r S; do D="$(echo $S | sed -E 's|ahb/apb/bus@1e78a000/i2c-bus|ahb/apb@1e780000/bus@1e78a000/i2c|')"; mkdir -p "$(dirname "$D")"; git mv "$S" "$D"; done
git ls-files -- :/:*/ahb/apb/pwm-tacho-controller@1e786* | while read -r S; do D="$(echo "$S" | sed -E 's|ahb/apb/pwm-tacho-controller@1e786000|ahb/apb@1e780000/pwm-tacho-controller@1e786000|')"; mkdir -p "$(dirname "$D")"; git mv "$S" "$D"; done
sed -Ei 's|ahb/apb/|ahb/apb@1e780000/|g' meta-ibm/meta-palmetto/recipes-phosphor/sensors/phosphor-hwmon_%.bbappend meta-ibm/meta-romulus/recipes-phosphor/sensors/phosphor-hwmon_%.bbappend meta-ibm/recipes-phosphor/sensors/phosphor-hwmon_%.bbappend meta-ingrasys/meta-zaius/recipes-phosphor/sensors/phosphor-hwmon_%.bbappend meta-quanta/meta-f0b/recipes-phosphor/sensors/phosphor-hwmon_%.bbappend meta-quanta/meta-q71l/recipes-phosphor/sensors/phosphor-hwmon_%.bbappend
sed -Ei 's/i2c-bus@/i2c@/g' meta-ibm/dynamic-layers/meta-openpower/recipes-phosphor/vpd/openpower-fru-vpd_%.bbappend meta-ibm/meta-palmetto/recipes-phosphor/sensors/phosphor-hwmon_%.bbappend meta-ibm/meta-romulus/recipes-phosphor/logging/files/callouts.yaml meta-ibm/meta-romulus/recipes-phosphor/sensors/phosphor-hwmon_%.bbappend meta-ibm/recipes-phosphor/chassis/vrm-control/ibm-ac-server/ir35221-unbind-bind.sh meta-ibm/recipes-phosphor/fans/phosphor-fan-presence-config/swift/config.yaml meta-ibm/recipes-phosphor/logging/ibm-logging/ibm-ac-server/policyTable.json meta-ibm/recipes-phosphor/sensors/phosphor-hwmon_%.bbappend meta-ingrasys/meta-zaius/recipes-phosphor/sensors/phosphor-hwmon_%.bbappend meta-quanta/meta-f0b/recipes-phosphor/sensors/phosphor-hwmon_%.bbappend meta-quanta/meta-q71l/recipes-phosphor/sensors/phosphor-hwmon_%.bbappend
sed -Ei 's|/ahb:apb/|/1e780000.apb/|g' $(git grep -lE 'ahb:apb/.*1e78')
sed -Ei 's|ahb:apb:i2c@1e78a000|1e780000.apb:bus@1e78a000|g' $(git grep -l ahb:apb:i2c@1e78a000)
sed -Ei 's|ahb:apb|1e6e0000.apb|g' meta-phosphor/recipes-phosphor/video/uart-render-controller/uart-render-controller.service
git ls-files -- :/:meta-ibm/*/ahb--apb/* | while read -r S; do D="$(echo "$S" | sed -E 's/ahb--apb/1e780000.apb/g')"; mkdir -p "$(dirname "$D")"; git mv "$S" "$D"; done; sed -Ei 's/ahb--apb/1e780000.apb/g' $(git grep -l ahb--apb -- meta-ibm)
git ls-files -- :/:*i2c-bus* | while read -r S; do D="$(echo "$S" | sed -E 's/(1e78a..0).i2c-bus/\1.i2c/g')"; mkdir -p "$(dirname "$D")"; git mv "$S" "$D"; done; sed -Ei 's/(1e78a..0).i2c-bus/\1.i2c/g' $(git grep -lE '(1e78a..0).i2c-bus')
```
What couldn't be automated was cleaning up the ibm-logging policy table,
which had many duplicate keys that needed manual inspection to remove.
It's worth noting that even for the remaining, non-duplicate keys, the
sysfs paths are not correct: The physical address of the I2C controllers
needs to map to the assigned device index, but in the file they all have
address 1e78a100 despite changes to the device index.
From eslint in CI:
```
meta-ibm/recipes-phosphor/logging/ibm-logging/ibm-ac-server/policyTable.json
9999:10 error Duplicate object key json/*
10020:10 error Duplicate object key json/*
10041:10 error Duplicate object key json/*
10062:10 error Duplicate object key json/*
10083:10 error Duplicate object key json/*
10104:10 error Duplicate object key json/*
17097:10 error Duplicate object key json/*
17118:10 error Duplicate object key json/*
17139:10 error Duplicate object key json/*
17160:10 error Duplicate object key json/*
17181:10 error Duplicate object key json/*
17202:10 error Duplicate object key json/*
17223:10 error Duplicate object key json/*
17244:10 error Duplicate object key json/*
17265:10 error Duplicate object key json/*
17286:10 error Duplicate object key json/*
17307:10 error Duplicate object key json/*
17328:10 error Duplicate object key json/*
17349:10 error Duplicate object key json/*
17370:10 error Duplicate object key json/*
17391:10 error Duplicate object key json/*
17412:10 error Duplicate object key json/*
17433:10 error Duplicate object key json/*
17454:10 error Duplicate object key json/*
19575:10 error Duplicate object key json/*
19596:10 error Duplicate object key json/*
```
Andrew Jeffery (1):
ARM: dts: aspeed: Rework APB nodes
Cosmo Chou (3):
dt-bindings: vendor-prefixes: add asteralabs
dt-bindings: trivial-devices: add Astera Labs PT5161L
hwmon: Add driver for Astera Labs PT5161L retimer
Potin Lai (2):
ARM: dts: aspeed: catalina: add pdb cpld io expander
ARM: dts: aspeed: catalina: update io_expander7 & io_expander8 line name
Rob Herring (Arm) (1):
arm: dts: aspeed: Use standard 'i2c' bus node name
Change-Id: I32ca7579eabd59ed20da3f481da687589a361763
Signed-off-by: Andrew Jeffery <andrew@codeconstruct.com.au>
|
|
phosphor-led-manager no longer assumes any default led priority.
The behavior was changed in
https://github.com/openbmc/phosphor-led-manager/commit/5a9f651949c133ed0868d72734c4ac82c75a7656
and
https://github.com/openbmc/phosphor-led-manager/commit/55badf7979f57d5f9ed2db726eebf35e84d0dac5
The led priority needs to be configured, so updating all configurations
here.
Before, led-manager assumed a default led priority of 'Blink'.
So this patch is just making that explicit in the existing
configurations. There should be no behavior change from this patch.
Change-Id: I74aabb567ccf6682d56877f8bb42e636fc1746f6
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>
|
|
/sys/class/i2c-adapter is a deprecated path, use the i2c bus path to
access the eeprom instead.
Change-Id: Idcf980994c4ea2a1ef16e667ddc57df2620ff694
Signed-off-by: William A. Kennington III <wak@google.com>
|
|
The busctl tree and grep used to get the number of zones is too wide and
the tree returns fans and sensors within the zone:
~# busctl tree xyz.openbmc_project.State.FanCtrl
`- /xyz
`- /xyz/openbmc_project
`- /xyz/openbmc_project/settings
`- /xyz/openbmc_project/settings/fanctrl
|- /xyz/openbmc_project/settings/fanctrl/zone0
| |- /xyz/openbmc_project/settings/fanctrl/zone0/cputemp
...
...
`- /xyz/openbmc_project/settings/fanctrl/zone1
|- /xyz/openbmc_project/settings/fanctrl/zone1/nvme0
...
...
Which means running the grep gives more than the 2 zones that were meant
to be gotten resulting in trying to set the failsafe for many more
zones:
Jul 11 11:49:08 busctl[9642]: Failed to set property Target on interface xyz.openbmc_project.Control.FanSpeed: Unknown object '/xyz/openbmc_project/settings/fanctrl/zone4'.
Jul 11 11:49:09 busctl[9693]: Failed to set property Target on interface xyz.openbmc_project.Control.FanSpeed: Unknown object '/xyz/openbmc_project/settings/fanctrl/zone4'.
Jul 11 11:49:10 busctl[9744]: Failed to set property Target on interface xyz.openbmc_project.Control.FanSpeed: Unknown object '/xyz/openbmc_project/settings/fanctrl/zone4'.
Jul 11 11:49:11 busctl[9767]: Failed to set property Target on interface xyz.openbmc_project.Control.FanSpeed: Unknown object '/xyz/openbmc_project/settings/fanctrl/zone4'.
Jul 11 11:49:12 gbs-set-failsafe.sh[8710]: Failure setting zone4 fan failsafe to 255
...
...
Tested:
With the change, the grep correctly gives "2" as the number of zones:
~# busctl tree xyz.openbmc_project.State.FanCtrl | grep -c -E 'zone[0-9]$'
2
Signed-off-by: Brandon Kim <brandonkim@google.com>
Change-Id: I1efd09b8d99ec00027489a05b647b43b1186c038
|
|
Apply the upstream security and bug fixes.
And modify KERNEL_DEVICETREE with prefix `nuvoton` for gbs build.
Signed-off-by: Tim Lee <timlee660101@gmail.com>
Change-Id: I4e39260615be2f421aa28c16c93e609133b973a0
Signed-off-by: Tim Lee <timlee660101@gmail.com>
|
|
Upstream poky made a commit that adds an `/etc/motd` with a strong
warning that meta-poky is not intended to be used for production
purposes. There isn't anything we use directly from meta-poky anymore,
so clean up all references to it.
See poky commit a226865c8683398b5f58628ba2ec5aee1ee6c19d for additional
context.
We could potentially switch from using the `poky` subtree to picking
up OECore directly, but right now it also provides us a qualified set of
OECore + bitbake.
Tested: Built yosemite4 and confirmed `motd` output is absent when
SSHing in.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I9c08650d2278b3ba26794c009adad3e593767f58
|
|
Update the interface name, path and other attributes for the
health monitor IPMI interface for s6q.
Change-Id: I35f746b863803a887803fac06855a72d7e98f2e5
Signed-off-by: Jagpal Singh Gill <paligill@gmail.com>
|
|
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>
|
|
Tested: can build
Change-Id: Id99ae67689f13b2f71fbe8ab3edad03700f9291b
Signed-off-by: Tom Tung <shes050117@gmail.com>
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I28ee50fa55ae47dd7fd8c99f8f6db8f5f6dfa53d
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I85272779478b66452acd11be93f5fea99e4c3a34
|
|
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
|
|
Since Linux 5.6, the entropy generation provided by rng-tools is now
in the kernel directly. Yocto has removed all dependencies on rng-tools
for packages that use to use it, such as ssh[1]. There isn't any reason
for us to explicitly include rng-tools any longer. Remove the feature
and package group that we had that included it.
[1]: https://lore.kernel.org/openembedded-core/20221209070538.72901-1-xiangyu.chen@eng.windriver.com/
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I85cbe7f30bb41045b7fd1170391a083db9e378b7
|
|
The webui-vue package is now enabled by default. Clean up all the meta
layers to clean up the explicit enablement.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I4895be56c70d2c8666dd96cec18d2c097f0c05a3
|
|
This moves the OpenBMC kernel to a v6.5 base for ASPEED.
The 6.5 kernel moved all 32-bit ARM device trees, so our BMC device
tress are now under an 'aspeed' subdirectory. This change means systems
must describe the KERNEL_DEVICETREE with the aspeed directory prefix.
There are 78 patches in the tree, with 40 of those patches not
yet queued for merging in v6.6.
The remaining out of tree patches include:
Andrew Jeffery (6):
dt-bindings: hwmon: pmbus: Add Maxim MAX31785 documentation
pmbus (max31785): Add support for devicetree configuration
pmbus (core): One-shot retries for failure to set page
pmbus (max31785): Wrap all I2C accessors in one-shot failure handlers
ARM: dts: aspeed: witherspoon: Update max31785 node
ipmi: kcs_bmc: Add a "raw" character device interface
Cédric Le Goater (1):
/dev/mem: add a devmem kernel parameter to activate the device
Eddie James (16):
ARM: dts: aspeed: bonnell: Add reserved memory for TPM event log
dt-bindings: soc: Add Aspeed XDMA Engine
soc: aspeed: Add XDMA Engine Driver
soc: aspeed: xdma: Add user interface
soc: aspeed: xdma: Add reset ioctl
soc: aspeed: xdma: Add trace events
i2c: core: Add mux root adapter operations
iio: si7020: Lock root adapter to wait for reset
eeprom: ee1004: Enable devices on multiple busses
dt-bindings: trivial-devices: Add Atmel AT30TSE004A serial eeprom
eeprom: ee1004: Add OF matching support
leds: pca955x: Refactor with helper functions and renaming
leds: pca955x: Use pointers to driver data rather than I2C client
leds: pca955x: Optimize probe led selection
leds: pca955x: Add HW blink support
leds: Ensure hardware blinking turns off when requested
Jae Hyun Yoo (1):
clk: ast2600: enable BCLK for PCI/PCIe bus always
Joel Stanley (14):
net: ftgmac100: Ensure tx descriptor updates are visible
ARM: aspeed: Add debugfs directory
ARM: soc: aspeed: Add secure boot controller support
dt-bindings: trivial-devices: Remove Infineon SLB9673 TPM
ARM: dts: nuvoton: npmc750-evb: Add default console
tpm: tis-i2c: Add more compatible strings
leds: pca955x: Revert "Remove the unused function pca95xx_num_led_regs()"
arm64: configs: Add Nuvoton NPCM defconfig
ARM: configs: aspeed: Add new FSI drivers
ARM: config: aspeed_g5: Enable SSIF BMC driver
ARM: config: aspeed: Remove FIRMWARE_MEMMAP
ARM: config: aspeed: Add Ampere SMPro drivers
ARM: config: Add openbmc defconfig
ARM: config: openbmc: Add HPE GPX and Nuvoton 7xx
Johannes Holland (1):
dt-bindings: tpm: Add schema for TIS I2C devices
Potin Lai (1):
mtd: spi-nor: winbond: Add support for w25q01jvq
Change-Id: Ib97af192391af6e71c96fe14e12cfc88c23d7a7d
Signed-off-by: Joel Stanley <joel@jms.id.au>
|
|
This is required to pick up the glome recipe that was moved from
meta-google to meta-security.
Change-Id: I52790cd2b792ed208498715748e5bed6fb120d20
Signed-off-by: Willy Tu <wltu@google.com>
|
|
The machine is no longer being supported.
Change-Id: I06cccfd7abdefc6ba93c43b1135b4c8942267eb6
Signed-off-by: Willy Tu <wltu@google.com>
|
|
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
|
|
Duke Du (1):
Filter some probe fail fru
Fran (8):
Merge pull request #6 from quanta-bmc/dev_harvey_retry_get
Merge pull request #7 from quanta-bmc/dev_harvey_retry_get
Merge pull request #9 from quanta-bmc/dev_scron_using_cmake_and_googletest_and_find_fru_bus
Merge pull request #10 from quanta-bmc/dev_scron_Update_README
Merge pull request #12 from quanta-bmc/dev_scron_0204_fixed_bug_when_empty_line_in_config
Merge pull request #11 from quanta-bmc/dev_scron_0107_Renamed_parameters
Merge pull request #13 from quanta-bmc/dev_scron_0205_Modified_mac_address
Merge pull request #16 from quanta-bmc/dev_duke_filter_multiple_same_fru_name_1001
Harvey Wu (4):
add retry get fru data loop & output log
only print first 256 bytes FRU data in log
Merge pull request #18 from wltu/dev-cstdint
Merge pull request #19 from wltu/dev-cmake-systemd-unit
Scron Chang (9):
Use Cmake, gtest, and add find-fru-bus.sh as well as config.txt.
Add new line in the end of find-fru-bus.sh
Add install config.txt and find-fru-bus.sh in CMake.
Add PERMISSIONS to find-fru-bus
Add new line to mac-address.cpp mac-address.hpp
Update README.md.
Renamed parameters
Fixed mac-address gets "is empty".
Improve the code efficiency and robustness.
Scron-Chang (1):
Merge branch 'master' into dev_scron_using_cmake_and_googletest_and_find_fru_bus
Willy Tu (2):
include cstdint for uint8_t
cmake: Update system unit dir location to install
Change-Id: I56abce1b6eca8e4a5a64a59c1949263e4eab462f
Signed-off-by: Willy Tu <wltu@google.com>
|
|
Removing obmc-dmtf-pmci to remove packagegroup-obmc-apps entirely.
Without doing so the gbs image will try to build the
packagegroup-obmc-apps packages.
Change-Id: I3e710cc904344fb86761f5ea5794ef94a506607c
Signed-off-by: Willy Tu <wltu@google.com>
|
|
Hao Zhou (1):
File System: Dependency Injection and Mock
John Wedig (1):
Add IPMI OEM command LinuxBootDone
Patrick Williams (1):
build: upgrade to C++23
Change-Id: I2d203a8ca9c5e0947b2d70f9d3aec3831c31fa64
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
Signed-off-by: John Wedig <johnwedig@google.com>
|
|
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>
|
|
The latest yocto update is flagging an error on machines using this
recipe. The reason appears to be because the templated services are
added to the SYSTEMD_SERVICE variable but they are not properly
installed into the filesystem.
Install the templated service file utilizing a more direct and tested
method.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I8f856e43244cc766fcec92ba9e21a6b98c9dbd03
|
|
The recipe previously had a hard coded removal of `/lib` which will no
longer exist under usrmerge. I think the purpose of this was to delete
the systemd units, but leave the library. Therefore I've adjusted the
removal to be systemd_unitdir instead of `/lib`.
This results in the following package structure:
```
$ find tmp/work/armv7a-openbmc-linux-gnueabi/google-ipmi-sys/0.1+gitAUTOINC+59445f5e21-r1/packages-split/google-ipmi-sys -type f
tmp/work/armv7a-openbmc-linux-gnueabi/google-ipmi-sys/0.1+gitAUTOINC+59445f5e21-r1/packages-split/google-ipmi-sys/usr/lib/systemd/system-preset/98-google-ipmi-sys.preset
tmp/work/armv7a-openbmc-linux-gnueabi/google-ipmi-sys/0.1+gitAUTOINC+59445f5e21-r1/packages-split/google-ipmi-sys/usr/lib/ipmid-providers/libgooglesys.so
```
The `98-google-ipmi-sys.preset` file is empty.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I05dd016fcc8dfd8dc33ba90acad100afafb18073
|
|
Use the libhoth recipe in meta-security instead of openbmc one. Set the
source rev override until that bump is submitted and subtree updated.
libhoth source bump in
- https://lore.kernel.org/yocto/20230517172730.3837643-1-jebr@google.com/T/#u
Change-Id: I6099d6351417dcef9eb3b4d297de43a5e229d7dd
Signed-off-by: Willy Tu <wltu@google.com>
|
|
This reverts commit ec7cab9378f548e94ee4af83946b2583d1fc7194.
ec7cab9378f5 ("Add socket-id for the first console") was only a partial
fixup of the obmc-console configurations found in the tree. For
historical reasons not all platforms that support multiple consoles
install a client configuration file for all consoles. Instead they relied
on some default behaviour that was removed in obmc-console's 4e7186918599
("Fixed broken dbus interface for multiple consoles"). Applying the
configuration changes in the manner of ec7cab9378f5 ("Add socket-id for
the first console") resulted in an asymmetric configuration between the
client and server, breaking some SSH SOL instances.
As of ae2460d0b8e8 ("obmc-console: Provide a default value for
`console-id`.") in obmc-console the requirement to specify
`socket-id`[^1] is lifted. Instead, the configuration can choose to
override a default value. This restores the original behaviour and
unbreaks SSH SOL.
[^1]: Now an alias of the `console-id` configuration key
Websocket and IPMI consoles remain broken. A fix is under development
that also takes a proper approach to supporting multiple consoles in
bmcweb, and the same solution pattern can be applied in ipmid.
More discussion of the problems involved can be found here:
https://amboar.github.io/notes/2023/05/08/happenings-in-obmc-console.html
Change-Id: I274284c791758f336da6d6301dc523bac2b5dd69
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I189baf142cc86d44ebbf615edd3bb0ec1785be8d
|
|
This drop adds socket-id for first console. This was left out before
but with the new design this parameter is required and enforced by
obmc-console code. It is decided to set "socket-id = console0"
initially because that is the current path used by bmcweb but in future
you can change this value to whatever is appropriate.
The console information is now available through the DBUS interface.
Each console server register it's object path on DBUS. The leaf node of
the object path is used to create a console path. For example in the
following console0 the leaf node "console0" is used to make up the
console path and the GUI will use console path "/console/console0" to
connect to the console.
busctl tree xyz.openbmc_project.Console.console0
`-/xyz
`-/xyz/openbmc_project
`-/xyz/openbmc_project/console
`-/xyz/openbmc_project/console/console0
Here is a discord thread where new design was discussed.
https://discord.com/channels/775381525260664832/1083551792094249051
Tested:
Tested on the rainier system.
Related commits:
1) phosphor-dbus-interface: https://gerrit.openbmc.org/c/openbmc/phosphor-dbus-interfaces/+/61486
2) obmc-console: https://gerrit.openbmc.org/c/openbmc/obmc-console/+/62496
3) bmcweb: https://gerrit.openbmc.org/c/openbmc/bmcweb/+/62525
Change-Id: I5fd06f3aac69872a3e5032d592a9b59711795cfe
Signed-off-by: Ninad Palsule <ninadpalsule@us.ibm.com>
|
|
The latest version of shellcheck is stricter. Fix a few warnings.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I7f94205aef6d348e9c53dca261ea996dd4f84a6a
|
|
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>
|
|
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I8e54833ac78e540e9dd5011533d53ff9a3af6763
|
|
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>
|
|
obmc-host-ctl is defined as a MACHINE, DISTRO, and IMAGE feature.
The IMAGE feature pulls in a virtual provider only if the COMBINED
feature (MACHINE + DISTRO) is enabled. This is only done on openpower
machines. There are a lot of other places where machines are half-way
disabling this feature and/or changing variables which have zero effect
anyhow. Clean up almost all references to `obmc-host-ctl`.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I090d782bb6a34dc4e3c43df97e8ccf6e1812fed5
|
|
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
|
|
Josh Lehan(1):
Caching services and values for fast performance
Harvey Wu(1):
Update to c++20, meson >= 0.57.0
Signed-off-by: Harvey.Wu <Harvey.Wu@quantatw.com>
Change-Id: I8ec80185c228bf50e0c373214ec607810800642b
|
|
Added do_install in base recipe for phosphor-health-monitor to
copy the json file from WORKDIR to target for monitoring the CPU
and Memory usage of BMC. Therefore, this can be removed in each
bbappend files.
Tested: Tested in Facebook YosemiteV2 platform.
Signed-off-by: Jayashree Dhanapal <jayashree-d@hcl.com>
Change-Id: I78eed7efb3e0be81a952c4bc3c3ae64579a48f7d
|
|
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I65febeeec11e6e5b40ee728f42cfbe77023dae1e
|
|
We also need to move the conf-notes.txt files.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ic44e015c0216b526de4fec277ad42f162bca1f33
|
|
The latest poky commit is requiring us to have all of
our template configs in a subdirectory instead of directly in
the `conf` directory. Without this we end up with errors during
setup like:
```
Error: TEMPLATECONF value (which is .../openbmc/meta-facebook/meta-bletchley/conf) must point to meta-some-layer/conf/templates/template-name
```
Fix this by moving all of our template files into the 'default'
template subdirectory (following the pattern of poky) and modifying
`setup` as necessary to follow.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Iecefde73d55acbb6bc63ae3d68c4311adaf327ae
|