openbmc / phosphor-dbus-interfaces

YAML descriptors of standard dbus interfaces

Apache License 2.0

39 stars 65 forks source link

mctpd integrate miss dependency #16

Closed chenhongbo2022 closed 2 months ago

chenhongbo2022 commented 2 years ago

Compile mctpd miss dependency，this open source depends the third library phosphor-dbus-interfaces，such as：

include <xyz/openbmc_project/MCTP/Base/server.hpp>

include <xyz/openbmc_project/MCTP/SupportedMessageTypes/server.hpp>

But I did't find the corresponding file from phosphor-dbus-interfaces，Where can i get the dependent files？ Am i looking for the right one？Not uploaded to openbmc GitHub？

I want to integrate thie to BMC. How should i integrate mctpd to bmc?

williamspatrick commented 2 years ago

We don’t have any mctpd that I am aware of. Can you point me to this “open source”. If it is something in another organization, they have probably hijacked our project dbus interface names without contributing them back to us.

rebeccasf commented 2 months ago

Hi, I had the same issue when building https://github.com/Intel-BMC/mctpd, I think this repo was outdated? Or does the implementation moved to somewhere within openbmc?

williamspatrick commented 2 months ago

The recommendation is to use the in-kernel MCTP subsystem and the userspace daemon from CodeConstruct. I don't know what the Intel mctpd was doing. The CodeConstruct one is for endpoint management only and not part of the protocol transport path.

amboar commented 2 months ago

Specifically, the userspace daemon @williamspatrick mentions is https://github.com/CodeConstruct/mctp. This is automatically installed in your image if you enable the mctp in DISTRO_FEATURES in OpenBMC. Enabling the mctp distro feature is best done by adding require require conf/distro/include/mctp.inc (https://github.com/openbmc/openbmc/blob/master/meta-phosphor/conf/distro/include/mctp.inc) to your platform config.