Age | Commit message (Collapse) | Author | Files | Lines |
|
Enable other distro config files to require/include a proper include
rather than directly including openbmc-phosphor.conf
Tested: Built a Witherspoon image
Change-Id: I2e94c9a7f9ecc1527ece86099407cdc4f34a5e36
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
These _appends can potentially crowd their neighboring entries.
Change-Id: I1dae77f9a17744110175916935c01773708652b0
Tested: Built a witherspoon image
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Add a missing pyyaml dependency to the
phosphor-ipmi-sensor-inventory-mrw-native recipe.
Tested: Built a Witherspoon image
Change-Id: Ia85eb4da107f855d8f2dd9483b4dd34b71674f24
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Change-Id: I4be962064668c937fce34b7814378c86a2dc994c
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I53d382a8c253338cf41ba4d6440b4a83e495e803
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I0eed1092768c02f68efe678a38ae012da14feae1
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
With this entry, code will be able to map a gpu_func sensor
that comes down in an eSEL to an inventory item so a callout
can be done.
Resolves openbmc/openbmc#2955
Tested: Checked that the GPU inventory items now show up
in inventory-sensor-gen.cpp
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: Ibb3a100c9bb129fe0a06fc82af40413101be80c5
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
|
|
1. Enable fan monitor;
2. Add FanPwm interface;
3. Set the factor, offset for calculation from pwm to rpm
Tested: Verifid fan monitor works fine;
Manually change pwm, fan monitor will set fan non-functional
after a while; change pwm back, it will set fan functional
again, as expected.
Change-Id: Ic56cbe291959368218b60c4a5b8e305f39ff7edb
Signed-off-by: Lei YU <mine260309@gmail.com>
|
|
This is a minor reorganization by grouping of the
overrides specified in the phosphor-defaults.inc file.
Change-Id: I1634552a286e0f0b04b4c0d3c7c07e4a0aa03c06
Signed-off-by: Patrick Venture <venture@google.com>
|
|
Ignore the return code of fw_printenv in case rwfs_size doesn't
exist, otherwise the function fails and doesn't get set.
Also check that the imgsize variable is set before calling
fw_setenv to prevent accidentally clearing the rwfs_size.
Tested: Verified that rwfs_size gets set when it didn't exist.
Also that it gets updated when the value is different than
the one passed. And that it doesn't get cleared if the function
is called without passing the imgsize argument.
Change-Id: I02e3414e3b22da55890b7343b8805402139b6bad
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
Change-Id: I0064c7475ed7f6b1d6509ab76d0f01683f9c7889
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I1b299ee8167a7c9a16b06fb58c3d13ed9c0d97a0
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I693eeb27becfd6290a3bd420e3b403d27a570e4e
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I17a657c313010c0cb25a333381c9a291442200d4
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: Idb4499cb078065998cf9fb3a47abe64bbed0a2e2
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: Id0a813aee8d4a7dee62bb48cf5e7934fd01a3614
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I76d3a7a54142dd36292e7d66b383ffda8fd31ffc
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I3b07d2ae145412a57a74ce1dfc153e81b5d331d3
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Upstream now points at the Yocto mirror. Drop.
Change-Id: I4cbc6ab8f51acaf2e9c11478ab5275c60f734fd6
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
We had an i2c-tools bbappend that removed the i2c-tools-misc package
as a runtime dependency from the i2c-tools package, but i2c-tools-misc
isn't a dependency of i2c-tools in the first place. Drop.
Change-Id: I1505a45c7bf84db484eb667f3f842d3835857a8e
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
This recipe is not used. Drop.
Change-Id: I3fa053e55241fbb8b1ded9bee88cc47c2780a8b4
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
This has been addressed upstream and no longer required.
This reverts commit 74cb437b8c4c4a5c7ebdc68d09de377bece59e5e.
Change-Id: I349c4b413da82610b709b8a6ee440652f3698368
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
The MAX31785revA specification does not match how it functions.
The MAX31785revA's functionality reports the rotor feedback speeds in a
slow/fast relationship instead of rotor positions, therefore the fan*_0
object will always represent the slowest rotor. In combination with how
the MAX31785revA latches its TACHSEL = 0 during a speed transition, it
made it difficult to fault isolate the rear rotor of a fan.
Using a combination of the fan monitor trust groups on all fan rotors
and enabling a delay timer to determine when a fan rotor becomes
functional, fan monitor is able to correctly reflect each fan rotor and
fan enclosure's functional state in inventory.
Fan faults are unable to be monitored on all fans when transitioning to
a target speed.
Tested:
Each fan rotor's functional state is updated when blocked
Correct amount of time to mark a fan nonfunctional is still honored
A fan is functional after remaining in spec for 5 seconds
Resolves openbmc/openbmc#2798
Change-Id: I834d63f6082eae4cb03ec501f0fb8272a9f5b673
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
|
|
Change-Id: I0cea4645f828be91eb7fcf0820d84f40c2b69e95
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
This changeset replaces phosphor rest with bmcweb for the s2600wf
target. bmcweb has several advantages over phosphor-rest.
1. Written in C++.
2. Supports Asyncronous transactions
3. Authentication backed by PAM
4. SSL key generation, and modern SSL cipher suites
See the application readme for more
To enable bmcweb, this patchset also:
1. Revs libtinyxml to the latest version 6.0.0 from the existing 3.0.0
2. Adds the boost-dbus recipe, to allow bmcweb to build. Bmcweb is a
header only dbus library.
Change-Id: I5b90427977709127c09fedcfc4bcb713908d356f
Signed-off-by: Ed Tanous <ed.tanous@intel.com>
|
|
Change-Id: I198e97fec23f00aa8b0cf3c5857b50ebca01e424
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I5ef42421c04ace6e87bf9402cc73bf89ffe490e1
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: Ic4ccebc7c809f051f702821b9c69c3c2c20c7117
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I2daa0cbf688fcb9f842c739363350b3f7cffaede
Signed-off-by: Ken Chen <chen.kenyy@inventec.com>
|
|
If an environment variable doesn't exist, fw_printenv returns 1
(error). With the addition of "set -eo pipefail" to the
obmc-flash-bmc script, this error causes the service that sets
the priorities to fail:
witherspoon systemd[1]: Starting Set U-Boot environment variable...
witherspoon obmc-flash-bmc[3883]: ## Error: "66089bab" not defined
witherspoon systemd[1]: obmc-flash-bmc-setenv@66089bab\x3d0.service: Main process exited, code=exited, status=1/FAILURE
witherspoon systemd[1]: Failed to start Set U-Boot environment variable.
Ignore the return code instead, the variable will be empty
if fw_printenv doesn't return a value.
Change-Id: I78b559e0ccb74ae52c73f1cc37b06a139a0d0e14
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
This tells phosphor-hwmon to create FanPwm interface for the fan
sensors.
Tested: Verified it is able to set Target to change the fan pwm.
Change-Id: I79c4cb8fe00812659a2c536d9d2e328b5438e0c6
Signed-off-by: Lei YU <mine260309@gmail.com>
|
|
This revision adds the FanPwm interface support which is used by Romulus
as well.
Change-Id: I44c7ba66028eb54630268019acc78a9e3720e621
Signed-off-by: Lei YU <mine260309@gmail.com>
|
|
Change-Id: I233142268f40237b0bb96bbd0e508ea3f0f91922
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I1e9e01dd769c29a13b078d84a10792401b2f32c9
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I4ecedf890b2d19d1345fa80f9278ed198429a588
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: Ia242be45464d724f600b3d0d2127a9b44858fc48
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Resolves openbmc/openbmc#2500
Change-Id: I7caaf07f1d9f6d54d2e92a382d675ef450db2c8e
Signed-off-by: Jayanth Othayoth <ojayanth@in.ibm.com>
|
|
In older versions of coreutils install with -D and -t together has
different behavior such that the destination directory is not created.
Later versions of coreutils this does work but manually install the
target directory so builds on RHEL7 can work.
Change-Id: I42c46198321e4bf8085898b6d7485a3425634aca
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
The default size of the UBI read-write volume is currently set to 4MB.
There are no plans to store more than one BMC rootfs image on the chip
so in a 32MB flash chip, accounting for the current rootfs image size
of ~15MB with plans to grow to ~20MB with the addition of redfish and
other packages, it is safe to increase the read-write volume size to
6MB since the current size is already almost completely taken up if
there are multiple error logs / dumps on the system.
In addition, make the size configurable from a recipe so that the
size can be changed in a per-system basis. And during code update,
check the current size and update it if it's different, this allows
systems to be able to be resized to a new size by performing factory
reset after a code update that sets the new size.
Tested:
- Booted on QEMU and verified non-ubi system (romulus) retains the
current 4MB read-write volume, and ubi system (witherspoon) has a
6MB volume.
- Code updated to an image that has these changes and verified the
rwfs_size env variable changes to 6MB, and that a subsequent factory
reset rebuilds the volume with size 6MB.
Change-Id: I995eb560c1bd87ee95712c731e3d6e55bc0b2735
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
The phosphor-software-manager-updater depends on bash. Add
the runtime dependency.
Change-Id: I619cf8d965be520785752ad6073419a5eb645618
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Provide a Turbo mode setting and the ability to enable/disable
the processor Turbo mode. Default value is enabled.
Change-Id: I17537f845dc17fb850e349a63cd2f93ad25cf225
Signed-off-by: Jayanth Othayoth <ojayanth@in.ibm.com>
|
|
The handcoded harware sensor yaml file is merged with the final
sensor yaml file.
phosphor-ipmi-sensor-inventory native recipe finds all the
yaml files with extension ".hardcoded.yaml" and merges them
to the final sensor yaml file
Resolves openbmc/openbmc#2613
Change-Id: I3e25f1312a813fcba0070a3e778915f52edba7aa
Signed-off-by: Marri Devender Rao <devenrao@in.ibm.com>
|
|
With openbmc commit c03b9c4a5d5f8e089297d8d4c13540d6b2c9421f
a new U-Boot environment variable was added to perform a
factory reset, and logic was added to obmc-flash-bmc to
always check if this new variable existed so that systems
running older versions of BMC firmware would have this new
variable added since the environment variables are not updated
during a firmware update.
A few openbmc tags have been created since then, and only
witherspoon currently uses this code update script, so it's a
good time to remove this workaround.
Change-Id: I1cb7fdf0e657d24a7641cc4d18776ea20d206c7c
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
Change-Id: Ib78f9f24dc4a33cda8c4268178425df42c394364
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: Ied6bfe428fe5bc6a19fdb3adf606cbc18b245b3e
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: Idcbe38497ff10223faa931853c697c3ae405af0b
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Change-Id: I4e34219fc78cadc11c6122b960b73b976f3c43b9
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
In migrating to support better fan failure/missing shutdown scenarios,
currently defined shutdown scenarios should monitor the functional
states of the fan enclosures and do a soft poweroff.
A system hard poweroff will occur for the following condition:
- More than two fan rotors are nonfunctional
Change-Id: Ie5c769f6b4f272f4ff39c7f3ca036db275723997
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
|
|
An increase to the fan watchdog timeout was done to mediate a BMC
performmance issue, which has now been resolved, so reducing the fan
watchdog timeout. The timeout should be set to 5 seconds (instead of
originally 3 seconds) to functionally align with the 5 second timer that
fan control uses in adjusting fan speeds.
The fan watchdog timeout increase was done under:
https://gerrit.openbmc-project.xyz/#/c/8398/
The performance issue with the OCC patches:
http://patchwork.ozlabs.org/project/openbmc/list/
(Series = Locking fixes for FSI, SBEFIFO, OCC)
Tested:
The fan-watchdog.conf file in BMC image contains 5 second timeout
Change-Id: I1f99cf33ce3f17b291bf0997f792e2409ec5fb2b
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
|
|
All the fan services associated with controlling, monitoring and
presence detection of fans need to remain active until a poweroff
completes. This helps ensure thermal safety while a system is in
transition to a powered off state.
With the introduction of this new target (which runs last in
the power off path), need to move the host reset service to
running after it to ensure the power on in the reboot path
does not start until the last power off target runs.
Resolves openbmc/openbmc#2762
Change-Id: I4ca671d2c7de66e8bfa4bba607b06c88c0a6e7f9
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
|