Differential D55934

sound: Retire SND_MULTICHANNEL
ClosedPublic
Actions

Authored by christos on Mar 19 2026, 1:28 PM.

Tags

None

Referenced Files

	F167752924: D55934.id.diff
	Mon, Aug 24, 7:19 AM

	Unknown Object (File)
	Fri, Aug 21, 8:15 PM

	Unknown Object (File)
	Fri, Aug 21, 8:14 PM

	Unknown Object (File)
	Thu, Aug 20, 11:28 PM

	Unknown Object (File)
	Wed, Aug 19, 8:38 AM

	Unknown Object (File)
	Wed, Aug 19, 7:53 AM

	Unknown Object (File)
	Tue, Aug 18, 8:50 PM

	Unknown Object (File)
	Mon, Aug 17, 11:38 PM

View All 69 Files

Subscribers

Details

Reviewers

markj
emaste

Commits

rGd6b36b860770: sound: Retire SND_MULTICHANNEL
rG712bd9b2750a: sound: Retire SND_MULTICHANNEL

Summary

SND_MULTICHANNEL is always defined, so SND_CHN_MAX will also always be

Apart from the fact that there is no other place in the code that

touches SND_MULTICHANNEL, there is also no good reason nowadays to set
SND_CHN_MAX to 2.

Sponsored by: The FreeBSD Foundation
MFC after: 1 week

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 71507
Build 68390: arc lint + arc unit

Event Timeline

christos created this revision.Mar 19 2026, 1:28 PM

Herald added a subscriber: imp. · View Herald TranscriptMar 19 2026, 1:28 PM

christos requested review of this revision.Mar 19 2026, 1:28 PM

Harbormaster completed remote builds in B71507: Diff 173895.Mar 19 2026, 1:28 PM

christos added a parent revision: D55933: sound: Retire SND_OLDSTEREO.Mar 19 2026, 1:28 PM

markj added inline comments.Mar 20 2026, 2:17 AM

sys/dev/sound/pcm/matrix.h
181	How is this a no-op? Please write commit log messages.

christos added inline comments.Mar 20 2026, 10:31 AM

sys/dev/sound/pcm/matrix.h
181	You are right. I only didn't write one because I thought it is fairly obvious in this case. SND_MULTICHANNEL is defined in the file above, so this condition is always true since there is no undef for it in the code. That being said, perhaps no-op is not a good word for this.

christos retitled this revision from sound: Retire no-op SND_MULTICHANNEL to sound: Retire SND_MULTICHANNEL.Mar 20 2026, 10:54 AM

christos edited the summary of this revision. (Show Details)

markj accepted this revision.Mar 21 2026, 3:27 AM

This revision is now accepted and ready to land.Mar 21 2026, 3:27 AM

Closed by commit rG712bd9b2750a: sound: Retire SND_MULTICHANNEL (authored by christos). · Explain WhyMar 24 2026, 10:39 AM

This revision was automatically updated to reflect the committed changes.

christos added a commit: rG712bd9b2750a: sound: Retire SND_MULTICHANNEL.

christos added a commit: rGd6b36b860770: sound: Retire SND_MULTICHANNEL.Mar 31 2026, 9:55 AM

Revision Contents
Changeset List

Path

Size

sys/

dev/

sound/

pcm/

7 lines

Diff 173895

sys/dev/sound/pcm/matrix.h

Loading...