Repository navigation
Conversation
Keep the boot loader DMA context for a running J613 MTP session. Admit only the T8122 MTP DART at 0x2fa808000, exactly stream 1, and verify that its control register still selects bypass before skipping reset. Select an identity domain and reject paging, blocked-domain attachment and runtime suspend while the inherited session runs. Retain the clock and bypass state at removal. This permits the existing mtp-live-v1 RTKit adoption path to keep serving input without another INIT. The C-diag11 implementation passed owner input testing on J613 with firmware 26.6.2. The rebased series requires a separate owner boot test. Signed-off-by: Satya Benson <satya@benson.earth>
The J613 and J615 25G83 device trees list the MTP as "apple,t8122-mtp", "apple,t8122-rtk-helper-asc4", "apple,mtp" without the generic "apple,rtk-helper-asc4" fallback, and say that Linux binds the SoC-specific compatible. The helper only matches the generic one, so on those boards it never probes, DockChannel HID defers forever and the built-in keyboard and trackpad stay absent. Match "apple,t8122-rtk-helper-asc4" as well. Signed-off-by: Satya Benson <satya@benson.earth>
Keep the adopted MTP firmware session and identity DMA stream powered through s2idle. Check ASC RUN and bypass before and after sleep, reject deeper sleep, and refuse to reset inherited DMA on loss. Leave non-J613 helpers and ordinary DART runtime PM unchanged. Exercise the real PM callbacks with host fixtures, including abort unwinding and non-J613 peers. Keep successful retention messages at debug level. Signed-off-by: Satya Benson <satya@benson.earth>
satchlj
requested review from
chriskearney,
maralcbr and
scottjones
as code owners
November 3, 2026 18:31
Author
|
Claude Fable 5.2 on @satchlj's behalf Closing in favour of aurora-silicon/linux#972, where this change is now maintained against aurora-silicon/linux integration/aurora (still open there). |
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.
The live J613 MTP adoption path cannot restart its inherited firmware/DMA session and rejects runtime suspend. Add separate system-sleep callbacks that retain the session through s2idle, mark represented supplier paths awake, and check ASC RUN and DART bypass before and after sleep. Reject deeper sleep and refuse destructive recovery after state loss. Ordinary runtime PM and other machines retain their existing policy.
Depends on #328, whose original commits remain in the branch. The exact head builds Image, modules and DTBs successfully. Actual-body fixtures cover normal and aborted cycles, non-J613 peers, deeper-sleep refusal and state loss without reset or INIT. The companion display branch passed the 57-case Linux classic harness. Owner C13h source testing passed docked/undocked s2idle and built-in input recovery; the rearranged head still needs an owner-attended hardware retest.