chore(core): libtropic - rename timeout to timeout_ms [no changelog]
What changed, and why it matters
This is a simple variable rename from 'timeout' to 'timeout_ms' in a single function. It does not change any behavior, logic, or security properties of the code. There is no security issue here.
No security action required. This is a non-functional rename.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit renames the parameter ‘timeout’ to ‘timeout_ms’ in lt_port_spi_transfer() and updates its single call site in HAL_SPI_TransmitReceive(). The type remains uint32_t and no functional changes are made. This is a cosmetic/readability change only.
Changed components
core/embed/sec/tropic/stm32/tropic01.cInspect captured patch +2 / −2
diff --git a/core/embed/sec/tropic/stm32/tropic01.c b/core/embed/sec/tropic/stm32/tropic01.c
index cb675cbfb..3df6bf1b3 100644
--- a/core/embed/sec/tropic/stm32/tropic01.c
+++ b/core/embed/sec/tropic/stm32/tropic01.c
@@ -160,14 +160,14 @@ lt_ret_t lt_port_spi_csn_high(lt_handle_t *h) {
}
lt_ret_t lt_port_spi_transfer(lt_handle_t *h, uint8_t offset, uint16_t tx_len,
- uint32_t timeout) {
+ uint32_t timeout_ms) {
tropic01_hal_driver_t *drv = &g_tropic01_hal_driver;
if (offset + tx_len > LT_L1_LEN_MAX) {
return LT_L1_DATA_LEN_ERROR;
}
int ret = HAL_SPI_TransmitReceive(&drv->spi, h->l2.buff + offset,
- h->l2.buff + offset, tx_len, timeout);
+ h->l2.buff + offset, tx_len, timeout_ms);
if (ret != HAL_OK) {
return LT_FAIL;
}
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.