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

Lighten OMNIBUS (F3) target #3394

Merged
merged 1 commit into from Jun 20, 2018

Conversation

Projects
None yet
2 participants
@shellixyz
Copy link
Collaborator

commented Jun 18, 2018

The OMNIBUS target (F3) is close to flash saturation.

Remove things probably nobody uses: external I2C baro (already has a baro on board) and HCSR04 rangefinder

Before (b665c48) :
Memory region Used Size Region Size %age Used
FLASH: 255408 B 252 KB 98.98%
FLASH_CONFIG: 0 GB 4 KB 0.00%
RAM: 36744 B 40 KB 89.71%
CCM: 7804 B 8 KB 95.26%
MEMORY_B1: 0 GB 0 GB -nan%

After:
Memory region Used Size Region Size %age Used
FLASH: 251352 B 252 KB 97.41%
FLASH_CONFIG: 0 GB 4 KB 0.00%
RAM: 36352 B 40 KB 88.75%
CCM: 7800 B 8 KB 95.21%
MEMORY_B1: 0 GB 0 GB -nan%

Lighten OMNIBUS (F3) target
The OMNIBUS target (F3) is close to flash saturation.

Remove things probably nobody uses: external I2C baro (already has a baro on board) and HCSR04 rangefinder

Before:
Memory region         Used Size  Region Size  %age Used
           FLASH:      255408 B       252 KB     98.98%
    FLASH_CONFIG:          0 GB         4 KB      0.00%
             RAM:       36744 B        40 KB     89.71%
             CCM:        7804 B         8 KB     95.26%
       MEMORY_B1:          0 GB         0 GB      -nan%

After:
Memory region         Used Size  Region Size  %age Used
           FLASH:      251352 B       252 KB     97.41%
    FLASH_CONFIG:          0 GB         4 KB      0.00%
             RAM:       36352 B        40 KB     88.75%
             CCM:        7800 B         8 KB     95.21%
       MEMORY_B1:          0 GB         0 GB      -nan%
@fiam

This comment has been minimized.

Copy link
Member

commented Jun 18, 2018

Just noticed this too. We’re like 4K away from overflow. An easy way to shave 10K is removing some CMS entries. The strings take a lot of flash.

@shellixyz

This comment has been minimized.

Copy link
Collaborator Author

commented Jun 18, 2018

For a start it will shave some out when #3376 is merged

@fiam fiam added this to the 2.0 milestone Jun 20, 2018

@fiam

fiam approved these changes Jun 20, 2018

@fiam fiam merged commit 406e1dc into iNavFlight:development Jun 20, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.