commit a1a91463bfd74aab6b4dd12e4cb80fbfb7391ff4
author Alexander Hansen <alexander.hansen@9elements.com> Tue Apr 08 11:48:22 2025 +0200
committer Alexander Hansen <alexander.hansen@9elements.com> Thu Apr 10 11:27:41 2025 +0200
tree 6b1d52cb0505e7bad2821fa67e6fb85d284f5216
parent 6e0fc042b6873cd6e959787d60bcbe522f512e1f

schemas: legacy: move INA233, ADM1281, LTC4282

These devices are covered by PSUSensor in pending changes [1],[2],[3].

If they end up being supported elsewhere, they may be moved to a
different schema at that point.

A new property 'InScaleFactor' has been added to PSUSensor which is
assumed supported since it has been configured, and the code suggests it
may be supported [4].

Also remove several properties from 'Aggregate' schema which are now
unused.

References:

[1] https://gerrit.openbmc.org/c/openbmc/dbus-sensors/+/66747
[2] https://gerrit.openbmc.org/c/openbmc/dbus-sensors/+/66751
[3] https://gerrit.openbmc.org/c/openbmc/dbus-sensors/+/58436
[4] https://github.com/openbmc/dbus-sensors/blob/408061153b93d3cf1699a019bc76dc5cc2b0c356/src/psu/PSUSensorMain.cpp#L889

Tested: Schema Validator Pass

Change-Id: I16fdd6c99cec23246fa396697a05d0245568fc9b
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

schemas/legacy.json

diff --git a/schemas/legacy.json b/schemas/legacy.json
index ca7c7ad..8fe6a05 100644
--- a/schemas/legacy.json
+++ b/schemas/legacy.json
@@ -847,6 +847,9 @@
                 "CurrScaleFactor": {
                     "$ref": "#/$defs/Types/CurrScaleFactor"
                 },
+                "InScaleFactor": {
+                    "$ref": "#/$defs/Types/InScaleFactor"
+                },
                 "Labels": {
                     "$ref": "#/$defs/Types/Labels"
                 },
@@ -875,6 +878,7 @@
                         "ADM1272",
                         "ADM1275",
                         "ADM1278",
+                        "ADM1281",
                         "ADM1293",
                         "ADS1015",
                         "ADS7830",
@@ -888,6 +892,7 @@
                         "DPS800",
                         "INA219",
                         "INA230",
+                        "INA233",
                         "INA238",
                         "IPSPS1",
                         "IR35221",
@@ -903,6 +908,7 @@
                         "ISL69260",
                         "LM25066",
                         "LTC2945",
+                        "LTC4282",
                         "LTC4286",
                         "LTC4287",
                         "MAX5970",
@@ -1080,39 +1086,17 @@
                 "Bus": {
                     "$ref": "#/$defs/Types/Bus"
                 },
-                "CurrScaleFactor": {
-                    "$ref": "#/$defs/Types/CurrScaleFactor"
-                },
-                "InScaleFactor": {
-                    "$ref": "#/$defs/Types/InScaleFactor"
-                },
                 "Labels": {
                     "$ref": "#/$defs/Types/Labels"
                 },
-                "PowerScaleFactor": {
-                    "$ref": "#/$defs/Types/PowerScaleFactor"
-                },
                 "PowerState": {
                     "$ref": "#/$defs/Types/PowerState"
                 },
-                "SlotId": {
-                    "$ref": "#/$defs/Types/SlotId"
-                },
                 "Thresholds": {
                     "$ref": "#/$defs/Types/Thresholds"
                 },
-                "pin_Scale": {
-                    "$ref": "#/$defs/Types/ScaleValue"
-                },
                 "Type": {
-                    "enum": [
-                        "INA233",
-                        "ADM1281",
-                        "ISL69269",
-                        "LTC4282",
-                        "XDPE152C4D",
-                        "PSM7"
-                    ]
+                    "enum": ["ISL69269", "XDPE152C4D", "PSM7"]
                 }
             },
             "required": ["Name", "Type"],