Skip to content

tkey-runapp: reset TKey if it is not in firmware mode - #32

Open
dehanj wants to merge 3 commits into
mainfrom
add_reset
Open

dehanj wants to merge 3 commits into
mainfrom
add_reset

Conversation

@dehanj

@dehanj dehanj commented Sep 16, 2026

Copy link
Copy Markdown
Member

Description

Updated tkey-runapp to use newer tkeyclient in order to test the new reset command, see tillitis/tkeyclient#31

Feature description:
Support resetting a TKey before loading the app, given that the TKey supports it and an app is already loaded.

If the app or the TKey does not support a reset, a manual re-plug is still needed.

Updates tkeyclient to pre-released version for the new reset, reconnect, and error types.

I think this should supersede #31.
Compared to that PR this is lacking a separate sub-command/flag to perform just a reset while not loading an app. It would need to support choosing which reset type, and a way to input nextAppData. I purpose that we add that in a later PR.

Type of change

  • Feature (non breaking change which adds functionality)

Submission checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my changes
  • I have tested and verified my changes on target
  • My changes are well written and CI is passing
  • I have squashed my work to relevant commits and rebased on main for linear history
  • I have added a "Co-authored-by: x" if several people contributed, either pair programming or by squashing commits from different authors.
  • I have updated the documentation where relevant (readme, dev.tillitis.se etc.)
  • QEMU is updated to reflect changes

dehanj and others added 2 commits September 16, 2026 15:05
Support resetting a TKey before loading the app, given that the TKey
supports it and an app is already loaded.

If the app or the TKey does not support a reset, a manual re-plug is
still needed.

Updates tkeyclient to pre-released version for the new reset, reconnect,
and error types.
Gives a slighlt higher value 33, than the limit. Turn off for now,
refactor later.
@dehanj
dehanj marked this pull request as ready for review September 16, 2026 13:29
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