Skip to content
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

Add printer gadget and example #13

Merged
merged 10 commits into from
Dec 6, 2024
Merged

Add printer gadget and example #13

merged 10 commits into from
Dec 6, 2024

Conversation

tuna-f1sh
Copy link
Contributor

Last one from me I think!

Gadget is fairly simple. Included example based on prn_example in kernel docs. I put some const I used in the example that I think are useful in the lib too; ioctl ID bytes.

Copy link
Owner

@surban surban left a comment

Choose a reason for hiding this comment

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

Minor changes.

examples/printer.rs Outdated Show resolved Hide resolved
examples/printer.rs Outdated Show resolved Hide resolved
examples/printer.rs Outdated Show resolved Hide resolved
examples/printer.rs Outdated Show resolved Hide resolved
src/function/printer.rs Outdated Show resolved Hide resolved
src/function/printer.rs Outdated Show resolved Hide resolved
src/function/printer.rs Show resolved Hide resolved
@surban surban self-assigned this Dec 5, 2024
@surban surban added the enhancement New feature or request label Dec 5, 2024
@tuna-f1sh
Copy link
Contributor Author

Thanks. I've fed these changes back in.

@surban
Copy link
Owner

surban commented Dec 6, 2024

Thanks.

Are there any gadgets left unimplemented? ;-)

@surban surban merged commit 340c514 into surban:master Dec 6, 2024
1 check passed
@tuna-f1sh
Copy link
Contributor Author

Haha, I don't think so! Thanks for your help and support on these though. Learnt some things.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants