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

Flysky PL18/PL18EV #2987

Closed
wants to merge 181 commits into from
Closed

Flysky PL18/PL18EV #2987

wants to merge 181 commits into from

Conversation

gagarinlg
Copy link
Member

Summary of changes:
Adds a file system abstraction layer
adds preliminary PL18 support

rotorman and others added 30 commits November 13, 2022 20:22
Removed SD presence detection to get Trim 2 Up button also free, assuming SD card is always there.
Added trim 1 and 2 sampling plus emulated keys from physical trim buttons 3 to 8.
Trim 6 up == ENTER
Trim 6 down == RTN/Exit
… analogs diagnose page.

Unify the variable name used to display touch chip firmware version with GT911.
TODO from the analogs are the VRC and both side sliders.
After these changes, the radio behaves awkwardly (e.g. start screen has no sliders & trims).
I had to extend the memory size in radio/src/storage/yaml/yaml_datastructs_funcs.cpp to make it build, but apparently something is not yet correct with this commit/patch.
@pafleraf could you please take a look. Thanks!
PL18/PL18EV key remap improvements, matching key diagnose values.
…ART7.

There were further changes necessary, as PL18 is the first color radio w/o an internal RF module.
SYS button of T16 Family is actually "RADIO" button. Changed labels accordingly.
USB charging working for PL18
display charge animation on PL18
fix compilation of PL18 target
untested
@richardclli richardclli changed the title Fs pl18 Flysky PL18/PL18EV Mar 22, 2023
This was referenced Aug 24, 2023
@richardclli
Copy link
Collaborator

Replaced by PR #3963

@pfeerick pfeerick deleted the fs_pl18 branch August 2, 2024 05:00
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.

4 participants