5d82f474 | 22-Jul-2022 |
Patrick Williams <patrick@stwcx.xyz> |
sdbusplus: use shorter type aliases
The sdbusplus headers provide shortened aliases for many types. Switch to using them to provide better code clarity and shorter lines. Possible replacements are
sdbusplus: use shorter type aliases
The sdbusplus headers provide shortened aliases for many types. Switch to using them to provide better code clarity and shorter lines. Possible replacements are for: * bus_t * exception_t * manager_t * match_t * message_t * object_t * slot_t
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: Ibd2a0b512bfb7caf65bfab64b271d194da520aac
show more ...
|
7d7a6858 | 25-Jul-2022 |
Patrick Williams <patrick@stwcx.xyz> |
avoid non-virtual-dtor warnings
``` ../subprojects/phosphor-host-ipmid/include/ipmid/handler.hpp:157:7: error: base class ‘class ipmi::HandlerBase’ has accessible non-virtual destructor [-Werror=non
avoid non-virtual-dtor warnings
``` ../subprojects/phosphor-host-ipmid/include/ipmid/handler.hpp:157:7: error: base class ‘class ipmi::HandlerBase’ has accessible non-virtual destructor [-Werror=non-virtual-dtor] ../subprojects/phosphor-host-ipmid/include/ipmid/filter.hpp:37:7: error: ‘class ipmi::FilterBase’ has virtual functions and accessible non-virtual destructor [-Werror=non-virtual-dtor] ```
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: Ib1f8befabc6ea4564524c474118b78f05403413d
show more ...
|
b69b2066 | 25-Jul-2022 |
Patrick Williams <patrick@stwcx.xyz> |
meson: refactor generators
Using 'run_command' is not the appropriate way to generate code since it only runs when meson itself runs and doesn't expose to meson the correct dependency information.
meson: refactor generators
Using 'run_command' is not the appropriate way to generate code since it only runs when meson itself runs and doesn't expose to meson the correct dependency information. 'custom_target' is the more appropriate way to generate code, so switch to using it.
This was noticed because when trying to build as a subproject, the depending repository failed with: ``` subprojects/phosphor-host-ipmid/meson.build:229:0: ERROR: Sandbox violation: Tried to grab file sensor-gen.cpp outside current (sub)project. ```
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: I555c6e1a7bfaabae63d757042ddc1c5a548295aa
show more ...
|
37b9f414 | 22-Jul-2022 |
Patrick Williams <patrick@stwcx.xyz> |
OWNERS: switch 'matches' to 'matchers'
The original OWNERS template had a mistake which used 'matches' instead of the field supported by the Gerrit plugin 'matchers'. Update the OWNERS file to have
OWNERS: switch 'matches' to 'matchers'
The original OWNERS template had a mistake which used 'matches' instead of the field supported by the Gerrit plugin 'matchers'. Update the OWNERS file to have the correct field.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: I5443b95e89867b1f2892409f4094acb1fef61fad
show more ...
|
bfd3a17f | 31-May-2022 |
Willy Tu <wltu@google.com> |
apphandler: Add firmware_revision support for ipmi config
Allow us to configure the firmware revision if needed. The field is optional and will override the existing firmware revision. This provide
apphandler: Add firmware_revision support for ipmi config
Allow us to configure the firmware revision if needed. The field is optional and will override the existing firmware revision. This provide us a way to customize the firmware revision to anything we want.
Tested: With firmware_revision in dev_id.json ``` $ cat /usr/share/ipmi-providers/dev_id.json $ ipmitool mc info { "id": 0, "revision": 0, "addn_dev_support": 0, "firmware_revision": { "major": 35, "minor": 16 }, "manuf_id": 11129, "prod_id": 14426, "aux": 0 }
ipmitool mc info Device ID : 0 Device Revision : 0 Firmware Revision : 35.10 IPMI Version : 2.0 Manufacturer ID : 11129 Manufacturer Name : Google, Inc. Product ID : 14426 (0x385a) Product Name : Unknown (0x385A) Device Available : yes Provides Device SDRs : no Additional Device Support : Aux Firmware Rev Info : 0x00 0x00 0x00 0x00 ```
Change-Id: Ia05ceb9ff5ca46b104e70ad9230432e79fc70d07 Signed-off-by: Willy Tu <wltu@google.com>
show more ...
|
bef102a3 | 09-Jun-2022 |
Willy Tu <wltu@google.com> |
dbus-sdr: Update bitset usage for VR profile status
The assertions for getVrEventStatus is not used properly. It used to be different with uint8_t which is set to 1 << index. With bitset, we can jus
dbus-sdr: Update bitset usage for VR profile status
The assertions for getVrEventStatus is not used properly. It used to be different with uint8_t which is set to 1 << index. With bitset, we can just set it to the index directly. Wrapped with htole16 just in case.
Tested: The profile status still works. With correct response this time ``` $ ipmitool raw 0x4 0x2b 0xd3 00 02 00 00 00
// Update the profile $ ipmitool raw 0x4 0x30 0xd3 0 0 4 0 0 0 0 0 0
$ ipmitool raw 0x4 0x2b 0xd3 00 04 00 00 00 ```
Change-Id: Ic59e88b65ff9b45ba759a8c62a141cf14e8bdc20 Signed-off-by: Willy Tu <wltu@google.com>
show more ...
|
1a2e1506 | 07-Jul-2022 |
George Liu <liuxiwei@inspur.com> |
Fix project name misspelling
Signed-off-by: George Liu <liuxiwei@inspur.com> Change-Id: I11a2565cdf424fc98a193fe722643efdf8ae9424 |
ff1b4bf1 | 28-Jun-2022 |
Jian Zhang <zhangjian.3032@bytedance.com> |
meson: Fix channel command invaild
Fix channel command invaild, `channelcommands.cpp` file is missing in meson.build.
Tested:
ipmitool channel info 1 Channel 0x1 info: Channel Medium Type : 80
meson: Fix channel command invaild
Fix channel command invaild, `channelcommands.cpp` file is missing in meson.build.
Tested:
ipmitool channel info 1 Channel 0x1 info: Channel Medium Type : 802.3 LAN Channel Protocol Type : IPMB-1.0 Session Support : multi-session Active Session Count : 0 Protocol Vendor ID : 7154 Volatile(active) Settings Alerting : enabled Per-message Auth : enabled User Level Auth : enabled Access Mode : always available Non-Volatile Settings Alerting : enabled Per-message Auth : enabled User Level Auth : enabled Access Mode : always available
Change-Id: I36436a78b4344087b4d079fa1775383005a0bd3d Signed-off-by: Jian Zhang <zhangjian.3032@bytedance.com>
show more ...
|
f5684a12 | 01-Jun-2022 |
Willy Tu <wltu@google.com> |
README: Update build instruction to based on meson
Copied the instructions from bmcweb in https://github.com/openbmc/bmcweb/blob/master/README.md
Change-Id: I9446113bb87e8f530647d2a6a64a72806e355d6
README: Update build instruction to based on meson
Copied the instructions from bmcweb in https://github.com/openbmc/bmcweb/blob/master/README.md
Change-Id: I9446113bb87e8f530647d2a6a64a72806e355d62 Signed-off-by: Willy Tu <wltu@google.com>
show more ...
|
d09db492 | 29-May-2022 |
Tim Lee <timlee660101@gmail.com> |
Fix UnknownProperty error when getting sensor data record
Symptom: When getting sensor data record by ipmitool sdr command, there is error from dbus-monitor debug log as below: method call time=1653
Fix UnknownProperty error when getting sensor data record
Symptom: When getting sensor data record by ipmitool sdr command, there is error from dbus-monitor debug log as below: method call time=1653875077.002911 sender=:1.56 -> destination=xyz.openbmc_project.Hwmon-3446998316.Hwmon1 serial=69975 string "xyz.openbmc_project.Sensor.Threshold.Warning" string "WarningAlarmlow" error time=1653875077.002975 sender=:1.52 -> destination=:1.56 error_name=org.freedesktop.DBus.Error.UnknownProperty re string "Unknown interface xyz.openbmc_project.Sensor.Threshold.Warning or property WarningAlarmlow." method return time=1653875077.003043 sender=:1.56 -> destination=:1.107 serial=69976 reply_serial=11
Root cause: The property "WarningAlarmlow" name is case sensitive
Solution: Correct the property name "WarningAlarmlow" to "WarningAlarmLow"
Verified: Run ipmitool sdr command and check dbus-monitor debug log
Signed-off-by: Tim Lee <timlee660101@gmail.com> Change-Id: Ic92b4849373756bf94a4960d803fae5269d5b8e9
show more ...
|
b4905919 | 17-Jun-2022 |
Tim Lee <timlee660101@gmail.com> |
ipmid: message: fix pack/unpack compile error at aarch64 platform
After debugging with boost/multiprecision owner jzmaddock. We have found the root cause why boost v1.79.0 got this compile error in
ipmid: message: fix pack/unpack compile error at aarch64 platform
After debugging with boost/multiprecision owner jzmaddock. We have found the root cause why boost v1.79.0 got this compile error in ipmid. More detail you can refer from https://github.com/boostorg/multiprecision/issues/472
Root cause: Boost changed all bit counts from unsigned to std::size_t, specifically for platforms like arm64 (and windows!) where unsigned is narrower than size_t so that the maximum representable number isn't unnecessarily constrained.
This then changes the interface for cpp_int_backend to use size_t rather than unsigned for the bit counts. On most platforms and most use cases, this makes no perceptible difference, but unfortunately this appears to be one situation where it really does make a difference.
Apparently this is true even though:
template <unsigned N> using fixed_uint_t = boost::multiprecision::number<boost::multiprecision::cpp_int_backend< N, N, boost::multiprecision::unsigned_magnitude, boost::multiprecision::unchecked, void>>;
Is declared with an unsigned parameter, when partially specializing for fixed_uint_t you need to use the actual type of the template parameter in the underlying template, not the type used in the template alias!
Solution: Change all usage of unsigned for bitcounts to a new typedef bitcount_t which is size_t for Boost-1.79 and later, and unsigned for Boost-1.78 and earlier.
Verified: No compile error at aarch64 platform and test ipmitool sdr command is pass.
Signed-off-by: jzmaddock <john@johnmaddock.co.uk> Signed-off-by: Tim Lee <timlee660101@gmail.com> Change-Id: Id7a7c86ef854f4b9c06fc4da054c8021f76812b8
show more ...
|
8d90f975 | 04-Jul-2022 |
George Liu <liuxiwei@inspur.com> |
meson: Expose libchannellayer and libuserlayer
Since other repos need to depend on channellayer and userlayer (for example: phosphor-net-ipmi, and plan to switch to meson), it is necessary to expose
meson: Expose libchannellayer and libuserlayer
Since other repos need to depend on channellayer and userlayer (for example: phosphor-net-ipmi, and plan to switch to meson), it is necessary to expose libchannellayer and libuserlayer for other repos to depend on.
Signed-off-by: George Liu <liuxiwei@inspur.com> Change-Id: I6af6064be71f5a7390940b929d20f4c9e138f60a
show more ...
|
25035b88 | 16-Jun-2022 |
Patrick Williams <patrick@stwcx.xyz> |
gitignore: minor fixup for symlink workflow
It is convenient to have the subprojects source populated by symlinking to a common workspace so that cross-repository development can be done. The previo
gitignore: minor fixup for symlink workflow
It is convenient to have the subprojects source populated by symlinking to a common workspace so that cross-repository development can be done. The previous .gitignore caused 'untracked file' for each subproject symlink because it was a symlink instead of a directory. Modify the .gitignore to match what most other meson repositories have done.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: I303effd3c8e7099aaade4a3ac1e81ac7f55cdec0
show more ...
|
bbc6ad48 | 16-Jun-2022 |
Patrick Williams <patrick@stwcx.xyz> |
meson: minor dependency fix ups for wrap-builds
Add a few missing dependencies to enable full wrap-based builds.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: I0eab0c37acaad4be5b63
meson: minor dependency fix ups for wrap-builds
Add a few missing dependencies to enable full wrap-based builds.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: I0eab0c37acaad4be5b63d9804200566405f40c0b
show more ...
|
3d8d793f | 16-Jun-2022 |
Patrick Williams <patrick@stwcx.xyz> |
remove std::experimental usage
C++17 moved this support out of std::experimental and into std directly. Fix up the code and dependencies to reflect this.
Signed-off-by: Patrick Williams <patrick@s
remove std::experimental usage
C++17 moved this support out of std::experimental and into std directly. Fix up the code and dependencies to reflect this.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: I1cd852831fc2608bd3a36f047aa6ac14741da270
show more ...
|
c8fc728f | 16-Jun-2022 |
Patrick Williams <patrick@stwcx.xyz> |
meson: add nlohmann-json wrapper and deps
A number of locations within the repository do not appropriately specify the dependency on nlohmann-json. Fix up the wrap file, add appropriate dependency
meson: add nlohmann-json wrapper and deps
A number of locations within the repository do not appropriately specify the dependency on nlohmann-json. Fix up the wrap file, add appropriate dependency detection in the root meson, and add the library as a dependency when needed. This gets closer to allowing full subproject-based builds.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: If70888c6de5651ca355f84616e41408ddde9fc08
show more ...
|
422385da | 16-Jun-2022 |
Patrick Williams <patrick@stwcx.xyz> |
meson: simplify sdbusplus dependency
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: If34a35bd8984c5ef54fb88f2805570748cc08515 |
6b4cf435 | 16-Jun-2022 |
Patrick Williams <patrick@stwcx.xyz> |
meson: add wrap for mapper
The phosphor-objmgr repository, which supplies libmapper, now compiles using meson. Add a wrap file for it to allow builds outside of Yocto or an SDK.
Signed-off-by: Pat
meson: add wrap for mapper
The phosphor-objmgr repository, which supplies libmapper, now compiles using meson. Add a wrap file for it to allow builds outside of Yocto or an SDK.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: Ic0c9fa2880a86116e03c5a26b51de4dd032baf4b
show more ...
|
0e7a8af0 | 10-Jun-2022 |
Harvey.Wu <Harvey.Wu@quantatw.com> |
dbus-sdr: fix the vector size in sdrWriteTable and sdrStatsTable
- The original padEntries size is only 0-255 because the input index size is uint8. But if sensor number > 255, the size of padEn
dbus-sdr: fix the vector size in sdrWriteTable and sdrStatsTable
- The original padEntries size is only 0-255 because the input index size is uint8. But if sensor number > 255, the size of padEntries will be different from the size of sensor map, so the result value when using getWritePermission will be incorrect.
TEST: Before: ipmitool raw -l 0 0x04 0x30 0x19 0xff 127 0x00 0x00 0x00 0x00 0x00 0x00 0x00 Unable to send RAW command (channel=0x0 netfn=0x4 lun=0x0 cmd=0x30 rsp=0xce): Command response could not be provided ipmitool raw -l 1 0x04 0x30 0x19 0xff 127 0x00 0x00 0x00 0x00 0x00 0x00 0x00 Unable to send RAW command (channel=0x0 netfn=0x4 lun=0x1 cmd=0x30 rsp=0xce): Command response could not be provided
After: ipmitool raw -l 0 0x04 0x30 0x19 0xff 127 0x00 0x00 0x00 0x00 0x00 0x00 0x00 ipmitool raw -l 1 0x04 0x30 0x19 0xff 127 0x00 0x00 0x00 0x00 0x00 0x00 0x00 Unable to send RAW command (channel=0x0 netfn=0x4 lun=0x1 cmd=0x30 rsp=0xce): Command response could not be provided
Signed-off-by: Harvey.Wu <Harvey.Wu@quantatw.com> Change-Id: Idf6f4f31a1a9d2dd3cc87ebe3a4dc4ca4b283f7c
show more ...
|
4376cdf8 | 16-Nov-2021 |
Harvey.Wu <Harvey.Wu@quantatw.com> |
dbus-sdr: fix SEL record Generator ID to fit SPEC
- According the SEL Event Record format(IPMI SPEC Table 32), the Generator ID has 2 bytes, and LUN is saved at the lowest two bits in the High b
dbus-sdr: fix SEL record Generator ID to fit SPEC
- According the SEL Event Record format(IPMI SPEC Table 32), the Generator ID has 2 bytes, and LUN is saved at the lowest two bits in the High byte.
In the origin code, the LUN is saved in the low byte. It will let the sensor name cannot be found when call sel elist like below: ~# ipmitool sel elist 1 | Pre-Init |0000000063| Power Unit #0x0d | AC lost | Asserted 2 | Pre-Init |0000000071| Power Unit #0x0d | Power off/down | Deasserted ~# ipmitool sel elist -v Loading IANA PEN Registry... Running Get VSO Capabilities my_addr 0x20, transit 0, target 0 Invalid completion code received: Invalid command Discovered IPMB address 0x0 SEL Record ID : 0001 Record Type : 02 Timestamp : 00:01:03 GMT 00:01:03 GMT Generator ID : 0021 EvM Revision : 04 Sensor Type : Power Unit Sensor Number : 0d Event Type : Sensor-specific Discrete Event Direction : Assertion Event Event Data : 04ffff Description : AC lost
SEL Record ID : 0002 Record Type : 02 Timestamp : 00:01:11 GMT 00:01:11 GMT Generator ID : 0021 EvM Revision : 04 Sensor Type : Power Unit Sensor Number : 0d Event Type : Sensor-specific Discrete Event Direction : Deassertion Event Event Data : 00ffff Description : Power off/down
After fixing Generator_Id problem: ~# ipmitool sel elist 1 | Pre-Init |0000000063| Power Unit chassis0 | AC lost | Asserted 2 | Pre-Init |0000000071| Power Unit chassis0 | Power off/down | Deasserted ~# ipmitool sel elist -v Loading IANA PEN Registry... Running Get VSO Capabilities my_addr 0x20, transit 0, target 0 Invalid completion code received: Invalid command Discovered IPMB address 0x0 SEL Record ID : 0001 Record Type : 02 Timestamp : 00:01:03 GMT 00:01:03 GMT Generator ID : 0120 EvM Revision : 04 Sensor Type : Power Unit Sensor Number : 0d Event Type : Sensor-specific Discrete Event Direction : Assertion Event Event Data : 04ffff Description : AC lost
SEL Record ID : 0002 Record Type : 02 Timestamp : 00:01:11 GMT 00:01:11 GMT Generator ID : 0120 EvM Revision : 04 Sensor Type : Power Unit Sensor Number : 0d Event Type : Sensor-specific Discrete Event Direction : Deassertion Event Event Data : 00ffff Description : Power off/down
Signed-off-by: Harvey.Wu <Harvey.Wu@quantatw.com> Change-Id: I3a7745fd3a3040375bc03bef7da7ae84ae22e04c
show more ...
|
6eab420a | 09-Jun-2022 |
Willy Tu <wltu@google.com> |
build: Remove dynamicsensor library if it is not enabled
Use the meson option to not include the dynamiccmds lib if it is disabled.
Without removing this, the dbus-sdr implmentation will always tak
build: Remove dynamicsensor library if it is not enabled
Use the meson option to not include the dynamiccmds lib if it is disabled.
Without removing this, the dbus-sdr implmentation will always take over the other handler implmentation.
Change-Id: I15f51dbd3dcb9e9b78e61f3f6b11991f75b42cc7 Signed-off-by: Willy Tu <wltu@google.com>
show more ...
|
d22ab5b0 | 08-Jun-2022 |
Vernon Mauery <vernon.mauery@linux.intel.com> |
fix header install
The meson changes moved where header files were installed, causing other projects that depend on the headers being present to fail to build.
This installs the header files like t
fix header install
The meson changes moved where header files were installed, causing other projects that depend on the headers being present to fail to build.
This installs the header files like they were before.
Tested: can build the intel-ipmi-oem code again
Change-Id: Icd826b2f264603b8f7e32f7d7eaeb2c319b381cb Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>
show more ...
|
ba9bbb6f | 01-Jun-2022 |
Willy Tu <wltu@google.com> |
build: move all shared_library/module to library with version suffix
The version suffix will create `.so`, `.so.0`, and `.so.0.1` and will be compatiable with existing bitbake setup for phosphor-ipm
build: move all shared_library/module to library with version suffix
The version suffix will create `.so`, `.so.0`, and `.so.0.1` and will be compatiable with existing bitbake setup for phosphor-ipmi-host.
Also moved the dbus-sdr library output the `dbus-sdr` folder since the shared module depends on libipmi20, but it can't find it due to the it being in a different folder. Moving it to the same meson file fixes that issue.
Also removed all unused declare_dependency. Built fine without it.
Tested: dbus-sdr works now ``` ipmitool sdr cpu0 | disabled | ns cpu0_core0 | disabled | ns cpu0_core1 | disabled | ns cpu0_core2 | disabled | ns ... ```
Change-Id: Ib13a79c1b27a7c122c5a29237716898cd2e29b4c Signed-off-by: Willy Tu <wltu@google.com>
show more ...
|
c710b975 | 11-Aug-2021 |
Willy Tu <wltu@google.com> |
build: Add meson build for phosphor-host-ipmi
Create meson and meson_options for the existing features.
Removed sample.h/cpp since that is not used at all.
Tested: Ipmi command works fine the clea
build: Add meson build for phosphor-host-ipmi
Create meson and meson_options for the existing features.
Removed sample.h/cpp since that is not used at all.
Tested: Ipmi command works fine the cleanup. ``` $ systemctl status phosphor-ipmi-host ● phosphor-ipmi-host.service - Phosphor Inband IPMI Loaded: loaded (/lib/systemd/system/phosphor-ipmi-host.service; enabled; vendor preset: enabled) Active: active (running) since Thu 1970-01-08 21:20:56 UTC; 21s ago Main PID: 24987 (ipmid) CGroup: /system.slice/phosphor-ipmi-host.service └─24987 /tmp/ipmid
Jan 08 21:20:56 $HOST ipmid[24987]: Registering OEM:[0X002B79], Cmd:[0X30] for Ethstats Commands Jan 08 21:20:56 $HOST ipmid[24987]: Registering OEM:[0X00C2CF], Cmd:[0X30] for Ethstats Commands Jan 08 21:20:56 $HOST ipmid[24987]: Registering OEM:[0X002B79], Cmd:[0X32] for Sys Commands Jan 08 21:20:56 $HOST ipmid[24987]: Registering OEM:[0X002B79], Cmd:[0X02] for I2C Jan 08 21:20:56 $HOST ipmid[24987]: Registering OEM:[0X00C2CF], Cmd:[0X02] for I2C Jan 08 21:20:56 $HOST ipmid[24987]: I2C_WHITELIST_CHECK is disabled, do not populate whitelist Jan 08 21:20:56 $HOST ipmid[24987]: Registering OEM:[0X00C2CF], Cmd:[0X04] for Manual Zone Control Jan 08 21:20:56 $HOST systemd[1]: Started Phosphor Inband IPMI. Jan 08 21:20:56 $HOST ipmid[24987]: Loading whitelist filter Jan 08 21:20:56 $HOST ipmid[24987]: Set restrictedMode = true (reverse-i-search)`ipmi': systemctl status phosphor-^Cmi-host
$ ipmitool mc info Device ID : 32 Device Revision : 1 Firmware Revision : 0.00 IPMI Version : 2.0 Manufacturer ID : 7244 Manufacturer Name : Quanta Computer Inc. Product ID : 14426 (0x385a) Product Name : Unknown (0x385A) Device Available : yes Provides Device SDRs : yes Additional Device Support : Sensor Device SEL Device FRU Inventory Device Chassis Device Aux Firmware Rev Info : 0x00 0x00 0x00 0x00 ```
Change-Id: Ibaea9eea802b063a0207320654cbb817fde30c2a Signed-off-by: Willy Tu <wltu@google.com>
show more ...
|
11d68897 | 20-Jan-2022 |
Willy Tu <wltu@google.com> |
cleanup: Remove all warning errors for the Meson build support
Remove all the build warning to prepare for the meson build. Meson build will be in https://gerrit.openbmc-project.xyz/c/openbmc/phosph
cleanup: Remove all warning errors for the Meson build support
Remove all the build warning to prepare for the meson build. Meson build will be in https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-host-ipmid/+/47748
Tested: Ipmi command works fine the cleanup. ``` $ systemctl status phosphor-ipmi-host ● phosphor-ipmi-host.service - Phosphor Inband IPMI Loaded: loaded (/lib/systemd/system/phosphor-ipmi-host.service; enabled; vendor preset: enabled) Active: active (running) since Thu 1970-01-08 21:20:56 UTC; 21s ago Main PID: 24987 (ipmid) CGroup: /system.slice/phosphor-ipmi-host.service └─24987 /tmp/ipmid
Jan 08 21:20:56 $HOST ipmid[24987]: Registering OEM:[0X002B79], Cmd:[0X30] for Ethstats Commands Jan 08 21:20:56 $HOST ipmid[24987]: Registering OEM:[0X00C2CF], Cmd:[0X30] for Ethstats Commands Jan 08 21:20:56 $HOST ipmid[24987]: Registering OEM:[0X002B79], Cmd:[0X32] for Sys Commands Jan 08 21:20:56 $HOST ipmid[24987]: Registering OEM:[0X002B79], Cmd:[0X02] for I2C Jan 08 21:20:56 $HOST ipmid[24987]: Registering OEM:[0X00C2CF], Cmd:[0X02] for I2C Jan 08 21:20:56 $HOST ipmid[24987]: I2C_WHITELIST_CHECK is disabled, do not populate whitelist Jan 08 21:20:56 $HOST ipmid[24987]: Registering OEM:[0X00C2CF], Cmd:[0X04] for Manual Zone Control Jan 08 21:20:56 $HOST systemd[1]: Started Phosphor Inband IPMI. Jan 08 21:20:56 $HOST ipmid[24987]: Loading whitelist filter Jan 08 21:20:56 $HOST ipmid[24987]: Set restrictedMode = true (reverse-i-search)`ipmi': systemctl status phosphor-^Cmi-host
$ ipmitool mc info Device ID : 32 Device Revision : 1 Firmware Revision : 0.00 IPMI Version : 2.0 Manufacturer ID : 7244 Manufacturer Name : Quanta Computer Inc. Product ID : 14426 (0x385a) Product Name : Unknown (0x385A) Device Available : yes Provides Device SDRs : yes Additional Device Support : Sensor Device SEL Device FRU Inventory Device Chassis Device Aux Firmware Rev Info : 0x00 0x00 0x00 0x00 ```
Change-Id: I372c9433a274bc633a24d213b7a1bc205531e33d Signed-off-by: Willy Tu <wltu@google.com>
show more ...
|