Age | Commit message (Collapse) | Author | Files | Lines |
|
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
|
|
The BOARDVER_PVT_LAST is used to indicate the last value of PVT version.
The GPIO line name (BACKUP_SCP_SEL) replaces GPIO number 168.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I9057371597996cdcf620cb79d94b7e72356a535b
|
|
Add the missing shellcheck source path to kudo-lib.sh for the
references to I2C_BMC_CPLD and I2C_MB_CPLD arrays.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I97606afdf59caea3695428839d8806a96198ee96
|
|
Editing kudo_entity_association_map json so that PCIe Instance
number matches the i2c bus number.
Signed-off-by: Grant Williams <grant.williams@fii-na.com>
Change-Id: I2c480ae288f9a9a4281d54bd4e1b3131bb427836
|
|
Upstream phosphor-state-manager doesn't want to support
obmc-chassis-powercycle target.
Signed-off-by: Avenash Asai Thambi <avenash.thambi@fii-usa.com>
Change-Id: Idb955d3f0bee0701bd87d265fac679e7209ff87b
|
|
The upper critical value of CPU0_MEM_PW has been changed to 100
and the max value has been set to 200 to support this change.
Signed-off-by: Avenash Asai Thambi <avenash.thambi@fii-usa.com>
Change-Id: I1ee84cbeca688a8203e49c891b8a8a870db693fe
|
|
Adjust CPLD aux version by shifting byte from 2 to 5 to
1 to 4
cmd: kudo.sh fw
Before change: BMC_CPLD: 0.0.15.0
[5], [4], [3], [2]. the [5] is null
Post change: BMC_CPLD: 0.15.0.0
[4], [3], [2], [1]
Signed-off-by: Grant Williams <grant.williams@fii-na.com>
Change-Id: I518ddb5e739fb73876f325b44b0104210a48c09c
|
|
Removed the 2p sensor support from the phosphor-virtual-sensor
and modified the recipe accordingly to support the new config.
Signed-off-by: Avenash Asai Thambi <avenash.thambi@fii-usa.com>
Change-Id: Ie9896ed6473844f54ba0fb3e422de946bec52892
|
|
The custom phosphor-pid-control.service is not necessary because PID
control still operates with the default phosphor-pid-control.service.
Signed-off-by: Avenash Asai Thambi <avenash.thambi@fii-usa.com>
Change-Id: Ic0f1447c5a20f275cec9e51d26018f6f55101ed0
|
|
The version value used to be interpreted as hexadecimal for IPMI
and then was changed to decimal to match Redfish.
This will change the interpretation to decimal and increment to
the next version.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I9e3513c7e51186bdbdbc2f3526a8e1dd0edf808d
|
|
kudo.sh: Show which SCP EEPROM is used in (fw) option
kudo-lib.sh: Added get_scp_eeprom() function
Signed-off-by: Karthikeyan Sundaram <karthikeyan.sundaram@fii-na.com>
Change-Id: I686ff6b3bf9ab8ef7434ab44f0668912bfffb9b2
|
|
The phosphor-watchdog-host-reset service used the deprecated
force_reset command instead of the host_reset.
Signed-off-by: Ashwin Murali <ashwin.murali@fii-na.com>
Change-Id: I329006b23ce7ac50d659dd8cf59704256b2bffaa
|
|
Replace append directive with += in usb-network bb recipe
Signed-off-by: Grant Williams <grant.williams@fii-na.com>
Change-Id: I57d40a40a3f9f50e0ec52f4f53a10b261f655097
|
|
Replaces the hardcoded board version limits with reference to name
in kudo-lib.sh for Kudo layer.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I69a2a157d5f49f474b06bbc15d15377371fffcf2
|
|
The only recipes which use content from meta-perl are openpower
machines:
```
meta-openpower/recipes-bsp/pdata/pdata_git.bb: libxml-libxml-perl-native \
meta-yadro/meta-nicole/recipes-phosphor/logging/openpower-esel-parser_git.bb: libxml-libxml-perl-native \
```
Remove meta-perl from the bblayers of every layer except openpower
machines.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I51f1a8fbfbe879295c64d2339fc115dbd8823681
|
|
Use GPIO instead of IPMI for host condition.
Adopt the multi-host support and syntax for phosphor-state-manager
services.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I58ff4291ce3ec055579df5a2463c178e53d93625
|
|
This was added with cde0f094f for libseccomp, but shortly after that
upstream moved libseccomp from meta-security to core (241c7d2e6). As
such, meta-security is no longer used or required.
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
Change-Id: I371e54b11f6336720dfc6edf0ef733d22b7fb4f2
|
|
Adds the I2C bus and address in array form to kudo-lib.sh, and
replaces the hardcoded values with references across Kudo layer.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I69bbff444e013e61d7f7fd5051612a6d4adc8ded
|
|
The method to monitor the power and reset buttons through the
ampere_power_util.sh with phosphor-gpio-monitor was a temporary
solution. It was a temporary workaround until there is support for
Nuvoton GPIO in phosphor-buttons.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I34541c0f56887962ad780b811eae44f1d851059d
|
|
Replaces hard-coded GPIO throughout the Kudo layer with a reference
to its line name from DTS.
The process to dynamically obtain the GPIO number for Get is added
to kudo-lib.sh,
1. Use `gpiofind <Line Name>` to obtain gpiochip# and pin#
2. If the gpiochip# exceeds the number of GPIO banks of Nuvoton,
then it is an I/O Expander.
a. Use `gpiodetect` and extract the I2C bus-addr
b. Obtain GPIO base value from sysfs using bus-addr
c. The GPIO number is then the (base + pin#)
3. If the gpiochip# is within Nuvoton GPIO banks, then calculate
the GPIO number as ((gpiochip#)*32 + pin#)
Then the GPIO number can be used in the export to obtain the value.
Tested with Nuvoton:
- Can read/set Nuvoton GPIO with defined line names
- Can read/set GPIO from I/O Expanders with defined line names
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I651c0707a8bbc170d3c5a5fc261e5f9c62453e9f
|
|
We want it to be possible to order units prior to the powercycle event
if state needs to be persisted prior to a powercycle.
We also no longer need the logic to handle waiting for the delay as this
is now part of the global target logic.
Change-Id: Icba008e66c81834a65099a25b1af4f7bde4e94b5
Signed-off-by: William A. Kennington III <wak@google.com>
|
|
Replace append directive with += in kudo-scp-update bb recipe
Signed-off-by: Ashwin Murali <ashwin.murali@fii-na.com>
Change-Id: I3d207f3aae5a504ac570472d0848e40a4699cbe7
|
|
Replace append directive with += in kudo-cpld-updates bb recipe
Signed-off-by: Grant Williams <grant.williams@fii-na.com>
Change-Id: If5c63210a5195033b182aade3c995e39225a6597
|
|
The NBM 2G and 4G chipset version is selected using expressions in
the json based on the chipset used and available in SDR.
Signed-off-by: Avenash Asai Thambi <avenash.thambi@fii-usa.com>
Change-Id: Ibd3c683fde94206254c37c3552db828a6b78e982
|
|
Per [1][2], Yocto is starting to deprecate ABORT and has replaced
the "ABORT" action in BB_DISKMON_DIRS entries with "HALT".
1. https://wiki.yoctoproject.org/wiki/Inclusive_language
2. https://git.yoctoproject.org/poky/commit/?id=4f77505d94a8f6260933f457e9848d1d2fa98ce5
Tested:
Built obmc-phosphor-image successfully and eliminate the following
warnings:
```
WARNING: The BB_DISKMON_DIRS "ABORT" action has been renamed to
"HALT", update configuration
```
Signed-off-by: George Liu <liuxiwei@inspur.com>
Change-Id: If57d0ded9fac41e23b31b01e2a0e309ac7388148
|
|
Fixes bash shellcheck errors so that the exemptions in run-repotest
can be removed.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I6afb059cf1151c871ddfcaf67efc6a63183f7130
|
|
The tla2024 support has been moved into internal repo.
Signed-off-by: Avenash Asai Thambi <avenash.thambi@fii-usa.com>
Change-Id: Ia77538f10015b033f989b9371c69cfe5d286ee52
|
|
Fix an issue where the SubMajor version was not converted
correctly from Hex to Dec.
Signed-off-by: Ashwin Murali <ashwin.murali@fii-na.com>
Change-Id: I38d9d7e93652696b9674a8719d751325160b4f13
|
|
Replace append directive with += in hotswap-power-cycle bb recipe
Signed-off-by: Ashwin Murali <ashwin.murali@fii-na.com>
Change-Id: I64e3f2c2dd96b5f42b089d380c3568d1f0f9d544
|
|
Replace append directive with += in kudo-bmc-update bb recipe
Signed-off-by: Ashwin Murali <ashwin.murali@fii-na.com>
Change-Id: I91e02e7c671783a885015610a59e3383a2bf3f60
|
|
Replace append directives with += in kudo-bios-update bb recipe
Signed-off-by: Ashwin Murali <ashwin.murali@fii-na.com>
Change-Id: I9ced6841b6f9d68d3697bb51be5e36186262b0ce
|
|
add comment on sel-logger to match kudo code "enable clear sel method"
Signed-off-by: Grant Williams <grant.williams@fii-na.com>
Change-Id: Iae39238880c5fd9ddf86c40743d8dd6c5be73041
|
|
The following are not used in Kudo and can be removed.
- obmc-bmc-service-quiesce@.target
- ${PN}-chassis-check-power-status
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: Iad82935b3b0c621ec6b1c81cd0ed56a6be55baaf
|
|
Update Kudo Version ID to v0.10-00-kudo
Signed-off-by: Karthikeyan Sundaram <karthikeyan.sundaram@fii-na.com>
Change-Id: I86c0e0142b197b1be7acf426fc1f68e7f8482217
|
|
Upstream has removed the image-prelink bbclass[1], so remove it from any
local.conf.sample that still has it enabled.
1. https://git.openembedded.org/openembedded-core/commit/?id=857baaf9e3d181ca18264e85d90b899fd94acff9
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I70f42f8816f5dea207fcf5c34837d6796b3f6bb6
|
|
* Deprecate N-1 release (hardknott).
* Enable N+1 release (kirkstone).
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I39e027e02dab64b4390b46ffbd9c299c858f403e
|
|
Add CPU detect to support 1p/2p virtual sensor json configuration
Add new thermal sensors
Add virtual sensor json selection support based on NBM chipset version
Signed-off-by: Mustafa Shehabi <mustafa.shehabi@fii-na.com>
Signed-off-by: Avenash Asai Thambi <avenash.thambi@fii-usa.com>
Change-Id: Ib5a442cc35f2402c521557d908a90a86dd091cc0
|
|
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
|
|
The package, phosphor-hostlogger, is no longer required.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I17bede02ae32c47c5078368899248c734683c6a3
|
|
avenash.thambi (1):
foxconn-ipmi-oem: Add Maintainers and OWNERS into foxconn-ipmi-oem
Change-Id: I93a509b724a9531511dd7fd31d0a7d715e4525a9
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
e.g.
WARNING: Duplicate inclusion for /opt/openbmc/meta-phosphor/conf/distro/openbmc-phosphor.conf in /opt/openbmc/meta-google/conf/distro/gbmc.conf
WARNING: Duplicate inclusion for /opt/openbmc/meta-phosphor/conf/distro/include/phosphor-base.inc in /opt/openbmc/meta-phosphor/conf/distro/openbmc-phosphor.conf
WARNING: Duplicate inclusion for /opt/openbmc/meta-phosphor/conf/distro/include/phosphor-defaults.inc in /opt/openbmc/meta-phosphor/conf/distro/include/phosphor-base.inc
WARNING: Duplicate inclusion for /opt/openbmc/meta/conf/distro/include/security_flags.inc in /opt/openbmc/meta-phosphor/conf/distro/include/phosphor-defaults.inc
WARNING: Duplicate inclusion for /opt/openbmc/meta/conf/distro/include/no-static-libs.inc in /opt/openbmc/meta-phosphor/conf/distro/include/phosphor-base.inc
WARNING: Duplicate inclusion for /opt/openbmc/meta/conf/distro/include/yocto-uninative.inc in /opt/openbmc/meta-phosphor/conf/distro/include/phosphor-base.inc
Signed-off-by: Peter Foley <pefoley@google.com>
Change-Id: I74856075638fe243974ebb4818e80f8e7e2022e6
|
|
The server.ttyS1.conf and server.ttyS3.conf files were missing from
the SRC_URI.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: Ic2762f8cfdb1a5b6e2965a77890a98afdc1ae31e
|
|
With the usage of new fans there is a delay in the rate of change
of fan pwm from 255 to lower level so the pwm-init.sh is updated to
configure the fan controller settings at boot.
Signed-off-by: avenash.thambi <avenash.thambi@fii-usa.com>
Change-Id: If43dd1de39702128bf61fa6483ea2d210410f67e
|
|
This file is no longer needed since we moved away from
using the intel-ipmi-oem.
Signed-off-by: avenash.thambi <avenash.thambi@fii-usa.com>
Change-Id: Ifadd213d9b26a11d88caeab37b4b5cb079ba0f52
|
|
e.g.
| CMake Error at /opt/openbmc/build/kudo/tmp/work/armv7a-openbmc-linux-gnueabi/fii-ipmi-oem/git-r0/recipe-sysroot-native/usr/share/cmake-3.21/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
| Could NOT find PkgConfig (missing: PKG_CONFIG_EXECUTABLE)
Change-Id: Ie9667094c782072614ff5dccd61a1cf31ebd5589
Signed-off-by: Peter Foley <pefoley@google.com>
|
|
ERROR: Layer 'kudo-layer' depends on layer 'fii-layer', but this layer is not enabled in your configuration
Signed-off-by: Peter Foley <pefoley@google.com>
Change-Id: I419a42a8e5e8fd3452e5bdc1073aee8dd1699c37
|
|
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
|
|
The obmc-console recipe changes from
https://gerrit.openbmc-project.xyz/c/openbmc/openbmc/+/49336
add support for multiple port configurations. Therefore, the configs
do not need to occur in the bbappend.
Removed the additional configuration steps to configure multiple
ports and used OBMC_CONSOLE_TTYS override instead.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: Iba14a8b5935cd3cd8718a8bf94d5eed18de9db9e
|