#
91e360eb |
| 10-Jun-2025 |
Alexander Hansen <alexander.hansen@9elements.com> |
entity-manager: extract getDBusType
For read+write properties the numbers are represented as doubles. Extract that quirk into a separate function (separation of concerns).
Tested: on Tyan s8030 boa
entity-manager: extract getDBusType
For read+write properties the numbers are represented as doubles. Extract that quirk into a separate function (separation of concerns).
Tested: on Tyan s8030 board
Inventory is properly exposed and object paths appear as expected.
``` root@s8030-bmc-30303035c0c1:~# busctl tree xyz.openbmc_project.EntityManager `- /xyz `- /xyz/openbmc_project |- /xyz/openbmc_project/EntityManager `- /xyz/openbmc_project/inventory `- /xyz/openbmc_project/inventory/system |- /xyz/openbmc_project/inventory/system/board | `- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/CPU0_Power_Consumption | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/CPU0_Temp | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/GARBO_SENSOR | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_18_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_CORE_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_MEM_ABCD | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_MEM_EFGH | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_SOC_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VBAT_33 | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_12_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_33_DUAL | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_33_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_5_DUAL | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_5_RUN | `- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/X550 ```
DBus interfaces appear as expected
``` root@s8030-bmc-30303035c0c1:~# busctl introspect xyz.openbmc_project.EntityManager /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VBAT_33 NAME TYPE SIGNATURE RESULT/VALUE FLAGS org.freedesktop.DBus.Introspectable interface - - - .Introspect method - s - org.freedesktop.DBus.Peer interface - - - .GetMachineId method - s - .Ping method - - - org.freedesktop.DBus.Properties interface - - - .Get method ss v - .GetAll method s a{sv} - .Set method ssv - - .PropertiesChanged signal sa{sv}as - - xyz.openbmc_project.Configuration.ADC interface - - - .Index property t 10 emits-change .Name property s "VBAT_33" emits-change .PowerState property s "Always" emits-change .ScaleFactor property d 0.3333 emits-change .Type property s "ADC" emits-change xyz.openbmc_project.Configuration.ADC.Thresholds0 interface - - - .Delete method - - - .Direction property s "greater than" emits-change .Name property s "upper critical" emits-change .Severity property d 1 emits-change .Value property d 3.507 emits-change xyz.openbmc_project.Configuration.ADC.Thresholds1 interface - - - .Delete method - - - .Direction property s "less than" emits-change .Name property s "lower critical" emits-change .Severity property d 1 emits-change .Value property d 2.688 emits-change ```
Change-Id: I9e6855de4ac98a79978422d42b58a355c601d38a Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com> Signed-off-by: Ed Tanous <etanous@nvidia.com>
show more ...
|
#
17f8e6ae |
| 10-Jun-2025 |
Alexander Hansen <alexander.hansen@9elements.com> |
entity-manager: populateInterfacePropertyFromJson
Extract populateInterfacePropertyFromJson from populateInterfaceFromJson because of the lengthy handling of the different property types.
Tested: o
entity-manager: populateInterfacePropertyFromJson
Extract populateInterfacePropertyFromJson from populateInterfaceFromJson because of the lengthy handling of the different property types.
Tested: on Tyan s8030 board
Inventory is properly exposed and object paths appear as expected.
``` root@s8030-bmc-30303035c0c1:~# busctl tree xyz.openbmc_project.EntityManager `- /xyz `- /xyz/openbmc_project |- /xyz/openbmc_project/EntityManager `- /xyz/openbmc_project/inventory `- /xyz/openbmc_project/inventory/system |- /xyz/openbmc_project/inventory/system/board | `- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/CPU0_Power_Consumption | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/CPU0_Temp | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/GARBO_SENSOR | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_18_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_CORE_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_MEM_ABCD | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_MEM_EFGH | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_SOC_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VBAT_33 | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_12_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_33_DUAL | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_33_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_5_DUAL | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_5_RUN | `- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/X550 ```
DBus interfaces appear as expected
``` root@s8030-bmc-30303035c0c1:~# busctl introspect xyz.openbmc_project.EntityManager /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VBAT_33 NAME TYPE SIGNATURE RESULT/VALUE FLAGS org.freedesktop.DBus.Introspectable interface - - - .Introspect method - s - org.freedesktop.DBus.Peer interface - - - .GetMachineId method - s - .Ping method - - - org.freedesktop.DBus.Properties interface - - - .Get method ss v - .GetAll method s a{sv} - .Set method ssv - - .PropertiesChanged signal sa{sv}as - - xyz.openbmc_project.Configuration.ADC interface - - - .Index property t 10 emits-change .Name property s "VBAT_33" emits-change .PowerState property s "Always" emits-change .ScaleFactor property d 0.3333 emits-change .Type property s "ADC" emits-change xyz.openbmc_project.Configuration.ADC.Thresholds0 interface - - - .Delete method - - - .Direction property s "greater than" emits-change .Name property s "upper critical" emits-change .Severity property d 1 emits-change .Value property d 3.507 emits-change xyz.openbmc_project.Configuration.ADC.Thresholds1 interface - - - .Delete method - - - .Direction property s "less than" emits-change .Name property s "lower critical" emits-change .Severity property d 1 emits-change .Value property d 2.688 emits-change ```
Change-Id: If0a86a21fabb6a28df7e5469399b3026c28cf402 Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>
show more ...
|
#
d7908697 |
| 10-Jun-2025 |
Alexander Hansen <alexander.hansen@9elements.com> |
entity-manager: extract addValueToDbus
Based on the type and if it's an array or not the json data will be exposed. There are 2 template functions 'addProperty' and 'addArrayToDbus' which can be cal
entity-manager: extract addValueToDbus
Based on the type and if it's an array or not the json data will be exposed. There are 2 template functions 'addProperty' and 'addArrayToDbus' which can be called.
Extract that branch into another function to avoid the duplication there.
Tested: on Tyan s8030 board
Inventory is properly exposed and object paths appear as expected.
``` root@s8030-bmc-30303035c0c1:~# busctl tree xyz.openbmc_project.EntityManager `- /xyz `- /xyz/openbmc_project |- /xyz/openbmc_project/EntityManager `- /xyz/openbmc_project/inventory `- /xyz/openbmc_project/inventory/system |- /xyz/openbmc_project/inventory/system/board | `- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/CPU0_Power_Consumption | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/CPU0_Temp | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/GARBO_SENSOR | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_18_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_CORE_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_MEM_ABCD | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_MEM_EFGH | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_SOC_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VBAT_33 | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_12_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_33_DUAL | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_33_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_5_DUAL | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_5_RUN | `- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/X550 ```
DBus interfaces appear as expected
``` root@s8030-bmc-30303035c0c1:~# busctl introspect xyz.openbmc_project.EntityManager /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VBAT_33 NAME TYPE SIGNATURE RESULT/VALUE FLAGS org.freedesktop.DBus.Introspectable interface - - - .Introspect method - s - org.freedesktop.DBus.Peer interface - - - .GetMachineId method - s - .Ping method - - - org.freedesktop.DBus.Properties interface - - - .Get method ss v - .GetAll method s a{sv} - .Set method ssv - - .PropertiesChanged signal sa{sv}as - - xyz.openbmc_project.Configuration.ADC interface - - - .Index property t 10 emits-change .Name property s "VBAT_33" emits-change .PowerState property s "Always" emits-change .ScaleFactor property d 0.3333 emits-change .Type property s "ADC" emits-change xyz.openbmc_project.Configuration.ADC.Thresholds0 interface - - - .Delete method - - - .Direction property s "greater than" emits-change .Name property s "upper critical" emits-change .Severity property d 1 emits-change .Value property d 3.507 emits-change xyz.openbmc_project.Configuration.ADC.Thresholds1 interface - - - .Delete method - - - .Direction property s "less than" emits-change .Name property s "lower critical" emits-change .Severity property d 1 emits-change .Value property d 2.688 emits-change ```
Change-Id: I3ee614847bed92ae5015d7ecb585cdfda54e3ea2 Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>
show more ...
|
#
d72dc33d |
| 10-Jun-2025 |
Alexander Hansen <alexander.hansen@9elements.com> |
entity-manager: extract checkArrayElementsSameType
This simplifies the populateInterfaceFromJson function.
Tested: on Tyan s8030 board
Inventory is properly exposed and object paths appear as expe
entity-manager: extract checkArrayElementsSameType
This simplifies the populateInterfaceFromJson function.
Tested: on Tyan s8030 board
Inventory is properly exposed and object paths appear as expected.
``` root@s8030-bmc-30303035c0c1:~# busctl tree xyz.openbmc_project.EntityManager `- /xyz `- /xyz/openbmc_project |- /xyz/openbmc_project/EntityManager `- /xyz/openbmc_project/inventory `- /xyz/openbmc_project/inventory/system |- /xyz/openbmc_project/inventory/system/board | `- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/CPU0_Power_Consumption | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/CPU0_Temp | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/GARBO_SENSOR | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_18_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_CORE_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_MEM_ABCD | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_MEM_EFGH | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/P0_VDD_SOC_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VBAT_33 | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_12_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_33_DUAL | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_33_RUN | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_5_DUAL | |- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VDD_5_RUN | `- /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/X550 ```
DBus interfaces appear as expected
``` root@s8030-bmc-30303035c0c1:~# busctl introspect xyz.openbmc_project.EntityManager /xyz/openbmc_project/inventory/system/board/Tyan_S8030_Baseboard/VBAT_33 NAME TYPE SIGNATURE RESULT/VALUE FLAGS org.freedesktop.DBus.Introspectable interface - - - .Introspect method - s - org.freedesktop.DBus.Peer interface - - - .GetMachineId method - s - .Ping method - - - org.freedesktop.DBus.Properties interface - - - .Get method ss v - .GetAll method s a{sv} - .Set method ssv - - .PropertiesChanged signal sa{sv}as - - xyz.openbmc_project.Configuration.ADC interface - - - .Index property t 10 emits-change .Name property s "VBAT_33" emits-change .PowerState property s "Always" emits-change .ScaleFactor property d 0.3333 emits-change .Type property s "ADC" emits-change xyz.openbmc_project.Configuration.ADC.Thresholds0 interface - - - .Delete method - - - .Direction property s "greater than" emits-change .Name property s "upper critical" emits-change .Severity property d 1 emits-change .Value property d 3.507 emits-change xyz.openbmc_project.Configuration.ADC.Thresholds1 interface - - - .Delete method - - - .Direction property s "less than" emits-change .Name property s "lower critical" emits-change .Severity property d 1 emits-change .Value property d 2.688 emits-change ```
Change-Id: I73aa40b8e430578b6073d5f01c5e0b43cec55c46 Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>
show more ...
|
#
59ef1e72 |
| 16-Apr-2025 |
Christopher Meis <christopher.meis@9elements.com> |
cleanup: Move em only utils code into em directory
More separation of code for improvement of repository maintainability.
Change-Id: I4e77b472355066483cf646cdcd29e7ed660294a3 Signed-off-by: Christo
cleanup: Move em only utils code into em directory
More separation of code for improvement of repository maintainability.
Change-Id: I4e77b472355066483cf646cdcd29e7ed660294a3 Signed-off-by: Christopher Meis <christopher.meis@9elements.com>
show more ...
|
#
fc9e7fda |
| 03-Apr-2025 |
Christopher Meis <christopher.meis@9elements.com> |
cleanup: move EM service code in EM directory
Improving repository structure increases maintainability.
Change-Id: I7f746a5d491dda256a06143f516e7d078a761c14 Signed-off-by: Christopher Meis <christo
cleanup: move EM service code in EM directory
Improving repository structure increases maintainability.
Change-Id: I7f746a5d491dda256a06143f516e7d078a761c14 Signed-off-by: Christopher Meis <christopher.meis@9elements.com>
show more ...
|