Skip to content

Adds support for Dell Active Pen PN7320A#872

Merged
whot merged 7 commits intolinuxwacom:masterfrom
joaommartins:master
Mar 24, 2025
Merged

Adds support for Dell Active Pen PN7320A#872
whot merged 7 commits intolinuxwacom:masterfrom
joaommartins:master

Conversation

@joaommartins
Copy link
Contributor

Includes support for Dell Latitude 7320 Detachable integrated Wacom pen sensor, and the accompanying pen.

@whot whot added the sysinfo needed Used to indicate sysinfo is missing from the PR label Mar 21, 2025
@github-actions
Copy link

Hi, there. I'm a bot and have been asked to have a look at this.

Please run the sysinfo script from https://github.com/linuxwacom/wacom-hid-descriptors and
file a new issue in that repo with the resulting sysinfo tarball. The README in that repo
has more information and instructions.

The .tablet files in this PR should then link to the sysinfo in this form:

# sysinfo.abCdE1234.tar.gz
# https://github.com/linuxwacom/wacom-hid-descriptors/issues/1234

A git grep sysinfo will show how this is currently used in existing data files.

See also https://github.com/linuxwacom/libwacom/wiki/Adding-a-new-device


This is an automated comment created by a bot. Responding to the bot or mentioning it won't have any effect.

Comment on lines 170 to 174
Group=isdv4-aes
Buttons=1
EraserType=Button
Axes=Pressure
Type=Mobile
Copy link
Member

Choose a reason for hiding this comment

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

With #871 merged now this can be:

Suggested change
Group=isdv4-aes
Buttons=1
EraserType=Button
Axes=Pressure
Type=Mobile
AliasOf=0x1

@joaommartins joaommartins requested a review from whot March 23, 2025 22:36
@whot whot removed the sysinfo needed Used to indicate sysinfo is missing from the PR label Mar 24, 2025
@github-actions
Copy link

Hi, there. I'm a bot and have been asked to have a look at this.

Thanks for providing the sysinfo. Someone will be along shortly to review the PR.


This is an automated comment created by a bot. Responding to the bot or mentioning it won't have any effect.

@whot
Copy link
Member

whot commented Mar 24, 2025

Alrighty, the remaining complaint is because this device was added in 792ce0f, so all we need here is the stylus. Can you drop the .tablet file from this MR please, thanks.

@joaommartins
Copy link
Contributor Author

Tablet definition dropped in favour of existing one.

@joaommartins joaommartins changed the title Adds support for Dell Latitude 7320 Detachable and Dell Active Pen PN7320A Adds support for Dell Active Pen PN7320A Mar 24, 2025
@whot whot merged commit 8b6a0c1 into linuxwacom:master Mar 24, 2025
17 checks passed
@whot
Copy link
Member

whot commented Mar 24, 2025

Thanks, squashed and merged

@joaommartins
Copy link
Contributor Author

Thanks @whot

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