lkml.org 
[lkml]   [2004]   [Mar]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: 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]
\
 
 \ /
  Last update: 2005-03-22 14:01    [W:0.092 / U:0.204 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site