Skip to content

Commit 9e551c9

Browse files
committed
remove _ff_overflowed() due to lack of use
1 parent c29b764 commit 9e551c9

File tree

1 file changed

+1
-12
lines changed

1 file changed

+1
-12
lines changed

src/common/tusb_fifo.c

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -331,17 +331,6 @@ uint16_t _ff_count(uint16_t depth, uint16_t wr_idx, uint16_t rd_idx)
331331
}
332332
}
333333

334-
// BE AWARE - THIS FUNCTION MIGHT NOT GIVE A CORRECT ANSWERE IN CASE WRITE POINTER "OVERFLOWS"
335-
// Only one overflow is allowed for this function to work e.g. if depth = 100, you must not
336-
// write more than 2*depth-1 items in one rush without updating write pointer. Otherwise
337-
// write pointer wraps and you pointer states are messed up. This can only happen if you
338-
// use DMAs, write functions do not allow such an error.
339-
TU_ATTR_ALWAYS_INLINE static inline
340-
bool _ff_overflowed(uint16_t depth, uint16_t wr_idx, uint16_t rd_idx)
341-
{
342-
return _ff_count(depth, wr_idx, rd_idx) > depth;
343-
}
344-
345334
// return remaining slot in fifo
346335
TU_ATTR_ALWAYS_INLINE static inline
347336
uint16_t _ff_remaining(uint16_t depth, uint16_t wr_idx, uint16_t rd_idx)
@@ -672,7 +661,7 @@ uint16_t tu_fifo_remaining(tu_fifo_t* f)
672661
/******************************************************************************/
673662
bool tu_fifo_overflowed(tu_fifo_t* f)
674663
{
675-
return _ff_overflowed(f->depth, f->wr_idx, f->rd_idx);
664+
return _ff_count(f->depth, f->wr_idx, f->rd_idx) > f->depth;
676665
}
677666

678667
// Only use in case tu_fifo_overflow() returned true!

0 commit comments

Comments
 (0)