[niteria]

Deep sleep (suspend-to-RAM) for the M4 MacBook Air (T8132, J713) at EL2, on dj's generic sleep driver, with Erik Bourget's radio and docked-sleep series.

Base: integration/aurora (aa68b3c81830; it merges cleanly with today's 4277bddd6312), which has dj's T8140 suspend-to-RAM stack since #1445 merged. #1445 superseded #1334 and has the same generic apple-sleep driver. T8132 is that driver's second SoC, t8132_sleep_soc with t8132_sleep_ops, in the per-SoC data and operations of Ryan's #1769, with the few fields and operations it needs added.

What's in it, 52 commits:

• The T8132 port (from iommu: apple-dart: don't require DAPF ranges on a DART that has none to arm64: dts: apple: t8132-j713: name the SEP's domain in the PMGR policy), including the system level cache write-back, the SMC's T8132 numbering of its wake reasons, and the PMP thermal samplers paused across a deep sleep.
• Erik Bourget's native radio series, arm64: dts: apple: Restore DCP address cells for J713 IOMMU validation #1385, [1/5] iommu: apple-dart: Preserve J713 radio translations across S2R #1378 and [2/5] soc: apple: Resolve radio PERST through the sleep controller PCIe phandle #1381 to [5/5] Bluetooth: bcm4377: Restore private PCI windows before J713 resume #1384: Wi-Fi and Bluetooth through a deep sleep. Erik has closed those PRs. Ahead of his brcmfmac commit, wifi: brcmfmac: don't cancel a bus reset that was never set up.
• Erik Bourget's docked series, [1/7] pmdomain: apple: Let a consumer switch the J713 external display domains #1414 to [7/7] drm/apple: m4: Stop the J713 external DCPs for a deep sleep and restart them #1420, as he force-pushed it after dj's review: Thunderbolt host router, CIO PHY, tunneled PCIe and the external DCPs through a deep sleep, behind pcie_apple.cio_s2r and thunderbolt_apple.cio_s2r (both off by default). That's 18 commits, among them the four the earlier copy here lacked: rolling back a partial consumer power-off in pmgr-pwrstate, the apple,clock-protection binding and its J713 DT, and apple,policy-machine for the platform-handoff entries. His kernel code is as on his branches in all 18, and 10 are identical outright. The rest differ in context, three of them (the DART, ATC CIO and Thunderbolt apple.c ones) after conflict resolution that keeps both sides, and in the host-test changes the notes below list.
• Two binding commits for properties the radio and docked series' device trees use: a root port's clkreq-gpios and the USB4 router's cio PHY.
• tools/asahi: run the J713 sleep tests from the host-test runner: registers the 17 host tests the two series added, which run-host-tests.py requires, and makes them run under its python3 -I.

Erik's commits keep his authorship and sign-off. Thirteen of them have a bracketed note after it: his Based-on-work-by: tags, which aren't kernel tags, moved into the text (five), checkpatch fixes in his host-test harnesses (four), older host tests' mocks updated for his changes (six, below), the review fixes in item 2 (three), and the fixes for dj's review in item 3 (four). If this PR is where they land, #1414 to #1420 can be closed too.

Eight older host suites copy functions this series changes into mocks, and stop compiling against it. Each mock gets the new state, inert as on other machines, in the commit that first needs it:

• apple-dart-fw and apple-dart-range (apple-dart.c): the radio sleep record, the sleep writer gate and, with the dock series, cio_active and the gate's new return value;
• brcmf-suspend and bcm4377-resume, with their mutation runs: the radio sleep hooks;
• t6030-clock and apple-dcp (atc.c): the CIO sleep state;
• rtkit-oslog, with its mutation runs: it now also copies apple_rtkit_reset_session(), which the cold-restart commit split out of apple_rtkit_reinit();
• asahi-m4: G16's publish() checks the sleep flag.

asahi-m4 also gets a case for a publish during a sleep, and t6030-clock two for the stale ACK a CIO sleep leaves. Erik's own brcmfmac-sleep-lifetime test copies brcmf_pcie_pm_enter_D3(), which now has #1653's WoWLAN checks; its mocks get that state too, never armed.

Two fixes the J713 needed in integration/aurora itself are merged now: #1669 (the G16 probe overflowed its 32 KB stack) and #1668 (the scaler and JPEG DARTs had no power domain, so the DART probe panicked at boot).

Review items

1. Signed-off-by: on every commit; Erik's carry his and then mine as the submitter.

2. Changes from a review of 19de60a3ee67, first pushed as c53fb99165a4. Each fix is in the commit that had the problem, or in a new commit where marked:

   • arm64: dts: apple: t8132: add the sleep controller: the node is disabled in t8132.dtsi and enabled in t8132-j713.dts. It bound on every T8132 board and made deep sleep the default there, but the radio, DART and DCP recovery is the J713's alone, so every deep sleep was refused and s2idle was no longer the default.
   • iommu: apple-dart: accept a display DART the sleep engine rebuilt: its mark that the sleep engine rebuilt a display DART is only set at resume now, and only the DCP's next power-up clears it. With the panel off across a deep sleep and then an s2idle or a sleep refused late, the mark was cleared while the DART still had the engine's streams on, and the next power-up failed: DCP SID23 retired: power-up: state lost (-5), internal panel dead until a reboot. Reproduced on the J713 before the fix, gone after it.
   • New, thermal: apple-pmp: pause sampling across a deep sleep: sampling ran on through the stopped PMP. After every wake, "sensor error -19; CPU minimum caps retained" (about 100 lines per test run), the CPUs held at their minimum frequency for about a second, and the GPU's power level stepped back from 7 over about 15 s. Now no sensor or telemetry error after any wake.
   • drm/asahi: G16: ride out dark sensors while the GPU is idle: an idle sample keeps the last cool reading current, so the 2 s allowance runs from when work starts rather than from the last valid reading. After a deep sleep the GPU takes no idle sample before its first work, so that case remains (see Known limitations).
   • New, mfd: macsmc: number T8132's wake reasons as its SMC does: the wake reasons were named by T8140's numbering, where the lid is bit 10; the J713's SMC uses bit 8. A lid closed during a deep sleep woke the J713 fully, user space included. Now: "woken for the lid, which is closed", a dark wake with the outputs off, and straight back to sleep; opening the lid wakes it.
   • dt-bindings: pci: apple,pcie: give T8132 its own register list: the T8132 branch rejected the third-port layout of the J604, J623, J624 and J773g (12 new CHECK_DTBS warnings). It accepts it now, and those boards' older reg-names warnings are gone too.
   • soc: apple: mcc: write T8132's cache back for a deep sleep: the cache is also written back when probe left it alone and when the driver is unbound, and an enable that isn't confirmed turns it off again: nothing else writes it back before a sleep.
   • Erik's iommu: apple-dart: preserve native radio translations across system sleep: sleep-spin.h reads CNTFRQ_EL0 with arch_timer_get_cntfrq(), as arch_timer_get_rate() isn't exported: with CONFIG_APPLE_DART=m (arm64 defconfig) modpost failed. The spin also reads the register once more at its deadline before it reports a timeout. The radio port commit's host-test mock follows.
   • Erik's thunderbolt: keep PCIe and USB tunnels across a host router power cycle: tb_switch_suspend_cio() took tb_switch_clx_disable()'s positive return, the CL states it disabled, for an error.
   • New, wifi: brcmfmac: don't cancel a bus reset that was never set up: a suspend or a remove before the firmware had loaded ran cancel_work_sync() on a work with no function, and the workqueue code warned.

3. Changes from dj's review of af8707ff5d61, each in the commit it concerns:

   • Erik's thunderbolt: apple: power-cycle the J713 host router for a deep sleep: only a sleep that may power-cycle the router expects its PCIe tunnel back. That's one with resume_recovery, as before, or the J713's native deep sleep. On any other host, a tunnel nobody asks for again after a wake no longer sets off the post-wake -ENOLINK cycle of ACIO and the PHY. Every host still pauses the DP-IN retries and defers a teardown's PCIe-C quiesce to .complete for a sleep, as tb-dock-pm-tests requires. A failed CIO prepare also restores the expectation. The test has cases and mutations for each rule.
   • The J715 no longer gets the J713's sleep wiring through the files they share. These now live in t8132-j713.dts, each moved in the commit that added it:
     • the SEP and its DART, and ps_sep (enable the SEP for system sleep, describe what a deep sleep needs);
     • the external DCPs' consumer-switched domains with their retention words (Erik's let the external DCPs switch their domains);
     • the ACIO hosts' cio PHY (Erik's give the ACIO hosts their PHY).
       name the SEP's domain in the PMGR policy now names it for the J713 only (apple,policy-machine), as the ACIO entries do. In the compiled J715 tree, those nodes match the base again.
   • describe what a deep sleep needs: the clusters' power gates are disabled in t8132.dtsi and enabled on the J713.
   • Erik's drm/apple: m4: stop the external DCPs for a deep sleep and restart them: a failed stop ends the sleep's PM epoch at once, as .resume would. The PM core doesn't resume a device whose suspend failed, and the odd epoch kept that display dark until a reboot.
   • drm/asahi: G16: stop the firmware for a deep sleep and restart it warm: if the commit can't take the GPU stop, .prepare refuses the sleep, as the PMP and SIO drivers do. It used to stop the firmware itself, and a later refusal then left the GPU stopped until the next real deep sleep.

