-
Notifications
You must be signed in to change notification settings - Fork 701
Add TWSC NVMe plugin foldernit__.py #2976
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Create new folder for TWSC NVMe plugin
|
Not sure where this is heading using Python. I want to see some more code before accepting this. I mark this PR as draft, please add some more code. Thanks |
Thanks for the feedback! I've added the three implementation files for the TWSC plugin: twsc-nvme.c, twsc-nvme.h, and twsc-types.h. They're ready for your review now. |
|
Sorry, for coming back so late. I was a bit away from the keyboard :) Could you rebase your work to the latest master? There were a bunch of changes in the build system and I think you need to look into this part as it doesn't compile right now. Please also have a look at the checkpatch warnings. FWIW, it looks fairly okay from a quick glance. |
Thank you for the review. I'll work on these points right away. |
|
I though it's done and did some reformatting. First I rebased all changed and let checkpatch fix the errors. After that I spend way too long adding newlines before I realized this plugin needs to be updated against the new APIs. There is no Anyway my attempt of reformatting is here: https://github.com/igaw/nvme-cli/tree/twsc Please update your code accordingly. Thanks. |
This PR adds a new folder for TWSC NVMe device support.