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

Cleanup all source files #343

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

elral
Copy link
Collaborator

@elral elral commented Nov 1, 2024

Description of changes

This is a major PR but it has only editorial changes. Functionality is not changed, tested by compiling this branch and the main branch. For all boards the hex size hasn't changed.

  1. All files are opend and autoformatted. So each file is affected by this PR
  2. Most files have included mobiflight.h which included severall more .h files. For most files, all include files are not required. So mobiflight.h is deleted and in each other file only the required include files are included.
  3. Board definition is done in two different locations, in MF_Boards.h for each board and platformio.ini. With this PR all board definitions are moved to platformio.ini and all MF_Boards.h files are deleted. All board.json files and reset.hex files are now in one folder. Therefore the copy_fw_files.py is also adapted.
  4. Mixed usage of ifdef and if defined() is commonly changed to if defined().

Copy link

github-actions bot commented Nov 1, 2024

Board and firmware folder for this pull request:
Mobiflight-Connector.zip

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.

1 participant