commit 262b9220e59434f8031d9dc24e481b28f420c73a
author Harvey Wu <Harvey.Wu@quantatw.com> Wed Jan 04 10:35:41 2023 +0800
committer Patrick Williams <patrick@stwcx.xyz> Tue Sep 19 17:26:47 2023 +0000
tree 5163e6e9eeb6ab37b249bb3ad62088780bf458ca
parent 8e360784fc4edcc4de5b4b0f1fce5ffa7bf327fb

Add new interface Debug.Pid.Thermal/Power

- According to the issue#27, adding new interface to record the driver
  sensor name, driver sensor value and PID loop result to dbus.

interface:
Debug.Pid.ThermalPower => Record the sensor name and reading with the
                          highest temperature or power in the input
                          list. And record the result of pid loop

Debug.Pid.Zone => Record the pid config name that that is driving the
                  fans

refs:
https://github.com/openbmc/phosphor-pid-control/issues/27

Change-Id: I67051616958e07c148eabee658165082855c674d
Signed-off-by: Harvey Wu <Harvey.Wu@quantatw.com>

yaml/xyz/openbmc_project/Debug/Pid/ThermalPower.interface.yaml

diff --git a/yaml/xyz/openbmc_project/Debug/Pid/ThermalPower.interface.yaml b/yaml/xyz/openbmc_project/Debug/Pid/ThermalPower.interface.yaml
new file mode 100644
index 0000000..eda0ce7
--- /dev/null
+++ b/yaml/xyz/openbmc_project/Debug/Pid/ThermalPower.interface.yaml
@@ -0,0 +1,41 @@
+description: >
+    Implement to show the states of the first PID control loop(thermal/power).
+
+properties:
+    - name: Leader
+      type: string
+      flags:
+          - readonly
+      description: >
+          This means the name of the thermal sensor that is the highest temp in
+          input sensor list.
+
+    - name: ClassType
+      type: string
+      flags:
+          - readonly
+      description: >
+          The class of this PID loop. It can be "Temperature", "Margin", "Power"
+          or "PowerSum".
+
+    - name: Setpoint
+      type: double
+      flags:
+          - readonly
+      description: >
+          This pid config's setpoint.
+
+    - name: Input
+      type: double
+      flags:
+          - readonly
+      description: >
+          The "Leader" sensor reading.
+
+    - name: Output
+      type: double
+      flags:
+          - readonly
+      description: >
+          This property is the thermal PID loop result. This data doesn't have
+          static unit, it can be RPM or PWM depending on PID parameter settings.

yaml/xyz/openbmc_project/Debug/Pid/Zone.interface.yaml

diff --git a/yaml/xyz/openbmc_project/Debug/Pid/Zone.interface.yaml b/yaml/xyz/openbmc_project/Debug/Pid/Zone.interface.yaml
new file mode 100644
index 0000000..076d4fc
--- /dev/null
+++ b/yaml/xyz/openbmc_project/Debug/Pid/Zone.interface.yaml
@@ -0,0 +1,11 @@
+description: >
+    Implement to show the states of the zone.
+
+properties:
+    - name: Leader
+      type: string
+      flags:
+          - readonly
+      description: >
+          This means the name of the thermal/Power PID config that is driving
+          the fans.