diff options
author | Harald Welte <laforge@gnumonks.org> | 2018-12-12 23:20:29 +0100 |
---|---|---|
committer | Harald Welte <laforge@gnumonks.org> | 2018-12-23 10:38:23 +0000 |
commit | a4f562240863393fb6c1dc3b9261add253794ddf (patch) | |
tree | e85eb35326d310576dc56c667840270072267a1c /src/osmo-bts-trx/loops.h | |
parent | 3379250b518dd236ec38dc999f40205b9c3fab8b (diff) |
osmo-bts-trx/loops.[ch]: Make functions "void" if they always return 0
The concept of a return value only makes sense if there's actually ever
something non-constant to return, and if the caller actually processes
that return value. If we always "return 0" and ignore it on the caller
side, functions should be of "void" type.
Change-Id: I3575a2cef75f3fd4c3f95eddb40719d28a055b54
Related: OS#1622, OS#1851
Diffstat (limited to 'src/osmo-bts-trx/loops.h')
-rw-r--r-- | src/osmo-bts-trx/loops.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/osmo-bts-trx/loops.h b/src/osmo-bts-trx/loops.h index f9e69c84..50a658d0 100644 --- a/src/osmo-bts-trx/loops.h +++ b/src/osmo-bts-trx/loops.h @@ -13,15 +13,15 @@ * loops api */ -int trx_loop_sacch_input(struct l1sched_trx *l1t, uint8_t chan_nr, +void trx_loop_sacch_input(struct l1sched_trx *l1t, uint8_t chan_nr, struct l1sched_chan_state *chan_state, int8_t rssi, int16_t toa); -int trx_loop_sacch_clock(struct l1sched_trx *l1t, uint8_t chan_nr, +void trx_loop_sacch_clock(struct l1sched_trx *l1t, uint8_t chan_nr, struct l1sched_chan_state *chan_state); -int trx_loop_amr_input(struct l1sched_trx *l1t, uint8_t chan_nr, +void trx_loop_amr_input(struct l1sched_trx *l1t, uint8_t chan_nr, struct l1sched_chan_state *chan_state, float ber); -int trx_loop_amr_set(struct l1sched_chan_state *chan_state, int loop); +void trx_loop_amr_set(struct l1sched_chan_state *chan_state, int loop); #endif /* _TRX_LOOPS_H */ |