Differential D55933

sound: Retire SND_OLDSTEREO
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F167900366: D55933.id173894.diff
	Tue, Aug 25, 8:26 AM

	Unknown Object (File)
	Sun, Aug 23, 11:27 AM

	Unknown Object (File)
	Fri, Aug 21, 4:55 AM

	Unknown Object (File)
	Thu, Aug 20, 5:32 AM

	Unknown Object (File)
	Wed, Aug 19, 10:19 AM

	Unknown Object (File)
	Tue, Aug 18, 11:38 PM

	Unknown Object (File)
	Mon, Aug 17, 8:02 PM

	Unknown Object (File)
	Sun, Aug 16, 4:18 PM

View All 67 Files

Subscribers

Details

Reviewers

markj
emaste

Commits

rGea4182876ac5: sound: Retire SND_OLDSTEREO
rGbf0e5d987ecd: sound: Retire SND_OLDSTEREO

Summary

This is a legacy option and does not serve a good purpose anymore.

Sponsored by: The FreeBSD Foundation
MFC after: 1 week

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

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

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

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

Harbormaster completed remote builds in B71506: Diff 173894.Mar 19 2026, 1:28 PM

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

christos added a child revision: D55934: sound: Retire SND_MULTICHANNEL.

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

sys/dev/sound/pcm/matrix.h
33	How is this "unused"?

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

sys/dev/sound/pcm/matrix.h
33	It never gets defined, so this condition is always true.

markj added inline comments.Mar 21 2026, 3:25 AM

sys/dev/sound/pcm/matrix.h
33	It's a kernel option, right? So a user might define it by configuring "options SND_OLDSTEREO" in their kernel config. If you want to remove the option, then fine, but "unused" is not correct I think.

christos retitled this revision from sound: Retire unused SND_OLDSTEREO to sound: Retire SND_OLDSTEREO.Mar 23 2026, 12:48 PM

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

christos added inline comments.Mar 23 2026, 12:50 PM

sys/dev/sound/pcm/matrix.h
33	I forgot about this. But still I'm pretty sure no one is using this option.

markj accepted this revision.Mar 23 2026, 1:58 PM

This revision is now accepted and ready to land.Mar 23 2026, 1:58 PM

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

This revision was automatically updated to reflect the committed changes.

christos added a commit: rGbf0e5d987ecd: sound: Retire SND_OLDSTEREO.

christos added a commit: rGea4182876ac5: sound: Retire SND_OLDSTEREO.Mar 31 2026, 9:55 AM

Revision Contents
Changeset List

Path

Size

sys/

conf/

4 lines

1 line

dev/

sound/

pcm/

3 lines

Diff 174214

sys/conf/NOTES

Loading...

sys/conf/options

Loading...

sys/dev/sound/pcm/matrix.h

Loading...