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

PS/2 Rewrite #57

Open
wants to merge 6 commits into
base: development
Choose a base branch
from
Open

PS/2 Rewrite #57

wants to merge 6 commits into from

Conversation

Gegy
Copy link
Member

@Gegy Gegy commented May 9, 2018

Putting this here for visibility and tracking

@Restioson
Copy link
Member

Shoot. I still have some keyboard enhancements stuff. I'll push them soon.

@Restioson Restioson assigned Restioson and Gegy and unassigned Restioson May 15, 2018
@Restioson Restioson self-requested a review May 15, 2018 05:25
@Restioson
Copy link
Member

In line with #37 we should note that keyboard key ids are only loosely based on row/cols, or whatever we choose as a solution.

@Restioson
Copy link
Member

@gegy1000 acpi is merged!

Copy link
Member

@Restioson Restioson left a comment

Choose a reason for hiding this comment

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

A nit

kernel/src/drivers/keyboard/mod.rs Show resolved Hide resolved
@Gegy Gegy changed the title [NOT READY FOR MERGE] PS/2 Rewrite [WIP] PS/2 Rewrite Oct 9, 2018
@Gegy
Copy link
Member Author

Gegy commented Oct 9, 2018

Status update on this for visibility, we're going to implement the simpler PIC and PIT instead of APIC and HPET. These will still be implemented in the future, but for the purposes of this PR we will go with the simpler alternative. I'm currently planning to work on those, though can only start next week. This will hopefully mean that PS/2 support can be finally completed.

@Gegy Gegy changed the title [WIP] PS/2 Rewrite PS/2 Rewrite Apr 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants