arno/trezor-firmware
1
0
Fork 0
mirror of https://github.com/trezor/trezor-firmware.git synced 2025-01-15 18:00:59 +00:00
Code Issues Releases Wiki Activity

fix(core): fix wrong RSOD color on some older Model T devices

Browse Source
This commit is contained in:
tychovrahe 2025-01-12 15:35:14 +01:00 committed by TychoVrahe
parent b726e1e203
commit 34e033fd4e
4 changed files with 26 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
core/.changelog.d/4491.fixed Normal file
View File

@ -0,0 +1 @@
[T2T1] Fix wrong RSOD color on some older Model T devices

6
core/embed/io/display/st-7789/display_driver.c
View File

@ -61,6 +61,12 @@ bool display_init(display_content_mode_t mode) {
#endif #endif
if (mode == DISPLAY_RESET_CONTENT) { if (mode == DISPLAY_RESET_CONTENT) {
#if defined TREZOR_MODEL_T2T1 && !defined BOARDLOADER
// This is required for the model T to work correctly.
// Boardloader does this by constant in binary, other stages need to read
// this from the display
display_panel_preserve_inversion();
#endif
display_io_init_gpio(); display_io_init_gpio();
display_io_init_fmc(); display_io_init_fmc();
display_panel_init(); display_panel_init();

18
core/embed/io/display/st-7789/display_panel.c
View File

@ -47,14 +47,26 @@
#ifdef KERNEL_MODE #ifdef KERNEL_MODE
#ifdef TREZOR_MODEL_T2T1
#ifdef BOARDLOADER
// using const volatile instead of #define results in binaries that change // using const volatile instead of #define results in binaries that change
// only in 1-byte when the flag changes. // only in 1-byte when the flag changes.
// using #define leads compiler to over-optimize the code leading to bigger // using #define leads compiler to over-optimize the code leading to bigger
// differencies in the resulting binaries. // differences in the resulting binaries.
const volatile uint8_t DISPLAY_ST7789V_INVERT_COLORS2 = 1; const volatile uint8_t DISPLAY_ST7789V_INVERT_COLORS2 = 1;
#else
volatile uint8_t DISPLAY_ST7789V_INVERT_COLORS2 = 0;
void display_panel_preserve_inversion(void) {
DISPLAY_ST7789V_INVERT_COLORS2 = display_panel_is_inverted();
}
#endif
#endif
// Window padding (correction) when using 90dg or 270dg orientation // Window padding (correction) when using 90dg or 270dg orientation
// (internally the display is 240x320 but we use only 240x240) // (internally the display is 240x320, but we use only 240x240)
static display_padding_t g_window_padding; static display_padding_t g_window_padding;
#ifdef DISPLAY_IDENTIFY #ifdef DISPLAY_IDENTIFY
@ -232,7 +244,7 @@ void display_panel_init(void) {
} }
void display_panel_reinit(void) { void display_panel_reinit(void) {
// reinitialization is needed due to original sequence is unchangable in // reinitialization is needed due to original sequence is unchangeable in
// boardloader // boardloader
#ifdef TREZOR_MODEL_T2T1 #ifdef TREZOR_MODEL_T2T1
// model TT has new gamma settings // model TT has new gamma settings

4
core/embed/io/display/st-7789/display_panel.h
View File

@ -55,4 +55,8 @@ void display_panel_set_window(uint16_t x0, uint16_t y0, uint16_t x1,
uint16_t y1); uint16_t y1);
void display_panel_rotate(int angle); void display_panel_rotate(int angle);
#if defined TREZOR_MODEL_T2T1 && !defined BOARDLOADER
void display_panel_preserve_inversion(void);
#endif
#endif // TREZORHAL_ST7789_PANEL_H #endif // TREZORHAL_ST7789_PANEL_H