Remove #warning ignore

Isn't an issue anymore.

Tested: It compiles

Change-Id: I59e343db79459c55538f4d9828ac269237e1b015
Signed-off-by: James Feist <james.feist@linux.intel.com>

Implement PSU Power Sensor

Implement Power Supply Input Power Sensor.

Tested By:
With the related change in entity-manager, after run psusensor in
BMC console, xyz.openbmc_project.PSUSensor has bee

Implement PSU Power Sensor

Implement Power Supply Input Power Sensor.

Tested By:
With the related change in entity-manager, after run psusensor in
BMC console, xyz.openbmc_project.PSUSensor has been created and
PSU pin dbus interface has been created with correct Thresholds
and value. Ipmitool sensor list can show PSU pin sensor.

Change-Id: Ib057a9ecca7bf317eb8d98af1ddb8be39841a54f
Signed-off-by: Cheng C Yang <cheng.c.yang@linux.intel.com>

show more ...

Add association to Fan Redundancy

This also needs an association.

Change-Id: I6b0f2f4c5fdc2143ee4e2336d0f9fb9976cec629
Signed-off-by: James Feist <james.feist@linux.intel.com>

Add associations for pwm

Pwm doesn't inheirt from Sensor as it is special, so
it needs it to be added seperate.

Tested: Saw associations on dbus

Change-Id: I2178e9346dcae845b47a013a2cc15f62e9ed9b3

Add associations for pwm

Pwm doesn't inheirt from Sensor as it is special, so
it needs it to be added seperate.

Tested: Saw associations on dbus

Change-Id: I2178e9346dcae845b47a013a2cc15f62e9ed9b31
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...

Add associations to sensors

This adds associations to all the sensors on dbus.

Tested: Mapper has lots of sensor associations now.

Change-Id: I9a1c51d6b139e45a228507d29637c29a84b302d9
Signed-off-b

Add associations to sensors

This adds associations to all the sensors on dbus.

Tested: Mapper has lots of sensor associations now.

Change-Id: I9a1c51d6b139e45a228507d29637c29a84b302d9
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...

Destroy redundancy sensor before re-creation.

The mapper gets confused if a removed comes too soon
before an add and removes the interface. Fix this.

Tested: Showed up in the mapper on boot.

Chang

Destroy redundancy sensor before re-creation.

The mapper gets confused if a removed comes too soon
before an add and removes the interface. Fix this.

Tested: Showed up in the mapper on boot.

Change-Id: Iad11cc5f30f8a1de594810cd19cbb415799a1fb8
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...

Trim insignificant figures from exit air temp

Trim to whole digit precision since this is based off
of a calculation.

Change-Id: If375d5255a3bd2df800303d0b599249290dcab12
Signed-off-by: James Feist

Trim insignificant figures from exit air temp

Trim to whole digit precision since this is based off
of a calculation.

Change-Id: If375d5255a3bd2df800303d0b599249290dcab12
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...

Fix threshold conversion

Converting to float first makes the values get extra
garbage data. Go directly to double.

Change-Id: I1c4af05d89e6adcabcf1a3b1bef523c61c0515fd
Signed-off-by: James Feist <j

Fix threshold conversion

Converting to float first makes the values get extra
garbage data. Go directly to double.

Change-Id: I1c4af05d89e6adcabcf1a3b1bef523c61c0515fd
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...

Filter ADC threshold crossings

When power goes down we get bad events because the
voltage rail spins down faster than pgood. Add in
a delay to check power 2 seconds after the event to
make sure powe

Filter ADC threshold crossings

When power goes down we get bad events because the
voltage rail spins down faster than pgood. Add in
a delay to check power 2 seconds after the event to
make sure power isn't going down.

Tested-by: power cycled and extra SEL went away

Change-Id: Ib820ccb50d1a949b8096f08e2711ad7a7c36087b
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...

ipmbsensor: Add delay to init cmd

Sometimes we run the init cmd too early and we start
getting sensor readings that are out of range. Normally
this is only 1-2 sensors. Add in a delay so that hopefu

ipmbsensor: Add delay to init cmd

Sometimes we run the init cmd too early and we start
getting sensor readings that are out of range. Normally
this is only 1-2 sensors. Add in a delay so that hopefully
this won't happen. Also don't print errors too often.

