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

ESP8266 miner ESP01 compatibility #1581

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

mlesniew
Copy link

So far the official ESP8266 miner wouldn't run on ESP-01 because the result binary was to big. Disabling features reduced the result binary size, but not sufficiently to fit onto the 500k flash.

This PR fixes this by using the preprocessor to disable unneeded features.

Besides features, which have already been configurable (using global variables), this PR introduces the USE_OTA macro. If it's not defined, OTA will be disabled.

I was able to reduce the binary size to around 400k by disabling:

  • the web dashboard (comment out #define WEB_DASHBOARD)
  • OTA upgrades (comment out #define USE_OTA)
  • DHT support (comment out #define USE_DHT)
  • MQTT support (comment out #define USE_MQTT)

@mlesniew mlesniew mentioned this pull request Nov 21, 2022
@revoxhere
Copy link
Owner

Please make sure it doesn't break the official configurator found on the website.

@revoxhere revoxhere self-assigned this Nov 28, 2022
@revoxhere revoxhere added the enhancement New feature or request label Nov 28, 2022
@revoxhere revoxhere added the in progress This is currently being worked on label May 3, 2023
@revoxhere revoxhere added postponed May be done during better times and removed in progress This is currently being worked on enhancement New feature or request labels Jul 10, 2023
@revoxhere revoxhere added enhancement New feature or request in progress This is currently being worked on and removed postponed May be done during better times labels Nov 30, 2023
@revoxhere revoxhere added postponed May be done during better times and removed in progress This is currently being worked on labels Dec 16, 2023
@revoxhere
Copy link
Owner

Hello, if you're still interested in pushing these changes to main codebase, you'll need to update the changes to the all-new and universal ESP Code released with version 4.0

@ffrediani
Copy link

@mlesniew this improvement seems interesting for those boards with 512kB flash. Are you planning to review the pending issues so the changes can get trough the main codebase.

I have found recently and be aware that ESP01 has at least 3 variants: 1) Blue ESP-01 with 512kB of flash, 2) Black ESP-01 with 1MB of flash and 3) Black ESP-01S both with 1M of flash. I am not entirely sure how to differentiate the Blue and Black versions in in the code for building proposes.

@mlesniew
Copy link
Author

Hey, @ffrediani. To be honest, I had completely forgotten about this pull request. I briefly explored DuinoCoin around 1.5 years ago just to try it out, and this PR was the result of my experimenting.

Unfortunately, completing this PR requires a lot more work, as noted in the comments above. I never found the time to follow through, and my interest in the project has waned since then.

If you'd like to continue the work I've started, please feel free to do so. I'm happy to support you and help where I can.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request postponed May be done during better times
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants