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

Update keyboard.c #216

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hyunjun1234
Copy link

function to read keyboard input

I want to know the exact request!!
@dreamos82
Copy link
Owner

Hi thanks for the PR, i did a quick review, and i think how it is implemented it is not working.

So far the kernel doesn't have scanf fgets and similar functions, they aren't implemented yet (my bad).

Also for the output probably better to us pretty_logf (for logging into serial) and _fb_printStrAndNumber (i don't remember the exact right name right now) if we want to read a line, we need to implement the logic for it, there is a getCh function that is reading a character, so maybe this can be used to implement a getString logic, for now i would avoid using scanf and fscanf, since we are far from implementing them, you can probably implement the getString and for now get away with it.

I also would suggest if possible to check that it compiles and works at least on qemu. In the docs folder you should find the build instructions.

Let me know if you want to try and implement it, and feel free to ask any question i will be happy to help!

Thanks,
Ivan

@hyunjun1234
Copy link
Author

hyunjun1234 commented Jun 6, 2024 via email

@inuyasha82
Copy link

inuyasha82 commented Jun 6, 2024

Yes, for now yes. (it's still me @dreamos82 from another account!)

@hyunjun1234
Copy link
Author

hyunjun1234 commented Jun 6, 2024 via email

@inuyasha82
Copy link

So i strongly suggest first to set-up a cross-compile environment on your machine, you can follow this guide here; First you : https://github.com/dreamportdev/Osdev-Notes/blob/master/99_Appendices/E_Cross_Compilers.md, if you never done it i strongly advise to use llvm and clang, since it should be easier.

Then you should install qemu too if you don't have it already installed.

This is the first time i'm guiding someone else through the process of building Dreamos, so it could be also a good help for me to see if the documentation and the build process is straightforward.

If you already have gcc or clang, and qemu let me know and we can do couple of tests to check if it build on a default non cross-dev environment.

There is also the section on how to build dreamos (if you have everything intsalled) in the Readme: https://github.com/dreamos82/Dreamos64?tab=readme-ov-file#compiling-and-running-the-os, try to see if it is clear to you (also in the Building Document pointed on that section make sure to read through the psf part, this step needs to be done) if you want to be able to compile it.

Feel free to ask any question, as i said, it is a good exercise also for the documentation i wrote.

@hyunjun1234
Copy link
Author

hyunjun1234 commented Jun 6, 2024 via email

@dreamos82
Copy link
Owner

i've enabled the dreamos workflow builds on your pr, now you should be able to check at least if it compiles when making changes.

@dreamos82
Copy link
Owner

ALso if you have problems in finding a psf font, let me know i'll send you the file i'm using for dreamos.

@hyunjun1234
Copy link
Author

hyunjun1234 commented Jun 11, 2024 via email

@dreamos82
Copy link
Owner

You can ask any question if you face any problem, or anything is not clear from the documentation, this help me also in writing a better documentation.

And lastly thanks to you in taking time to contribute to my small project, i really appreciate it.

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.

3 participants