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

PlatformIO support #4

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

maxgerhardt
Copy link

Adds a builder script and package.json metadata so that we can easily include and work with it in PlatformIO.

Already running nicely in https://github.com/Community-PIO-CH32V/platform-ch32v/tree/develop/examples/blinky-arduino-ch32v003.

grafik

Keeping this as a draft because I see this core as fast changing and I don't wanna have the build options get out of sync quickly.

I also deviate with not using -flto because it actually results in larger binary sizes with the propretiary toolchain. But even so, a lot size optimization potential is still there when looking at the included functions and objects in the map file (Serial) is there even though not used in sketch, IPAddress conversion / print functions, ... :()

@maxgerhardt maxgerhardt marked this pull request as draft May 22, 2023 01:22
@AlexanderMandera
Copy link
Owner

Hi, thank you for your contribution! This looks very promising.

When ch32v003fun is included as a submodule to get all upstream changes,
I will continue to work on the optimizations.

I will definitely take a closer look at LTO, the unused functions and more optimization inside the GPIO code as well.

@maxgerhardt maxgerhardt changed the base branch from i2c to master November 27, 2024 18:04
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