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

Extend .blit header with api/device info #822

Merged
merged 5 commits into from
Jun 1, 2024

Conversation

Daft-Freak
Copy link
Collaborator

Some patches from the pico launcher/api stuff. Adds a device id and the api version to the .blit file header, taking advantage of there being an unused constant 00 00 00 90 at the end for backwards compatibility (first byte is repurposed as device id, non-0 means the api info is there).

I have no idea where the name "STM32Blit" came from though. Should probably be something like STH32H7 (existing .blits are built for cortex-m7 and assume STM32 RAM layout, though there isn't a loader for anything that isn't a 32blit...)

@Gadgetoid
Copy link
Contributor

I have no idea where the name "STM32Blit" came from though.

I have no earthly clue, it's not even a nice round number of bytes 😆

@Daft-Freak
Copy link
Collaborator Author

I can only assume it has something to do with not being able to have a number at the start...

(Kinda forgot about this one, it's mostly relevant for the pico launcher stuff... but I'm not entirely sure what my plan for that is...)

@Daft-Freak
Copy link
Collaborator Author

Oh, it does allow checking files for compatibility before launching them as well. (WIP: Daft-Freak@99fb098)

Want to use these in an assembly file
@Daft-Freak
Copy link
Collaborator Author

Okay, now this works:
Launcher0
(Those are pico-blits)

@Daft-Freak Daft-Freak marked this pull request as ready for review May 31, 2024 22:04
@Daft-Freak Daft-Freak merged commit b43a2e3 into 32blit:master Jun 1, 2024
9 checks passed
@Daft-Freak Daft-Freak deleted the blit-header-extend branch June 1, 2024 22:02
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.

2 participants