Tested-by: Power cycled multiple times and never saw NA
for sensor readings.

Change-Id: Ib1d336fc5b04b2c96e41b5de2a89d26c7a6d3e83
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...

Adjust DIMM detection interval

In case a CPU doesn't have any DIMM installed, cpusensor service
will keep scanning to detect DIMM. Currently, the detection
interval is 1 second which raises CPU reso

Adjust DIMM detection interval

In case a CPU doesn't have any DIMM installed, cpusensor service
will keep scanning to detect DIMM. Currently, the detection
interval is 1 second which raises CPU resource consumption of
cpusensor service up to 20% so this commit adjusts the detection
interval to 3 seconds.

Tested:
Observe CPU resource usage using top -d1. cpusensor service
shouldn't eat much resources.

Change-Id: I443a5428eaa2c72a38fd4f5e2c1973c821944084
Signed-off-by: Jae Hyun Yoo <jae.hyun.yoo@linux.intel.com>

show more ...

Exit Air: Allow destruction

In the current mode a configuration reload would not
cause thresholds to update. Allow destruction so we
reload the interfaces each time.

Change-Id: I1fec9c6fc9ae1a20a95

Exit Air: Allow destruction

In the current mode a configuration reload would not
cause thresholds to update. Allow destruction so we
reload the interfaces each time.

Change-Id: I1fec9c6fc9ae1a20a95477157ad399c73e3f9c75
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...

Improve cpusensor performance

The cpusensor service is still making some redundant accesses for
reading hwmon attributes on every scanning. Actually, threshold
update is needed only when CPU package

Improve cpusensor performance

The cpusensor service is still making some redundant accesses for
reading hwmon attributes on every scanning. Actually, threshold
update is needed only when CPU package's Tcontrol target is
changed so this commit make it do that only when Tcontrol value
is changed.

Tested:
Compare cpu resource consumption of cpusensor service using top.
It should eat less cpu resources than before.

Change-Id: Ieb582996fc5c9c07abbfc7ac0d1b37f593269d00
Signed-off-by: Jae Hyun Yoo <jae.hyun.yoo@linux.intel.com>

show more ...

Fix threshold updating

The object type was never stored so it was never
updated. Also change threshold size to count of
thresholds for if anything has multiple thresholds.

Tested-by: set thresholds

Fix threshold updating

The object type was never stored so it was never
updated. Also change threshold size to count of
thresholds for if anything has multiple thresholds.

Tested-by: set thresholds for front panel temp

Change-Id: I0949261b87336eb4a85cf596805c02683fb30d9b
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...

cmakelists: Make sensors able to be disabled

Some vendors may not want all sensors, make them
able to be disabled. Also alphabetize them.

Change-Id: Idf6cec39cd1b521784cc3528945c5069e1e43763
Signed

cmakelists: Make sensors able to be disabled

Some vendors may not want all sensors, make them
able to be disabled. Also alphabetize them.

Change-Id: Idf6cec39cd1b521784cc3528945c5069e1e43763
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...

Add chassis intrusion sensor daemon

Chassis intrusion signal is read from PCH via I2C or from GPIO.
Create a new daemon to poll its status and expose to dbus.

Related patches to run test:
- meta-ph

Add chassis intrusion sensor daemon

Chassis intrusion signal is read from PCH via I2C or from GPIO.
Create a new daemon to poll its status and expose to dbus.

Related patches to run test:
- meta-phosphor: dbus-sensors: Enable new service of intrusion sensor
https://gerrit.openbmc-project.xyz/#/c/openbmc/meta-phosphor/+/17063/
- redfish: chassis: add property of physical security
https://gerrit.openbmc-project.xyz/#/c/openbmc/bmcweb/+/17691/

Tested-by: Check intrusion status value by redfish,
verified PCH solution on WFP and GPIO solution on STP.

Change-Id: Id5e67abbd80bbf2ef502db49fa183d92d0d31bda
Signed-off-by: Qiang XU <qiang.xu@linux.intel.com>

show more ...

Modify PECI macro uses to prepare PECI header update

This commit is for preparation of PECI header update which has
'PECI_' prefix on all PECI relating macros. The temporary
re-definition code will

