Linux-kernel mailing list archive 1997-49,: Re: Sound Config.in patch/cleanup

# Re: Sound Config.in patch/cleanup

Michael Elizabeth Chastain (mec@shout.net)
Mon, 8 Dec 1997 08:33:39 -0600

- Messages sorted by: [ date ][ thread ][ subject ][ author ]
- Next message: mlord: "Re: PnP in 2.2 ?"
- Previous message: linux kernel account: "Re: PnP in 2.2 ? (novel concept)"
- Maybe in reply to: Brian Gerst: "Sound Config.in patch/cleanup"
- Next in thread: Alan Cox: "Re: Sound Config.in patch/cleanup"
- Next in thread: Brian Gerst: "Re: Sound Config.in patch/cleanup"
- Reply: Alan Cox: "Re: Sound Config.in patch/cleanup"

Hi Brian,

Combining the 'if' statements looks good. I suggest moving these
'if' statements to match the corresponding dep_tristate, like this:

dep_tristate 'ProAudioSpectrum 16 support' CONFIG_PAS $CONFIG_SOUND
if [ "$CONFIG_PAS" = "y" -o "$CONFIG_PAS" = "m" ]; then
int 'PAS16 IRQ 3, 4, 5, 7, 9, 10, 11, 12, 14 or 15' PAS_IRQ 10
int 'PAS16 DMA 0, 1, 3, 5, 6 or 7' PAS_DMA 3
fi

dep_tristate '100%% Sound Blaster compatibles (SB16/32/64, ESS, Jazz16) support' CONFIG_SB $CONFIG_SOUND
if [ "$CONFIG_SB" = "y" -o "$CONFIG_SB" = "m" ]; then
hex 'I/O base for SB Check from manual of the card' SBC_BASE 220
int 'Sound Blaster IRQ Check from manual of the card' SBC_IRQ 7
int 'Sound Blaster DMA 0, 1 or 3' SBC_DMA 1
int 'Sound Blaster 16 bit DMA (SB16, Jazz16, SMW) 5, 6 or 7 (use 1 for 8 bit cards)' SB_DMA2 5
hex 'MPU401 I/O base of SB16, Jazz16 and ES1688 Check from manual of the card' SB_MPU_BASE 330
comment 'MPU401 IRQ is only required with Jazz16, SM Wave and ESS1688.'
comment 'Enter -1 to the following question if you have something else such as SB16/32.'
int 'SB MPU401 IRQ (Jazz16, SM Wave and ES1688) Check from manual of the card' SB_MPU_IRQ -1
fi

...

There is a more serious problem. PAS_IRQ, PAS_DMA, SBC_BASE, and so on,
are ignored for modules. They are used only in dev_table.h and they are
compiled into dev_table.h only for resident drivers.

Is anyone working on fixing this?

Michael Chastain
<mailto:mec@shout.net>
"love without fear"

- Next message: mlord: "Re: PnP in 2.2 ?"
- Previous message: linux kernel account: "Re: PnP in 2.2 ? (novel concept)"
- Maybe in reply to: Brian Gerst: "Sound Config.in patch/cleanup"
- Next in thread: Alan Cox: "Re: Sound Config.in patch/cleanup"
- Next in thread: Brian Gerst: "Re: Sound Config.in patch/cleanup"
- Reply: Alan Cox: "Re: Sound Config.in patch/cleanup"