Skip to content

HID: magicmouse: report the J616s trackpad's palms - #1510

Merged
RyanTheTide merged 1 commit into
aurora-silicon:integration/aurorafrom
jgoyvaerts:m4pro/j616s-trackpad-palms
Nov 13, 2026
Merged

RyanTheTide merged 1 commit into
aurora-silicon:integration/aurorafrom
jgoyvaerts:m4pro/j616s-trackpad-palms

Conversation

@jgoyvaerts

@jgoyvaerts jgoyvaerts commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

The J616s (MacBook Pro 16" M4 Pro) trackpad sends the standard MTP report and classes its contacts like the T6041 one (#636), in the low byte of a record's second word. This reports its palms as MT_TOOL_PALM through the T6041 path.

Recorded on a J616s, three 12 s sessions:

session class touch_major tool_major
fingers 2, 3 (some 4) <= 227 <= 2475
thumb 1 <= 673 <= 2094
palm 6, 7 1861-2358 2754-2905

The class rule (6 and above with a tool area over 1400) separates these. The T6041 rule that makes a contact a palm once the tool area alone passes 2200 stays T6041 only (new mtp_palm_size), because J616s fingers reach 2475. Only machines whose root is apple,j616s change; T6041 behaves as before.

Testing: on the J616s daily kernel since 11-07, where palm rejection was checked by hand in normal use. Builds clean on integration/aurora 451dc70 (W=1); checkpatch clean.

🤖 Generated with Claude Code

The J616s (MacBook Pro M4 Pro) trackpad sends the standard MTP report
and classes its contacts like the T6041 one, in the low byte of a
record's second word. Recorded on a J616s, three 12 s sessions:

  session  class          touch_major  tool_major
  fingers  2, 3 (some 4)  <= 227       <= 2475
  thumb    1              <= 673       <= 2094
  palm     6, 7           1861-2358    2754-2905

Report its palms as MT_TOOL_PALM through the T6041 path. Its rule that
makes a contact a palm once the tool area alone passes 2200 stays
T6041 only: J616s fingers reach 2475. The class rule (6 and above with
a tool area over 1400) separates the sessions above. Only machines
whose root is apple,j616s change.

Signed-off-by: Jonathan Goyvaerts <jonathan.goyvaerts@gmail.com>
Assisted-by: LLM

@RyanTheTide RyanTheTide 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.

The class rule and the T6041-only size rule read right. We will run a trackpad check on our J616s (palms while typing, taps, scroll, gestures, thumb) and come back with the result.

@RyanTheTide

Copy link
Copy Markdown
Contributor

Ran this on our J616s (M4 Pro), merged on integration e661192, as a one-shot test kernel.

  • The Apple MTP multi-touch device now advertises ABS_MT_TOOL_TYPE with max 15 (production: max 0), so palm contacts can be reported as MT_TOOL_PALM.
  • Keyboard and trackpad bind as before, with no HID/SPI errors. They still work after 3 s2idle cycles.
  • 0 AVC, 0 kernel errors.

Nobody was at the machine, so we did not do a hands-on palm/tap/gesture pass. The capability change and the unchanged binding are what we could check remotely.

@RyanTheTide
RyanTheTide merged commit 7392e90 into aurora-silicon:integration/aurora Nov 13, 2026
1 check 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