Repository navigation
[Joshua's agent] arm64: dts: apple: t6050-k114c: Describe the left-front and right USB-C ports - #1714
Conversation
Enable ATC1 on the K114c: the PD controller hpm1 on nub-spmi-a0 (SN201202x, SPMI address 0xa, port-location "left-front"), the ATC1 combo PHY, usb-drd1 and their mailbox and DARTs. The K114cAP firmware ADT has the same hpm0, hpm1, hpm5, atc-phy and usb-drd nodes as the J714cAP one, so the port block is the J714c left-front one. t6050-k114-k116.dtsi keeps ATC1 disabled for the boards that share it; the K114c enables it. The port's USB4/Thunderbolt router and its DisplayPort routes are not described; the PHY runs in USB mode. The left-back port (ATC0) is the boot link's debug port and stays out, as on the J714c. Booted on a K114c (14-inch MacBook Pro, M5 Max): sn201202x binds on spmi-1 (1-0a), typec port0 registers, dwc3-apple binds usb@2aa280000, the ATC1 PHY and both DARTs probe, no oops or SError. A device on the port is not yet tested. Signed-off-by: Joshua Warren <816217+joshuaswarren@users.noreply.github.com>
efa263a to
63f0671
Compare
… MagSafe controller The right port's PD controller sits on nub-spmi-a1 (hpm2, SPMI address 0xc) and drives ATC2. Describe the controller and its connector, the ATC2 PHY and the dwc3 controller for USB. The USB4 tunnel (ACIO2), PCIe tunnelling and DisplayPort alt mode are not described. nub-spmi-a1 takes no interrupt: the driver polls the PD controllers and refuses an interrupt on this bus, as on the J714s, so the node deletes it. nub-spmi-a0 gets its generation 4 fault queue window (ADT reg[1]), so a fault the left-front controller raises is drained and logged. The MagSafe controller (hpm5, SPMI address 0x8) is described without a connector node: it has no USB data path and no liquid detection. Signed-off-by: Joshua Warren <816217+joshuaswarren@users.noreply.github.com>
63f0671 to
6fd895b
Compare
|
[Joshua's agent] w8B: result on a second kernel, and moving this out of draft. Booted on the same 14-inch M5 Max MacBook Pro (K114c) with a newer kernel set (7.3.0-k114c, the one our integration build now defaults to), with this DT:
Not tested: a USB data device on the right port. The right port currently holds the power supply, so nothing has enumerated there yet; this is the one open item, and it needs someone to plug a device in. Marking ready for review now because the left-front port is verified on two kernels and the right-port description is the J714s one; if the right-port data test turns out to fail, I will follow up in this PR rather than leave it silent. Not in this PR: the left-back port (ATC0, the boot and debug link), USB4 tunnels (ACIO, #1579 is the base), PCIe tunnelling and DisplayPort alt mode. |
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
[Joshua's agent] w8B: the right-port data test, which was the open item, is done and passes. On the 14-inch M5 Max MacBook Pro (K114c), same kernel as the previous comment, the power supply was unplugged from the right port and a USB 3.0 flash drive plugged in:
So both described ports now have a USB 3 data device enumerating on this DT: left-front (two kernels) and right (one kernel, tested after the earlier comment). The right-port role swap to host works with the polled PD bus. Not tested: hot-plug of a device while the supply is attached (a supply-and-drive hub), and the intermittent hpm2 boot miss Dj described (#1713); this PR does not claim to fix it. |
RyanTheTide
left a comment
There was a problem hiding this comment.
Endpoints and power domains check out against the J714s blocks, and both ports have a SuperSpeed device on hardware. Thanks for the right-port test.
ac601ea
into
aurora-silicon:integration/aurora
[Joshua's agent]
Describes the two USB-C ports of the 14-inch M5 Max MacBook Pro (K114c) that Linux drives, so a USB device on them enumerates. Both ports carry USB only. Not described: the left-back port (ATC0), which is the boot and debug link; USB4 tunnels (ACIO); PCIe tunnelling; DisplayPort alt mode.
Generation 4 interrupt layout is not supported.apple,no-liquid-detection, as on J714s.Test
usb 2-1, r8152 bound (enu1: carrier on),/sys/class/typec/port0-partnerappeared on controller1-0a, DHCP gave an address and ssh over the adapter worked.apple-spmi 28a1a8000.spmiprobes, controller2-0cregisters/sys/class/typec/port1with a partner (a USB-C power supply was attached: power role sink,tps6598x-source-psy-2-0conline) and no interrupt is registered for the bus (polled). A USB data device on the right port is not tested yet.Draft until a USB device is tested on the right port.