TPDU support, take 2 #324

Merged
PeterCxy merged 5 commits from tpdu into master 2026-03-25 22:44:40 +01:00
Owner

We now know that some readers won't actually work with TPDU even if they advertise it. So, reintroduce the changes but now we only enable if the reader is a known one that requires TPDU or if the user enables it via a setting.

We now know that some readers won't actually work with TPDU even if they advertise it. So, reintroduce the changes but now we only enable if the reader is a known one that requires TPDU or if the user enables it via a setting.
resolves #37

For TPDU based readers like USB 2.0-CRW 2 additional commands are needed in initialisation. Add them

Reviewed-on: #295
Reviewed-by: septs <github@septs.pw>
Co-authored-by: Vladimir Serbinenko <phcoder@gmail.com>
Co-committed-by: Vladimir Serbinenko <phcoder@gmail.com>
Prefer APDU mode when advertised by CCID reader
All checks were successful
Build Debug APKs / build-debug (push) Successful in 4m23s
002c90ba11
This reverts commit 002c90ba11.
Make TPDU mode optional, enabled via settings OR known device quirks
All checks were successful
Build Debug APKs / build-debug (push) Successful in 4m32s
35be370c16
Log USB vendorId:productId correctly
All checks were successful
Build Debug APKs / build-debug (push) Successful in 4m25s
9b635d05c3
PeterCxy merged commit 1c70ca7a70 into master 2026-03-25 22:44:40 +01:00
PeterCxy deleted branch tpdu 2026-03-25 22:44:40 +01:00
PeterCxy referenced this pull request from a commit 2026-03-25 22:44:42 +01:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
PeterCxy/OpenEUICC!324
No description provided.