From nobody Thu Nov 20 13:31:38 2025 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4dBzlZ4wQfz6HP2x for ; Thu, 20 Nov 2025 13:31:38 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R12" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4dBzlZ1Pp4z44XY for ; Thu, 20 Nov 2025 13:31:38 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1763645498; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=H2GmjTeLx2EYOyernw0bYwlRZ58wq0UbqB2s0ejeyXQ=; b=Qft0/0XVtA1bVg6I799/U/Xepc2Ib/KjMZg93zr3Bu2Xn1nyWvv4ZPSFzQwFz/QYbmUyM5 Id8xy9ixUp5naqGLAO8BJj1RCGw4Q6Dl0599j7xM+2zUDJtF9kSWfBTyVG0VFjwGTz217I bEYzAn13QqaBmWRcE1Zc6EbOx3M9IjScrTtpaZPSGMD1uIo5DicpkDgfvuEAH3xdcBNwLR 1UhQIIjFxKnOJJiGS9lazGhMgbrujVHu/kDJFvF2g7FuCUVu05wpp1SdbHkVsRFLNVQFJo Uaa22NstczrggIExfLDoieFmDpNoxgb6EJBFfpPD5p/hyMzEkTr+NbJf4eZ3xQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1763645498; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=H2GmjTeLx2EYOyernw0bYwlRZ58wq0UbqB2s0ejeyXQ=; b=R4bdbKtuJAkqXA5lrEKAcA0dx+qbPv8fuRpQMHdQj1olBeNzAmGz87BppY4pEs+Pd+u4FA pXSIcyVK1Qgva8JvyzXOpE5VCkNzDsIa5If1DGcQBaOdmvwo0q43crMfXzVqy4AO96i0lY B44g4PVW/ui78muY9kxNGEia4MGyBreBV0m19F7kVSWZv0y7LmIVbxEok/msKGkdp4V6Je 8ZMRoe6ovCZFszMYgfb87n668k0LX6rTKTmjLJg2uYhkSYBFwKdfETVqFj/u0vvh7vvhNW u5EnvVmOCJekVLGDmZwTA7nRAFg5ftDlPd6ZNDd530/vgB4eww8JHTUJfPojUw== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1763645498; a=rsa-sha256; cv=none; b=p3dHYgFufkoo+2yPuz3/xHxnNF5MHo0rOwWM57VHW9ykmxWJmU1W5a4qeyaDvObzPs9W9/ 7toDDeOsC3nCdxsE5QGhUdamDClg8p2b4apmW1jydr1vCOp5N7c/ELuI8/CbN/Qba51nE6 a00VsD9zPxpSfrNoCsoIr6YTV1joT/xdAQccUAoAbfs26YtPhXnonJBNvj0ckpu/iC9ZbN SKWU9vGDkMz3sFGUTPKyzwbUp1dxievmIgAQDLzomgFsAiZcBHRJLV5vlqLt1tHbt6ycX5 /yLsEHkyVmXTXxe/73Boifx7kSZ+t+35bBe8mSuE4nBztD5wiPpo64225HtDTQ== ARC-Authentication-Results: i=1; mx1.freebsd.org; none Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) by mxrelay.nyi.freebsd.org (Postfix) with ESMTP id 4dBzlZ11SgzC8Y for ; Thu, 20 Nov 2025 13:31:38 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from git (uid 1279) (envelope-from git@FreeBSD.org) id 2d492 by gitrepo.freebsd.org (DragonFly Mail Agent v0.13+ on gitrepo.freebsd.org); Thu, 20 Nov 2025 13:31:38 +0000 To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org From: Christos Margiolis Subject: git: 80873ecdc11a - stable/15 - sound: Remove dead midi code List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: X-BeenThere: dev-commits-src-all@freebsd.org Sender: owner-dev-commits-src-all@FreeBSD.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: christos X-Git-Repository: src X-Git-Refname: refs/heads/stable/15 X-Git-Reftype: branch X-Git-Commit: 80873ecdc11ac673bf5a3e7d7120191f40ef97f0 Auto-Submitted: auto-generated Date: Thu, 20 Nov 2025 13:31:38 +0000 Message-Id: <691f183a.2d492.79ccaacd@gitrepo.freebsd.org> The branch stable/15 has been updated by christos: URL: https://cgit.FreeBSD.org/src/commit/?id=80873ecdc11ac673bf5a3e7d7120191f40ef97f0 commit 80873ecdc11ac673bf5a3e7d7120191f40ef97f0 Author: Christos Margiolis AuthorDate: 2025-11-13 12:10:04 +0000 Commit: Christos Margiolis CommitDate: 2025-11-20 13:28:14 +0000 sound: Remove dead midi code Sponsored by: The FreeBSD Foundation MFC after: 1 week (cherry picked from commit e4b31985b54b626b2bbf486a82dd789f3dd381db) --- sys/dev/sound/midi/midi.c | 137 +++++----------------------------------------- 1 file changed, 14 insertions(+), 123 deletions(-) diff --git a/sys/dev/sound/midi/midi.c b/sys/dev/sound/midi/midi.c index eed7ce89a919..b39288675ea6 100644 --- a/sys/dev/sound/midi/midi.c +++ b/sys/dev/sound/midi/midi.c @@ -62,12 +62,6 @@ #include MALLOC_DEFINE(M_MIDI, "midi buffers", "Midi data allocation area"); -#define MIDI_DEV_MIDICTL 12 - -enum midi_states { - MIDI_IN_START, MIDI_IN_SYSEX, MIDI_IN_DATA -}; - #define MIDI_NAMELEN 16 struct snd_midi { KOBJ_FIELDS; @@ -86,12 +80,6 @@ struct snd_midi { struct selinfo rsel, wsel; int hiwat; /* QLEN(outq)>High-water -> disable * writes from userland */ - enum midi_states inq_state; - int inq_status, inq_left; /* Variables for the state machine in - * Midi_in, this is to provide that - * signals only get issued only - * complete command packets. */ - struct proc *async; struct cdev *dev; TAILQ_ENTRY(snd_midi) link; }; @@ -326,10 +314,8 @@ static int midi_lengths[] = {2, 2, 2, 2, 1, 1, 2, 0}; int midi_in(struct snd_midi *m, uint8_t *buf, int size) { - /* int i, sig, enq; */ int used; - /* uint8_t data; */ MIDI_DEBUG(5, printf("midi_in: m=%p size=%d\n", m, size)); /* @@ -341,111 +327,22 @@ midi_in(struct snd_midi *m, uint8_t *buf, int size) used = 0; mtx_lock(&m->qlock); -#if 0 - /* - * Don't bother queuing if not in read mode. Discard everything and - * return size so the caller doesn't freak out. - */ - - if (!(m->flags & M_RX)) - return size; - - for (i = sig = 0; i < size; i++) { - data = buf[i]; - enq = 0; - if (data == MIDI_ACK) - continue; - - switch (m->inq_state) { - case MIDI_IN_START: - if (MIDI_IS_STATUS(data)) { - switch (data) { - case 0xf0: /* Sysex */ - m->inq_state = MIDI_IN_SYSEX; - break; - case 0xf1: /* MTC quarter frame */ - case 0xf3: /* Song select */ - m->inq_state = MIDI_IN_DATA; - enq = 1; - m->inq_left = 1; - break; - case 0xf2: /* Song position pointer */ - m->inq_state = MIDI_IN_DATA; - enq = 1; - m->inq_left = 2; - break; - default: - if (MIDI_IS_COMMON(data)) { - enq = 1; - sig = 1; - } else { - m->inq_state = MIDI_IN_DATA; - enq = 1; - m->inq_status = data; - m->inq_left = MIDI_LENGTH(data); - } - break; - } - } else if (MIDI_IS_STATUS(m->inq_status)) { - m->inq_state = MIDI_IN_DATA; - if (!MIDIQ_FULL(m->inq)) { - used++; - MIDIQ_ENQ(m->inq, &m->inq_status, 1); - } - enq = 1; - m->inq_left = MIDI_LENGTH(m->inq_status) - 1; - } - break; - /* - * End of case MIDI_IN_START: - */ - - case MIDI_IN_DATA: - enq = 1; - if (--m->inq_left <= 0) - sig = 1;/* deliver data */ - break; - case MIDI_IN_SYSEX: - if (data == MIDI_SYSEX_END) - m->inq_state = MIDI_IN_START; - break; - } - - if (enq) - if (!MIDIQ_FULL(m->inq)) { - MIDIQ_ENQ(m->inq, &data, 1); - used++; - } - /* - * End of the state machines main "for loop" - */ + MIDI_DEBUG(6, printf("midi_in: len %jd avail %jd\n", + (intmax_t)MIDIQ_LEN(m->inq), + (intmax_t)MIDIQ_AVAIL(m->inq))); + if (MIDIQ_AVAIL(m->inq) > size) { + used = size; + MIDIQ_ENQ(m->inq, buf, size); + } else { + MIDI_DEBUG(4, printf("midi_in: Discarding data qu\n")); + mtx_unlock(&m->qlock); + return 0; } - if (sig) { -#endif - MIDI_DEBUG(6, printf("midi_in: len %jd avail %jd\n", - (intmax_t)MIDIQ_LEN(m->inq), - (intmax_t)MIDIQ_AVAIL(m->inq))); - if (MIDIQ_AVAIL(m->inq) > size) { - used = size; - MIDIQ_ENQ(m->inq, buf, size); - } else { - MIDI_DEBUG(4, printf("midi_in: Discarding data qu\n")); - mtx_unlock(&m->qlock); - return 0; - } - if (m->rchan) { - wakeup(&m->rchan); - m->rchan = 0; - } - selwakeup(&m->rsel); - if (m->async) { - PROC_LOCK(m->async); - kern_psignal(m->async, SIGIO); - PROC_UNLOCK(m->async); - } -#if 0 + if (m->rchan) { + wakeup(&m->rchan); + m->rchan = 0; } -#endif + selwakeup(&m->rsel); mtx_unlock(&m->qlock); return used; } @@ -480,11 +377,6 @@ midi_out(struct snd_midi *m, uint8_t *buf, int size) m->wchan = 0; } selwakeup(&m->wsel); - if (m->async) { - PROC_LOCK(m->async); - kern_psignal(m->async, SIGIO); - PROC_UNLOCK(m->async); - } } mtx_unlock(&m->qlock); return used; @@ -526,7 +418,6 @@ midi_open(struct cdev *i_dev, int flags, int mode, struct thread *td) m->rchan = 0; m->wchan = 0; - m->async = 0; if (flags & FREAD) { m->flags |= M_RX | M_RXEN;