diff options
| author | Leah Rowe <leah@libreboot.org> | 2026-03-12 04:53:05 +0000 |
|---|---|---|
| committer | Leah Rowe <leah@libreboot.org> | 2026-03-12 04:53:05 +0000 |
| commit | 1953812efd3544934b7d1df8f6c5aa300f012a80 (patch) | |
| tree | 9946981ebd0aeaa35fb87f4d9d0ede52bc0be2af | |
| parent | a31ccd8c3dc87010ab44cac6027477b62ec446b3 (diff) | |
util/spkmodem-recv: split up handle_audio()
the signal check should be its own function,
for clearer understanding
Signed-off-by: Leah Rowe <leah@libreboot.org>
| -rw-r--r-- | util/spkmodem_recv/spkmodem-recv.c | 15 |
1 files changed, 11 insertions, 4 deletions
diff --git a/util/spkmodem_recv/spkmodem-recv.c b/util/spkmodem_recv/spkmodem-recv.c index 29832919..58f59294 100644 --- a/util/spkmodem_recv/spkmodem-recv.c +++ b/util/spkmodem_recv/spkmodem-recv.c @@ -56,6 +56,7 @@ struct decoder_state { static const char *argv0; static void handle_audio(struct decoder_state *st); +static int valid_signal(struct decoder_state *st); static void decode_pulse(struct decoder_state *st); static int set_ascii_bit(struct decoder_state *st); static void print_char(struct decoder_state *st); @@ -123,10 +124,7 @@ handle_audio(struct decoder_state *st) if (st->sample_count > (3 * SAMPLES_PER_FRAME)) reset_char(st); - if ((st->freq_separator > FREQ_SEP_MIN) && - (st->freq_separator < FREQ_SEP_MAX) && - (st->freq_data > FREQ_DATA_MIN) && - (st->freq_data < FREQ_DATA_MAX)) { + if (valid_signal(st)) { if (set_ascii_bit(st) < 0) print_char(st); @@ -141,6 +139,15 @@ handle_audio(struct decoder_state *st) } } +static int +valid_signal(struct decoder_state *st) +{ + return (st->freq_separator > FREQ_SEP_MIN && + st->freq_separator < FREQ_SEP_MAX && + st->freq_data > FREQ_DATA_MIN && + st->freq_data < FREQ_DATA_MAX); +} + static void decode_pulse(struct decoder_state *st) { |