Repository navigation
Add TG5040 and TG5050 targets: TrimUI handhelds running NextUI - #282
Closed
realBytewav wants to merge 2 commits into
Closed
realBytewav wants to merge 2 commits into
realBytewav wants to merge 2 commits into
Conversation
Covers the TrimUI Brick and Brick Pro (1024x768) and the Smart Pro (1280x720) as "tg5040", plus "tg5050" (1280x720). NextUI builds those two platforms from different toolchains and sysroots, so each needs its own binary, but the source is identical and both reuse the existing SDL2 adapter set unchanged - the same combination RG35XXPLUS already uses. Button indices come from NextUI's own platform.h and are the same on both platforms: A is b1 and B is b0, crossed over relative to SDL's positional naming so that A lands on the button physically printed "A". The d-pad is a hat. SELECT and the menu button are left unmapped, the former because there is no /event/select and the latter because the firmware wants it. SCREENMULT is 3, the largest whole scale that suits every panel involved: 960x720 centred on 1024x768, and exactly full height on 1280x720. One change outside the new files: the SDL2 window takes the panel's full mode on these platforms. Left windowed_ true, SDL opens a 320x240*mult window - 960x720 on a 1024x768 device - instead of letting appAnchorX_/Y_ centre the grid in the real screen. DEVKIT follows the /opt convention of the MIYOO, BITTBOY and RG35XXPLUS targets, and is ?= so it can be overridden from the environment. Tested on a TrimUI Brick running NextUI-20260719: builds clean, resolves SDL2 from the device's /usr/trimui/lib, opens at 1024x768 on the mali driver, and all nine mappings attach. tg5050 is built but untested - I do not have the hardware. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
- BUILD_COUNT 0-bacon20 -> 0-bacon21, with a matching CHANGELOG entry. - docs/wiki/What-is-LittlePiggyTracker.md gains a trimui bullet in the per-platform "getting lgpt to run" list. - Self-review: the platform branch I had added to SDLGUIWindowImp.cpp duplicated the #else branch verbatim apart from one line. It is now a separate #if that only sets windowed_, matching the #if defined(RS97) block immediately below it that does the same thing for bitDepth_. - Spelled out in Makefile.TG5040 that tg5040 is the platform name for the whole A133p family rather than one device, since the absence of a Brick-specific target is otherwise surprising. NextUI distinguishes devices at runtime through DEVICE=brick/brickpro/smartpro. No new compiler warnings: building X64 from upstream master and from this branch both produce the same 113 distinct warnings, none added or removed. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Owner
|
Thank you for contributing to Little Piggy Tracker. I currently have very little time for maintaining this fork, thank you for understanding. From what I can tell, this device combination is compatible with portmaster that can install Piggy. Maintaining a device is continuous work, rarely a one-shot. What does this merge request contribute that can't be achieved using Portmaster? |
Author
|
ah I just didn't know it was listed on portmaster. Lets just close this |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi,
Worth flagging up front: this patch was written with AI assistance (Claude), with me directing the work and reviewing the result. I'd rather you know than find out, and it should colour how you read the testing claims.
What I can stand behind:
The button indices and resolutions aren't invented — they're read out of NextUI's own workspace/tg5040/platform/platform.h and tg5050/platform.h.
TrimUITracker.mp4
Covers the TrimUI Brick and Brick Pro (1024x768) and the Smart Pro (1280x720) as "tg5040", plus "tg5050" (1280x720). NextUI builds those two platforms from different toolchains and sysroots, so each needs its own binary, but the source is identical and both reuse the existing SDL2 adapter set unchanged - the same combination RG35XXPLUS already uses.
Button indices come from NextUI's own platform.h and are the same on both platforms: A is b1 and B is b0, crossed over relative to SDL's positional naming so that A lands on the button physically printed "A". The d-pad is a hat. SELECT and the menu button are left unmapped, the former because there is no /event/select and the latter because the firmware wants it.
SCREENMULT is 3, the largest whole scale that suits every panel involved: 960x720 centred on 1024x768, and exactly full height on 1280x720.
One change outside the new files: the SDL2 window takes the panel's full mode on these platforms. Left windowed_ true, SDL opens a 320x240*mult window - 960x720 on a 1024x768 device - instead of letting appAnchorX_/Y_ centre the grid in the real screen.
DEVKIT follows the /opt convention of the MIYOO, BITTBOY and RG35XXPLUS targets, and is ?= so it can be overridden from the environment.
Tested on a TrimUI Brick running NextUI-20260719: builds clean, resolves SDL2 from the device's /usr/trimui/lib, opens at 1024x768 on the mali driver, and all nine mappings attach. tg5050 is built but untested - I do not have the hardware.
Description
Thank you for contributing to Little Piggy Tracker!
For fixes, please include a summary of the changes and any related issue.
For features, please include a detailed description of the new feature and how it works.
All PR:s are run through line-by-line clang-format and builds for all supported platforms.
clang-format can be applied locally on commit by installing pre-commit from repo root:
cp pre-commit .git/hooks/
In section below, delete any option that isn't relevant.
Type of change
How Has This Been Tested?
Please describe the tests procedure by which you verified your changes.
Provide instructions on how to reproduce.
Test Configuration:
Checklist:
sources/Application/Model/Project.h