Skip to content

rpi led and serial#218

Merged
SamTebbs33 merged 21 commits intoZystemOS:feature/aarch64-portfrom
markfirmware:rpi-led-serial
Sep 5, 2020
Merged

rpi led and serial#218
SamTebbs33 merged 21 commits intoZystemOS:feature/aarch64-portfrom
markfirmware:rpi-led-serial

Conversation

@markfirmware
Copy link

@markfirmware markfirmware commented Aug 5, 2020

zig build -Dtarget=aarch64-freestanding

Then copy rpi-sdcard.img to an unused sd card and insert in 3b+

Will flash rpi3b+ activity led quickly on exception. Pins 14/15 have serial tx/rx.

@SamTebbs33 SamTebbs33 self-requested a review August 6, 2020 18:17
Copy link
Collaborator

@SamTebbs33 SamTebbs33 left a comment

Choose a reason for hiding this comment

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

Good job on getting it working in hardware! You're not going to like me for requesting this but could you add doc comments above each function, struct, struct field and public variable? Documentation is important :) Have a look at src/kernel/filesystem/vfs.zig for inspiration.

Copy link
Author

@markfirmware markfirmware left a comment

Choose a reason for hiding this comment

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

Some of the comments can be resolved. @SamTebbs33

@SamTebbs33
Copy link
Collaborator

Other then the comments from my latest review, it looks like all that is needed is some documentation and comments in initSerial.

@markfirmware
Copy link
Author

Other then the comments from my latest review, it looks like all that is needed is some documentation and comments in initSerial.

Yes including a few questions that you made that I have not yet answered.

@SamTebbs33
Copy link
Collaborator

Thanks for cleaning up the image creation code in build.zig 👍

@SamTebbs33
Copy link
Collaborator

Thanks for replacing the wget and bash 👍 Including the firmware files here means that we need to replicate their copyright notices in the README, but I can do that before finishing the port. I think there are still a few unresolved comments left then we can go ahead and merge.

@markfirmware
Copy link
Author

I have added the initSerial comments and removed the now unnecessary assembly code for the led.

I am now starting the /// commentary.

@markfirmware
Copy link
Author

@SamTebbs33 Please check 997d157 for satisfaction.

Copy link
Collaborator

@SamTebbs33 SamTebbs33 left a comment

Choose a reason for hiding this comment

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

Thanks @markfirmware 👍. It's great to have your changes to make it work on real hardware!

@SamTebbs33 SamTebbs33 merged commit 51f9d72 into ZystemOS:feature/aarch64-port Sep 5, 2020
@markfirmware markfirmware deleted the rpi-led-serial branch September 7, 2020 17:37
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