History log of /openbmc/phosphor-host-ipmid/ (Results 226 – 250 of 1339)
Revision Date Author Comments
(<<< Hide modified files)
(Show modified files >>>)
5d82f47422-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 ...

7d7a685825-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 ...

b69b206625-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 ...

37b9f41422-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 ...

bfd3a17f31-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 ...

bef102a309-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 ...

1a2e150607-Jul-2022 George Liu <liuxiwei@inspur.com>

Fix project name misspelling

Signed-off-by: George Liu <liuxiwei@inspur.com>
Change-Id: I11a2565cdf424fc98a193fe722643efdf8ae9424

ff1b4bf128-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 ...

f5684a1201-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 ...

d09db49229-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 ...

b490591917-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 ...

8d90f97504-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 ...

25035b8816-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 ...

bbc6ad4816-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 ...

3d8d793f16-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 ...

c8fc728f16-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 ...

422385da16-Jun-2022 Patrick Williams <patrick@stwcx.xyz>

meson: simplify sdbusplus dependency

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: If34a35bd8984c5ef54fb88f2805570748cc08515

6b4cf43516-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 ...

0e7a8af010-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 ...

4376cdf816-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 ...

6eab420a09-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 ...

d22ab5b008-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 ...

ba9bbb6f01-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 ...

c710b97511-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 ...

11d6889720-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 ...

12345678910>>...54