diff mbox series

[03/19] core/pldm: PLDM over MCTP Binding

Message ID 20220225142848.93747-4-clombard@linux.vnet.ibm.com
State Superseded
Headers show
Series Implement MCTP and PLDM features | expand

Commit Message

Christophe Lombard Feb. 25, 2022, 2:28 p.m. UTC
Enable the mctp binding over LPC bus interface and new wrappers to send
and receive PLDM messages over the mctp library.

PLDM is supported as a message type over MCTP. PLDM over MCTP binding
defines the format of PLDM over MCTP messages.

An MCTP Endpoint is the terminus for MCTP communication. A physical device
that supports MCTP may provide one or more MCTP Endpoints. Endpoints are
addressed using a logical address called the Endpoint ID, or EID. EIDs in
MCTP are analogous to IP Addresses in Internet Protocol networking.

The BMC EID default is 8.

First byte of the PLDM over MCTP Message Fields identifies the MCTP
message as carrying a PLDM message:
Message Type (7 bits)  PLDM = 0x01 (000_0001b).

Signed-off-by: Christophe Lombard <clombard@linux.vnet.ibm.com>
---
 core/Makefile.inc | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

Comments

Christophe Lombard March 1, 2022, 9:08 a.m. UTC | #1
Le 25/02/2022 à 15:28, Christophe Lombard a écrit :
> Enable the mctp binding over LPC bus interface and new wrappers to send
> and receive PLDM messages over the mctp library.
>
> PLDM is supported as a message type over MCTP. PLDM over MCTP binding
> defines the format of PLDM over MCTP messages.
>
> An MCTP Endpoint is the terminus for MCTP communication. A physical device
> that supports MCTP may provide one or more MCTP Endpoints. Endpoints are
> addressed using a logical address called the Endpoint ID, or EID. EIDs in
> MCTP are analogous to IP Addresses in Internet Protocol networking.
>
> The BMC EID default is 8.
>
> First byte of the PLDM over MCTP Message Fields identifies the MCTP
> message as carrying a PLDM message:
> Message Type (7 bits)  PLDM = 0x01 (000_0001b).
>
> Signed-off-by: Christophe Lombard <clombard@linux.vnet.ibm.com>
> ---
>   core/Makefile.inc | 6 +++++-
>   1 file changed, 5 insertions(+), 1 deletion(-)
Hum. The patch was poorly built. There is a mix with patch 04.
To be reviewed. Will be re-done in v2
> diff --git a/core/Makefile.inc b/core/Makefile.inc
> index f80019b6..263a0e50 100644
> --- a/core/Makefile.inc
> +++ b/core/Makefile.inc
> @@ -22,8 +22,12 @@ endif
>
>   CORE=core/built-in.a
>
> +ifeq ($(CONFIG_PLDM),1)
> +include $(SRC)/core/pldm/Makefile.inc
> +endif
> +
>   CFLAGS_SKIP_core/relocate.o = -pg -fstack-protector-all
>   CFLAGS_SKIP_core/relocate.o += -fstack-protector -fstack-protector-strong
>   CFLAGS_SKIP_core/relocate.o += -fprofile-arcs -ftest-coverage
>
> -$(CORE): $(CORE_OBJS:%=core/%)
> +$(CORE): $(CORE_OBJS:%=core/%) $(PLDM)
diff mbox series

Patch

diff --git a/core/Makefile.inc b/core/Makefile.inc
index f80019b6..263a0e50 100644
--- a/core/Makefile.inc
+++ b/core/Makefile.inc
@@ -22,8 +22,12 @@  endif
 
 CORE=core/built-in.a
 
+ifeq ($(CONFIG_PLDM),1)
+include $(SRC)/core/pldm/Makefile.inc
+endif
+
 CFLAGS_SKIP_core/relocate.o = -pg -fstack-protector-all
 CFLAGS_SKIP_core/relocate.o += -fstack-protector -fstack-protector-strong
 CFLAGS_SKIP_core/relocate.o += -fprofile-arcs -ftest-coverage
 
-$(CORE): $(CORE_OBJS:%=core/%)
+$(CORE): $(CORE_OBJS:%=core/%) $(PLDM)