summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* system_manager: wait for objects with mapper waitBrad Bishop2016-07-261-48/+7
| | | | | | | | This is much faster than the current introspection based mechanism. Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com> Change-Id: I1a737cab00311e9e0c0b63f4d8a487085cdf44e0
* system_manager: Remove bus from lookup responseBrad Bishop2016-07-261-14/+3
| | | | | | | | Remove the busname field from the 'GetObject' method responses. Applications should get the busname information from the mapper. Change-Id: Id548f3d5d6afda8334ad1163d66f69832591511a Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* pcie: Use mapper calls to lookup busnamesBrad Bishop2016-07-261-8/+68
| | | | | | | | | This is a reaction to a /org/openbmc/managers/System API update. The update removes the bus name field from the 'GetObjectByxxx' method responses. Change-Id: I061f8071097b4e870031198a509abe48a4128d82 Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* system_manager: Remove dead codeBrad Bishop2016-07-241-20/+0
| | | | | | | | Remove a reference to 'BMC_INIT' state which isn't used. Remove heartbeat code that isn't used. Change-Id: I2de0c881372085519a15235703881054c5a9adfa Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* system_manager: Run pep8 on system_manager.pyBrad Bishop2016-07-221-248/+246
| | | | | Change-Id: I976228399027420f255f09622b02352e1799fe91 Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* Remove workbook dependency from download mgrBrad Bishop2016-07-106-8/+3
| | | | | | | | The download path for temporary files isn't a property of the board. Hardcode /tmp into the application. Change-Id: Ib375ddee9a4c50aa8817980562fbd5b1191b5542 Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* Remove unnecessary InterfacesAdded signalsBrad Bishop2016-07-106-7/+0
| | | | | | | | | | Remove ObjectManager signals emitted before a bus name is owned. This isn't necessary now that the system_manager can detect objects via NameOwnerChanged. Change-Id: Ia2b626a926392b50d20c26474e6c9a9fdb3ca2f0 Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* Export objects prior to bus name grabBrad Bishop2016-07-107-0/+12
| | | | | | | | | The default behavior in pyphosphor changed such that ObjectManager and Properties signals are not emitted by default. Enable signals just prior to bus name grab. Change-Id: I27a2de4b60482dd843702396f33c7cdccaf17277 Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* Defer busname ownership until init completeBrad Bishop2016-07-107-7/+7
| | | | | | | | | Avoid race conditions where applications expecting an object to be present upon recipt of NameOwnerChanged find that the object hasn't been created yet. Change-Id: I4236a3898055a84b00e1eaf37f32e0192b683a0f Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* system_mananger: Add a name owner handlerBrad Bishop2016-07-101-0/+34
| | | | | | | | | Add a bus name handler so the system manager can detect objects implemented by services that create objects before owning a bus name. Change-Id: Ic311133a8e532e7f2441102689632f5821dbf589 Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* system_manager: Add try_next_state functionBrad Bishop2016-07-101-16/+19
| | | | | | | | Add a re-usable try_next_state function. A handful of idiomatic python improvements. Change-Id: I435aaaeb5760a677f8fd292f396cdbdae1195674 Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* pyflashbmc: Remove unused variableBrad Bishop2016-07-101-1/+0
| | | | | Change-Id: I0c795151f723fdfa41aa24d9213ffb8490b03349 Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* hostcheckstop: export signals prior to bus grabBrad Bishop2016-07-101-2/+1
| | | | | | | To be consistant with the other gdbus app implementations. Change-Id: Ib427a61c4ef9ae0cf593aec1deafd9d3e7aac586 Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* Add checkstop gpio to Firestone, Garrison, WitherspoonAdriana Kobylak2016-07-053-0/+24
| | | | | | | | | Firestone, Garrison, and Witherspoon were missing the checkstop gpio handling. Add the checkstop handler process and the gpio pin number per the system schematics. Change-Id: Id53e2f8888a7c706bf30a95fe56223628a2ba70a Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
* Merge pull request #111 from mdmillerii/flash-bmcPatrick Williams2016-07-011-12/+177
|\ | | | | Flash bmc updates
| * flash_bmc: Add Apply method to flash updates from runtimeMilton Miller2016-06-301-9/+154
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit adds methods to Validate (not exposed), Apply, GetUpdateProgress, and Abort along with an auto_apply attribute. A PrepareForUpdate method is also added that will write bootloader environment to instruct the initramfs init script to copy necessary file system state into RAM to allow the flash to be updated. Together these methods take the prior Update method and chain on the Validate phase which checks the image can be applied concurrently (and could be extended to check signatures). If validation fails auto_apply is cleared. If auto_apply remains true then Apply is called which will perform the check again (implicitly in the update script) and update the flash contents, saving its output to a temporary file. The GetUpdateProgress will return the status variable followed by the output from the update script, with status that was overwritten with a carriage return stripped. As each new phase (erase, write, and verify) of each image is encountered the message will be extended. An Abort method will kill an in-progress update, erase the logfile, and remove any pending images. A new image can be unpacked at this time. Note: the current method to clear all persistent files removes the whitelist and all future calls will clear all files from the read write layer. This maybe addressed in a future update. Signed-off-by: Milton Miller <miltonm@us.ibm.com>
| * bmc_update: allow whitelist to be files in a directoryMilton Miller2016-06-301-1/+21
| | | | | | | | | | | | | | | | | | | | The whitelist is moveing from a file to a set of files in a directory. Use try when deleting whitelist entries. It would be better to call update without saving the files in the first place but this will suffice for now. Signed-off-by: Milton Miller <miltonm@us.ibm.com>
| * bmc_update: save runtime modified u-boot environmentMilton Miller2016-06-301-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Copy the version saved by our initfs and pointed to by our fw_setenv tools via /etc/fw_setenv. Do this copy by default. The /etc/fw_env.cnf config file points to /run/fw_env in our systems. The bmc_update object was copying /dev/mtd2. That could be very system specific, and it misses any changes made from elsewhere in the system. This variable causes the u-boot environment to be written back after writing the whole image. This will save the ethernet mac which for systems which don't store it elsewhere. Signed-off-by: Milton Miller <miltonm@us.ibm.com>
* | Merge pull request #120 from geissonator/debug-mode-watchdogPatrick Williams2016-07-011-192/+216
|\ \ | | | | | | Disable watchdog timer if debug_mode is set
| * | Disable watchdog timer if debug_mode is setAndrew Geissler2016-07-011-5/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | resolves openbmc/openbmc#246 debug_mode is used when the host is not going to be started. In this case, we do not want the watchdog timer to be started because there is no one to keep it from resetting. This code change prevents the watchdog from being started during poweron. Potential side affects could be if we do start the host in this code path. The watchdog is un-tested when it's not started but software comes in and tries to reset it. The code in control_host_obj.c currently enforces the no-host-power-on when debug_mode is set.
| * | Reformat pychassisctl/chassis_control.py with pep8 standardsAndrew Geissler2016-06-301-192/+200
| | | | | | | | | | | | This commit is formatting only, no functional changes.
* | | Merge pull request #119 from anoo1/vpdPatrick Williams2016-07-011-2/+0
|\ \ \ | | | | | | | | Remove Chassis and Product area from IO Board
| * | | Remove Product and Chassis area from IOBoardAdriana Kobylak2016-07-011-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove Product and Chassis area for IO board vpd since only board area is present in the latest version of the IO eeprom. Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
* | | | Merge pull request #125 from bradbishop/revert-fixupPatrick Williams2016-07-011-0/+1
|\ \ \ \ | |_|_|/ |/| | | hwmon: attribute is not polled after failure
| * | | hwmon: attribute is not polled after failureBrad Bishop2016-07-011-0/+1
|/ / / | | | | | | | | | | | | | | | | | | This line of code was removed with revert bd01c24d (original efc68970) but should not have been. The line was originally added with 6956bbd4. Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* | | Merge pull request #122 from vishwabmc/masterPatrick Williams2016-06-301-0/+6
|\ \ \ | |_|/ |/| | Add the missing binary into Palmetto config file
| * | Add the missing binary into Palmetto config fileVishwa2016-06-301-0/+6
|/ / | | | | | | | | | | | | | | | | | | | | Palmetto.py did not have discover_system_state.py and this patch is to update that. discover_system_state.py is responsible for looking at power restoration policy set by the user and then taking appropriate actions when BMC comes back from any reboot / power cycle and reaches BMC_READY state. Actions could be : 1) Leave the system in power off state. 2) Power on the box. 3) Restore the last state which can be either #1 or #2 above.
* | Merge pull request #118 from shenki/gcc5-build-fixesPatrick Williams2016-06-292-1/+3
|\ \ | |/ |/| GCC 5 build fixes
| * hostcheckstop: Fix missing headerJoel Stanley2016-06-291-0/+2
| | | | | | | | | | | | | | | | | | | | | | Fixes the following error: host_checkstop_obj.c: In function ‘is_host_booted’: host_checkstop_obj.c:46:10: error: implicit declaration of function ‘strcmp’ [-Werror=implicit-function-declaration] if ((strcmp(system_state, "HOST_BOOTED") == 0) || Signed-off-by: Joel Stanley <joel@jms.id.au>
| * libopenbmc_intf: Fix non-arm devmem implementationJoel Stanley2016-06-291-1/+1
|/ | | | | | | | | | Fixes the following error: In file included from button_reset_obj.c:4:0: ../libopenbmc_intf/openbmc.h:39:15: error: return type defaults to ‘int’ [-Werror=implicit-int] static inline devmem(uint32_t val, uint32_t reg) Signed-off-by: Joel Stanley <joel@jms.id.au>
* Merge pull request #115 from bradbishop/refactoringPatrick Williams2016-06-2821-36563/+36561
|\ | | | | More skeleton refactoring
| * Conditionally use system provided libopenbmc_intfBrad Bishop2016-06-273-15/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch structures the make rules such that building from the top will build the in-tree libopenbmc_intf and link to that. For example: $make $make bmcctl would both result in ../libopenbmc_intf being built and the applications will link to that. Alternatively when building applications directly like: $make -C bmcctl the default compiler/linker search paths will be used. The latter facilitates package managers and the former retains a level of flexibility for developers working from the repository. Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
| * bin targets are rebuilt every timeBrad Bishop2016-06-271-6/+6
| | | | | | | | | | | | | | Address an issue where the targets specified in the BINS variable are rebuilt every time. Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
| * libopenbmc_intf rules simplificationBrad Bishop2016-06-272-27/+28
| | | | | | | | | | | | | | | | | | Don't use rules.mk for libopenbmc_intf. There isn't enough overlap here to warrant the extra complexity. Fix an issue where the library is rebuilt for every make invocation. Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
| * Create a libopenbmc_intf symlinkBrad Bishop2016-06-271-0/+2
| | | | | | | | | | | | | | | | | | Create a libopenbmc_intf symlink in the build target to enable things like: -L ../libopenbmc_intf -lopenbmc_intf Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
| * Update and run libopenbmc_intf codegenBrad Bishop2016-06-272-4/+4
| | | | | | | | | | | | | | Update codegen to reflect new path xml/output locations. Run updated codegen. Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
| * Move libopenbmc_intf codegen/xmlBrad Bishop2016-06-276-36493/+36491
| | | | | | | | | | | | To the libopenbmc_intf directory. Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
| * libopenbmc_intf include path updateBrad Bishop2016-06-2714-30/+29
| | | | | | | | | | | | | | | | | | Remove the interfaces prefix from applications including openbmc_intf.h. Use brackets rather than quotes in prep for a libopenbmc_intf SDK package. Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
| * Add force link option to install targetBrad Bishop2016-06-271-1/+1
| | | | | | | | | | | | This allows 'make install' to be invoked more than once. Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
| * Add install-headers targetBrad Bishop2016-06-272-1/+7
|/ | | | | | | Install the libopenbmc_intf includes in prep for a devel package. Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* Merge pull request #114 from bradbishop/openbmc-1.0Patrick Williams2016-06-242-19/+8
|\ | | | | dbus.Double and revert sensor cache
| * Cast dbus.Double for python-jsonBrad Bishop2016-06-241-0/+3
| | | | | | | | | | | | | | | | | | | | At some point a floating point value was added to a sensor object. python-json doesn't handle the dbus wrapper types like dbus.Double. Cast dbus.Double to a float, similar to integers. Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
| * Revert "Don't make pointless sensor update method calls"Brad Bishop2016-06-241-19/+5
| | | | | | | | | | | | | | | | | | This reverts commit efc689707d21f36c0ed8f0b1af474d2087ef8f95. This enhancement breaks the ability to set the sensor value via user-interface. Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
| * Revert "Fix broken attempt at hwmon caching"Brad Bishop2016-06-241-1/+1
|/ | | | | | This reverts commit 8224ff87c914861d28ed0dc6a251ba213a9ee607. Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* Merge pull request #109 from williamspatrick/parallel-build-issuePatrick Williams2016-06-221-1/+1
|\ | | | | rules.mk: Fix parallel build for 'make install'
| * rules.mk: Fix parallel build for 'make install'Patrick Williams2016-06-211-1/+1
| | | | | | | | | | | | install-bins was missing a dependency on the binaries themselves. Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
* | Merge pull request #110 from bradbishop/enumeratePatrick Williams2016-06-224-1157/+119
|\ \ | |/ |/| remove org.openbmc.Object.Enumerate interface
| * Run gdbus codegen for removed ObjectAdded signalBrad Bishop2016-06-212-1141/+99
| | | | | | | | | | | | This is the result of running gdbus-codegen with the latest XML. Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
| * Remove org.openbmc.Object.Enumerate interfaceBrad Bishop2016-06-211-6/+0
| | | | | | | | | | | | | | All applications are using freedesktop ObjectManager now so this can be removed. Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
| * obmcutil: Use freedesktop ObjectManagerBrad Bishop2016-06-211-5/+19
| | | | | | | | | | | | | | The org.openbmc.Object.Enumerate interface is deprecated in favor of the freedesktop ObjectManager. Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
OpenPOWER on IntegriCloud