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

Arduino Code is too large for Atmega48 (88+ is ok) #3

Closed
pyr0ball opened this issue Jan 10, 2019 · 3 comments
Closed

Arduino Code is too large for Atmega48 (88+ is ok) #3

pyr0ball opened this issue Jan 10, 2019 · 3 comments
Assignees
Labels
bug Something isn't working wontfix This will not be worked on

Comments

@pyr0ball
Copy link
Owner

Current sketch uses 5900 bytes, well over the 4kb limit of the mega48.

Refactor using low-level code will be required to fit the sketch on the smaller flash size, or switch to mega88+ may be required

@pyr0ball pyr0ball self-assigned this Jan 10, 2019
@pyr0ball pyr0ball added the bug Something isn't working label Jan 10, 2019
@pyr0ball pyr0ball added this to Backlog in Pyr0-Piezo Z-Probe Sensor via automation Feb 5, 2019
@teemuatlut
Copy link

Hey I saw your project on reddit and browsed this github page for a bit. Is this issue still ongoing for you?

I spent a few minutes making changes and with the serial stuff taken out, I got it to 2426B of progmem and 47 of sram.
I suppose it might be possible to include the serial parser as well considering an empty sketch takes 382B and with Serial.begin(9600); added it increased to 1480B.

@pyr0ball
Copy link
Owner Author

pyr0ball commented Mar 3, 2019

I'm actually migrating over to the ATSAMD21E series of MCU's after the beta testing is complete, which has enough memory not to need optimization (though that's not the main reason I'm switching).

Also serial + i2c are going to be required services in the post-beta versions.

I'm trying to keep the code and designs as easy to work with as possible so that other community members can futz around with them.

I do appreciate the offer to help though!

@pyr0ball pyr0ball added the wontfix This will not be worked on label Sep 25, 2019
@pyr0ball
Copy link
Owner Author

Closing as the firmware has definitely reached a point where 4K will never be enough

Pyr0-Piezo Z-Probe Sensor automation moved this from Backlog to Done Sep 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working wontfix This will not be worked on
Projects
Development

No branches or pull requests

2 participants