hwmontempsensor: add support for tmp411 temp sensors

tmp411 support is handled with the tmp401 driver[1]

[1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=fce0758f59

hwmontempsensor: add support for tmp411 temp sensors

tmp411 support is handled with the tmp401 driver[1]

[1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=fce0758f5990b34af4ffba589b23f25c700beb2f

TESTED:

Confirmed sensors of these types are initialized and
picked up by HwmonTempSensor daemon on a live system with OBMC.

'''
> busctl tree xyz.openbmc_project.HwmonTempSensor
`- /xyz
`- /xyz/openbmc_project
`- /xyz/openbmc_project/sensors
`- /xyz/openbmc_project/sensors/temperature
|- /xyz/openbmc_project/sensors/temperature/TMP411__i2c_dev_49_004c
'''

'''
>busctl introspect xyz.openbmc_project.HwmonTempSensor /xyz/openbmc_project/sensors/temperature/TMP411__i2c_dev_49_004c

NAME TYPE SIGNATURE RESULT/VALUE FLAGS
org.freedesktop.DBus.Introspectable interface - - -
.Introspect method - s -
...
xyz.openbmc_project.Sensor.Value interface - - -
.MaxValue property d 127 emits-change
.MinValue property d -128 emits-change
.Unit property s "xyz.openbmc_project.Sensor.Value.Uni... emits-change
.Value property d 24.078 emits-change
...
'''

Change-Id: I1322b2ed968cb8fb97082caaddbf55dfcae47cb2
Signed-off-by: Chris Sides <Christopher.Sides@hpe.com>

show more ...

src: Convert logging to lg2

Change-Id: I28bf669e5139062f0f6145277db65d6821d48e78
Signed-off-by: George Liu <liuxiwei@ieisystem.com>

nvme: Convert logging to lg2

Change-Id: Iea6313ac06e6cd807330c5a4785ee52662f4e110
Signed-off-by: George Liu <liuxiwei@ieisystem.com>

mcu: Convert logging to lg2

Change-Id: I0599375329d0a18ad35e3cfe39c01be9fc7bc706
Signed-off-by: George Liu <liuxiwei@ieisystem.com>

ipmb: Convert logging to lg2

Change-Id: Ic7984c1ad6b5553e2ea2c7a9d09619051d181bd1
Signed-off-by: George Liu <liuxiwei@ieisystem.com>

intrusion: Convert logging to lg2

Change-Id: I372adbb3ca3ee163df6243061fea37403b8bfa76
Signed-off-by: George Liu <liuxiwei@ieisystem.com>

intel-cpu: Convert logging to lg2

Change-Id: I54869ba638de055e2f115b90c05e0f90354c3980
Signed-off-by: George Liu <liuxiwei@ieisystem.com>

hwmon-temp: Convert logging to lg2

Change-Id: Ie4f9a2b48a42ae060cc7f89167ae913af17a9f38
Signed-off-by: George Liu <liuxiwei@ieisystem.com>

fan: Convert logging to lg2

Change-Id: I622dff33087039c460ce64ea06634a0e07555230
Signed-off-by: George Liu <liuxiwei@ieisystem.com>

external: Convert logging to lg2

Change-Id: I3d426c85be0a6fef0c4600ec784a56952871e4c1
Signed-off-by: George Liu <liuxiwei@ieisystem.com>

exit-air: Convert logging to lg2

Change-Id: I66d8bb0328fa98915d5bc910072a72f5b50e9cac
Signed-off-by: George Liu <liuxiwei@ieisystem.com>

adc: Convert logging to lg2

Change-Id: I6408725aa3a0eb79b0d2ee7fd0ee46d0f553d9ed
Signed-off-by: George Liu <liuxiwei@ieisystem.com>

psu: Convert logging to lg2

Change-Id: I223d888a7c8f8bfa64c1fbdf3170fa511fb696d8
Signed-off-by: George Liu <liuxiwei@ieisystem.com>

Fix for PSU input sensors

PSU1_input_power and PSU1_In_Voltage sensors are not populated in
busctl tree xyz.openbmc_project.PSUSensor.

Its happening as the sensor names searched with "sensorNameSub

Fix for PSU input sensors

PSU1_input_power and PSU1_In_Voltage sensors are not populated in
busctl tree xyz.openbmc_project.PSUSensor.

Its happening as the sensor names searched with "sensorNameSubStr" in
code and it was populating PSUProperty as "Output Power" for Power Input
sensors(pin) as well.

So update "sensorNameSubStr" with "labelHead" and add the strings to
labelMatch as per the PSU configuration file to populate pin sensors.

Tested:
Verified using ipmitool sensor list.
~#ipmitool sensor list | grep -i psu
PSU1_In_Current | 1.872 |Amps |ok|na | na|na|na|na| na
PSU1_Out_Current | 30.000 |Amps |ok|na |na|na| 164.000|180.000|na
Pwm_PSU1_Fan_1|39.984 |unspecified |ok|na|na|na|na|na| na
Pwm_PSU1_Fan_2|39.984 |unspecified |ok|na|na|na|na|na| na
PSU1_Fan_Speed_1 | 8142.000|RPM |ok|na|na|na|na|na| na
PSU1_Fan_Speed_2 | 6962.000|RPM |ok|na|na|na|na|na| na
PSU1_Input_Power | 401.200 |Watts|ok|na|na|na| 1947.000| 2147.600 | na
PSU1_Out_Power|354.000 |Watts |ok|na|na|na|na|na| na
PSU1_Temperature | 26.000 |degrees C |ok|na|na|na| 73.000 |78.000 |na
PSU1_In_Voltage | 207.680 |Volts |ok|na|na|na|na|na| na

Signed-off-by: Jitendra Tripathy <jitendra.kumarx.tripathy@intel.com>
Signed-off-by: Jayaprakash Mutyala <mutyalax.jayaprakash@intel.com>
Change-Id: Idc7099e28632342c3c6591a63da6cd2129cd9f63

show more ...

Update Vernon Mauery email

Change-Id: Icbee474eb6528128df46bb0d35d5582655d7c525
Signed-off-by: Vernon Mauery <vernon.mauery@gmail.com>

clang-format: update latest spec and reformat

Copy the latest format file from the docs repository and apply.

Change-Id: I81aff1005be025ddb4405f384513c8e5e20bf6f4
Signed-off-by: Patrick Williams <p

clang-format: update latest spec and reformat

Copy the latest format file from the docs repository and apply.

Change-Id: I81aff1005be025ddb4405f384513c8e5e20bf6f4
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

meson: reformat with meson formatter

Apply the `meson format` results.

Change-Id: Ia881ec1745179e1ced79cc1d6cddbb2743f390f2
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

Apply misc-include-cleaner fixes

misc-include-cleaner seems to have found a lot more things all of a
sudden. Unclear why the result has changed, but we can just fix the
issues.

Tested: Clang-tidy

Apply misc-include-cleaner fixes

misc-include-cleaner seems to have found a lot more things all of a
sudden. Unclear why the result has changed, but we can just fix the
issues.

Tested: Clang-tidy now passes. Code compiles.

Change-Id: Iab045cc183f0daef663b4d5ac901200a42807987
Signed-off-by: Ed Tanous <etanous@nvidia.com>

show more ...

Deprecate std::filesystem namespace include

Clang-tidy is complaining about our using fs = ... directives. These
were originally added to handle boost::filesystem vs
std::experimental::filesystem v

Deprecate std::filesystem namespace include

Clang-tidy is complaining about our using fs = ... directives. These
were originally added to handle boost::filesystem vs
std::experimental::filesystem vs std::filesystem between the various
compilers. Now that we're all on std::filesystem, this redefinition can
be removed.

```
ChassisIntrusionSensor.hpp:11:11: error: namespace alias decl 'fs' is unused [misc-unused-alias-decls,-warnings-as-errors]
11 | namespace fs = std::filesystem;
```

Tested: code compiles

Change-Id: If4b968ff9cceb4038f526321d3c7c11563ffbce9
Signed-off-by: Ed Tanous <etanous@nvidia.com>

show more ...

psusensor: add new CRPS185 power supply

The CRPS185 is a new power supply for certain IBM systems.
The device driver has been upstreamed in the Linux kernel.

Tested:
- Confirmed expected sensors ap

psusensor: add new CRPS185 power supply

The CRPS185 is a new power supply for certain IBM systems.
The device driver has been upstreamed in the Linux kernel.

Tested:
- Confirmed expected sensors appeared on dbus

Change-Id: I460fbf81f5f67f47f5dd56b9a1435f217b144f80
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>

show more ...

Utils: Improve debug print

Currently the log contains lines like:
"psusensor[1464]: Error getting [...]: retries left4294967295"

Make debug print more readable:
- Add missing space between 'retries

Utils: Improve debug print

Currently the log contains lines like:
"psusensor[1464]: Error getting [...]: retries left4294967295"

Make debug print more readable:
- Add missing space between 'retries left' and number of
remaining retries
- Skip printing leftover retries when retries is zero as it
would underflow

Change-Id: I467053b43642fffb58771e525b773e2f1a0c24f3
Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com>

show more ...

adcsensor: prevent reading when power is not ready

Currently, the adcsensor is reading all of the sensors even their power
state is not ready. It made the system waste more resources to read
files.

adcsensor: prevent reading when power is not ready

Currently, the adcsensor is reading all of the sensors even their power
state is not ready. It made the system waste more resources to read
files.

This commit checks the power state before requesting to read I/O files.
It prevents reading sensors when the power state is not ready.

Tested:
1. Turn off the power.
$ipmitool power off
2. Check the adc sensors that have the "PowerState" property in
Entity-Manager is "On"/"ChassisOn" via dbus interfaces.
3. Those sensors have the "Available" property is "false" and the
"Value" is "nan".
4. The hwmon files that corresponding to those sensors are not read
5. Turn on the power.
$ipmitool power on
6. Check the adc sensors that have the "PowerState" property in
Entity-Manager is "On"/"ChassisOn" via dbus interfaces.
7. Those sensors have the "Available" property is "true" and the
"Value" is not "nan".

Change-Id: I52219f40fdd3b478425956c176ac40e3e16c10d1
Signed-off-by: Thang Tran <thuutran@amperecomputing.com>

show more ...

clang-format: re-format for clang-19

clang-format-19 isn't compatible with the clang-format-18 output, so we
need to reformat the code with the latest version. A few parameters
in clang-tidy have b

clang-format: re-format for clang-19

clang-format-19 isn't compatible with the clang-format-18 output, so we
need to reformat the code with the latest version. A few parameters
in clang-tidy have been deprecated, so adjust the style file
accordingly.

See Ie2f6eb3b043f2d655c9df806815afd7971fd0947 for updated style.
See I88192b41ab7a95599a90915013579608af7bc56f for clang-19 enablement.

Change-Id: I1e9a5994d067f97a54388024184eb450938fe7b3
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

PSUSensor: add SY24655 device

Summary:
# Description:
Add SY24655 in dbus-sensors

# Motivation:
- Support SY24655.

Test Plan:
- Check the corresponding sensors

# Test Log:
root@bmc:~# busctl intr

PSUSensor: add SY24655 device

Summary:
# Description:
Add SY24655 in dbus-sensors

# Motivation:
- Support SY24655.

Test Plan:
- Check the corresponding sensors

# Test Log:
root@bmc:~# busctl introspect xyz.openbmc_project.EntityManager \
> .../Yosemite_4_Spider_Board/SPIDER_PMON_NIC0_12V
NAME RESULT/VALUE
org.freedesktop.DBus.Introspectable -
.Introspect s
org.freedesktop.DBus.Peer -
.GetMachineId s
.Ping -
org.freedesktop.DBus.Properties -
.Get v
.GetAll a{sv}
.Set -
.PropertiesChanged -
xyz.openbmc_project.Configuration.SY24655 -
.Address 65
.Bus 14
.Labels 3 "in0" "curr1" "power1"
.Name "SPIDER_PMON_NIC0_12V"
.Type "SY24655"
.curr1_Name "SPIDER_PMON_NIC0_12V_CURR_A"
.in0_Name "SPIDER_PMON_NIC0_12V_VOLT_V"
.power1_Name "SPIDER_PMON_NIC0_12V_PWR_W"

Change-Id: I79a2c004c2418f3e8582aef6aa59ae0193b05bac
Signed-off-by: Rush Chen <rush.chen.wiwynn@gmail.com>

show more ...

hwmontempsensor: check active state before activation

The activate() function can be called by the createSensors() when host
power state changes, when host goes off and on again, the sensor will be

hwmontempsensor: check active state before activation

The activate() function can be called by the createSensors() when host
power state changes, when host goes off and on again, the sensor will be
activated repeatedly and cause coredump.

This commit add the check activation state before activating the sensor,
which reference from PSUSensor[1].

[1] https://gerrit.openbmc.org/c/openbmc/dbus-sensors/+/74727

Tested on Catalina, no more hwmontempsensor service crash when host
power state from off to running.

Change-Id: I72af303fe808538e20b60a5619c036973b3eac91
Signed-off-by: Potin Lai <potin.lai@quantatw.com>

show more ...