summaryrefslogtreecommitdiffstats
path: root/dbus
Commit message (Collapse)AuthorAgeFilesLines
...
* [dbus-writer] limit d-bus trafficJames Feist2018-10-082-1/+13
| | | | | | | | | | | Limit d-bus traffic by only sending updates, instead of every value. Tested-by: used d-bus monitor and noticed significantly reduced messages. Change-Id: Ie677feaedab4e9ebb14950392b9588d4d82c0058 Signed-off-by: James Feist <james.feist@linux.intel.com>
* [dbusconfiguration] Allow partial configurationsJames Feist2018-10-061-99/+124
| | | | | | | | | | | | | | | In a few cases (cpu sensors, etc.) we can add sensor during runtime, that aren't an error to be missing when the system is off, or are optional (add-in cards). Add a dbus match to restart on sensors added, and don't add configurations that are missing a sensor. Tested-by: Turned on debug, noticed on cpu sensors being added daemon restarted correctly. Also noticed daemon didn't crash with no configuration. Change-Id: Ide0bd03c12e380e5aad56b1da06e34a5fc5cdb9f Signed-off-by: James Feist <james.feist@linux.intel.com>
* Add hysteresis to stepwise controllerJames Feist2018-09-211-0/+16
| | | | | | | | Tested-by: Ran on platform monitoring output and wrote unit test Change-Id: I74a1d21544c1a9cb4c1cb26dd4a353cbff0442d0 Signed-off-by: James Feist <james.feist@linux.intel.com>
* Add stepwise controllerJames Feist2018-09-121-133/+252
| | | | | | | | | | | | | | | This adds the ability to use stepwise curves alongside pid control. This creates a base controller class that pidcontroller and stepwise controller inherit from. Note: Hysteresis to come in follow-on patch Tested-by: Created a stepwise controller and noticed that when it crossed a threshold that it contributed to the pwm setting. Change-Id: I6cf842f80eaccafc905d620970afe91e2092d568 Signed-off-by: James Feist <james.feist@linux.intel.com>
* DbusConfiguration Flatten configurationJames Feist2018-09-121-86/+123
| | | | | | | | | | | | | It didn't make much sense to have the ILimit and OutLimit be their own interface. Entity Manager change to be submitted shortly. Tested-By: Used configuration from review https://gerrit.openbmc-project.xyz/#/c/12082/ and verified swampd still controlled fans properly. Change-Id: Id183cd09eba43f65fb80c9a4af7092f5e2e0cc08 Signed-off-by: James Feist <james.feist@linux.intel.com>
* clang-format: fix pointer alignment, sort usingPatrick Venture2018-09-043-43/+43
| | | | | Change-Id: Id433d0ecc10c62807594b3a637e591045223faa6 Signed-off-by: Patrick Venture <venture@google.com>
* add .clang-formatPatrick Venture2018-08-318-135/+114
| | | | | Change-Id: I6627b5569c2e0f730be7331403218b823a2c622f Signed-off-by: Patrick Venture <venture@google.com>
* dbusConfiguration: restart on configuration changeJames Feist2018-08-221-0/+21
| | | | | | | | | | | | Watch for dbus interface changes and stop the daemon when they happen. Systemd should restart us if we have the correct service file written. Tested-by: Used new add object entity-manager interface and noticed daemon restart when new config was added. Change-Id: I708447dbb6ed2ab23b60036d310606ef7a1a7d3b Signed-off-by: James Feist <james.feist@linux.intel.com>
* Add double support for sensor interfaceJames Feist2018-08-103-12/+42
| | | | | | | | | Add support for reading sensors that produce a double as the value. Use a visitor to pull out the sensor value so type doesn't matter. Change-Id: I37c05e7077ead6f3084ab9704a1b2c62becb5e19 Signed-off-by: James Feist <james.feist@linux.intel.com>
* Add ability to configure from dbusJames Feist2018-07-235-0/+517
| | | | | | | | | | | | | | This allows configuration of PID control from dbus using entity manager. Sample configuration will be pushed to entity-manager repo shortly. Tested-by: Used yaml configuration and dbus / entity-manager based configuration and pid seemed to work the same. Verified printout of configuration matched generated cpp files. Change-Id: Ia7b016e53262791ffcccdb9b21c1ccddae2926bc Signed-off-by: James Feist <james.feist@linux.intel.com>
* test: dbus: passive read interfacePatrick Venture2018-06-284-1/+32
| | | | | | | | | | Added basic unit-tests and added a factory for creating the DbusPassive read interface so that it can be nicely error checked. This is handled via a valid type check where the only valid types are 'fan' and 'temp'. Change-Id: I558ed09bf509d26f20c6e431bb0789074e9aa841 Signed-off-by: Patrick Venture <venture@google.com>
* dbus: transition to interface for testingPatrick Venture2018-06-226-38/+77
| | | | | | | | | | To make testing easier, the dbus objects now receive a helper interface to use instead of calling into a helper module. Tested: Ran on quanta-q71l board and it behaved as expected. Change-Id: I2521d9c75aec76e92d2e26dc044d01011e44d552 Signed-off-by: Patrick Venture <venture@google.com>
* dbus: dbuspassive refactorPatrick Venture2018-06-221-17/+16
| | | | | | | | | Moved the sensor value handler into a sub routine to enable testing. Tested: Ran on quanta-q71l board and it behaved as expected. Change-Id: I4873558cfc1a48a92fb60cd173dc69e237813683 Signed-off-by: Patrick Venture <venture@google.com>
* dbus: util: Remove explicit element countPatrick Venture2018-05-101-5/+1
| | | | | | | | | | The dbus interface can have new properties added, and it's improbable to have the three we care about removed. Therefore, let's remove the explicit count check on the number of properties returned. Change-Id: Ia292c73dc889a33a4b1f334585bbce910f20a580 Signed-off-by: Patrick Venture <venture@google.com>
* Sensor ObjectsPatrick Venture2018-03-087-0/+388
This includes all the sensor objects including a few implementations, including dbus and sysfs sensors. Change-Id: I9897c79f9fd463f00f0e02aeb6c32ffa89635dbe Signed-off-by: Patrick Venture <venture@google.com>
OpenPOWER on IntegriCloud