Fix spelling mistakes using codespell

This commit corrects various spelling mistakes throughout the
repository. The corrections were made automatically using `codespell`[1]
tool.

[1]: https://githu

Fix spelling mistakes using codespell

This commit corrects various spelling mistakes throughout the
repository. The corrections were made automatically using `codespell`[1]
tool.

[1]: https://github.com/codespell-project/codespell

Change-Id: I0d3acfeb002b6e3e8356b4f1c2a960c53ecadf2e
Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>

show more ...

Update language in ipmbbridge to be more inclusive

Use more inclusive terminology inside of ipmbbridge repo.

Tested: No functional change. Verified on Intel system that ipmb.service
runs and Ipmb c

Update language in ipmbbridge to be more inclusive

Use more inclusive terminology inside of ipmbbridge repo.

Tested: No functional change. Verified on Intel system that ipmb.service
runs and Ipmb channel still gets published on DBus.

Change-Id: I70577a3b28e209a38bcd3d2ec99dd17b09a2d723
Signed-off-by: Matt Simmering <matthew.simmering@intel.com>

show more ...

clean up missing-field-initializers warnings

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

fix different signedness comparison

../ipmbbridged.cpp:368:12: warning: comparison of integer expressions of different signedness: ‘int’ and ‘const size_t’ {aka ‘const long unsigned int’} [-Wsign-co

fix different signedness comparison

../ipmbbridged.cpp:368:12: warning: comparison of integer expressions of different signedness: ‘int’ and ‘const size_t’ {aka ‘const long unsigned int’} [-Wsign-compare]

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

show more ...

simplify sequence number processing

Newer GCC didn't like the complexity of the sequence calculation,
so simplify it.

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

simplify sequence number processing

Newer GCC didn't like the complexity of the sequence calculation,
so simplify it.

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

show more ...

clang-format: update with latest

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

Change io_service to io_context

This was renamed a while back in boost to be compliant with the
std::executors proposal.

Change-Id: I2bda7bf0e3e010a4162073b579bb744901597062
Signed-off-by: Ed Tanou

Change io_service to io_context

This was renamed a while back in boost to be compliant with the
std::executors proposal.

Change-Id: I2bda7bf0e3e010a4162073b579bb744901597062
Signed-off-by: Ed Tanous <edtanous@google.com>

show more ...

Ipmbbridge: Add support for multiple sub channel.

Added Multiple sub channels support for ipmb and me
type to communicate with more than one device of same type.

ChannelType sup

Ipmbbridge: Add support for multiple sub channel.

Added Multiple sub channels support for ipmb and me
type to communicate with more than one device of same type.

ChannelType supports only two channels ME and ipmb.
Using combination of devIndex and channelType approach,
can support multiple sub channels without affecting the
existing functionalities.

Single host or channel, "devIndex" will be set default index as 0.

For multiple channel under same type need to provide devIndex in
json configurations. This approach uses the sendRequest method's
first parameter values for MSB 6 bits as "devIndex" and
the remaining 2 bits as "channelType".

TESTED : Built Facebook YosemiteV2 & Tiogapass images and loaded on
the target hardware. We are able to see BMC to all
host communications through Ipmb.

Signed-off-by: Kumar Thangavel <thangavel.k@hcl.com>
Change-Id: I20a7231993a0dd1f9d396dde114837380302dc76

show more ...

Fix includes

sdbusplus dropped the asio.hpp #include, which this daemon was relying
on. Fix to only include the things we need.

Tested:
Code builds. Do not have system to

Fix includes

sdbusplus dropped the asio.hpp #include, which this daemon was relying
on. Fix to only include the things we need.

Tested:
Code builds. Do not have system to test ipmb functionality, but
unlikely to be broken.

Signed-off-by: Ed Tanous <ed@tanous.net>
Change-Id: I4bdd097bd3509f9b08e2863c2421108aa0b0c658

show more ...

Customize ipmbbridge as per upstream IPMB driver

Mqueue slave driver is not upstream and there is a new IPMB
driver which is upstream in kernel and this application has
been modified

Customize ipmbbridge as per upstream IPMB driver

Mqueue slave driver is not upstream and there is a new IPMB
driver which is upstream in kernel and this application has
been modified as per driver packet format.

Also using same node for sending IPMB data to device. No need to
define master device.

Signed-off-by: Vijay Khemka <vijaykhemka@fb.com>
Change-Id: I8286c41970f680aae2c452331739fa934e41fd4a

show more ...

Support broadcast message

1. Support send/recv broadcast message
2. Fix an issue of inconsistent addr between src addr of incoming message
and target addr of response

Tested

Support broadcast message

1. Support send/recv broadcast message
2. Fix an issue of inconsistent addr between src addr of incoming message
and target addr of response

Tested:
1. CMC broadcast message can be sent and received (BNP)
2. IPMI command injected to i2c bus0 via IPMB header can return correct value (WFP)

Change-Id: Ic03ee4ef6552a40d20594c4abb337f71ae5763eb
Signed-off-by: Qiang XU <qiang.xu@linux.intel.com>

show more ...

Support dynamically update BMC slave address

On multi-node platform, the BMC slave address is determined by node ID,
so the BMC slave address needs to be updated after node ID detection.

Support dynamically update BMC slave address

On multi-node platform, the BMC slave address is determined by node ID,
so the BMC slave address needs to be updated after node ID detection.
This patch makes slave-mqueue configurable at run time using 'new_device'
and 'delete-device' sysfs interfaces.
The sample to change BMC slave addr:
(Assume original slave addr is 0x20, but it should be 0x22)
echo "0x1010" > /sys/bus/i2c/devices/i2c-0/delete_device
echo "slave-mqueue 0x1011" > /sys/bus/i2c/devices/i2c-0/new_device

Dependency:
[DTS] remove ipmb0 node to support dynamically update BMC slave address

Tested:
1. The I2C slave message can be received correctly after CMC sends signal
of "updateBmcSlaveAddr" (BNP)
2. Check sys file of slave device exists by command like
"ls -l /sys/bus/i2c/devices/0-1011/slave-mqueue" (BNP)
3. IPMI command injected to bus0 via IPMB header can return correct value (WFP)

Change-Id: Ie55715cf940f19e10d265ae7efec4d4bf55744a2
Signed-off-by: Qiang XU <qiang.xu@linux.intel.com>

show more ...

Make use of new IPMI method call API

tested: checked communication by injecting I2C traffic via aardvark,
everything is working fine
aardvark<->I2C<->IPMB<->IPMI

Change-Id:

Make use of new IPMI method call API

tested: checked communication by injecting I2C traffic via aardvark,
everything is working fine
aardvark<->I2C<->IPMB<->IPMI

Change-Id: Ie2a982e710473cfa594838328730b98affc2cb39
Signed-off-by: Dawid Frycki <dawid.frycki@intel.com>

show more ...

Read channel configuration from JSON file

Read channel information from /usr/share/ipmbbridge/ipmb-channels.json
allowing platforms to override this information.

Signed-off-by:

Read channel configuration from JSON file

Read channel information from /usr/share/ipmbbridge/ipmb-channels.json
allowing platforms to override this information.

Signed-off-by: Amithash Prasad <amithash@fb.com>
Change-Id: I161cf72840ff211c270fe56a86e6d3ebd65f170c

show more ...

ipmbbridge initial implementation

ipmbbridge gives the ability for openbmc to host ipmb transactions over
smbus

Change-Id: I3744dbef5a6db0b2ff4f2b691e68ca8dc3b1d24b
Signed-o

ipmbbridge initial implementation

ipmbbridge gives the ability for openbmc to host ipmb transactions over
smbus

Change-Id: I3744dbef5a6db0b2ff4f2b691e68ca8dc3b1d24b
Signed-off-by: Dawid Frycki <dawid.frycki@intel.com>
Signed-off-by: Ed Tanous <ed.tanous@intel.com>

show more ...