4. Rebased from f173f7c8537a onto 028d76246b00, then bf00add09e22, 673f61842546 and aa68b3c81830. Five conflicts: four in Erik's commits, one of which also needed a host-test change, and soc: apple: sleep: per-SoC data and operations for the sleep sequence #1769 in the two T8132 sleep commits:

   • soc: apple: sleep: per-SoC data and operations for the sleep sequence #1769 (per-SoC data and operations for the sleep sequence) and soc: apple: sleep: add T8132 (M4 Air, J713) support and soc: apple: sleep: get T8132 through a full sleep and wake: T8132 is ported onto it.

     • t8132_sleep_ops has T8132's own last_mile and wake, and uses T8140's deep WFI and AIC operations.
     • t8132_sleep_soc fills soc: apple: sleep: per-SoC data and operations for the sleep sequence #1769's fields: the cluster and clock-mux words, a CPU start stride of 6, the PMU's boot-stage register, two MCC channels and two AMCC planes, and the wake block.
     • The shared commit gains what T8132 needs and soc: apple: sleep: per-SoC data and operations for the sleep sequence #1769 has no place for, without changing T8140's path: a lower_io operation (T8140's is apple_sleep_lower_io()), the resume entry the mailbox gives aura (T8140's is cpu_resume), an optional pre_commit step before the commit word, and a delay before the boot core's WFI.
     • This PR's own per-SoC register list and commit/rewake hooks are gone.
     • T8132's sequence is unchanged: the same steps, order and values.

   • wifi: brcmfmac: pcie: keep Apple firmware in D0 across s2idle #1722 (Apple firmware kept in D0 across s2idle) and wifi: brcmfmac: coordinate retained firmware with J713 system sleep: only brcmf-suspend-tests' mock struct conflicted; it has both sides' fields. In pcie.c the new s2idle-D0 branch runs only for s2idle, so the J713's deep-sleep path (the lease, the D3 inform, apple_radio_sleep_endpoint_suspended()) is as before. brcmfmac-sleep-lifetime-tests copies the same function, and its mocks get the s2idle-D0 state, never taken in its deep-sleep cases.

   • PCI: apple: t8140: bring the radios back after a sleep off resume_noirq #1717 (T8140 radio resume off resume_noirq) and PCI: apple: stop and restart the J713 tunneled port for a deep sleep: both added an apple_pcie_config_read(). It's one function now: the Neo's D0 answer while its radio link comes back, then the J713 tunneled port's cio_cfg_lock. PCI: apple: t8140: bring the radios back after a sleep off resume_noirq #1717's T8140 resume is unchanged, and async resume stays the Neo host's: the J713 radio host still disables it.

   • T8132: DisplayPort alt mode on the J604's USB-C ports #1678 (DisplayPort alt mode on T8132) and phy: apple: atc: keep a T8132 CIO link when its USB host stops and phy: apple: atc: take the T8132 CIO PHY through a deep sleep: T8132: DisplayPort alt mode on the J604's USB-C ports #1678's t8132_dp_alt and the CIO sleep state are neighbouring fields, in atc.c and in the host tests' copies of its struct (atc-usb3-tests, and the DCP selftests' phy.py and dwc3/mocks.c). Each keeps both. They concern different modes: the CIO sleep is only prepared in a USB4 or Thunderbolt mode, and only the J604 marks a port for DP alt mode, and deep sleep stays off there.

   • wifi: brcmfmac: pcie: reset the chip when the firmware dies #1729 (reset the chip when the brcmfmac firmware dies) and wifi: brcmfmac: coordinate retained firmware with J713 system sleep: brcmf_pcie_reset() first refuses while the J713 sleep owns the radio, before it touches anything. Then it marks the device down and drains, as wifi: brcmfmac: pcie: reset the chip when the firmware dies #1729 does. wifi: brcmfmac: pcie: reset the chip when the firmware dies #1729 also runs the reset work on a freezable workqueue, so a reset armed during a sleep now waits until the processes thaw, after .complete has given the radio back to the driver. That also covers the dropped reset in item 6 of dj's review of af8707ff5d61.

   Nothing else the base gained (163 commits, 257 with merges) conflicted. pmdomain: apple: pulse a latched-off domain's PS reset on T8140 only #1757 now pulses a latched-off domain's PS reset only on T8140. It never fired in our J713 runs, where it would have logged "off with BUSY latched".

   Earlier: rebased from Apple SoC sleep: one driver for all Apple SoCs, matched only on tested ones #1334 onto integration/aurora after Apple SoC suspend-to-RAM on T8140 #1445 merged, with Erik's docked series replaced by his current one. Of the 31 other older patches, git range-diff against this PR's previous head shows 20 unchanged and three changed only in context. Five conflicted, and each keeps both sides:

   • soc: apple: sleep: add T8132 ...: Apple SoC suspend-to-RAM on T8140 #1445 now panics when a commit returns, and that check follows the per-SoC commit hook; T8132's commit doesn't return either. Apple SoC suspend-to-RAM on T8140 #1445's report of coprocessor stop errors sits next to T8132's report of power words still on, and the latched power words dj's later change records sit next to T8132's quiesce word. Ryan's AVE complex change (soc: apple: sleep: power the AVE complex as the encoders' parent domain #1663) skips .prepare's complex power-off once genpd owns the complex; T8132 has no complex, so that power-off also stays T8140's (has_ave), as before. A comment left over from removed bring-up code is gone.
   • iommu: apple-dart: accept a display DART the sleep engine rebuilt: a neighbouring bitfield.
   • Erik's iommu: apple-dart: preserve native radio translations ...: the DART mock's struct also gets Apple SoC suspend-to-RAM on T8140 #1445's fw_tree fields, and apple_dart_suspend() returns early for the live J613 MTP first, then for a radio DART.
   • Erik's soc: apple: resolve radio PERST through the sleep controller's PCIe host: it drops radio_pcie from the per-SoC data, next to soc: apple: sleep: power the AVE complex as the encoders' parent domain #1663's new AVE fields.
   • Erik's wifi: brcmfmac: coordinate retained firmware with J713 system sleep, against Ryan's WoWLAN in s2idle (wifi: brcmfmac: Wake-on-WLAN in s2idle for the BCM4387/BCM4388 Apple laptops #1653). The suspend refuses WoWLAN without wakeup permission or outside s2idle before the J713 sleep lease is taken, so a deep sleep never reaches the WoWLAN path; the resume checks the lease first, then takes WoWLAN's hot path. brcmf-suspend's mocks have both sides' fields, and it passes with all its mutations.

   Three changed on purpose:

   • phy: apple: atc: bring the T8132 PHY back after a deep sleep no longer sets the PIPE handler's dummy PHY enable itself. Friedrich's phy: apple: atc: enable the dummy PHY on T8132 SuperSpeed ports (phy: apple: atc: enable the dummy PHY on T8132 SuperSpeed ports #1492) now sets it at the same point for every T8132 port with its SuperSpeed banks, which both J713 ports have, and after a deep sleep this commit has the PIPE handler set up again.
   • arm64: dts: apple: t8132: describe what a deep sleep needs: its message said the driver falls back to s2idle without the APCIe tables. Since Apple SoC suspend-to-RAM on T8140 #1445 it refuses a deep sleep that stops ANS instead.
   • The registration commit registers the docked series' current host tests.

   The rest of the base's sleep changes need nothing for T8132. The SEP's SLEEP request goes out at the platform's last returnable step. The USB host's DRAM vote stays T8140's alone (apple,sleep-ddr). The latched power-word clearing and the T8140 domain changes take per-SoC tables T8132 doesn't fill. The refusal over an unbound running coprocessor and NVMe's shutdown notification at suspend apply to T8132 as well, and were in the tests below. So were the scaler and both JPEG engines that integration/aurora now enables on the J713: bound and runtime-suspended through every sleep. Its video decoder didn't bind in the tests (no apple/avd-fw-v5-t1.bin on this Air), so no sleep has run with it bound.

5. KUnit, built with GCC and run in QEMU: apple-sleep-soc-data (T8132 in its tested list, and in soc: apple: sleep: per-SoC data and operations for the sleep sequence #1769's layout and operations checks) and apple-mcc pass, 9 of 9, and sep.kunitconfig's 294 apple_sep* tests pass with CONFIG_GPIOLIB added, which sensor_shim.c needs under GCC, on the base too. The SMC's macsmc-parser suite passes 15 of 16 with the new T8132 wake-reason cases; the one failure, macsmc_sleep_order_test, fails the same way on integration/aurora.

6. Bring-up removed: the DO NOT MERGE commit, the progress marks in the sleep driver, PM core and arm64 resume path, the panic and die notifiers, the comparison with macOS's power words, every bringup_* parameter (their defaults kept), and cpu-reset.o without KEXEC: the board's kernel config now has CONFIG_KEXEC=y. The T8132 commit no longer adds the PMU commands macOS sends after the commit; sleep and wake work without them.

7. t8132-j713-apcie-tunables.dtsi is gone: the bootloader passes ANS's APCIe tables. Aura 1.0.0b11 publishes them from the ADT on every chip: Publish APCIe ANS recovery tables with atomic packed-DT export aura#112's final revision opens a copy of the tree with the buffer's full capacity, so a tree prepare_dt() has already packed has room for them too. On the J713, 1.0.0b11 logs FDT: /arm-io/apcie tables published, 261 records, and the ten tables are byte-identical to the copy this PR used to carry. With an Aura that predates it (1.0.0b10 as tagged, say), nothing breaks, but every deep sleep is refused at the driver's last checks ("no tunables to bring ANS's PCIe block back") and resumes.

8. checkpatch: every commit is clean apart from two prototypes in .c files (the resume stub's, next to its asm(), and one the Rust side of G16 calls, like its neighbours) and the MAINTAINERS reminders for new files.

9. One copy: see above.

Known limitations

• USB devices disconnect at every deep sleep. For a deep sleep on a port without a fixed hub, dwc3-apple stops the USB host before the freezer runs and brings it back after the wake, as the base does; a mounted USB disk loses its block device, and no USB device can wake the Air.
• A deep sleep is refused unless brcmfmac and hci_bcm4377 are both bound: Erik's radio sleep needs both drivers to take the radio through its power loss.
• Docked with cio_s2r off, the default: a USB4 or Thunderbolt device in a port blocks every suspend, s2idle too. DWC3 stops its host for the default deep sleep, Erik's phy: apple: atc: keep a T8132 CIO link when its USB host stops keeps the PHY in a CIO mode, and atcphy_suspend() refuses. On integration/aurora that check still let a docked s2idle through.
• After some wakes the GPU runs at reduced clocks for up to about 16 s: 4 of 35 wakes on the pre-rebase test boot, then 1, none, 2, none and 2 of 10 on the last five heads' runs (af8707ff5d61, d0ac23c1cfe4, 67c358b5b63d, 025305c64266 and this one). Twice the GPU's power budget in apple-pmp-thermal started at its lowest level and released one step a second, and twice G16's own sensor backstop found the die sensors dark at its first sample after the wake. Neither came back in 18 sleeps looked at closely (the budget polled every 50 ms), so their causes aren't settled yet.
• Speakers with speakerguardd 0.1 (the Speaker Volume Unlock daemon the J713 runs today) come back almost silent after a deep sleep and take about 90 s to recover. At suspend, macaudio clips the speaker volumes ("Speaker volumes locked: System suspend"). A volume write from the daemon that lands in that window fails with EINVAL, and the daemon gives up ("leaving the speakers at the card's safe level"). After the wake it takes the lock back at about 97 dB of gain reduction and eases off from there. The kernel side behaves as intended: the pre-sleep volumes are restored at the unlock, and the amps keep their state through the sleep (their temperature registers survive it). speakerguardd v2 handles system sleep (packaging: orderly sleep, graph before logins, supervisor after sessions speakerguardd#84, Moved · 🧩 T6034 · M3 Max, 14-core — board index #85, PCI: apple: enable experimental J700 radio endpoint bootstrap #151) but doesn't support the J713 yet.

Tested

On the J713 (24 GB, Linux at EL2) as a full system build with CONFIG_KEXEC=y, radios, GPU driver and cache on, Aura 1.0.0b11 as Stage 2 (a one-boot aura-try trial from the installed b9 Stage 1, with this branch's DTB), with this head's kernel, all on one boot:

• 8 plain systemctl suspend sleeps from a lit desktop (40 s, and 3 min), each woken by its RTC alarm. In 7 of them a process rewrote and checked 256 MB up to the freeze: no bad blocks. After each, the radio recovered from the power loss, Wi-Fi carried 20 and 300 1400-byte pings to the gateway without loss, Bluetooth answered Read Local Version, and the GPU firmware was back and rendered. Then 2 sleeps refused at the last returnable step with apple_sep.sleep_refuse_before_send: each refused before SEP SLEEP was sent (-ECANCELED) and rolled back, and systemd's immediate second attempt then slept and woke normally.
• The panel off across a deep sleep, then a sleep refused late, then the panel on: it came back. On 19de60a3ee67 the same sequence left it dead until a reboot.
• Over the boot: the radio and the GPU back after every wake, no oops, panic, SError or WARNING, and no thermal sensor or power telemetry error after any wake (on 19de60a3ee67, about 100 "sensor error -19" lines in a run of 13 sleeps).
• The AOP's sensor services that the base now describes on T8132 (light, lid angle, motion and board temperature) were bound, and their drivers started again after every wake.

Mesa was built against this base's GPU UAPI, which has GEM userptr. The same run passed on the previous heads: af8707ff5d61 (on 028d76246b00), d0ac23c1cfe4 and 67c358b5b63d (on bf00add09e22), and 025305c64266 (on 673f61842546).

Item 3's G16 refusal, on a scratch build of 67c358b5b63d with a test switch that fails the GPU's commit-time stop registration (not part of this PR): with the switch on, both of systemd's attempts were refused in G16's .prepare ("no GPU stop at the commit (EAGAIN): refusing the sleep"), and the GPU rendered afterwards. With it off, the same boot slept and woke normally, the GPU back in 715 µs.

Before these rebases, on c53fb99165a4 (the same patches on f173f7c8537a), one boot also had:

• The lid closed during a deep sleep: "woken for the lid, which is closed", a dark wake with the outputs off, and straight back to sleep; opening the lid woke the Air. On 19de60a3ee67 the same close woke it fully.
• 22 more sleeps, plain and with the 256 MB writer, with the thermal driver's power budget read every 50 ms.
• In all, 37 sleep attempts, 35 wakes counting the dark wake, the radio and the GPU back after every one, and no fault.

The speaker amps keep their state through a deep sleep: their temperature registers, which a power-on reset clears, still held their last readings after it, and sound played about 4 s after the wake.

The docked series' DT changes the apple,bridge-link format, so the bootloader has to pass this branch's DTB: with the previous one, two pmgr-pwrstate domains fail to probe ("bad bridge link").

Before the rebase, on #1334: the same 8 sleeps and 2 refusals (on a boot without apple_sep) with the tables in the device tree, and 8 more sleeps with them from the bootloader.

Also, at this head: it builds (Image, modules, DTBs) with no warning beyond the base's, the whole run-host-tests.py passes (319 of 319 runs; the base passes its 302), KUnit is as in item 5, and CHECK_DTBS on all six T8132 boards (J604, J623, J624, J713, J715, J773g) shows no new warnings and ten fewer than the base: the T8132 PCIe host's reg-names now match its binding (with ti,cd2e22.yaml, via,vl122.yaml and apple,pmp.yaml set aside: their uint8-array apple,tunable crashes dtschema's decoder, on the base too). Before these rebases, on c53fb99165a4, every commit built and passed the whole host suite, and the head also built with the DART, PCIe, NVMe, SMC, DWC3, MCC and ATC PHY drivers as modules (19de60a3ee67 failed modpost there: arch_timer_get_rate).

Not tested here

• The docked paths: this Air has a USB-C adapter, not a Thunderbolt dock, and this boot chain doesn't hand over the external DCPs (external DCP not handed over; Aura does that only with chosen.asahi,j713-dcpext=1). Since Erik's own dock test, the base gained Ryan's thunderbolt: apple: Restart the port when the peer link is lost (e847be1). It cycles a port whose peer stays gone for 3 s, from a freezable workqueue. The docked sleep path hasn't run with it.

🤖 Generated with Claude Code

[RyanTheTide]

What this branch needs before it can be merged, on head 715a022:

1. Signed-off-by. None of the 19 commits from 297d7a47d7c9 (iommu: apple-dart: don't require DAPF ranges on a DART that has none) to 715a0229e1b8 (soc: apple: mcc: write T8132's cache back for a deep sleep), excluding the six signed radio commits 6a720bcba1b3..b37219c61cbd, carries a Signed-off-by: line. Please add yours to each (git rebase -x 'git commit --amend --no-edit -s' feat/t8140-s2r) and push.

2. Rebase onto the current feat/t8140-s2r (31fd6d6). The branch forked at 5b1a671; the base has since merged #1311 and the merge now conflicts in drivers/pci/controller/pcie-apple.c (with b724b53 "PCI: apple: attach and retain all host power domains in s2idle" and f31c0a3 "PCI: apple: preserve probe without power-management support").

3. Build break. bringup_va_mark in arch/arm64/kernel/sleep.S (added by b6f4482) loads t8132_spmi_va, which is defined in drivers/soc/apple/t8140-sleep.c and only exists when CONFIG_APPLE_T8140_SLEEP=y. Any config without that driver built in fails to link (ld.lld: error: undefined symbol: t8132_spmi_va, see the kunit job of https://github.com/aurora-silicon/linux/actions/runs/42695306042, which builds this branch's code). This goes away with item 4.

4. Remove the bring-up parts listed in the description: 669e8db10c18 (DO NOT MERGE workarounds), the progress marks and panic/die notifiers in b6f448244d44, the bringup_* module parameters, and cpu-reset.o built without KEXEC (enable CONFIG_KEXEC instead).

5. arch/arm64/boot/dts/apple/t8132-j713-apcie-tunables.dtsi is 295 lines of calibration copied from the ADT. Please have the bootloader pass these on instead of carrying the table in the tree, as the description already plans.

6. checkpatch on every commit.

7. The radio commits 6a720bcba1b3..b37219c61cbd are also open as #1385, #1378, #1381, #1382, #1383 and #1384. Keep them in one place so the two copies cannot diverge.

This PR is based on #1208, which has to land first.

[erikwb]

Please merge PR 1378, 13{81-84}, 1414-20 into this branch before diverging it too much - should work smoothly on your machine, tested on what I believe is an identical machine!

[niteria]

Thanks for the list. Pushed an updated branch; where each item stands:

1. Signed-off-by: on every commit; Erik's carry his and then mine as the submitter.
2. Rebased, now onto Apple SoC sleep: one driver for all Apple SoCs, matched only on tested ones #1334 (feat/apple-soc-sleep-generic) rather than Deep sleep (suspend-to-RAM) for the MacBook Neo (T8140) at EL2 #1208 alone: T8132 is a struct apple_sleep_soc entry in Apple SoC sleep: one driver for all Apple SoCs, matched only on tested ones #1334's generic driver, so on Deep sleep (suspend-to-RAM) for the MacBook Neo (T8140) at EL2 #1208 it would need redoing once Apple SoC sleep: one driver for all Apple SoCs, matched only on tested ones #1334 lands. This PR's base is changed to that branch. Nothing here needs t8140-sleep: take the deep path when an SLP_DDR wake would stall (camera ran, AOP crashed) #1358 or later.
3. The t8132_spmi_va link failure went with the instrumentation. sep.kunitconfig builds on this branch and its 210 apple_sep* tests pass; apple-sleep-soc-data (now listing T8132 as tested) and apple-mcc pass too. Built with GCC here, which on Apple SoC sleep: one driver for all Apple SoCs, matched only on tested ones #1334 too needs CONFIG_GPIOLIB for sensor_shim.c.
4. Bring-up removed: the DO NOT MERGE commit, the progress marks (sleep driver, PM core, arm64 resume path), the panic and die notifiers, the power-word comparison, and every bringup_* parameter, with their defaults kept as code. cpu-reset.o is back under KEXEC_CORE; the J713 board config now sets CONFIG_KEXEC=y.
5. The APCIe tables now come from the bootloader and t8132-j713-apcie-tunables.dtsi is gone. aura#112 already publishes them, but off T8140 prepare_dt() has packed the tree before the publisher runs, so it found no room ("APCIe tables not published"). The fix reopens the tree around it, as dt_disable_j713_pcie() does (kboot: publish the APCIe tables into a reopened tree off T8140 aura#145, stacked on Moved · J414C · MacBook Pro 14-inch (M2 Max / T6021) #112). With it the J713 gets all ten tables, byte-identical to the copy this PR carried, and sleeps on them.
6. checkpatch: every commit is clean apart from two prototypes in .c files (the resume stub's, next to its asm() block, and one G16's Rust side calls, like its neighbours) and the MAINTAINERS reminders. Erik's dock commits needed fixes in their host-test harnesses and their Based-on-work-by: tags; both are in his commits with a note.
7. Erik's radio (arm64: dts: apple: Restore DCP address cells for J713 IOMMU validation #1385, [1/5] iommu: apple-dart: Preserve J713 radio translations across S2R #1378, [2/5] soc: apple: Resolve radio PERST through the sleep controller PCIe phandle #1381-[5/5] Bluetooth: bcm4377: Restore private PCI windows before J713 resume #1384) and docked ([1/7] pmdomain: apple: Let a consumer switch the J713 external display domains #1414-[7/7] drm/apple: m4: Stop the J713 external DCPs for a deep sleep and restart them #1420) series are in this branch with his authorship kept: conflict resolution, and the item 6 fixes. Two new binding commits document what his device trees use (a root port's clkreq-gpios, the routers' cio PHY). Erik, if this is to be the one place, those PRs can be closed.

Also fixed on the way: the 17 host tests the radio and dock series added weren't listed in run-host-tests.py, which fails a run on an unlisted test; three couldn't import their helper under python3 -I, and one needed libfdt. T8132's sleep binding lacked ps-prot, and its PCIe host's register names never matched the binding. The dt-check job can't show any of that: its dtschema crashes decoding apple,tunable (typed uint8-array by ti,cd2e22.yaml and via,vl122.yaml) on base and head alike.

Tested on the J713, as a full system with radios, GPU and cache on: 8 of 8 sleeps with a 256 MB check (no bad blocks), radios and GPU back after each, 2 of 2 last-moment refusals rolled back; then 8 of 8 again with the APCIe tables from the bootloader. Every commit builds; the host tests, the sleep, MCC and apple_sep KUnit suites, and CHECK_DTBS on the T8132 boards (no new warnings) pass.

[niteria]

A correction to my reply above. "The host tests ... pass" covered only the 17 tests the radio and dock series added and the suites that read the sleep driver; I hadn't run the whole tools/asahi/run-host-tests.py. On the previous head (d9ec74bc) it fails 17 runs in seven older suites. Each copies a function this series changes into a mock that lacks something the new code uses:

• apple-dart-fw and apple-dart-range (apple-dart.c): the radio sleep record, the sleep writer gate, cio_active.
• brcmf-suspend and bcm4377-resume, with their mutation runs: the radio sleep hooks.
• t6030-clock and apple-dcp (atc.c): the CIO sleep state.
• asahi-m4: G16's publish() now checks the sleep flag.

None of them is a behaviour change. Each mock now has the new state, inert as on other machines, in the commit that first needs it. asahi-m4 also gets a case for a publish during a sleep, and t6030-clock two for the stale ACK a CIO sleep leaves. Only host-test files changed; the kernel is the one tested on the J713.

The whole runner now passes on #1334 (228 of 228 runs) and at every commit of this branch: 228 of 228 up to the last commit, which registers the series' 17 new tests, and 245 of 245 at the head. Before the registration the runner refuses to start, so for those per-commit runs that check was relaxed and the unregistered tests were skipped. Without the registration (on #1420 alone, say) it stops before running anything, which is why none of this showed earlier. CI doesn't run it here either: host-tests.yml only runs on PRs into integration/aurora and aurora-wip, and this one targets #1334's branch.

[niteria]

Rebased onto aa68b3c81830: 04f43c7fd9b8. Ryan's #1769 (per-SoC data and operations for the sleep sequence) conflicted with our two T8132 sleep commits, soc: apple: sleep: add T8132 (M4 Air, J713) support and soc: apple: sleep: get T8132 through a full sleep and wake, so T8132 is now ported onto it. In git range-diff against 025305c64266, the other 50 commits are identical, and nothing else conflicted.

• T8132 is t8132_sleep_ops and t8132_sleep_soc. It has its own last_mile and wake, and uses T8140's deep WFI and AIC operations. Its data fills soc: apple: sleep: per-SoC data and operations for the sleep sequence #1769's fields:

  • the cluster and clock-mux words;
  • a CPU start stride of 6;
  • the PMU's boot-stage register, 0xf801;
  • two MCC channels and two AMCC planes, mapped through soc: apple: sleep: per-SoC data and operations for the sleep sequence #1769's numbered names;
  • T8140's wake-block layout.

  This PR's own per-SoC register list, the commit and rewake hooks, cpu_p_first_bit and pmu_boot_stage are gone.

• T8132 needs four things soc: apple: sleep: per-SoC data and operations for the sleep sequence #1769 has no place for. They are added to the shared code, and T8140's path doesn't change:

  • a lower_io operation, for what the commit lowers once the other cores stop. T8140's is apple_sleep_lower_io(). T8132's cleans the caches first, as before, then lowers ATC0's complex and ANS;
  • a resume operation, the entry point the mailbox gives aura: cpu_resume on T8140, T8132's WD2 stub;
  • an optional pre_commit operation, between the mailbox and the commit word, where T8132 cleans its stack and the driver's state;
  • boot_wfi_delay_ms, 3 ms on T8132.

  ps-prot is mapped only for a SoC that lowers unbound displays, and apple_sleep_test_ops also checks lower_io and resume.

• T8132's sequence is unchanged: the same steps, order and values. Its path gains only two shared steps, the still-on polls and the latch clearing, and both do nothing on T8132 because its lists are empty.

Tested at this head:

• run-host-tests.py: 319 of 319. The base passes its 302.
• KUnit: apple-sleep-soc-data, now with soc: apple: sleep: per-SoC data and operations for the sleep sequence #1769's layout and operations checks over T8132 too, and apple-mcc: 9 of 9. The apple_sep* suites: 294. macsmc-parser is 15 of 16, failing as it does on the base.
• CHECK_DTBS on the six T8132 boards: no new warnings.
• It builds with no new warning, and sleep.o is also clean with W=1. checkpatch is as before.
• On the J713, the same run as before:
  • 8 deep sleeps, 7 of them with the 256 MB memory check: no bad blocks.
  • 2 sleeps refused late, each followed at once by a sleep that woke normally.
  • After every wake, the radio, Wi-Fi, Bluetooth and the GPU came back. Two wakes had the known slow GPU start.
  • The panel came back after the display-off sequence.
  • No faults, and the boot's kernel errors and warnings are the same as on 025305c64266.

integration/aurora has since moved to 4277bddd6312 (#1820, #1826, #1827 and #1763). None of them touches a file this PR changes, this head merges with it cleanly, and the Apple DCP, SoC and AVE/ISP drivers build on the merge with no new warning.

Good to merge from my side once CI is green on this head.

[DjDeveloperr]

Re-review of 04f43c7fd9b8, as merged with integration/aurora 4277bddd6312 (1ea434e36b51, the merge CI tested). Thanks for the quick turnaround.

1. Thunderbolt: resolved.
   • The PCIe-tunnel expectation is latched only with resume_recovery or on a native deep sleep (anhi->cio_dart with PM_SUSPEND_MEM). cio_dart is bound only with cio_s2r on a T8132 host of an apple,j713.
   • A failed CIO prepare now restores the expectation.
   • Pausing the DP-IN retries and deferring the PCIe-C quiesce for every host is now in the commit message, and .complete undoes both.
2. J715: resolved. In the shared t8132-j713-j715.dtsi the only change left is the DCP's address cells. The SEP, the domains and the cio PHY moved to t8132-j713.dts.
3. Cluster power gate: resolved. It is disabled in t8132.dtsi and enabled in t8132-j713.dts.
4. and 5. Fixed as suggested. Thanks also for the hardware check of 5.

The #1769 port on T8140. The new per-SoC fields reproduce the previous behaviour:

• Two displays, the external one lowered first, with the same pmc_dcp counts and the same power-gate pair.
• apple_sleep_lower_io(), cpu_resume, no pre_commit and no extra boot-WFI delay.
• The AVE handling and the DCS acknowledgement are kept.
• Probe still requires apple,wake-enable, so the new "no wake source" refusal can't trigger there.
• The radio PERST# host now comes from apple,apcie. On T8140 that is pcie0, the only apple,t8140-pcie node, so it resolves to the same host as before.
• Splitting out apple_rtkit_reset_session() leaves apple_rtkit_reinit() unchanged.

brcmfmac with #1729. brcmf_pcie_reset() refuses only while the J713 sleep holds the radio. Without an active J713 transaction, every radio-sleep hook returns 0 or false, so other machines keep #1729's behaviour.

Item 6 stays open for follow-ups, as agreed. On G17's matching fallback: thanks, we'll handle that on our side.

Before the merge I'll post two more results here:

• the Neo (T8140) rows on a kit built from 1ea434e36b51: boot, three DRAM-kept wakes and one deep sleep, with the radios and the kernel log checked after each, then the default restored;
• the M5 Pro boot smoke on the same merge.

If integration/aurora moves before then, I'll redo the merge check myself, so there's no need to rebase again.

[erikwb]

Works with the TB4 patches.

Kernel: #1314's head (04f43c7) plus our three patches (both cio_s2r defaults on, plus the brcmfmac fix).
Boot: a one-boot bundle with Aura 1.0.0b11 and the branch's DTB.
Proof it's the default: nothing on the boot line sets cio_s2r, and the trial refuses to start if anything does. Both module parameters read Y on the running kernel.
Sleep: with the TS5 attached, the trial ran the kernel's simulated suspend first, then one real deep sleep woken by a 30 s RTC alarm. It woke after 33 s.
What it logged: every step of the CIO stop and restore, the dock's PCIe tunnel coming back, and the Wi-Fi recovery after the power loss. No warnings or faults, and the kernel stayed untainted.
After the wake: the dock was back at 20 Gb/s with Ethernet up, and the external displays were connected.
Log noise: "DP IN handshake failed -116" at suspend entry also shows up in the earlier verified trials 291 and 293. The USB3 PHY and revalidation errors are on the debug-cable port.

[DjDeveloperr]

Merged as 3d015d3d4e7e onto integration/aurora 1d2ef2185056. Thanks @niteria for the series and the quick fixes and rebases, and thanks to Erik for the J713 sleep coordination underneath it.

What the merge rests on

• CI: all 8 checks were green on head 04f43c7fd9b8, merged with 4277bddd6312. Post-merge CI on 3d015d3d4e7e is running.
• Review: items 1-5 are resolved (see the re-review above). Item 6 stays open as follow-ups.
• J713: your runs at this head.
• G17's matching sleep fallback is tracked in drm/asahi: g17: refuse a sleep the commit can't stop #1839.

Hardware regression runs after the merge, on the landed tree

• M5 Pro: one boot, checking display, Wi-Fi, Bluetooth, USB-C, dock/Thunderbolt, NVMe and the kernel log.
• J700 (T8140): a kit built from 3d015d3d4e7e. It boots, does three DRAM-kept wakes and one deep sleep with the radios and the kernel log checked after each, then the default is restored.

I'll post both results here. Anything attributable to this series gets a fix-forward or a revert.