Skip to content

Conversation

@romanz
Copy link
Contributor

@romanz romanz commented Nov 10, 2025

Otherwise, the device may get "stuck" with an older seq_bit while Suite is waiting for a newer one (when the last THP ACK is lost and the host USB connection is closed).

@romanz romanz self-assigned this Nov 10, 2025
@trezor-bot trezor-bot bot added this to Firmware Nov 10, 2025
@github-project-automation github-project-automation bot moved this to 🔎 Needs review in Firmware Nov 10, 2025
@romanz romanz added the T3W1 Trezor Safe 7 label Nov 10, 2025
@github-actions
Copy link

github-actions bot commented Nov 10, 2025

en main(all)

model device_test click_test persistence_test
T2T1 test(all) main(all) test(all) main(all) test(all) main(all)
T3B1 test(all) main(all) test(all) main(all) test(all) main(all)
T3T1 test(all) main(all) test(all) main(all) test(all) main(all)
T3W1 test(all) main(all) test(all) main(all) test(all) main(all)

Latest CI run: 19297545534

@romanz romanz force-pushed the romanz/thp-seq-bit branch from 3c0f997 to 728ce00 Compare November 10, 2025 15:01
@romanz romanz marked this pull request as ready for review November 10, 2025 15:44
@romanz romanz requested a review from M1nd3r November 10, 2025 15:44
Copy link
Member

@mmilata mmilata left a comment

Choose a reason for hiding this comment

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

utACK but let's maybe also wait for @M1nd3r

Also pls make sure to notify Suite devs that this is coming 🙏

Make it compatible with Suite behaviour in case the last THP ACK is lost.
@romanz romanz force-pushed the romanz/thp-seq-bit branch from 728ce00 to df0ce90 Compare November 11, 2025 15:05
@romanz romanz marked this pull request as draft November 12, 2025 10:40
@romanz romanz marked this pull request as ready for review November 12, 2025 12:31
@romanz romanz requested a review from mmilata November 12, 2025 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T3W1 Trezor Safe 7

Projects

Status: 🔎 Needs review

Development

Successfully merging this pull request may close these issues.

3 participants