Fix file descriptor checking

'0' is also a valid file descriptor so this commit fixes incorrect
checks on fd after opening a file. Actually, fd 0, 1 and 2 will be
assigned to stdin, stdout and stderr respectively so this patch
doesn't fix any issue but it's for code consistency.

Tested: Detecting and reading sensors worked well.

Change-Id: I2626a0b92352257d1c294a61ca385f95568dae7d
Signed-off-by: Jae Hyun Yoo <jae.hyun.yoo@linux.intel.com>
6 files changed
tree: 64b40334b5c896de0bb2e6d446d6fcea0226f825
  1. cmake/
  2. include/
  3. service_files/
  4. src/
  5. tests/
  6. .clang-format
  7. .clang-ignore
  8. .gitignore
  9. cmake-format.json
  10. CMakeLists.txt
  11. Jenkinsfile
  12. LICENSE
  13. MAINTAINERS
  14. README.md
README.md

dbus-sensors

dbus-sensors is a collection of sensor applications that provide the xyz.openbmc_project.Sensor collection of interfaces. They read sensor values from hwmon, d-bus, or direct driver access to provide readings. Some advance non-sensor features such as fan presence, pwm control, and automatic cpu detection (x86) are also supported.

key features

  • runtime re-configurable from d-bus (entity-manager or the like)

  • isolated: each sensor type is isolated into its own daemon, so a bug in one sensor is unlikely to affect another, and single sensor modifications are possible

  • async single-threaded: uses sdbusplus/asio bindings

  • multiple data inputs: hwmon, d-bus, direct driver access