Skip to content

feat(tg5040): keep device awake while connected as a USB gadget#783

Merged
frysee merged 1 commit into
LoveRetro:mainfrom
Pobega:tg5040-usb-keepawake
Jul 19, 2026
Merged

feat(tg5040): keep device awake while connected as a USB gadget#783
frysee merged 1 commit into
LoveRetro:mainfrom
Pobega:tg5040-usb-keepawake

Conversation

@Pobega

@Pobega Pobega commented Jul 17, 2026

Copy link
Copy Markdown

Add an opt-in "Keep awake over USB" setting that prevents screen-off and deep sleep while the device is enumerated as a USB gadget by a host, as distinct from merely charging.

Detection reads /sys/class/udc/*/state and treats "configured" as connected (PLAT_isUSBConnected), so it reflects an active data connection rather than power presence. This matters when the battery is full: charging already blocks sleep, but a full battery on USB would otherwise let the device sleep and drop the connection.

The state is polled on the existing battery thread into a new pwr.is_usb_connected atomic and, gated by CFG_getKeepAwakeWhenUSB, is factored into both PWR_preventAutosleep() (no screen-off) and the PWR_waitForWake() deep-sleep deferral, mirroring how is_charging is handled.

Setting defaults to off and is exposed in Settings under the tg5040-only block. tg5050 and desktop get PLAT_isUSBConnected stubs returning 0.

Disclosure: LLMs were used to develop this code.

@Pobega

Pobega commented Jul 17, 2026

Copy link
Copy Markdown
Author

I tested this on a TrimUI Brick. I do not own any other tg5040 devices to test on.

Add an opt-in "Keep awake over USB" setting that prevents screen-off and
deep sleep while the device is enumerated as a USB gadget by a host, as
distinct from merely charging.

Detection reads /sys/class/udc/*/state and treats "configured" as
connected (PLAT_isUSBConnected), so it reflects an active data connection
rather than power presence. This matters when the battery is full: charging
already blocks sleep, but a full battery on USB would otherwise let the
device sleep and drop the connection.

The state is polled on the existing battery thread into a new
pwr.is_usb_connected atomic and, gated by CFG_getKeepAwakeWhenUSB, is
factored into both PWR_preventAutosleep() (no screen-off) and the
PWR_waitForWake() deep-sleep deferral, mirroring how is_charging is handled.

Setting defaults to off and is exposed in Settings under the tg5040-only
block. tg5050 and desktop get PLAT_isUSBConnected stubs returning 0.

Disclosure: LLMs were used to develop this code.
@frysee
frysee force-pushed the tg5040-usb-keepawake branch from 66bce80 to ed1c708 Compare July 19, 2026 14:55
@frysee
frysee merged commit aa3e624 into LoveRetro:main Jul 19, 2026
62 checks passed
apommel added a commit to apommel/NextUI that referenced this pull request Jul 20, 2026
apommel added a commit to apommel/NextUI that referenced this pull request Jul 20, 2026
pvaibhav added a commit to pvaibhav/NextUI that referenced this pull request Jul 20, 2026
Upstream FN action (LoveRetro#788) and USB keep-awake (LoveRetro#783) APIs require
platform definitions that were only added for other platforms.
Provide H700 FN stubs (no assignable buttons) and a zeroing
PLAT_isUSBConnected so the post-rebase build links again.

Document that FN1/FN2/HOME pak actions and USB keep-awake are
not supported on H700.
carroarmato0 added a commit to carroarmato0/NextUI that referenced this pull request Jul 26, 2026
… pak fixes

New upstream features merged:
- Color palettes: 18 bundled palettes + user-loadable palette dir (LoveRetro#787)
- FN1/FN2/HOME can now launch user-assigned tool paks (LoveRetro#788)
- Keep device awake while connected as USB gadget (LoveRetro#783)
- RetroAchievements identification for custom paks and multi-system emulators (LoveRetro#775)
- Brick Pro rumble limited to 2.5V (was 3.3V, too strong)
- Align Brick/Pro rumble strength

Conflict resolutions (C++20 branch vs upstream):
- nextui.cpp: kept RAII core::SurfacePtr, took upstream THEME_COLOR7 for preview fill
- minput.c: kept GFX_getFonts() and pad-> accessors, took upstream button rendering
  (ALT_BUTTON_TEXT_COLOR, text_color local, GFX_blitAssetColor, THEME_COLOR2 for sticks)
- tg5040 platform.c: accepted conditional MAX_VOLTAGE for Brick Pro; RUMBLE_PATH
  not needed (branch uses deviceModel->rumble_gpio_path from DeviceDescriptor)
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.

2 participants