Update lib/tinyusb digest to 5c0e31c - #12
Closed
renovate[bot] wants to merge 1 commit into
Closed
Conversation
|
|
renovate
Bot
force-pushed
the
renovate/lib-tinyusb-digest
branch
from
August 19, 2026 01:31
12832a5 to
231d36a
Compare
renovate
Bot
force-pushed
the
renovate/lib-tinyusb-digest
branch
from
August 19, 2026 02:01
231d36a to
b63b8d1
Compare
4 tasks
jamesarich
added a commit
that referenced
this pull request
Aug 19, 2026
* fix: bump lib/nrfx and lib/tinyusb together, fix the fallout Renovate's PR #11 (nrfx digest) and #12 (tinyusb digest) each failed CI on every board. Investigated both: - nrfx alone (bumped to Renovate's target, nrfx 4.5.0/1b7bedb) breaks the build immediately: nrfx 4.0 restructured its whole layout (mdk/ -> bsp/stable/mdk/, etc.), so this repo's Makefile IPATHs can't find nrf.h at all. That's a real rework, not a digest bump - pinned to v3.14.0 instead, the last tag before the 4.0 restructure. - tinyusb alone (Renovate's target, 5c0e31c) fails to compile against the old 2019 nrfx pin: its nrf5x USB port calls nrfx's chip-errata functions (nrf52_errata_199()) that don't exist yet at that nrfx version. The two submodules move together. With nrfx at v3.14.0 and tinyusb at 5c0e31c, three more issues surfaced, all fixed here: - tinyusb merged device/usbd_control.c into usbd.c; the Makefile's hardcoded tinyusb C_SRC list still named the now-gone file. - nrfx renamed nrf_wdt_started()/nrf_wdt_request_status() in the WDT HAL (see lib/nrfx/CHANGELOG.md) - the one in-repo call site needed updating. - nrfx's gcc_startup_<mcu>.S (since nrfx 3.x) copies .data via __data_start/__data_end (no trailing underscore) instead of this repo's __data_start__/__data_end__, plus three new RAM-loaded regions (.sdata/.tdata/.fast) nothing here uses. Added alias symbols and zero-length filler for the unused regions to linker/nrf_common.ld - deliberately not swapping in nrfx's own updated linker script, which also redefines .bss/.noinit placement and would fight this repo's board .ld files' fixed-address NOINIT region (BLE bond exchange across a DFU reset). Verified: all 14 boards build and link clean via tools/build_all.py. Not done here: real hardware testing. This is still the multi-year boot-critical submodule bump AGENTS.md already flagged as needing it before merge - a clean build is necessary, not sufficient. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * docs: record the CURRENT.UF2 restore gotcha found during hardware testing Real RAK4631 hardware testing for this branch (see PR #19) hit this: restoring an app via ghostfat's live CURRENT.UF2 dump left the device hung on boot even though the bytes matched what was there before. A normal release UF2 or OTA-DFU flash both worked fine immediately after. Recording it so it doesn't get mistaken for a regression from this PR's actual change (nrfx/tinyusb) next time someone hits it. * docs: update the CURRENT.UF2 gotcha now that it's fixed, not untriaged #20 root-caused and fixed what this note previously flagged as an open mystery hit during this PR's hardware testing. --------- Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
Collaborator
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
2 participants
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.
This PR contains the following updates:
9775e76→5c0e31cConfiguration
📅 Schedule: (UTC)
* 0-3 * * *)🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.