Age | Commit message (Collapse) | Author | Files | Lines |
|
As part of performing a reset instead of DC cycling when reboot is
called from the host. Remove the check on host status since previously
it would prevent a host reboot.
Change-Id: Id4b7888ebcb87c42cd46363b883d88ffa851d67e
Signed-off-by: Jonico Eustaquio <jonico.eustaquio@fii-na.com>
|
|
1. Fix syntax error resulting in "command not found" when calling
"kudo.sh usb"
2. Correct device address typo in print statement
3. Display only the udc that exist
Signed-off-by: Kyle Nieman <kyle.nieman@fii-na.com>
Change-Id: Ic9446eccf864fc518043fc0f1f16098d50415202
|
|
Willy Tu (1):
build: Migrate to meson and add clang-format
Change-Id: I846dae5a4a17cebb869e23b1ff507a31f2fe4877
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Willy Tu (1):
Remove Foxconn owners to test Gerrit Owner plugins
Change-Id: I5681d9a35ab8566ad1f1e01b4c0b91022541d031
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
As noted in the following mailing list thread:
https://lists.ozlabs.org/pipermail/openbmc/2022-November/032457.html
and implemented via this series of commits:
https://gerrit.openbmc.org/q/topic:op-power-control-ref
Some systemd services had their names changed. Update the service files
in this repo to reflect this.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I3667e6aed0d6c7ac01b0f2d1c7dee7e645df8235
|
|
This power control implementation is not specific to openpower systems
(as seen by the variety of systems including it) so rename it to
something more appropriate.
This is using code from skeleton which is not ideal but that's a change
for another day.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Iffe6427b5494558b53cd6f5716a08cce82cb90d8
|
|
ADM1266 executables now take I2C bus as a command line argument instead
of being hardcoded, so they can be utilized by additional projects.
Overview of changes:
1.)kudo-fw.sh
- Modify call to ADM1266 to pass I2C bus
2.)kudo.sh
- Modify call to ADM1266 to pass I2C bus
Signed-off-by: Kyle Nieman <kyle.nieman@fii-na.com>
Change-Id: I266818d3d57dd019ca0b8a78638e0f5f5a1d60b9
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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.
| 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>
|
|
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
|
|
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
|
|
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
|
|
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>
|
|
The ampere_power_util.sh file has a syntax error of missing double
quotes in a echo print.
This is a follow up to the gerrit commit
https://gerrit.openbmc-project.xyz/c/openbmc/openbmc/+/48477
Signed-off-by: Mustafa Shehabi <mustafa.shehabi@fii-na.com>
Signed-off-by: avenash.thambi <avenash.thambi@fii-usa.com>
Change-Id: Iff734c390507201c2d539dc6468de09a888623c7
|
|
Adds support for BIOS verification process to execute before
issuing a host reset and boot from AC power.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I31c019a5ea8cb296749f671df15c658a7e2a3b9f
|
|
Adds support for Power On for power button press and Power Off when
holding the power button. This is a temporary and rudimentary method
to handle button presses. The intention is to eventually use
phosphor-buttons with Nuvoton support.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: Iede64f89c716102a688880280c11ff7f3068021b
|
|
Instead of a host or chassis restart for a Chassis PowerCycle,
have both the BMC and host perform a hard reset. This requires
changes to phosphor-state-manager to add the handling of
xyz.openbmc_project.State.Chassis.Transition.PowerCycle. Currently,
there is handling for On and Off, but not PowerCycle.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I3d934c47a68899b2fd8fcba966f8d92c97554a77
|
|
Summary of changes:
1. Adjust start time of xyz.openbmc_project.State.Chassis.service
to fix initial chassis state
2. Add shutdown ack service
3. Remove reset ack service
4. Remove heartbeat mode
5. Remove sysreset pin control from power_control.exe
6. Remove all phosphor-fan services that caused power off during
a system reset
7. Adjustments to ampere_power_util.sh (See below)
ampere_power_util.sh:
1. Use GPIO instead of dbus for host state
2. Wait 3 seconds before checking power status for power on
3. Wait 6 seconds before checking power status for power off
4. Have force_off act GPIO while power_off sets Chassis state
5. Graceful shutdown acts GPIO
6. Check host status before issuing system reset
7. Set Watchdog Expire Action to None before power off and reset
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: Ie2720ee60d489c4160165c6827a78a400ae4d30b
|
|
Corrects the nvparm to disable LPI mode NV_SI_CPU_LPI_FREQ_DISABLE
for SCP 1.07 and newer.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I43f4d090fae3ff202834d39c9161802f20e396e1
|
|
The other system components are not using this the MON_BMC_ALIVE
signal, so it no longer requires assertion.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: Ieedc2650d230dc3eff8c2d76d2f1ce70b68919cb
|
|
The obmc-system-mgmt feature is currently used in the image to trigger
inclusion of a virtual-provider which provides a number of packages many
systems need. Partially revert the removal of this feature so that
the outcome is:
1. The empty obmc-phosphor-sysd package is still removed.
2. By default the 'obmc-system-mgmt' feature is included, unless
specifically exempted.
3. All EVB platforms remove the 'obmc-system-mgmt' feature since
they have no system they are managing.
This partially reverts commit 060ad3ff7fcc30aff78a9e504efee9d8fa0d4526.
Tested:
* Built `bletchley` and confirmed `packagegroup-fb-apps-system` and
`entity-manager` are present.
```
entity-manager armv7ahf-vfpv4d16 0.1+git0+6bf41588ab-r0
packagegroup-fb-apps-system all 1.0-r1
```
* Built `witherspoon` and confirmed `packagegroup-op-apps-system` and
`pdbg` are present.
```
packagegroup-op-apps-system noarch 1.0
pdbg arm1176jzs 3.3
```
* Ran `bitbake -p` on `evb-ast2600` to confirm the undefined
`virtual-obmc-system-mgmt` is not being included in the image.
Change-Id: I8b7804d5101cc84a2c57473b3f85672bf7767c67
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
|
|
Every machine layer treats 'system-management' as either part of a
package-group or removes the feature. The sample implementation in
meta-phosphor is a do-nothing shell script (and up until recently was a
Python script). There appears to be no useful purpose to this feature
as a stand-alone concept, so remove it.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I20ca1fa8ff3cb01cac2d07d4ded84e0769e4514b
|
|
1. Based on newer version of Ampere's nvparm spec updated nvparm
Signed-off-by: XP Chen <xiao-peng.chen@fii-na.com>
Change-Id: I47a338fcb483c5c22f3a9799088c54de1a9a5153
|
|
Signed-off-by: XP Chen <xiao-peng.chen@fii-na.com>
Change-Id: I97ee9317df93480f68cd69ee3462f970970ba885
|
|
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
|