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

Build firmware with PlatformIO, enable CI #4

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

maxgerhardt
Copy link

@maxgerhardt maxgerhardt commented Nov 4, 2022

To make SMC firmware builds reproducable so that not only the original author can compile it with their computer, I added PlatformIO configuration file (platformio.ini) to the project.

It enables the firmware code to be built for the Uno (ATmega328P) and ATTiny861 using ATTinyCore 2.0.0-dev with debugging enabled and disabled (USE_SERIAL_DEBUG) and the NMI button enabled or disabled (ENABLE_NMI_BUT).

It also enables Github Actions to build the firmware and archive the results (all .elf and .hex files).

Not tested on real hardware because I have none.

Sources the ATTiny 2.0.0-dev package from the PlatformIO trusted registry.

Example CI run: see here.

grafik

@maxgerhardt maxgerhardt changed the title Build firmware with PlatformIO, enable CI [WIP] Build firmware with PlatformIO, enable CI Nov 4, 2022
@maxgerhardt maxgerhardt changed the title [WIP] Build firmware with PlatformIO, enable CI Build firmware with PlatformIO, enable CI Nov 5, 2022
@maxgerhardt
Copy link
Author

PR is ready for review + merge.

I've also learnt of the fork https://github.com/jburks/x16-smc/tree/latest. If this code is merged, you can remove the lib_deps = mathertel/[email protected] line in the platformio.ini, since that version has pulled the (modified) library directly into the repo.

@maxgerhardt
Copy link
Author

..Hello?

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.

1 participant