Skip to content

Commit 233214f

Browse files
committed
Modified the TxTimeout workaround comment to be hopefully more clear.
1 parent 7ba49c4 commit 233214f

File tree

2 files changed

+12
-8
lines changed

2 files changed

+12
-8
lines changed

src/radio/sx1272/sx1272.c

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1225,10 +1225,12 @@ void SX1272OnTimeoutIrq( void* context )
12251225
break;
12261226
case RF_TX_RUNNING:
12271227
// Tx timeout shouldn't happen.
1228-
// But it has been observed that when it happens it is a result of a corrupted SPI transfer
1229-
// it depends on the platform design.
1230-
//
1231-
// The workaround is to put the radio in a known state. Thus, we re-initialize it.
1228+
// Reported issue of SPI data corruption resulting in TX TIMEOUT
1229+
// is NOT related to a bug in radio transceiver.
1230+
// It is mainly caused by improper PCB routing of SPI lines and/or
1231+
// violation of SPI specifications.
1232+
// To mitigate redesign, Semtech offers a workaround which resets
1233+
// the radio transceiver and putting it into a known state.
12321234

12331235
// BEGIN WORKAROUND
12341236

src/radio/sx1276/sx1276.c

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1386,10 +1386,12 @@ void SX1276OnTimeoutIrq( void* context )
13861386
break;
13871387
case RF_TX_RUNNING:
13881388
// Tx timeout shouldn't happen.
1389-
// But it has been observed that when it happens it is a result of a corrupted SPI transfer
1390-
// it depends on the platform design.
1391-
//
1392-
// The workaround is to put the radio in a known state. Thus, we re-initialize it.
1389+
// Reported issue of SPI data corruption resulting in TX TIMEOUT
1390+
// is NOT related to a bug in radio transceiver.
1391+
// It is mainly caused by improper PCB routing of SPI lines and/or
1392+
// violation of SPI specifications.
1393+
// To mitigate redesign, Semtech offers a workaround which resets
1394+
// the radio transceiver and putting it into a known state.
13931395

13941396
// BEGIN WORKAROUND
13951397

0 commit comments

Comments
 (0)