Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Uclogic XP-Pen Aritst 22HD all over again #133

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

miharenthas
Copy link

Hi again,

I found a little time to fiddle with abisaro's code, presented in pull request #73
As a quick refresher: there were two report descriptor proposed, both rather lengthy and the last status was "let's make the driver a little cleaner".

It turns out that the first report descriptor (_rdesc0) doesn't do much: by using just that the monitor is recognized, but not really the pen --and thus the stylus is also non mappable to a particular output (which I suppose it's usually needed, since I doubt these devices are used as the only monitor of a computer).

The second report descriptor, on the other hand, runs the whole thing: the monitor is recognized (as a mouse?) and, as soon as it makes contact with the screen, the pen is also listed. And it works, pressure sensitivity included.

I wouldn't classify that as a progress, though: I essentially copy and pasted half abisaro's effort in the new format and didn't actually simplify anything --and the weird double "pen pen (0)" device is still on.

But it works just as the in the old pull request, so at least I didn't ruin anything. Yet.

I'm awaiting instructions to make this good enough to be merged!

It's only one of the interfaces found in albisaro's driver --the firts one seems to be redundant, and doesn't actually drive the pen.
Still the weird mouse and pen seen as separate devices --with the pen appearing only after the first contact with the monitor. Have no idea why that happens.
@spbnick
Copy link
Member

spbnick commented Apr 16, 2018

Hi Lorenzo, thank you for your efforts. Please take a look at the new version of the driver I'm working on right now, and see if you would be able to do anything with it.

@miharenthas
Copy link
Author

Hi Nikolai,
Possibly: I'll attempt to integrate the rdesc this evening.

@miharenthas
Copy link
Author

miharenthas commented Apr 17, 2018

So: I made an attempt to make the code work with the hires_refactor branch, but I didn't succeed yet.
The device appears duplicate, both time listed as "UGEE TABLET MONITOR Mouse", and there's no pen.

I think I need to understand your newer driver more deeply before being actually able to make this work --reading the code, it seems that the pen is treated somewhat separately. Something that I don't yet know how to do.

You can see what I did here.

Also, there's a "tablet 47" that has the same device code as the xppen artist22hd. Which might be weird, but I can't really tell.

@spbnick
Copy link
Member

spbnick commented May 4, 2018

Everyone involved with this PR, you're welcome to join the new #DIGImend channel on irc.freenode.net and discuss this PR, other issues and help each other! Come and say "Hi!"

…nt about a missing flag --that looked like it came out of the Kernel itself. But who knows. Seem to work now.
@spbnick
Copy link
Member

spbnick commented Sep 8, 2018

@miharenthas, @abisaro, do you still have your XP-Pen Artist 22HD tablets? Could you please try the latest digimend-kernel-drivers? It seems that your tablet should be supported without modifying the code now.

@abisaro
Copy link

abisaro commented Sep 8, 2018 via email

@miharenthas
Copy link
Author

miharenthas commented Sep 9, 2018 via email

@abisaro
Copy link

abisaro commented Sep 9, 2018 via email

@spbnick
Copy link
Member

spbnick commented Sep 10, 2018

Sounds good, thank you for the report, @abisaro!

@miharenthas
Copy link
Author

miharenthas commented Oct 24, 2018

Hi Nikolai,

Not quick, but another XP-Pen Artist 22HD working here with the latest driver. Thanks a lot!

Cheers,
L.

@Deevad Deevad mentioned this pull request May 12, 2021
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.

3 participants