Age | Commit message (Collapse) | Author | Files | Lines |
|
As of 9a8f30ec5b58 ("obmc-console: Introduce console-id, deprecate
socket-id") in obmc-console `console-id` is preferred over `socket-id`.
Convert all in-tree configurations to `console-id` to enable removal of
support for the deprecated `socket-id` configuration option.
Change-Id: Ifd50603c378cfed5eb793b829b8296b7964e4194
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
|
|
phosphor-debug-collector is now used in the meta-mori layer for
faultlogs, so it needs to be included as part of the IMAGE_FEATURES.
Change-Id: I45390feec21ce4c99c7fd19118a44fe6604f5474
Signed-off-by: Ian Johnson <ian.johnson@fii-na.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>
|
|
Set fallback watchdog timer to 3 hours to guarantee host boot.
Change-Id: Ieb10ff5fee309163720f9c93baea2f5491fb8af5
Signed-off-by: Kyle Nieman <kyle.nieman@fii-na.com>
|
|
Remove inclusion of hotswap-power-cycle recipe since it has been
removed here:
https://gerrit.openbmc.org/c/openbmc/openbmc/+/62893
Change-Id: I213ea0f506f2f8490a0cd8190156dd7488fe1965
Signed-off-by: Jonico Eustaquio <jonico.eustaquio@fii-na.com>
|
|
The backup copy will be stored in Mobo FRU EEPROM.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: Id9a0b409f7cbe53ab91f7d5655e39f0a318ba9d1
|
|
LED_BMC_ALIVE is now managed by phosphor-led-manager instead of by
init_once.sh. Asserting LED_BMC_ALIVE is removed from init_once.sh since
the service obmc-led-group-start@bmc_booted.service now asserts the dbus
property for LED_BMC_ALIVE's led.yaml group to be used.
Signed-off-by: Jonico Eustaquio <jonico.eustaquio@fii-na.com>
Change-Id: Ib68be05246a2dc66458330d99721848f86a1ed50
|
|
Removes the hotswap-power-cycle recipe and service
Change-Id: Iee2dcf508760f57ea5faedb40eee3d8cf48103d0
Signed-off-by: Jonico Eustaquio <jonico.eustaquio@fii-na.com>
|
|
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 log-alarm PACKAGECONFIG is added so that sel-logger can monitor and
create SEL logs when virtual and pldm sensors cross thresholds.
Change-Id: Id786a26139b395d4a3a4aeed7934906a63167210
Signed-off-by: Jonico Eustaquio <jonico.eustaquio@fii-na.com>
|
|
An OpenBMC subtree update looks to have removed how dbus properties
could be seen from just using busctl introspect.
To get the actual value of the BIOS Version property, the script now
uses busctl get-property instead.
Change-Id: I3c6079026e9ea3193461fb86739892b178327fed
Signed-off-by: Jonico Eustaquio <jonico.eustaquio@fii-na.com>
|
|
Add watchdog service to monitor the host status and execute the expire
actions when host doesn't reset the watchdog timer.
Add fallback support to the watchdog with the interval set to 10
minutes. Timer is reset with changes to host boot progress. If the host
fails to reset the watchdog, then the watchdog action is performed
with fallback to AC cycle after 10 minutes.
Signed-off-by: Vincent Chou <Vincent.Chou@fii-na.com>
Change-Id: Id0905ebc72405713806bbe2d803e6ca559488495
|
|
The dependency obmc-host-reboot target has on obmc-host-shutdown and
phosphor-reboot-host must be removed to allow for the mori-bios-verify
service. The mori-bios-verify service will have a 60s sleep that
requires the host to hold its state for the 60s duration after it is
commanded to powercycle. Without these dependencies removed, the host
would shutdown immediately.
Signed-off-by: Jonico Eustaquio <jonico.eustaquio@fii-na.com>
Change-Id: If2df43c8276626a3623bd4e796d87ea6817c32c7
|
|
Removes the entity association map that is no longer used.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I4cb1dffea598bd27bc1cbbf94ef94332a04af2f6
|
|
The nvparm tool expects the BIOS partition to be named "hnor".
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I472bd32b7c70a5a8189cf16fcbf430b3d1094361
|
|
The host verification fails because the BIOS SPI is quad mode during the
flash. We must reset the BIOS SPI for the host to be able to read the
SPI for its verification stage.
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I0bf38a10ca939f7fcab338fe3f654303a4dea9f9
|
|
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
|
|
To fix the IPMI runtime error we need to remove this in kudo.
Signed-off-by: Avenash Asai Thambi <avenash.thambi@fii-usa.com>
Change-Id: I95ec7397812710992c5e4de3442e1d2ea7adbbeb
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I8ea79d284356629f5b7b24ad3a99336e9e2e297e
|
|
Add command to allow user to control GPIOs using mori.sh instead of
having to source mori-lib.sh and use functions.
Change-Id: I3cce175a41ca13bf3e14cea49495cde561348980
Signed-off-by: Kyle Nieman <kyle.nieman@fii-na.com>
|
|
The previous static-image building code only confirmed that the
u-boot image was smaller than the offset for the kernel image, but
most systems place the u-boot environment after the u-boot image.
It is possible that we have a u-boot image bigger than the u-boot
partition, which is then overwritten on boot by the u-boot env.
Check for this at image build time and prevent it from happening.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I3a1e6654272cedb3e840bbea4771ae4e464c4ecd
|
|
The functions in mori-lib.sh return the wrong values for get_gpio_num
and get_gpio_ctrl. get_gpio_num is not prepared to handle a device that
has more than 1 gpiobank associated with it. get_gpio_ctrl is dependent
on get_gpio_num to retrieve the correct identifying number for the gpio,
and the path location of the gpio state differs for gpios added by some
devices compared to the norm. mori-lib.sh is modify to account for these
differences.
Tested: I ran on real machine and checked that a gpio added by each type
of device is capable of being read with get_gpio_ctrl without error.
Signed-off-by: Kyle Nieman <kyle.nieman@fii-na.com>
Change-Id: I212ae4f14bbb57b9643befd0aadd8ac5ef4726da
|
|
Add command to allow user to enable or disable access for the host
to the RTC.
Change-Id: I0887cc18ed804191ffd80e5c6034d8ed1a0c3b66
Signed-off-by: Kyle Nieman <kyle.nieman@fii-na.com>
|
|
Willy Tu (1):
build: Migrate to meson and add clang-format
Change-Id: I846dae5a4a17cebb869e23b1ff507a31f2fe4877
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
The logging service is dbus activated now, so explicitly declaring
dependencies is unnecessary and redundant.
Change-Id: I87d61fbeb04a251caf88aa86cb21631e7fc2a1c4
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
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>
|
|
Willy Tu (1):
Remove Foxconn owners to test Gerrit Owner plugins
Change-Id: I5681d9a35ab8566ad1f1e01b4c0b91022541d031
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
|
|
Currently the gerrit review is not working for this repo due to Gerrit
Owner plugins disabling reviews after hitting owner issues. Remove one
of the owner to see if that is the cause of the issue.
Change-Id: If0191f2e550e4f0d0620911414ca17d138d6d872
Signed-off-by: Willy Tu <wltu@google.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
|
|
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I8e54833ac78e540e9dd5011533d53ff9a3af6763
|
|
There's a bug between the OWNERS file and how openbmc gerrit rules parse
that file. The bug seems to be around gerrits handling of a "." in the
first part of a persons email address.
To try and work around this issue, utilize gerrit names instead of email
addresses in the OWNERS file.
This commit will need to be directly pushed into master as it can not be
scored in gerrit due to the bug.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Iff8fe23e080d148b2837534f12398b361eb0b6f1
|
|
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
|
|
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.)mori-fw.sh
- Modify call to ADM1266 to pass I2C bus
2.)mori.sh
- Modify call to ADM1266 to pass I2C bus
Signed-off-by: Kyle Nieman <kyle.nieman@fii-na.com>
Change-Id: Id9d5e7d829557a30d929dad119326ac16e901c9b
|
|
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
|
|
Summary of changes:
1. Create a yaml file to set up the action of SYSTEM BOOT STATUS LED
2. Add bb file to install yaml to phosphor led manager package
3. Add a script to control the SYSTEM BOOT STATUS LED corresponding
to the OS status
4. Set up phosphor led sysfs so the physical led can be controlled
5. Set CHASSIS TARGET to none to remove unnecessary service call to
obmc-led-group-start/stop@power_on/off.
Signed-off-by: Vincent Chou <vincent.chou@fii-na.com>
Change-Id: Ia7b136c52518a1e2af6662ffc42b4f72b3e15584
|
|
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
|
|
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I65febeeec11e6e5b40ee728f42cfbe77023dae1e
|
|
Add phosphor console mori_uart_mux_ctrl script,obmc-console service
files, and .conf files
- Added console feature for host & mpro
- Added mori_uart_mux_ctrl.sh to set gpio S0_UART0_BMC_SEL and
S0_UART1_BMC_SEL allowing routing between host & mpro consoles
to bmc
- Added host_console_uart_configuration.service so that it runs
before obmc-console. Set it to require multi-user.target, which
is also required by obmc-console.
Signed-off-by: Grant Williams <grant.williams@fii-na.com>
Change-Id: I5d7002c20c5d1b87fa5c78e69b0885fc635c6145
|
|
Summary of Changes:
- Add basic fw_env.config for Nuvoton system.
- Add u-boot configuration file
Signed-off-by: Ashwin Murali <ashwin.murali@fii-na.com>
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I3e0f44b965ee402c105e4020c41235ebebda375b
|
|
Summary of Changes:
- Add custom Chassis and Host services
- Enable Host Condition GPIO using host0-ready
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I74330028d4ce20d8abf4d6a5169c1bc6c491c7d4
|
|
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
|
|
Defined I2C bus and address for ADM1272(HOTSWAP_CTRL) and
PM6764(STBUCK). Added to prevent hardcoding and promote modularity.
Signed-off-by: Kyle Nieman <kyle.nieman@fii-na.com>
Change-Id: I6bd3e993d944e87b9242d6237850d2a4b0e30ada
|
|
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
|
|
For the upgrade to Kernel 5.15.29,
- Consolidate the configurations between those required to be
added and those already included
- Organize the configurations into categories
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: Ia0df58344772af576e9ddb5346d8d505ce11fd1b
|
|
Adds the packages to support the button handler and signals.
Add the GPIO definitions for obmc-libobmc-intf of the skeleton
repository.
The GPIOs configured are,
- PGOOD: Host power good
- POWER_BUTTON: Power button input
- RESET_BUTTON: Reset button input
Signed-off-by: Charles Boyer <Charles.Boyer@fii-usa.com>
Change-Id: I1a8ca713c00b7220793c58b1695ae680882a3404
|
|
Add pwm_init service to set fans to pwm mode and initialise them to a value of 50% on boot.
Signed-off-by: Ashwin Murali <ashwin.murali@fii-na.com>
Change-Id: I07963eda9b0c264e66d1dd4f2a0d31f4e74d831e
|
|
Adding Mori Entity Association Map .bb and .json files.
Signed-off-by: Grant Williams <grant.williams@fii-na.com>
Change-Id: I3e090f9cc4258d0215a1af836ea8d85e7b852483
|