Messages in this thread | | | Date | Mon, 22 Mar 2004 23:53:27 +0200 | From | Muli Ben-Yehuda <> | Subject | Re: OSS: cleanup or throw away |
| |
On Mon, Mar 22, 2004 at 10:32:20PM +0100, Jos Hulzink wrote: > On Monday 22 March 2004 21:22, Muli Ben-Yehuda wrote: > > In my not so humble opinion, throwing OSS away will be a big mistake, > > as long as there are people willing to maintain it. Keep it there and > > let the users (or distributions) choose what to use. I've seen > > multiple bug reports of cards that work with OSS and don't work with > > ALSA (and vice versa), so keeping both seems the proper thing to > > do. Personally, I maintain one OSS driver, and fix bugs in others > > occasionally. > > Looking at the amount of warnings in the OSS drivers (depricated > check_region), the OSS drivers seem -with all respect for your hard > work- not so well maintained anymore. I'm willing to fix them all, > but not if entire OSS is ditched away Real Soon Now (tm).
Here's a log of a 'make allyesconfig', followed by 'make sound/oss/', with gcc-3.3.2. This is really not too bad, almost everything is trivial check_region -> request_region conversions. The great majority of drivers compile just fine, and at least some of them (those I have the hardware for) definitely work, too ;-)
I intend to go through these warnings and fix them, now that I'm aware of them. If you or anyone else want to collaborate on it, great. Andrew Morton is taking OSS patches - I sent one on Saturday night that's in 2.6.5-rc2-mm1 right now.
Cheers, Muli
cd /home/muli/kernel/linux-2.5/ make sound/oss/ make[1]: `arch/i386/kernel/asm-offsets.s' is up to date. CC sound/oss/gus_card.o sound/oss/gus_card.c: In function `probe_gus': sound/oss/gus_card.c:76: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) sound/oss/gus_card.c:78: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) sound/oss/gus_card.c:93: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) sound/oss/gus_card.c:94: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) CC sound/oss/gus_midi.o CC sound/oss/gus_vol.o CC sound/oss/gus_wave.o CC sound/oss/ics2101.o CC sound/oss/pas2_card.o CC sound/oss/pas2_midi.o CC sound/oss/pas2_mixer.o CC sound/oss/pas2_pcm.o CC sound/oss/sb_card.o CC sound/oss/sb_common.o sound/oss/sb_common.c: In function `sb_dsp_detect': sound/oss/sb_common.c:523: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) sound/oss/sb_common.c: In function `probe_sbmpu': sound/oss/sb_common.c:1224: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) CC sound/oss/sb_audio.o CC sound/oss/sb_midi.o CC sound/oss/sb_mixer.o CC sound/oss/sb_ess.o CC sound/oss/dev_table.o CC sound/oss/soundcard.o CC sound/oss/sound_syms.o CC sound/oss/audio.o CC sound/oss/audio_syms.o CC sound/oss/dmabuf.o CC sound/oss/midi_syms.o CC sound/oss/midi_synth.o CC sound/oss/midibuf.o CC sound/oss/sequencer.o CC sound/oss/sequencer_syms.o CC sound/oss/sound_timer.o CC sound/oss/sys_timer.o LD sound/oss/sound.o CC sound/oss/cs4232.o sound/oss/cs4232.c: In function `probe_cs4232': sound/oss/cs4232.c:141: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) sound/oss/cs4232.c:193: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) CC sound/oss/ad1848.o sound/oss/ad1848.c: In function `ad1848_detect': sound/oss/ad1848.c:1580: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) sound/oss/ad1848.c: In function `probe_ms_sound': sound/oss/ad1848.c:2530: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) sound/oss/ad1848.c: At top level: sound/oss/ad1848.c:2967: warning: `id_table' defined but not used CC sound/oss/aedsp16.o CC sound/oss/pss.o sound/oss/pss.c: In function `probe_pss': sound/oss/pss.c:191: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) sound/oss/pss.c: In function `configure_nonsound_components': sound/oss/pss.c:640: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) sound/oss/pss.c: In function `probe_pss_mpu': sound/oss/pss.c:710: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) sound/oss/pss.c: In function `probe_pss_mss': sound/oss/pss.c:1004: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) CC sound/oss/mpu401.o sound/oss/mpu401.c: In function `probe_mpu401': sound/oss/mpu401.c:1217: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) CC sound/oss/trix.o sound/oss/trix.c: In function `trix_set_wss_port': sound/oss/trix.c:85: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) sound/oss/trix.c: In function `probe_trix_wss': sound/oss/trix.c:147: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) sound/oss/trix.c: In function `probe_trix_sb': sound/oss/trix.c:292: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) LD sound/oss/sb_lib.o CC sound/oss/uart401.o CC sound/oss/opl3sa.o sound/oss/opl3sa.c: In function `probe_opl3sa_wss': sound/oss/opl3sa.c:114: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) sound/oss/opl3sa.c:122: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) CC sound/oss/sscape.o sound/oss/sscape.c: In function `detect_ga': sound/oss/sscape.c:737: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) sound/oss/sscape.c: In function `sscape_pnp_init_hw': sound/oss/sscape.c:1113: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) sound/oss/sscape.c: In function `detect_sscape_pnp': sound/oss/sscape.c:1132: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) sound/oss/sscape.c:1137: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) CC sound/oss/mad16.o sound/oss/mad16.c: In function `wss_init': sound/oss/mad16.c:322: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) CC sound/oss/opl3sa2.o LD sound/oss/pas2.o LD sound/oss/sb.o CC sound/oss/kahlua.o ( \ echo 'static unsigned char * maui_os = NULL;'; \ echo 'static int maui_osLen = 0;'; \ ) > sound/oss/maui_boot.h CC sound/oss/maui.o sound/oss/maui.c: In function `probe_maui': sound/oss/maui.c:307: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) CC sound/oss/uart6850.o LD sound/oss/gus.o CC sound/oss/adlib_card.o CC sound/oss/opl3.o CC sound/oss/v_midi.o CC sound/oss/sgalaxy.o sound/oss/sgalaxy.c: In function `probe_sgalaxy': sound/oss/sgalaxy.c:89: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) sound/oss/sgalaxy.c:97: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) CC sound/oss/ad1816.o CC sound/oss/ad1889.o sound/oss/ad1889.c: In function `ad1889_ac97_init': sound/oss/ad1889.c:853: warning: comparison is always false due to limited range of data type CC sound/oss/ac97_codec.o CC sound/oss/aci.o CC sound/oss/awe_wave.o CC sound/oss/via82cxxx_audio.o CC sound/oss/ymfpci.o CC sound/oss/nm256_audio.o CC sound/oss/ac97.o CC sound/oss/i810_audio.o CC sound/oss/sonicvibes.o CC sound/oss/cmpci.o sound/oss/cmpci.c: In function `cm_release_mixdev': sound/oss/cmpci.c:1465: warning: unused variable `s' sound/oss/cmpci.c: At top level: sound/oss/cmpci.c:2865: warning: `cmpci_pci_tbl' defined but not used CC sound/oss/es1370.o CC sound/oss/es1371.o CC sound/oss/esssolo1.o CC sound/oss/cs46xx.o CC sound/oss/maestro.o CC sound/oss/maestro3.o CC sound/oss/trident.o CC sound/oss/rme96xx.o CC sound/oss/btaudio.o CC sound/oss/ali5455.o CC sound/oss/forte.o CC sound/oss/ac97_plugin_ad1980.o CC sound/oss/cs4281/cs4281m.o LD sound/oss/cs4281/cs4281.o LD sound/oss/cs4281/built-in.o CC sound/oss/emu10k1/audio.o CC sound/oss/emu10k1/cardmi.o CC sound/oss/emu10k1/cardmo.o CC sound/oss/emu10k1/cardwi.o CC sound/oss/emu10k1/cardwo.o CC sound/oss/emu10k1/ecard.o CC sound/oss/emu10k1/efxmgr.o CC sound/oss/emu10k1/emuadxmg.o CC sound/oss/emu10k1/hwaccess.o CC sound/oss/emu10k1/irqmgr.o CC sound/oss/emu10k1/main.o CC sound/oss/emu10k1/midi.o CC sound/oss/emu10k1/mixer.o CC sound/oss/emu10k1/passthrough.o CC sound/oss/emu10k1/recmgr.o CC sound/oss/emu10k1/timer.o CC sound/oss/emu10k1/voicemgr.o LD sound/oss/emu10k1/emu10k1.o LD sound/oss/emu10k1/built-in.o LD sound/oss/built-in.o CC [M] sound/oss/wavfront.o sound/oss/wavfront.c: In function `detect_wavefront': sound/oss/wavfront.c:2427: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) sound/oss/wavfront.c: At top level: sound/oss/wavfront.c:2498: warning: `errno' defined but not used CC [M] sound/oss/wf_midi.o sound/oss/wf_midi.c: In function `detect_wf_mpu': sound/oss/wf_midi.c:788: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) CC [M] sound/oss/yss225.o LD [M] sound/oss/wavefront.o CC [M] sound/oss/msnd.o sound/oss/msnd.c: In function `msnd_register': sound/oss/msnd.c:63: warning: `MOD_INC_USE_COUNT' is deprecated (declared at include/linux/module.h:515) sound/oss/msnd.c: In function `msnd_unregister': sound/oss/msnd.c:84: warning: `MOD_DEC_USE_COUNT' is deprecated (declared at include/linux/module.h:527) CC [M] sound/oss/msnd_classic.o In file included from sound/oss/msnd_classic.c:3: sound/oss/msnd_pinnacle.c: In function `probe_multisound': sound/oss/msnd_pinnacle.c:1123: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) CC [M] sound/oss/msnd_pinnacle.o sound/oss/msnd_pinnacle.c: In function `probe_multisound': sound/oss/msnd_pinnacle.c:1123: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121) sound/oss/msnd_pinnacle.c: In function `msnd_init': sound/oss/msnd_pinnacle.c:1811: warning: `check_region' is deprecated (declared at include/linux/ioport.h:121)
Compilation finished at Mon Mar 22 23:48:37
-- Muli Ben-Yehuda http://www.mulix.org | http://mulix.livejournal.com/
[unhandled content-type:application/pgp-signature] | |