Age | Commit message (Collapse) | Author | Files | Lines |
|
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>
|
|
Added IPMI watchdog service to AMD platforms.
Change-Id: I7da12dd45eaaa5b9c92c9876cdca7cabc1f79d33
Signed-off-by: Rajaganesh Rathinasabapathi <Rajaganesh.Rathinasabapathi@amd.com>
|
|
Updated journald custom policy updates for
- size policy
- storage policy
tested: verified build and tested on AMD CRB.
Change-Id: Ib650ddeb968d9244ef818a7aa42e5ea048e0f8c9
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>
|
|
Added additional configuration files to support obmc-console
for all AMD CRB's.
Tested: verified build
Change-Id: Id31ec4e9953df32d5dc8468aafedbfa014f087f4
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>
|
|
Enabled phosphor state management based BMC state management.
This will also help to enable obmcutil tool support in AMD CRB.
Tested: verified on AMD CRB.
Change-Id: I14e6d0c5a96080a9ff79420e23871b8256338b73
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>
|
|
Enabled additional logging to support bmcweb issues debug.
- redfish-dbus-log: Enable DBUS log service transactions through Redfish
Change-Id: I089db5b4970febf1196809c9e0c05a278fdce547
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>
|
|
By default, bmcweb limits the size of the http request body to 30MB
AMD CRBs supports flash size 128MB. This commits helps to upload
higher size images.
Tested: verified image upload greater than 30MB.
Change-Id: I6dd5991023bb4e015bb564e3717f0168a25bfc03
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>
|
|
This is to keep all AMD layer common recipes at one place.
Change-Id: I5e407e9673437e203dc46d1bb4dea51387a91afd
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>
|
|
Code refactoring to support CRB's bring-up.
Configuration files "nohost" and "withhost" variants
introduced to support various stages of bring-up in
AMD CRBs.
Change-Id: Id52ac68edf365e6623bd71992fdd6d9fc8a3adf5
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>
|
|
Adds Jayanth as co-owner of meta-amd.
Change-Id: I7187e70d41265486adf858c71aa8aba2acaf1091
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I28ee50fa55ae47dd7fd8c99f8f6db8f5f6dfa53d
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I85272779478b66452acd11be93f5fea99e4c3a34
|
|
bmcweb is already added as a default in the meta-phosphor package
groups. A large number of machines have mistakenly also explicitly
added a RDEPEND on assorted package groups, which is not necessary.
Clean these up.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ifd3726d18aab45475f80d054a4640196ac0b71d2
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
New docker container is flagging some issues in this shell script:
```
SC2322 (style): In arithmetic contexts, ((x)) is the same as (x). Prefer only one layer of parentheses.
SC2046 (warning): Quote this to prevent word splitting.
SC2219 (style): Instead of 'let expr', prefer (( expr )).
```
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I46dac4da799a9e3256bb5f1829784253be67716f
|
|
To enable Serial-Over-LAN functionality via the standard "ipmitool sol"
command it is necessary to create a DBUS interface
"xyz.openbmc_project.Ipmi.SOL" that would be used for the communication
between the "phosphor-net-ipmid" and "phosphor-host-ipmid"
applications.
Change-Id: I8a63c233f321fe540d73ef33305beb0aa555796a
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
|
|
Current SoL settings cause 'ipmitool sol' to crash after some
working time. Increase RetryIntervalMS to fix the issue.
Change-Id: I650b7d642f7f3e13df1f2d433b612cee6e1f5616
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
|
|
Currently with the 'oneshot' unit type the 'ip-to-fpga.service' is
always in the 'running' state:
```
root@ethanolx:~# systemctl list-jobs
JOB UNIT TYPE STATE
88 obmc-led-group-start@bmc_booted.service start waiting
1 multi-user.target start waiting
126 ip-to-fpga.service start running
3 jobs listed.
```
This creates a problem since 'srvcfg-manager' service expects every
startup job to finish.
Change unit type to 'simple' to fix the issue.
Change-Id: I45840ef432ce33be400b547420ab5ceeca949ba0
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
|
|
'phosphor-net-ipmid' application now requires
'xyz.openbmc_project.Control.Service.Attributes' interface for the SoL
functionality to work correctly.
This interface is provided by the 'srvcfg-manager' application,
therefore include it to the build.
Change-Id: I0c0473df2d12cf7124dc07fa4c41e84d6e82bb5f
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
|
|
'SRC_URI:<machine> +=' syntax creates new variable 'SRC_URI:<machine>'.
'SRC_URI:append:<machine> =' syntax appends the variable 'SRC_URI' when
appropriate machine is selected.
hostlogger and obmc-console recipes incorrectly use the first syntax in
places where they should use the second one. This leads to build error,
when bitbake can't find added files.
Correct recipes to fix ethanolx/daytonax boards build.
Change-Id: Ibf424b95ae5aadf4c2ffd5915d746fb19f2a6d66
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I8ea79d284356629f5b7b24ad3a99336e9e2e297e
|
|
If switch SW2-7 is on, FPGA shows BMC IP address on the 7-segment
displays. For that functionality to work correctly BMC needs to write
its own IP address to the paticular FPGA I2C registers.
Add a service that performs this action on every IP change.
Change-Id: Iead978ef3651448f4c32936402607fbb3267ee18
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
|
|
Drop skeleton_workbook from the machine configuration. This is an
obsolete feature that shouldn't be used for modern systems.
Change-Id: I183044341548708265cd74f3d19c1b2266be3fc6
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
|
|
Move common machine configurations to the include file.
Change-Id: Iafd07c4d6dc1abdcffede7568e6aef16e2f53ac4
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
|
|
Add AMD DaytonaX CRB to the meta-amd layer.
Change-Id: I7df37f78640e9cc9c595da257bfc982d8f636acd
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
|
|
All configurations that are specific for the EthanolX board are needed
to have ":ethanolx" override.
Change-Id: I7b3381a1f486168bcd12a1ac0fb4756a31fb0487
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
|
|
"x86-power-control" doesn't allow to set BMC_READY signal, but besides
that everything is compatible with the upstream code.
Change-Id: Ic60bfa7b7cbb85494c542fea9b3fd480beec0a12
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
|
|
The "redfish/v1/Systems/system" page is necessary to fill the
"Server information" panel in the webui-vue application.
Change-Id: Id8ee7b6ab504cd38916f92fd6dda9d99dfdd898c
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
|
|
To enable Serial-Over-LAN functionality via the standard "ipmitool sol"
command it is necessary to create a DBUS interface
"xyz.openbmc_project.Ipmi.SOL" that would be used for the communication
between the "phosphor-net-ipmid" and "phosphor-host-ipmid"
applications.
Change-Id: I41eaac0eb53b66d135032c14e69abbe5e97509ce
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
|
|
Add support for dynamic fan RPM control based on the values from
temperature sensors.
Control is based on PIDs and is splitted to 3 thermal zones:
Zone 1:
- CPU0 SB-TSI temp with a setpoint 60.0 C
- P0 FAN
Zone 2:
- CPU1 SB-TSI temp with a setpoint 60.0 C
- P1 FAN
Zone 3:
- All LM75A sensors on board with a setpoint 35.0 C
- SYS FAN1 - SYS FAN6
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
Change-Id: Ifca359ba23777e2b1f46783272cf98ff0b08fe68
|
|
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I8e54833ac78e540e9dd5011533d53ff9a3af6763
|
|
It's not clear why these patches were added in this manner, but they
can't really remain here from a maintenance perspective. Pinning one
platform on an old version of x86-power-control, as well as setting a
specific patch means that it's impossible for people to test and use
upstream code with it. It is also explicitly called out as item #1 on
the meta layer guidelines for this reason:
https://github.com/openbmc/docs/blob/master/meta-layer-guidelines.md
To the maintainters of this meta layer, please work with Jason and Kwin
(the x86-power-control maintainers) to get your features accounted for
in the normal build, adding build flags and configuration entries where
appropriate.
Tested:
This almost certainly breaks power control on ethanolx, but given the
way this patch was merged, there's really no way to avoid that.
Signed-off-by: Ed Tanous <ed@tanous.net>
Change-Id: I1f6a40e5b01c36441ff8d66e8e73265e0e009a4f
|
|
Switch the u-boot branch from the 2016.07-aspeed-openbmc
to the 2019.04-aspeed-openbmc.
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
Change-Id: I7f17b31abe32c374b01cb037ba03ed2608c1190d
|
|
The code for obmc-op-control-host is only useful for OpenPower
machines, but somehow it is being included in many other machines.
Remove it from all of them except meta-openpower.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I4e7c7365eb320c3fb9ea7a57aca2ed0b1832e85e
|
|
In the AMD EthanolX board CPU0 UART1 is connected to the BMC UART0.
Therefore from the point of BMC UART0 is an upstream (host) UART.
'obmc-console' application gets this information from its configuration
file name ('server.ttyS0.conf').
On the other case setting 'local-tty' inside the configuration file is
intended for the downstream UART settings, which is not present on the
AMD EthanolX board. Therefore remove this setting.
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
Change-Id: I3daa1528cb5a7b5ff748189f29da3c3d91899ca4
|
|
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
|
|
Harvey.Wu (2):
dbus-sdr: fix SEL record Generator ID to fit SPEC
dbus-sdr: fix the vector size in sdrWriteTable and sdrStatsTable
Jian Zhang (1):
Add data types to Value to extend setDbusProperty
Vernon Mauery (1):
fix header install
Willy Tu (4):
cleanup: Remove all warning errors for the Meson build support
build: Add meson build for phosphor-host-ipmi
build: move all shared_library/module to library with version suffix
build: Remove dynamicsensor library if it is not enabled
Change-Id: I92503c6625d1ec2bf88c0a28afac9b4eeb8fbd75
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
Signed-off-by: Willy Tu <wltu@google.com>
|
|
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
|
|
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
|
|
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
|
|
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
|
|
The MAINTAINERS files are no longer used and are out of date (references
to IRC still exist in most of them). Remove them and rely on the OWNERS
exclusively.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I0f5d2719ad97e218ce03c4503efe1b1c92ac455e
|
|
This sets u-boot to the package containing the v2016.07-aspeed-openbmc
branch. This ensures there will be no change if the default changes in
the future.
Change-Id: Iea12e1691dbdda34a3f95d6d0862add0b15bcf37
Signed-off-by: Joel Stanley <joel@jms.id.au>
|