On Tue, 9 Sep 2025 14:32:57 +1000
Wilfred Mallawa <wilfred.opensou...@gmail.com> wrote:
> From: Wilfred Mallawa <wilfred.mall...@wdc.com>
>
> This header contains the transport encoding for an SPDM message that
> uses the SPDM over Storage transport as defined by the DMTF DSP0286.
>
> Signed-off-by: Wilfred Mallawa <wilfred.mall...@wdc.com>
> ---
> include/system/spdm-socket.h | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/include/system/spdm-socket.h b/include/system/spdm-socket.h
> index 29aa04fd52..c5f9f3611c 100644
> --- a/include/system/spdm-socket.h
> +++ b/include/system/spdm-socket.h
> @@ -88,6 +88,18 @@ bool spdm_socket_send(const int socket, uint32_t
> socket_cmd,
> */
> void spdm_socket_close(const int socket, uint32_t transport_type);
>
> +/**
This isn't the formal doc style, so /** not appropriate.
/* for a normal comment. This matches kernel style (same scripting is used).
Otherwise seems fine but I haven't checked the specs so no tag.
> + * Defines the transport encoding for SPDM, this information shall be passed
> + * down to the SPDM server, when conforming to the SPDM over Storage standard
> + * as defined by DSP0286.
> + */
> +typedef struct {
> + uint8_t security_protocol; /* Must be 0xE8 for SPDM
> Commands */
> + uint16_t security_protocol_specific; /* Bit[7:2] SPDM Operation
> + Bit[0:1] Connection ID */
> + uint32_t length; /* Length of the SPDM Message*/
> +} QEMU_PACKED StorageSpdmTransportHeader;
> +
> #define SPDM_SOCKET_COMMAND_NORMAL 0x0001
> #define SPDM_SOCKET_STORAGE_CMD_IF_SEND 0x0002
> #define SPDM_SOCKET_STORAGE_CMD_IF_RECV 0x0003
