Skip to content

fix: properly detect usb-c cards plugged in before booting#750

Merged
frysee merged 2 commits into
mainfrom
fix-usbc-dac-routing
May 27, 2026
Merged

fix: properly detect usb-c cards plugged in before booting#750
frysee merged 2 commits into
mainfrom
fix-usbc-dac-routing

Conversation

@frysee

@frysee frysee commented May 27, 2026

Copy link
Copy Markdown
Member

Fixes #715 and #530

@frysee frysee merged commit d4c01a3 into main May 27, 2026
62 checks passed
@frysee frysee deleted the fix-usbc-dac-routing branch May 27, 2026 21:17
ArjunKdaf pushed a commit to ArjunKdaf/kUI that referenced this pull request Jun 24, 2026
…#750)

* fix: properly detect usb-c cards plugged in before booting

* fix: apply the same patch to tg5050
carroarmato0 added a commit to carroarmato0/NextUI that referenced this pull request Jul 2, 2026
Brings the migration branch current with upstream (LoveRetro/NextUI). New
upstream work pulled in: white-point colour correction + displaycal for
tg5040 (LoveRetro#760), configurable font style and custom system fonts (LoveRetro#748/LoveRetro#729),
usb-c card detection before boot (LoveRetro#750), the game-switcher label fix for
entries without a save state (LoveRetro#751), readyResume on Quick Menu directory
nav (LoveRetro#739). Two upstream commits (LoveRetro#733 cpu governor, LoveRetro#728 Rewind_init) are
the same PRs our fork already carried under different SHAs; git merged them
cleanly with no duplicated logic.

Conflict resolution:

- nextui.cpp: upstream still edits the old nextui.c, which we renamed to
  .cpp. Git followed the rename. The one real conflict was the recent-game
  preview button hints -- upstream's LoveRetro#751 changed the logic (group 0 always
  BACK; group 1 shows "A OPEN" when there's no save state instead of
  RESUME/REMOVE). Took upstream's logic, kept our C++-friendly cast style.
- nextui.cpp: LoveRetro#739 added a readyResume call site into nextui.c using the old
  Array API (->count / ->items[]); rewrote it against std::vector to match
  the migrated container (top->entries[top->selected]).

Also fixes a latent const-correctness bug the full clean build surfaced:
the tg5040 device descriptor stores sysfs paths as const char*, but
getInt/putInt/putFile took char* -- so libbatmondb (which compiles
platform.c with -Werror=discarded-qualifiers) failed once its cached header
forced a platform.c recompile. Those helpers only read the path string, so
const-ifying the parameter is the correct fix.

Verified: full clean build green for both desktop and tg5040 (nextui,
minarch, settings).
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.

Multiple USB DACs don't work properly, which were previously reported as working

1 participant