Skip to content

nvme: use to show sanitize ns specific status code string#3056

Closed
ikegami-t wants to merge 1 commit intolinux-nvme:masterfrom
ikegami-t:show-error-status
Closed

nvme: use to show sanitize ns specific status code string#3056
ikegami-t wants to merge 1 commit intolinux-nvme:masterfrom
ikegami-t:show-error-status

Conversation

@ikegami-t
Copy link
Copy Markdown
Contributor

Since the status code is duplicated with the fw-commit status code.

Since the status code is duplicated with the fw-commit status code.

Signed-off-by: Tokunori Ikegami <[email protected]>
@igaw
Copy link
Copy Markdown
Collaborator

igaw commented Jan 12, 2026

I don't follow. There is no single user of nvme_show_opcode_status. Shouldn't we remove this function and add the feature to the nvme_show_err function you are working on?

@ikegami-t
Copy link
Copy Markdown
Contributor Author

Okay will do that. (But still considering as only nvme_show_opcode_status needs the paramters bool admin and __u8 opcode but for others not necessary for now so how to merge into the nvme_show_err.)

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