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

Possibility to complete disable CROSSFIRE / ELRS V2 / EXT-MODULE #176

Open
wimalopaan opened this issue Feb 27, 2022 · 4 comments
Open

Possibility to complete disable CROSSFIRE / ELRS V2 / EXT-MODULE #176

wimalopaan opened this issue Feb 27, 2022 · 4 comments

Comments

@wimalopaan
Copy link

To rediuce the flash and RAM size it should be possible to completely disable these two features.

I think this would be a good feature for user that do use the radios in a classical way.

@wimalopaan wimalopaan changed the title Possibility to complete disable CROSSFIRE / ELRS V2 Possibility to complete disable CROSSFIRE / ELRS V2 / EXT-MODULE Feb 27, 2022
@ajjjjjjjj
Copy link
Member

Elrs is already isolated, don't know about crossfire for sure.
What do you want that space to be used for?

@wimalopaan
Copy link
Author

As I understand, ELRS need CRSF. I just found, that deactivating PCBI6X_ELRSV2 does not remove the calls to the elrs/crsf stuff. So removing elrsV2.cpp and crossfire.cpp leads to undefined references.

The idea behind removing the support for ext. module was just, that there is no ext. module bay. So using an external module need some (minor) HW mod (at least find the pins), and that makes it the same king of mod as soldering an BT module.
I have some code I want to use for a simple protocol for the HolyBro SikV3 Modem.

@ajjjjjjjj
Copy link
Member

ajjjjjjjj commented Feb 28, 2022

PCBI6X_ELRSV2 removes all elrsv2 hooks for sure. CROSSFIRE define should also work. You may want to disable PCBI6X_USB_MSD which is also very resource consuming + remove declaration of uint8_t MSC_BOT_Data[MSC_MEDIA_PACKET] to save 0,5KB RAM. No need to delete files.
Full screen channels monitor is using about 1KB of FLASH space, can be define guarded.

How much space are you looking for?

@wimalopaan
Copy link
Author

wimalopaan commented Mar 1, 2022

Maybe you might habe a look a #178

Flash used: 113040 / 131072 (86%), left: 18032
RAM used: 15620 / 16384 (95%), left: 764

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

No branches or pull requests

2 participants