Skip to content

mctp-client: add cci message type code - #168

Merged
jk-ozlabs merged 1 commit into
CodeConstruct:mainfrom
LeoYangQuanta:main
Aug 18, 2026
Merged

mctp-client: add cci message type code#168
jk-ozlabs merged 1 commit into
CodeConstruct:mainfrom
LeoYangQuanta:main

Conversation

@LeoYangQuanta

Copy link
Copy Markdown
Contributor

According to Specification DSP0239 1.12.0
Add type code 8 for use with the CXL Type 3 Device Component Command Interface over MCTP.

Testing

cci commands

mctp-client net 1 eid 10 type cci data 00 00 00 00 03 00 00 00 00 00 cd ab
01 00 00 00 03 08 00 00 00 00 00 00 2B 01 AD 9C A4 8A D2 16

@jk-ozlabs jk-ozlabs left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change itself looks okay, just a comment on the chosen name here.

However, the commit message just needs a little work. The wording is a little off, and no need for the section break there, nor that Testing line (we assume that you have tested this before submission).

Maybe just:

Add type code 8 for CXL Type 3 Device Component Command Interface,
as defined by DSP0239 1.9.0 and later.

For example:

$ mctp-client eid 10 type cxl-cci data 00 00 00 00 03 00 00 00 00 00 cd ab
01 00 00 00 03 08 00 00 00 00 00 00 2B 01 AD 9C A4 8A D2 16

Comment thread src/mctp-client.c Outdated
Add type code 8 for CXL Type 3 Device Component Command Interface,
as defined by DSP0239 1.9.0 and later.

For example:
```
$ mctp-client eid 9 type cxl-cci data 00 00 00 00 03 00 00 00 00 00 cd ab
01 00 00 00 03 08 00 00 00 00 00 00 E5 52 F9 1E 50 21 D3 16
```

Signed-off-by: Leo Yang <Leo-Yang@quantatw.com>

@jk-ozlabs jk-ozlabs left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

@jk-ozlabs
jk-ozlabs merged commit 42646d1 into CodeConstruct:main Aug 18, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants