summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--util/nvmutil/nvmutil.c239
-rw-r--r--util/spkmodem_decode/.gitignore2
-rw-r--r--util/spkmodem_decode/Makefile (renamed from util/spkmodem_recv/Makefile)6
-rw-r--r--util/spkmodem_decode/spkmodem-decode.c496
-rw-r--r--util/spkmodem_recv/.gitignore1
-rw-r--r--util/spkmodem_recv/spkmodem-recv.c313
6 files changed, 710 insertions, 347 deletions
diff --git a/util/nvmutil/nvmutil.c b/util/nvmutil/nvmutil.c
index 3608c439..b718ac6e 100644
--- a/util/nvmutil/nvmutil.c
+++ b/util/nvmutil/nvmutil.c
@@ -15,6 +15,135 @@
* -Os -Wall -Wextra -Werror -pedantic -std=c90
*/
+/*
+ * Major TODO: split this into multiple files.
+ * This program has become quite large now, mostly
+ * due to all the extra sanity checks / portability.
+ * Make most of nvmutil a *library* for re-use
+ *
+ * TODO: gettimeofday not posible - use portable functions.
+ * TODO: uint32_t fallback: modify the program instead
+ * to run on 16-bit systems: smaller buffers, and do
+ * operations byte-based instead of word-based.
+ *
+ * TODO: _XOPEN_SOURCE 500 probably not needed anymore.
+ * the portable fallbacks alone are likely enough.
+ * e.g. i don't need stdint, and i don't use pwrite/pread
+ * anymore.
+ *
+ * TODO: version detection of various BSDs to detect
+ * arc4random, use that if available. but also work on
+ * older versions of those BSDs (also MacOS) that lack it.
+ *
+ * TODO: portability/testing on non-Unix systems:
+ * old DOS. all windows versions (probably irrelevant
+ * because you can use cygwin/wsl, whatever), classic MacOS,
+ * also test really old unix e.g. sunos and irix. Be/Haiku too!
+ *
+ * TODO: reliance on global variables for status. make
+ * functions use structs passed as args instead, make
+ * functions re-useable (including libraries), etc.
+ *
+ * TODO: bound checks for files per-command, e.g. only
+ * first 6 bytes for CMD_SETMAC
+ *
+ * TODO: clean up the do_rw function: make PSCHREIB and
+ * so on clearer, probably just define them inline and
+ * validate them inline (no define).
+ * TODO: in command sanitizer: verify that each given
+ * entry corresponds to the correct function, in the
+ * pointer (this check is currently missing)
+ *
+ * TODO: general modularisierung of the entire codebase.
+ * TODO: better explain copy/swap read inversion trick
+ * by improving existing comments
+ * TODO: lots of overwritten comments in code. tidy it up.
+ *
+ * TODO: use getopt for nvmutil args, so that multiple
+ * operations can be performed, and also on many
+ * files at once (noting limitations with cat)
+ * BONUS: implement own getopt(), for portability
+ *
+ * TODO: document fuzzing / static analysis methods
+ * for the code, and:
+ * TODO: implement rigorous unit tests (separate util)
+ * NOTE: this would *include* known good test files
+ * in various configurations, also invalid files.
+ * the tests would likely be portable posix shell
+ * scripts rather than a new C program, but a modularisiert
+ * codebase would allow me to write a separate C
+ * program to test some finer intricacies
+ * TODO: the unit tests would basically test regressions
+ * TODO: after writing back a gbe to file, close() and
+ * open() it again, read it again, and check that
+ * the contents were written correctly, providing
+ * a warning if they were. do this in the main
+ * program.
+ * TODO: the unit tests would include an aggressive set
+ * of fuzz tests, under controlled conditions
+ *
+ * TODO: also document the layout of Intel GbE files, so
+ * that wily individuals can easily expand the
+ * featureset of nvmutil.
+ * TODO: remove some clever code, e.g.:
+ * rw_type == PLESEN << 2
+ * make stuff like that clearer.
+ * ditto the invert copy/swap trick
+ * TODO: write a manpage
+ * TODO: simplify the command sanitization, implement more
+ * of it as build time checks, e.g. static asserts.
+ * generally remove cleverness from the code, instead
+ * prefyerring readibility
+ * TODO: also document nvmutil's coding style, which is
+ * its own style at this point!
+ * TODO: when all the above (and possibly more) is done,
+ * submit this tool to coreboot with a further change
+ * to their build system that lets users modify
+ * GbE images, especially set MAC addresses, when
+ * including GbE files in coreboot configs.
+ */
+/*
+ BONUS TODO:
+ CI/CD. woodpecker is good enough, sourcehut also has one.
+ tie this in with other things mentioned here,
+ e.g. fuzzer / unit tests
+*/
+
+/* Major TODO: reproducible builds
+Test with and without these:
+
+CFLAGS += -fno-record-gcc-switches
+CFLAGS += -ffile-prefix-map=$(PWD)=.
+CFLAGS += -fdebug-prefix-map=$(PWD)=.
+
+I already avoid unique timestamps per-build,
+by not using them, e.g. not reporting build
+time in the program.
+
+When splitting the nvmutil.c file later, do e.g.:
+
+SRC = main.c io.c nvm.c cmd.c
+OBJ = $(SRC:.c=.o)
+
+^ explicitly declare the order in which to build
+*/
+
+/*
+TODO:
+further note when fuzzing is implemented:
+use deterministic randomisation, with a
+guaranteed seed - so e.g. don't use /dev/urandom
+in test builds. e.g. just use normal rand()
+but with a static seed e.g. 1234
+*/
+/*
+TODO: stricter build flags, e.g.
+CFLAGS += -fstack-protector-strong
+CFLAGS += -fno-common
+CFLAGS += -D_FORTIFY_SOURCE=2
+CFLAGS += -fPIE
+*/
+
#ifndef _XOPEN_SOURCE
#define _XOPEN_SOURCE 500
#endif
@@ -215,6 +344,8 @@ static off_t gbe_x_offset(size_t part, const char *f_op,
const char *d_type, off_t nsize, off_t ncmp);
static ssize_t rw_file_exact(int fd, uint8_t *mem, size_t len,
off_t off, int rw_type);
+static ssize_t rw_file_once(int fd, uint8_t *mem, size_t len,
+ off_t off, int rw_type, size_t rc);
static ssize_t do_rw(int fd,
uint8_t *mem, size_t len, off_t off, int rw_type);
static ssize_t prw(int fd, void *mem, size_t nrw,
@@ -452,8 +583,8 @@ main(int argc, char *argv[])
#ifdef NVMUTIL_UNVEIL
if (pledge("stdio rpath wpath unveil", NULL) == -1)
err(errno, "pledge");
- if (unveil("/dev/null", "r") == -1)
- err(errno, "unveil '/dev/null'");
+ if (unveil("/dev/urandom", "r") == -1)
+ err(errno, "unveil /dev/urandom");
#else
if (pledge("stdio rpath wpath", NULL) == -1)
err(errno, "pledge");
@@ -1052,20 +1183,20 @@ fallback_rand(void)
static unsigned long
entropy_jitter(void)
{
- struct timeval a, b;
- unsigned long mix = 0;
- int i;
+ struct timeval a, b;
+ unsigned long mix = 0;
+ int i;
- for (i = 0; i < 8; i++) {
- gettimeofday(&a, NULL);
- getpid();
- gettimeofday(&b, NULL);
+ for (i = 0; i < 8; i++) {
+ gettimeofday(&a, NULL);
+ getpid();
+ gettimeofday(&b, NULL);
- mix ^= (unsigned long)(b.tv_usec - a.tv_usec);
- mix ^= (unsigned long)&mix;
- }
+ mix ^= (unsigned long)(b.tv_usec - a.tv_usec);
+ mix ^= (unsigned long)&mix;
+ }
- return mix;
+ return mix;
}
static void
@@ -1422,41 +1553,89 @@ gbe_x_offset(size_t p, const char *f_op, const char *d_type,
* be used on sockets or pipes, because 0-byte
* reads are treated like fatal errors. This
* means that EOF is also considered fatal.
+ *
+ * WARNING: Do not use O_APPEND on open() when
+ * using this function. If you do, POSIX allows
+ * write() to ignore the current file offset and
+ * write at EOF, which means that our use of
+ * lseek in prw() does not guarantee writing at
+ * a specified offset. So if using PSCHREIB or
+ * PLESEN, make sure not to pass a file descriptor
+ * with the O_APPEND flag. Alternatively, modify
+ * do_rw() to directly use pwrite() and pread()
+ * instead of prw().
*/
static ssize_t
rw_file_exact(int fd, uint8_t *mem, size_t len,
off_t off, int rw_type)
{
- ssize_t rval = 0;
- size_t rc = 0;
+ ssize_t rv;
+ size_t rc;
if (fd < 0 || !len || len > (size_t)SSIZE_MAX) {
errno = EIO;
return -1;
}
- while (rc < len) {
- rval = do_rw(fd, mem + rc, len - rc, off + rc, rw_type);
-
- if (rval < 0 && errno == EINTR) {
- continue;
- } else if (rval < 0) {
- errno = EIO;
+ for (rc = 0, rv = 0; rc < len; rc += (size_t)rv) {
+ if ((rv = rw_file_once(fd, mem, len, off, rw_type, rc)) == -1)
return -1;
- }
- if ((size_t)rval > (len - rc) /* Prevent overflow */
- || rval == 0) { /* Prevent infinite 0-byte loop */
- errno = EIO;
- return -1;
- }
-
- rc += (size_t)rval;
}
return rc;
}
static ssize_t
+rw_file_once(int fd, uint8_t *mem, size_t len,
+ off_t off, int rw_type, size_t rc)
+{
+ ssize_t rv;
+ size_t retries_on_zero = 0;
+ size_t max_retries = 10;
+
+read_again:
+ rv = do_rw(fd, mem + rc, len - rc, off + rc, rw_type);
+
+ if (rv < 0 && errno == EINTR) {
+ goto read_again;
+ } else if (rv < 0) {
+ errno = EIO;
+ return -1;
+ }
+
+ /*
+ * Theoretical bug: if a buggy libc returned
+ * a size larger than SSIZE_MAX, the cast may
+ * cause an overflow. Specifications guarantee
+ * this won't happen, but spec != implementation
+ */
+ if ((size_t)rv > SSIZE_MAX) {
+ errno = EIO;
+ return -1;
+ /* we will not tolerate your buggy libc */
+ }
+
+ if ((size_t)rv > (len - rc) /* Prevent overflow */
+ || rv == 0) { /* Prevent infinite 0-byte loop */
+ if (rv == 0) {
+ /*
+ * Fault tolerance against infinite
+ * zero-byte loop: re-try a finite
+ * number of times. This mitigates
+ * otherwise OK but slow filesystems
+ * e.g. NFS or slow media.
+ */
+ if (retries_on_zero++ < max_retries)
+ goto read_again;
+ }
+ errno = EIO;
+ return -1;
+ }
+
+ return rv;
+}
+
+static ssize_t
do_rw(int fd, uint8_t *mem,
size_t len, off_t off, int rw_type)
{
diff --git a/util/spkmodem_decode/.gitignore b/util/spkmodem_decode/.gitignore
new file mode 100644
index 00000000..42814fe4
--- /dev/null
+++ b/util/spkmodem_decode/.gitignore
@@ -0,0 +1,2 @@
+spkmodem-recv
+spkmodem-decode
diff --git a/util/spkmodem_recv/Makefile b/util/spkmodem_decode/Makefile
index 293d7475..b00c4f43 100644
--- a/util/spkmodem_recv/Makefile
+++ b/util/spkmodem_decode/Makefile
@@ -8,12 +8,12 @@ DESTDIR?=
PREFIX?=/usr/local
INSTALL?=install
-PROG=spkmodem-recv
+PROG=spkmodem-decode
all: $(PROG)
-$(PROG): spkmodem-recv.c
- $(CC) $(CFLAGS) spkmodem-recv.c -o $(PROG)
+$(PROG): spkmodem-decode.c
+ $(CC) $(CFLAGS) spkmodem-decode.c -o $(PROG)
install: $(PROG)
mkdir -p $(DESTDIR)$(PREFIX)/bin/
diff --git a/util/spkmodem_decode/spkmodem-decode.c b/util/spkmodem_decode/spkmodem-decode.c
new file mode 100644
index 00000000..9ba060d7
--- /dev/null
+++ b/util/spkmodem_decode/spkmodem-decode.c
@@ -0,0 +1,496 @@
+/*
+ * SPDX-License-Identifier: GPL-2.0-or-later
+ * Copyright (c) 2013 Free Software Foundation, Inc.
+ * Copyright (c) 2023, 2026 Leah Rowe <leah@libreboot.org>
+ *
+ * This program receives text encoded as pulses on the PC speaker,
+ * and decodes them via simple FSK (Frequency Shift Keying)
+ * demodulation and FIR (Finite Impulse Response) frequency
+ * discriminator.
+ *
+ * It waits for specific tones at specific intervals.
+ * It detects tones within the audio stream and reconstructs
+ * characters bit-by-bit as the encoded modem signal is received.
+ * This is performance-efficient on most CPUs, and has relatively
+ * high tolerance for noisy signals (similar to techniques used
+ * for data stored on audio cassette tapes).
+ *
+ * This is a special interface provided by coreboot and GNU GRUB,
+ * for computers that lack serial ports (useful for debugging).
+ * Note that GRUB and coreboot can both send these signals; this
+ * tool merely decodes them. This tool does not *encode*, only
+ * decode.
+ *
+ * Usage example (NOTE: little endian!):
+ * parec --channels=1 --rate=48000 --format=s16le | ./spkmodem-decode
+ *
+ * Originally provided by GNU GRUB, this version is a heavily
+ * modified fork that complies with the OpenBSD Kernel Source
+ * File Style Guide (KNF) instead of GNU coding standards; it
+ * emphasises strict error handling, portability and code
+ * quality, as characterised by OpenBSD projects. Several magic
+ * numbers have been tidied up, calculated (not hardcoded) and
+ * thoroughly explained, unlike in the original version.
+ *
+ * The original version was essentially a blob, masquerading as
+ * source code. This forked source code is therefore the result
+ * of extensive reverse engineering (of the GNU source code)!
+ * This cleaned up code and extensive commenting will thoroughly
+ * explain how the decoding works. This was done as an academic
+ * exercise in 2023, continuing in 2026.
+ *
+ * This fork of spkmodem-recv, called spkmodem-decode, is provided
+ * with Libreboot releases:
+ * https://libreboot.org/
+ *
+ * The original GNU version is here, if you're morbidly curious:
+ * https://cgit.git.savannah.gnu.org/cgit/grub.git/plain/util/spkmodem-recv.c?id=3dce38eb196f47bdf86ab028de74be40e13f19fd
+ *
+ * Libreboot's version was renamed to spkmodem-decode on 12 March 2026,
+ * since Libreboot's version no longer closely resembles the GNU
+ * version at all; ergo, a full rename was in order. GNU's version
+ * was called spkmodem-recv.
+ */
+
+#define _POSIX_SOURCE
+
+/*
+ * For OpenBSD define, to detect version
+ * for deciding whether to use pledge(2)
+ */
+#ifdef __OpenBSD__
+#include <sys/param.h>
+#endif
+
+#include <errno.h>
+#include <stdio.h>
+#include <stdarg.h>
+#include <stdlib.h>
+#include <string.h>
+#include <unistd.h>
+
+/*
+ * spkmodem is essentially using FSK (Frequency Shift Keying)
+ * with two primary tones representing encoded bits,
+ * separated by a framing tone.
+ * Very cheap on CPU cycles and avoids needing something more
+ * expensive like FFT or Goertzel filters, and tolerates
+ * weak/noisy signals.
+ */
+
+/*
+ * Frequency of audio in Hz
+ */
+#define SAMPLE_RATE 48000
+
+/*
+ * One analysis frame spans 5 ms.
+ *
+ * frame_time = SAMPLES_PER_FRAME / SAMPLE_RATE
+ *
+ * With the default sample rate (48 kHz):
+ *
+ * frame_time = N / 48000
+ * 0.005 s = N / 48000
+ * N = 0.005 × 48000 = 240 samples
+ */
+#define SAMPLES_PER_FRAME 240
+
+/*
+ * Number of analysis frames per second.
+ *
+ * Each increment in the frequency counters corresponds
+ * roughly to this many Hertz of tone frequency.
+ *
+ * With the default values:
+ * FRAME_RATE = 48000 / 240 = 200 Hz
+ */
+#define FRAME_RATE ((SAMPLE_RATE) / (SAMPLES_PER_FRAME))
+
+/*
+ * Two FIR windows are maintained; one for data tone,
+ * and one for the separator tone. They are positioned
+ * one frame apart in the ring buffer.
+ */
+#define MAX_SAMPLES (2 * (SAMPLES_PER_FRAME))
+
+/*
+ * Approx byte offset for ring buffer span, just for
+ * easier debug output correlating to the audio stream.
+ */
+#define SAMPLE_OFFSET ((MAX_SAMPLES) * (sizeof(short)))
+
+/*
+ * Expected tone ranges (approximate, derived from spkmodem).
+ * These values are intentionally wide because real-world setups
+ * often involve microphones, room acoustics, and cheap ADCs.
+ */
+#define SEP_TONE_MIN_HZ 1000
+#define SEP_TONE_MAX_HZ 3000
+
+#define DATA_TONE_MIN_HZ 3000
+#define DATA_TONE_MAX_HZ 12000
+
+/* Mid point used to distinguish the two data tones. */
+#define DATA_TONE_THRESHOLD_HZ 5000
+
+/*
+ * Convert tone frequency ranges into pulse counts within the
+ * sliding analysis window.
+ *
+ * pulse_count ≈ tone_frequency / FRAME_RATE
+ * where FRAME_RATE = SAMPLE_RATE / SAMPLES_PER_FRAME.
+ */
+#define FREQ_SEP_MIN ((SEP_TONE_MIN_HZ) / (FRAME_RATE))
+#define FREQ_SEP_MAX ((SEP_TONE_MAX_HZ) / (FRAME_RATE))
+
+#define FREQ_DATA_MIN ((DATA_TONE_MIN_HZ) / (FRAME_RATE))
+#define FREQ_DATA_MAX ((DATA_TONE_MAX_HZ) / (FRAME_RATE))
+
+#define FREQ_DATA_THRESHOLD ((DATA_TONE_THRESHOLD_HZ) / (FRAME_RATE))
+
+/*
+ * Sample amplitude threshold used to convert the waveform
+ * into a pulse stream. Values near zero are regarded as noise.
+ */
+#define THRESHOLD 500
+
+#define READ_BUF 4096
+
+struct decoder_state {
+ unsigned char pulse[MAX_SAMPLES];
+
+ signed short inbuf[READ_BUF];
+ size_t inpos;
+ size_t inlen;
+
+ int ringpos;
+ int sep_pos;
+
+ /*
+ * Sliding window pulse counters
+ * used to detect modem tones
+ */
+ int freq_data;
+ int freq_separator;
+ int sample_count;
+
+ int ascii_bit;
+ unsigned char ascii;
+
+ int debug;
+ int swap_bytes;
+};
+
+static const char *argv0;
+
+static int host_is_big_endian(void);
+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 signed short read_sample(struct decoder_state *st);
+static int set_ascii_bit(struct decoder_state *st);
+static void print_char(struct decoder_state *st);
+static void print_stats(struct decoder_state *st);
+static void reset_char(struct decoder_state *st);
+
+static void err(int errval, const char *msg, ...);
+static void usage(void);
+static const char *progname(void);
+
+int getopt(int, char * const *, const char *);
+extern char *optarg;
+extern int optind;
+extern int opterr;
+extern int optopt;
+
+int
+main(int argc, char **argv)
+{
+ struct decoder_state st;
+ int c;
+
+#if defined (__OpenBSD__) && defined(OpenBSD)
+#if OpenBSD >= 509
+ if (pledge("stdio", NULL) == -1)
+ err(errno, "pledge");
+#endif
+#endif
+
+ memset(&st, 0, sizeof(st));
+ st.ascii_bit = 7;
+
+ st.ringpos = 0;
+ st.sep_pos = SAMPLES_PER_FRAME;
+
+ argv0 = argv[0];
+
+ while ((c = getopt(argc, argv, "d")) != -1) {
+ if (c != 'd')
+ usage();
+ st.debug = 1;
+ break;
+ }
+
+ if (host_is_big_endian())
+ st.swap_bytes = 1;
+
+ setvbuf(stdout, NULL, _IONBF, 0);
+
+ for (;;)
+ handle_audio(&st);
+
+ return EXIT_SUCCESS;
+}
+
+static int
+host_is_big_endian(void)
+{
+ unsigned int x = 1;
+ return (*(unsigned char *)&x == 0);
+}
+
+static void
+handle_audio(struct decoder_state *st)
+{
+ int sample;
+
+ /*
+ * If the modem signal disappears for several frames,
+ * discard the partially assembled character.
+ */
+ if (st->sample_count >= (3 * SAMPLES_PER_FRAME))
+ reset_char(st);
+ if (!valid_signal(st)) {
+ decode_pulse(st);
+ return;
+ }
+
+ if (set_ascii_bit(st) < 0)
+ print_char(st);
+
+ st->sample_count = 0;
+ for (sample = 0; sample < SAMPLES_PER_FRAME; sample++)
+ decode_pulse(st);
+}
+
+/*
+ * Verify that the observed pulse densities fall within the
+ * expected ranges for spkmodem tones. This prevents random noise
+ * from being misinterpreted as data.
+ */
+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);
+}
+
+/*
+ * Main demodulation step (moving-sum FIR filter).
+ */
+static void
+decode_pulse(struct decoder_state *st)
+{
+ unsigned char old_ring, old_sep;
+ unsigned char new_pulse;
+ int ringpos;
+ int sep_pos;
+ signed short sample;
+
+ ringpos = st->ringpos;
+ sep_pos = st->sep_pos;
+
+ /*
+ * Sliding rectangular FIR (Finite Impulse Response) filter.
+ *
+ * After thresholding, the signal becomes a stream of 0/1 pulses.
+ * The decoder measures pulse density over two windows:
+ *
+ * freq_data: pulses in the "data" window
+ * freq_separator: pulses in the "separator" window
+ *
+ * Instead of calculating each window every time (O(N) per frame), we
+ * update the window sums incrementally:
+ *
+ * sum_new = sum_old - pulse_leaving + pulse_entering
+ *
+ * This keeps the filter O(1) per sample instead of O(N).
+ * The technique is equivalent to a rectangular FIR filter
+ * implemented as a sliding moving sum.
+ *
+ * The two windows are exactly SAMPLES_PER_FRAME apart in the ring
+ * buffer, so the pulse leaving the data window is simultaneously
+ * entering the separator window.
+ */
+ old_ring = st->pulse[ringpos];
+ old_sep = st->pulse[sep_pos];
+ st->freq_data -= old_ring;
+ st->freq_data += old_sep;
+ st->freq_separator -= old_sep;
+
+ sample = read_sample(st);
+
+ /*
+ * Convert the waveform sample into a pulse (0 or 1).
+ *
+ * The unsigned comparison creates a small dead zone near zero,
+ * suppressing small amplitude noise from microphones or
+ * cheap ADCs. Real PC speaker tones are far outside this
+ * range, so they still produce clean pulses.
+ */
+ if ((unsigned)(sample + THRESHOLD)
+ > (unsigned)(2 * THRESHOLD))
+ new_pulse = 1;
+ else
+ new_pulse = 0;
+
+ st->pulse[ringpos] = new_pulse;
+ st->freq_separator += new_pulse;
+
+ /*
+ * Advance both FIR windows through the ring buffer.
+ * The separator window always stays one frame ahead
+ * of the data window.
+ */
+ ringpos++;
+ if (ringpos >= MAX_SAMPLES)
+ ringpos = 0;
+ sep_pos++;
+ if (sep_pos >= MAX_SAMPLES)
+ sep_pos = 0;
+
+ st->ringpos = ringpos;
+ st->sep_pos = sep_pos;
+
+ st->sample_count++;
+}
+
+static signed short
+read_sample(struct decoder_state *st)
+{
+ size_t n;
+ signed short sample;
+ unsigned short u;
+
+ while (st->inpos >= st->inlen) {
+
+ n = fread(st->inbuf, sizeof(st->inbuf[0]),
+ READ_BUF, stdin);
+
+ if (n == 0) {
+ if (ferror(stdin))
+ err(errno, "stdin read");
+ if (feof(stdin))
+ exit(EXIT_SUCCESS);
+ }
+
+ st->inpos = 0;
+ st->inlen = n;
+ }
+
+ sample = st->inbuf[st->inpos++];
+
+ if (st->swap_bytes) {
+ u = (unsigned short)sample;
+ u = (u >> 8) | (u << 8);
+
+ sample = (signed short)u;
+ }
+
+ return sample;
+}
+
+/*
+ * Each validated frame contributes one bit of modem data.
+ * Bits are accumulated MSB-first into the ASCII byte.
+ */
+static int
+set_ascii_bit(struct decoder_state *st)
+{
+ if (st->debug)
+ print_stats(st);
+ if (st->freq_data < FREQ_DATA_THRESHOLD)
+ st->ascii |= (1 << st->ascii_bit);
+
+ st->ascii_bit--;
+ return st->ascii_bit;
+}
+
+static void
+print_char(struct decoder_state *st)
+{
+ if (st->debug)
+ printf("<%c,%x>", st->ascii, st->ascii);
+ else
+ putchar(st->ascii);
+
+ reset_char(st);
+}
+
+static void
+print_stats(struct decoder_state *st)
+{
+ long pos;
+
+ if ((pos = ftell(stdin)) == -1) {
+ printf("%d %d %d\n",
+ st->freq_data,
+ st->freq_separator,
+ FREQ_DATA_THRESHOLD);
+ return;
+ }
+
+ printf("%d %d %d @%ld\n",
+ st->freq_data,
+ st->freq_separator,
+ FREQ_DATA_THRESHOLD,
+ pos - SAMPLE_OFFSET);
+}
+
+static void
+reset_char(struct decoder_state *st)
+{
+ st->ascii = 0;
+ st->ascii_bit = 7;
+}
+
+static void
+err(int errval, const char *msg, ...)
+{
+ va_list ap;
+
+ fprintf(stderr, "%s: ", progname());
+
+ va_start(ap, msg);
+ vfprintf(stderr, msg, ap);
+ va_end(ap);
+
+ fprintf(stderr, ": %s\n", strerror(errval));
+ exit(EXIT_FAILURE);
+}
+
+static void
+usage(void)
+{
+ fprintf(stderr, "usage: %s [-d]\n", progname());
+ exit(EXIT_FAILURE);
+}
+
+static const char *
+progname(void)
+{
+ const char *p;
+
+ if (argv0 == NULL || *argv0 == '\0')
+ return "";
+
+ p = strrchr(argv0, '/');
+
+ if (p)
+ return p + 1;
+ else
+ return argv0;
+}
diff --git a/util/spkmodem_recv/.gitignore b/util/spkmodem_recv/.gitignore
deleted file mode 100644
index 2f5c946c..00000000
--- a/util/spkmodem_recv/.gitignore
+++ /dev/null
@@ -1 +0,0 @@
-spkmodem-recv
diff --git a/util/spkmodem_recv/spkmodem-recv.c b/util/spkmodem_recv/spkmodem-recv.c
deleted file mode 100644
index 4ff1fb5f..00000000
--- a/util/spkmodem_recv/spkmodem-recv.c
+++ /dev/null
@@ -1,313 +0,0 @@
-/*
- * SPDX-License-Identifier: GPL-2.0-or-later
- * Copyright (c) 2013 Free Software Foundation, Inc.
- * Copyright (c) 2023, 2026 Leah Rowe <leah@libreboot.org>
- *
- * This program receives text encoded as pulses on the PC speaker,
- * and decodes them. This is a special type of interface provided
- * by coreboot and GRUB, for computers that lack serial ports.
- *
- * Usage example:
- * parec --channels=1 --rate=48000 --format=s16le | ./spkmodem-recv
- *
- * Originally provided by GNU GRUB, this version is a heavily
- * modified fork that complies with the OpenBSD Kernel Source
- * File Style Guide (KNF) instead of GNU coding standards; it
- * emphasises strict error handling, portability and code
- * quality, as characterised by OpenBSD projects.
- *
- * This fork of spkmodem-recv is provided with Libreboot releases:
- * https://libreboot.org/
- */
-
-#define _POSIX_SOURCE
-
-/*
- * For OpenBSD define, to detect version
- * for deciding whether to use pledge(2)
- */
-#ifdef __OpenBSD__
-#include <sys/param.h>
-#endif
-
-#include <errno.h>
-#include <stdio.h>
-#include <stdarg.h>
-#include <stdlib.h>
-#include <string.h>
-#include <unistd.h>
-
-#define SAMPLES_PER_FRAME 240
-#define MAX_SAMPLES (2 * SAMPLES_PER_FRAME)
-
-#define FREQ_SEP_MIN 5
-#define FREQ_SEP_MAX 15
-
-#define FREQ_DATA_MIN 15
-#define FREQ_DATA_THRESHOLD 25
-#define FREQ_DATA_MAX 60
-
-#define THRESHOLD 500
-
-#define READ_BUF 4096
-
-struct decoder_state {
- signed short frame[MAX_SAMPLES];
- unsigned char pulse[MAX_SAMPLES];
-
- signed short inbuf[READ_BUF];
- size_t inpos;
- size_t inlen;
-
- int ringpos;
- int sep_pos;
-
- /*
- * Sliding window pulse counters
- * used to detect modem tones
- */
- int freq_data;
- int freq_separator;
- int sample_count;
-
- int ascii_bit;
- unsigned char ascii;
-
- int debug;
-};
-
-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 signed short read_sample(struct decoder_state *st);
-static int set_ascii_bit(struct decoder_state *st);
-static void print_char(struct decoder_state *st);
-static void print_stats(struct decoder_state *st);
-static void reset_char(struct decoder_state *st);
-
-static void err(int errval, const char *msg, ...);
-static void usage(void);
-static const char *progname(void);
-
-int getopt(int, char * const *, const char *);
-extern char *optarg;
-extern int optind;
-extern int opterr;
-extern int optopt;
-
-#ifndef errno
-extern int errno;
-#endif
-
-int
-main(int argc, char **argv)
-{
- struct decoder_state st;
- int c;
-
-#if defined (__OpenBSD__) && defined(OpenBSD)
-#if OpenBSD >= 509
- if (pledge("stdio", NULL) == -1)
- err(errno, "pledge");
-#endif
-#endif
-
- memset(&st, 0, sizeof(st));
- st.ascii_bit = 7;
-
- st.ringpos = 0;
- st.sep_pos = SAMPLES_PER_FRAME;
-
- argv0 = argv[0];
-
- while ((c = getopt(argc, argv, "d")) != -1) {
- if (c != 'd')
- usage();
- st.debug = 1;
- break;
- }
-
- setvbuf(stdout, NULL, _IONBF, 0);
-
- for (;;)
- handle_audio(&st);
-
- return EXIT_SUCCESS;
-}
-
-static void
-handle_audio(struct decoder_state *st)
-{
- int sample;
-
- if (st->sample_count > (3 * SAMPLES_PER_FRAME))
- reset_char(st);
- if (!valid_signal(st)) {
- decode_pulse(st);
- return;
- }
-
- if (set_ascii_bit(st) < 0)
- print_char(st);
-
- st->sample_count = 0;
- for (sample = 0; sample < SAMPLES_PER_FRAME; sample++)
- decode_pulse(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)
-{
- unsigned char old_ring, old_sep;
- unsigned char new_pulse;
-
- old_ring = st->pulse[st->ringpos];
- old_sep = st->pulse[st->sep_pos];
-
- st->freq_data -= old_ring;
- st->freq_data += old_sep;
- st->freq_separator -= old_sep;
-
- st->frame[st->ringpos] = read_sample(st);
-
- if ((unsigned)(st->frame[st->ringpos] + THRESHOLD)
- > (unsigned)(2 * THRESHOLD))
- new_pulse = 1;
- else
- new_pulse = 0;
-
- st->pulse[st->ringpos] = new_pulse;
- st->freq_separator += new_pulse;
-
- st->ringpos++;
- if (st->ringpos >= MAX_SAMPLES)
- st->ringpos = 0;
-
- st->sep_pos++;
- if (st->sep_pos >= MAX_SAMPLES)
- st->sep_pos = 0;
-
- st->sample_count++;
-}
-
-static signed short
-read_sample(struct decoder_state *st)
-{
- size_t n;
-
- while (st->inpos >= st->inlen) {
-
- n = fread(st->inbuf, sizeof(st->inbuf[0]),
- READ_BUF, stdin);
-
- if (n == 0) {
- if (feof(stdin))
- exit(EXIT_SUCCESS);
- err(errno, "stdin read");
- }
-
- st->inpos = 0;
- st->inlen = n;
- }
-
- return st->inbuf[st->inpos++];
-}
-
-static int
-set_ascii_bit(struct decoder_state *st)
-{
- if (st->debug)
- print_stats(st);
- if (st->freq_data < FREQ_DATA_THRESHOLD)
- st->ascii |= (1 << st->ascii_bit);
-
- st->ascii_bit--;
- return st->ascii_bit;
-}
-
-static void
-print_char(struct decoder_state *st)
-{
- if (st->debug)
- printf("<%c,%x>", st->ascii, st->ascii);
- else
- putchar(st->ascii);
-
- reset_char(st);
-}
-
-static void
-print_stats(struct decoder_state *st)
-{
- long pos;
-
- if ((pos = ftell(stdin)) == -1) {
- printf("%d %d %d\n",
- st->freq_data,
- st->freq_separator,
- FREQ_DATA_THRESHOLD);
- return;
- }
-
- printf("%d %d %d @%ld\n",
- st->freq_data,
- st->freq_separator,
- FREQ_DATA_THRESHOLD,
- pos - sizeof(st->frame));
-}
-
-static void
-reset_char(struct decoder_state *st)
-{
- st->ascii = 0;
- st->ascii_bit = 7;
-}
-
-static void
-err(int errval, const char *msg, ...)
-{
- va_list ap;
-
- fprintf(stderr, "%s: ", progname());
-
- va_start(ap, msg);
- vfprintf(stderr, msg, ap);
- va_end(ap);
-
- fprintf(stderr, ": %s\n", strerror(errval));
- exit(EXIT_FAILURE);
-}
-
-static void
-usage(void)
-{
- fprintf(stderr, "usage: %s [-d]\n", progname());
- exit(EXIT_FAILURE);
-}
-
-static const char *
-progname(void)
-{
- const char *p;
-
- if (argv0 == NULL || *argv0 == '\0')
- return "";
-
- p = strrchr(argv0, '/');
-
- if (p)
- return p + 1;
- else
- return argv0;
-}