Modify PECI macro uses to prepare PECI header update

This commit is for preparation of PECI header update which has
'PECI_' prefix on all PECI relating macros. The temporary
re-definition code will be removed later when the header file is
provisioned completely.

Change-Id: Ib66addb1c96c08f12dcfddbc6048dc425e0b2afb
Signed-off-by: Jae Hyun Yoo <jae.hyun.yoo@linux.intel.com>

show more ...

Add threshold value scanning logic into cpusensor

Threshold values of a CPU sensor can be changed at runtime by BIOS
based on the thermal policy changes of the host machine. So this
commit adds runt

Add threshold value scanning logic into cpusensor

Threshold values of a CPU sensor can be changed at runtime by BIOS
based on the thermal policy changes of the host machine. So this
commit adds runtime scanning logic for reading threshold values
from hwmon subsystem attributes.

Tested:
Check threshold properties in xyz.openbmc_projects.CPUSensor.
All threshold vaues should reflect peci hwmon values dynamically.

Change-Id: Ib5b0ce72f567437e69879a7e655d6f34f156013e
Signed-off-by: Jae Hyun Yoo <jae.hyun.yoo@linux.intel.com>

show more ...

Exclude non-sensor properties from CPU sensor creation

This commit makes CPU sensor service excludes non-sensor properties
while creating dbus sensors. Tcontrol, Tthrottle and Tjmax will be
filtered

Exclude non-sensor properties from CPU sensor creation

This commit makes CPU sensor service excludes non-sensor properties
while creating dbus sensors. Tcontrol, Tthrottle and Tjmax will be
filtered out.

Change-Id: I57a551bf576d74164a6f7ea4873ac8340c0696c3
Signed-off-by: Jae Hyun Yoo <jae.hyun.yoo@linux.intel.com>

show more ...

ipmbsensor: resend init command every boot

This stops the ipmi response from being na after
ipmitool power cycle.

Tested-by: Power cycled 5 times and sensors never
went to na

Change-Id: Ida6ac3978

ipmbsensor: resend init command every boot

This stops the ipmi response from being na after
ipmitool power cycle.

Tested-by: Power cycled 5 times and sensors never
went to na

Change-Id: Ida6ac397805b9eb47025e0227e2d15fcc26bb045
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...

Only show PWMs that are in the config

If we enable all the pwm from device tree, we should only
show the ones that are in the cofig to avoid confusion.

Tested-by: sensor list only showed pwms in th

Only show PWMs that are in the config

If we enable all the pwm from device tree, we should only
show the ones that are in the cofig to avoid confusion.

Tested-by: sensor list only showed pwms in the config

Change-Id: I9d3c1044045ee930b082ce2cbc4e55d33375b4d7
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...

Update clang format

Style only change

Change-Id: I41c97cbfb6a0ebf155a34166bc627f49a398442b
Signed-off-by: James Feist <james.feist@linux.intel.com>

Move variant usage to std

Since sdbusplus now uses std, it's cleaner just to
use the variant type directly not behind the namespace.

This was primarily done with sed commands

Tested-by: It builds

Move variant usage to std

Since sdbusplus now uses std, it's cleaner just to
use the variant type directly not behind the namespace.

This was primarily done with sed commands

Tested-by: It builds

Change-Id: I87a90a2942cfc6da312cb0e045cce7cd40a644e5
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...

Add bios post checking support

Some sensors aren't available until after bios post
add support to check for post when needed.

Tested-by: SSB temp no longer asserted thresholds before
bios post.

Ch

Add bios post checking support

Some sensors aren't available until after bios post
add support to check for post when needed.

Tested-by: SSB temp no longer asserted thresholds before
bios post.

Change-Id: I9251c2c501e082ef4bb0e9eb7909dc137771e120
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...

Fix power match

Power should be capitalized

Tested-by: Ipmb sensor had readings after reboot

Change-Id: I2bf64903ace851fa7b1aff58445c6ddf6457c712
Signed-off-by: James Feist <james.feist@linux.inte

Fix power match

Power should be capitalized

Tested-by: Ipmb sensor had readings after reboot

Change-Id: I2bf64903ace851fa7b1aff58445c6ddf6457c712
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...