Skip to content

Conversation

@ikegami-t
Copy link
Contributor

Added for NVMe 2.3 commands support.

@ikegami-t ikegami-t force-pushed the nvme-2.3-def branch 3 times, most recently from acd7603 to f691b99 Compare December 30, 2025 12:11
nvme-print.c Outdated
case NVME_LOG_LID_MEDIA_UNIT_STATUS:
return "Media Unit Status";
case NVME_LOG_LID_SUPPORTED_CAP_CONFIG_LIST:
return "Supported Capacity Configuration List";
Copy link
Collaborator

Choose a reason for hiding this comment

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

These two changes are unrelated to the change. Just whitespace changes. I suggest to keep the current formatting for this section consistent, so all in one line. Having different formatting styles is worse than the checkpatch warning.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fix the changes as mentioned. Thank you.

@igaw
Copy link
Collaborator

igaw commented Jan 2, 2026

Please update the formatting, the rest looks good.

Added for NVMe 2.3 commands support.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>
Supported by NVMe 2.3.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>
Since the status code is duplicated with the fw-commit status code.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>
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