Age | Commit message (Collapse) | Author | Files | Lines |
|
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
|
|
According to
https://github.com/openbmc/docs/blob/master/anti-patterns.md,
/usr/sbin is reserved for system administration executables. The
executables called by services or internal use are placed in
/usr/libexec/<package>.
Executables kept in /usr/sbin,
- kudo.sh
- kudo-ras.sh
- kudo-fw.sh
Executables moved to /usr/libexec,
- ampere-hostctrl/ampere_power_util.sh
- kudo-fw/kudo-fw-ver.sh
- kudo-fw/kudo-lib.sh
- kudo-boot/init_once.sh
- usb-network/usb_network.sh
- obmc-console/kudo_uart_mux_ctrl.sh
- pwm-init/pwm_init.sh
- phosphor-virtual-sensor/tla2024-enable.sh
Also removed /usr/bin/env from service files.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I270eee5cb9c0ca248f1b84a87d1cff5c6dc129f3
|
|
Restart psusensor service to ensure that the VBAT sensor
doesn't say "no reading" until it's second query after
a hotswap.
Signed-off-by: Ashwin Murali <ashwin.murali@fii-na.com>
Change-Id: I6bb8866bff011e4641e5f26b82a0b17e50cc5fc8
|
|
Modify the kudo.sh fw to display BMC version format in decimal
Signed-off-by: Mustafa Shehabi <mustafa.shehabi@fii-na.com>
Signed-off-by: XP Chen <xiao-peng.chen@fii-na.com>
Change-Id: Ifdc18e4b417bd764794750a2f7a28db849c17eca
|
|
1. ampere-host-power-cycle.service will force off host followed
by force on.
2. ampere-host-reset.service will pulse the sysreset pin
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I4736c82f8f462a21650d5abba55c036a05da8e25
|
|
`BBLAYERS_NON_REMOVABLE` is obsolete and no longer required.
As it said by Yocto documentation it can be used by `Hob`
https://www.yoctoproject.org/docs/1.5.2/ref-manual/ref-manual.html#var-BBLAYERS_NON_REMOVABLE
that already removed since Yocto-2.1
https://www.yoctoproject.org/tools-resources/projects/hob
Change-Id: Ibc2d8268a9d837a81e9cf6b0131dba8d0a030a3f
Signed-off-by: Alexander Filippov <a.filippov@yadro.com>
|
|
The ampere-host-reset.service and ampere-host-force-reset.service
are not used. Host is reset with ampere-host-power-cycle.service.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: Ie6c58c5c9d1179d79e5c5be307a37bdb263172ff
|
|
For shutdown, if it does not go through the power utility, the
Chassis state will not change.
For host reset, execute additional verification steps attached
to the power utility service performing the host reboot.
Signed-off-by: XP Chen <xiao-peng.chen@fii-na.com>
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I2f67a34ba8cfd80fd3dde999baa329d44288bdfe
|
|
The phosphor-pid-control package will be provided through
obmc-fan-mgmt, so it does not need to be provided through
obmc-fan-control as well.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I29ac8363056db866d6181d958d8a86e53605400d
|
|
The new yocto syntax was missing from,
https://gerrit.openbmc-project.xyz/c/openbmc/openbmc/+/46289
Also removed an extra empty line from the recipe.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: Id7684b24ecd2a51e7e2a28c10a76b61519ea0872
|
|
Avenash Asai Thambi (1):
Add boot count into motherboard eeprom and clear command
Change-Id: Iedc41527654482c2a45c414ec3c778a451199c93
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Change-Id: I8711fa16a624c9f132310357f973cebd25f9b1d3
Signed-off-by: William A. Kennington III <wak@google.com>
|