-
-
Notifications
You must be signed in to change notification settings - Fork 460
chore: put display settings in hwdefs JSON files. #7031
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
base: main
Are you sure you want to change the base?
Conversation
|
@philmoz looks good. FunctionSwitchGroups - I haven't dug around so just pointing out you changed the logic. Housekeeping: could you please alphabetise the capability enum and cases. Also checked and MaxAnalogs is redundant so can go. Its last use was for binary import and conversion. Love your work! |
Based on all the existing cases. Settings for custom switches need to go into JSON at some point (e.g. has RGB leds) so this could also go in the JSON if needed.
Done. |
I was 99.9% sure you knew what you were doing but hey doesn't hurt to double check. Anything and everything needs to go in hwdef files then Companion should only require maintenance when a new attribute type is added. I can dream can't I... |
|
Is it safe to remove FactoryInstalledPots and FactoryInstalledSwitches - don't appear to be used? |
From memory these are from conversion so if you delete and things compile go for it. There is a lot of housekeeping post removal of binary support and it is scattered everywhere. So to avoid monster PRs I just keep cleaning up in the areas I'm working on. |
|
I've added some more settings to the JSON files for custom switches and miscellaneous hardware settings. There are a few more that will take more time to figure out (HasIMU, HasTrainerModuleCPPM and HasTrainerModuleSBUS). |
|
Can you also look at the hacks in post load hwdefs in boardsjson which should not be required if hwdefs are complete. Fantastic work so far. I personally appreciate how this will make life easier maintaining Companion. |
I don't think those can go just yet - looks to still be some gaps in the other sections of the JSON files. |
|
I also need to check firmware capabilities in case any hardware related are still hiding out there. |
df96495 to
f697321
Compare
|
Reviewed OpenTxFirmware::getCapability and there are lots of hardware items that at a minimum need to be moved to Boards as a housekeeping task. I can foresee multiple PRs for the moves to Boards to keep the merges manageable. From there you can progressively add to hwdefs. |
Add the display related settings to the hwdefs files.
Update companion to use the settings from the hwdefs files.
@elecpower please check the Companion changes to make sure I've done it correctly.