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

add support for T3B1 #3728

Merged
merged 6 commits into from
Jul 16, 2024
Merged

add support for T3B1 #3728

merged 6 commits into from
Jul 16, 2024

Conversation

TychoVrahe
Copy link
Contributor

@TychoVrahe TychoVrahe commented Apr 17, 2024

Implements support for T3B1.

Refactored vendor headers a bit, basically just moved it to the common model folder.

DEVICE_CERT_CHAIN is mock for now, using values for T2B1. Also it should get same treatment as vendor headers..

Consumption mask still needs some timing adjustments.

Added missing font that affected/crashed bootloader for both T2B1 and T3B1.

fixes #3785

@TychoVrahe TychoVrahe self-assigned this Apr 17, 2024
@TychoVrahe TychoVrahe force-pushed the tychovrahe/t3b1/hw_support branch 6 times, most recently from 886229b to 001f374 Compare July 2, 2024 10:57
@TychoVrahe TychoVrahe marked this pull request as ready for review July 2, 2024 10:57
@TychoVrahe TychoVrahe requested review from cepetr and removed request for prusnak July 2, 2024 10:57
@TychoVrahe
Copy link
Contributor Author

@matejcik please review the production keys c9bbc71, device cert chain cdc2cd7, and conceptually vendor header restructuralization in 2993ad1

Copy link
Contributor

@cepetr cepetr left a comment

Choose a reason for hiding this comment

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

It looks good from my side. I just found a few minor issues.

core/Makefile Outdated Show resolved Hide resolved
core/embed/trezorhal/stm32u5/consumption_mask.c Outdated Show resolved Hide resolved
core/embed/models/T3B1/boards/trezor_t3b1_revB.h Outdated Show resolved Hide resolved
@TychoVrahe TychoVrahe requested a review from cepetr July 15, 2024 12:09
Copy link
Contributor

@cepetr cepetr left a comment

Choose a reason for hiding this comment

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

utACK

@TychoVrahe TychoVrahe force-pushed the tychovrahe/t3b1/hw_support branch from 28a002a to b14248b Compare July 16, 2024 12:47
@TychoVrahe TychoVrahe force-pushed the tychovrahe/t3b1/hw_support branch from b14248b to f3dbf05 Compare July 16, 2024 13:09
@TychoVrahe TychoVrahe merged commit 03199cd into main Jul 16, 2024
83 of 84 checks passed
@TychoVrahe TychoVrahe deleted the tychovrahe/t3b1/hw_support branch July 16, 2024 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Change manufacturer a product name on T3B1
3 participants