Page MenuHomeFreeBSD
Differential D34671

Retire ISA sound card drivers
ClosedPublic
Actions

Authored by emaste on Mar 25 2022, 1:51 PM.
Tags
None
Referenced Files
F110342507: D34671.diff
Mon, Feb 17, 2:26 AM
Unknown Object (File)
Thu, Jan 30, 8:55 AM
Unknown Object (File)
Sun, Jan 26, 9:49 PM
Unknown Object (File)
Sun, Jan 26, 9:50 AM
Unknown Object (File)
Sun, Jan 26, 9:41 AM
Unknown Object (File)
Sat, Jan 25, 9:10 AM
Unknown Object (File)
Sat, Jan 25, 9:09 AM
Unknown Object (File)
Wed, Jan 22, 11:17 PM
View All Files
Subscribers
None

Details

Reviewers
imp
mav
cy
Commits
rG9054e296819f: Retire ISA sound card DMA support
rGdf51e63eb5d7: Retire snd_ad1816 ISA sound card driver
rGaa83e9b189d6: Retire snd_ess ISA sound card driver
rG754decef384a: Retire snd_gusc ISA sound card driver
rG5126e5eeeb5e: Retire snd_mss ISA sound card driver
rG716924cb4832: Retire snd_sbc ISA sound card driver
Summary

These ISA sound cards are obsolete, and deprecation notices were added in e39ec8933be4.

Delete the now-unused files, but retain isa/sb.h as it is used by some PCI drivers.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

emaste requested review of this revision.Mar 25 2022, 1:51 PM
emaste created this revision.

D34604 for deprecation notices

emaste updated this revision to Diff 104188.Mar 25 2022, 2:00 PM

Delete the leftover files

emaste added inline comments.Mar 25 2022, 4:46 PM
sys/modules/sound/driver/Makefile
8–9

ess again removed locally
(accidentally reappeared in a rebase)

emaste updated this revision to Diff 104216.Mar 26 2022, 1:05 AM
emaste retitled this revision from Disconnect ISA sound cards from the to Retire ISA sound card drivers.
emaste edited the summary of this revision. (Show Details)
emaste added reviewers: imp, mav, cy.
mav added inline comments.Mar 26 2022, 1:31 AM
sys/dev/sound/pcm/buffer.h
34

SNDBUF_F_DMA and SND_DMA() are no longer used. SNDBUF_F_XRUN and SNDBUF_F_RUNNING seems just not used.

sys/dev/sound/pcm/channel.c
32

I guess this is no longer used?

sys/modules/sound/sound/Makefile
42

I guess if we remove opt_isa.h inclusion from channel.c we no longer need this.

emaste updated this revision to Diff 104222.Mar 26 2022, 1:36 AM

Fixups from @mav

mav accepted this revision.Mar 26 2022, 1:52 AM
mav added inline comments.
sys/dev/sound/pcm/buffer.h
52

dmachan and dir seem also no longer used, but I don't insist.

This revision is now accepted and ready to land.Mar 26 2022, 1:52 AM
emaste added a comment.Mar 31 2022, 5:45 PM

I will split this into a series of individual commits so that we could revert a single driver if desired

Closed by commit rG716924cb4832: Retire snd_sbc ISA sound card driver (authored by emaste). · Explain WhyJul 2 2022, 4:13 PM
This revision was automatically updated to reflect the committed changes.
emaste added a commit: rG716924cb4832: Retire snd_sbc ISA sound card driver.
emaste added a commit: rG5126e5eeeb5e: Retire snd_mss ISA sound card driver.Jul 2 2022, 4:23 PM
emaste added a commit: rG754decef384a: Retire snd_gusc ISA sound card driver.Jul 2 2022, 4:27 PM
emaste added a commit: rGaa83e9b189d6: Retire snd_ess ISA sound card driver.Jul 2 2022, 4:32 PM
emaste added a commit: rGdf51e63eb5d7: Retire snd_ad1816 ISA sound card driver.Jul 27 2022, 6:36 PM
emaste added a commit: rG9054e296819f: Retire ISA sound card DMA support.Sep 15 2022, 2:08 PM

Revision Contents
Changeset List

PathSize
9 lines
share/
man/
man4/
7 lines
19 lines
5 lines
sys/
conf/
18 lines
8 lines
dev/
sound/
7 lines
isa/
pcm/
6 lines
7 lines
modules/
sound/
driver/
8 lines
ad1816/
ess/
mss/
sb16/
sb8/
sbc/
sound/
12 lines

Diff 104216

View Options

ObsoleteFiles.inc

Loading...
View Options

share/man/man4/Makefile

Loading...
View Options

share/man/man4/pcm.4

Loading...
View Options

share/man/man4/snd_ad1816.4

Loading...
View Options

share/man/man4/snd_ess.4

Loading...
View Options

share/man/man4/snd_gusc.4

Loading...
View Options

share/man/man4/snd_mss.4

Loading...
View Options

share/man/man4/snd_sbc.4

Loading...
View Options

share/man/man4/snd_solo.4

Loading...
View Options

sys/conf/NOTES

Loading...
View Options

sys/conf/files

Loading...
View Options

sys/dev/sound/driver.c

Loading...
View Options

sys/dev/sound/isa/ad1816.h

Loading...
View Options

sys/dev/sound/isa/ad1816.c

Loading...
View Options

sys/dev/sound/isa/ess.c

Loading...
View Options

sys/dev/sound/isa/gusc.c

Loading...
View Options

sys/dev/sound/isa/mss.h

Loading...
View Options

sys/dev/sound/isa/mss.c

Loading...
View Options

sys/dev/sound/isa/sb16.c

Loading...
View Options

sys/dev/sound/isa/sb8.c

Loading...
View Options

sys/dev/sound/isa/sbc.c

Loading...
View Options

sys/dev/sound/isa/sndbuf_dma.c

Loading...
View Options

sys/dev/sound/pcm/buffer.h

Loading...
View Options

sys/dev/sound/pcm/channel.c

Loading...
View Options

sys/modules/sound/driver/Makefile

Loading...
View Options

sys/modules/sound/driver/ad1816/Makefile

Loading...
View Options

sys/modules/sound/driver/ess/Makefile

Loading...
View Options

sys/modules/sound/driver/mss/Makefile

Loading...
View Options

sys/modules/sound/driver/sb16/Makefile

Loading...
View Options

sys/modules/sound/driver/sb8/Makefile

Loading...
View Options

sys/modules/sound/driver/sbc/Makefile

Loading...
View Options

sys/modules/sound/sound/Makefile

Loading...