-
Notifications
You must be signed in to change notification settings - Fork 357
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
OLS cleanups #84
OLS cleanups #84
Conversation
It's nowhere documented and nowhere used.
…ure. This is a preparation to introduce "device flags" later that describe the device itself, not a specific operation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All my comments from PR #51 have been addressed. This subset is ready to go IMO.
I can pick up these improvements. @wsakernel shall I add your Acked-By? |
Technically, it would be more Reviewed-by. But I don't mind much which one... Thanks! |
Merged in 6f9234e, thank you. Have added wsa's Reviewed-By. Had to adjust the commit messages. Would be nice if your future commits would stick with the project's habit of prefixing the subject with the driver name. Makes it easier to navigate later during research. |
@gsigh: Sure, will do! Thanks for telling me. |
These clean-ups are taken from #51 and are the base to further refactor the driver and add support for advanced features of the DemonCore.