Skip to content

arm64: dts: apple: t6050: make die0's range cover the ISP watchdog - #1854

Merged
DjDeveloperr merged 1 commit into
aurora-silicon:integration/aurorafrom
MarianoMiguel:t6050-isp-wdt-ranges
Nov 13, 2026
Merged

DjDeveloperr merged 1 commit into
aurora-silicon:integration/aurorafrom
MarianoMiguel:t6050-isp-wdt-ranges

Conversation

@MarianoMiguel

Copy link
Copy Markdown
Contributor

The 16-inch M5 Max camera (J716c) only gave black frames, even with the camera light on.

Why: the camera chip has a watchdog register at 0x5c3b50000, just past the end of the soc@200000000 bus range (0x5c0000000). Linux drops it, nothing feeds the watchdog, and the camera firmware sends flat frames instead of the picture.

Fix: make the range a bit bigger, up to 0x600000000. Nothing else lives there.

Tested on a J716c (macOS 27.3 beta stub, Aura + integration/aurora):

  • before: 450 frames, all flat
  • after: the watchdog is fed about 30 times a second, and the camera streams a real image

All T6050 device trees build with no new warnings. #1756 (K114c) uses the same register and bus, so it probably needs this too.

The camera's ISP has a watchdog register at 0x5c3b50000, but the
soc@200000000 bus range stops at 0x5c0000000. The kernel drops that
register, nothing feeds the watchdog, and the ISP firmware sends flat
frames instead of the picture.

Extend the range to 0x600000000 so the register is reachable. Nothing
else sits in the added space.

Tested on a 16-inch M5 Max (J716c): before, 450 flat frames; after,
the watchdog is fed about 30 times a second and the camera streams a
real image.

Assisted-by: LLM
Signed-off-by: Mariano Miguel <mariano@mariano.io>

@DjDeveloperr DjDeveloperr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approve: no other T6050 node lives between 0x5c0000000 and 0x600000000, so widening die0's range only brings the ISP watchdog register into reach. The J716c streaming result covers it.

@DjDeveloperr
DjDeveloperr merged commit 31a03da into aurora-silicon:integration/aurora Nov 13, 2026
2 checks passed
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.

2 participants