Re: [alsa, pnp] more on opl3sa2

[Daniel Ritz <daniel.ritz@gmx.ch>]

ok, found that one...very simple:
against 2.5.59 + jaroslav's driver patch...


--- linux-2.5/sound/isa/opl3sa2.c~	2003-01-21 20:01:10.000000000 +0100
+++ linux-2.5/sound/isa/opl3sa2.c	2003-01-21 19:58:06.000000000 +0100
@@ -830,7 +830,8 @@
 	if ((err = snd_card_register(card)) < 0)
 		goto __error;
 
-	pnpc_set_drvdata(pcard, card);
+	if (pcard)
+		pnpc_set_drvdata(pcard, card);
 	snd_opl3sa2_cards[dev] = card;
 	return 0;
 

the card is not detected by pnp, that problem stays. is that a problem of the pnp layer or is
my toshiba laptop just so damn stupid??

rgds
-daniel



On Sunday 19 January 2003 19:07, Daniel Ritz wrote:
> more tests with snd-opl3sa2. i have to use the paramers to modprobe since
> pnp doesn't work. the reason:
> pnpc_register_driver sets driver.bus to its own static struct which doesn't
> have a device list. driver_attach tries to scan this device list and
> compare with the drivers device list. since there's no one for the bus it
> fails, the driver can't see it's devices.
>
> Call stack:
> driver_attach
> bus_add_driver
> driver_register
> pnpc_register_driver
> alsa_card_opl3sa_init
>
> ok, tried by hand with the config data from bios/lspnp:
> # modprobe snd-opl3sa2 isapnp=0 port=0x538 wss_port=0x530 sb_port=0x220
> fm_port=0x388 midi_port=0x330 irq=5 dma1=0 dma2=1 Segmentation fault
>
> Unable to handle kernel NULL pointer dereference at virtual address
> 00000144 printing eip:
> d08aecfb
> *pde = 00000000
> Oops: 0002
> CPU:    0
> EIP:    0060:[<d08aecfb>]    Not tainted
> EFLAGS: 00010246
> EIP is at snd_opl3sa2_probe+0x3bb/0x3e4 [snd_opl3sa2]
> eax: 00000000   ebx: cd9ace00   ecx: d089b734   edx: 00000000
> esi: 00000000   edi: cd9ace6c   ebp: c7e11f84   esp: c7e11f58
> ds: 007b   es: 007b   ss: 0068
> Process modprobe (pid: 1816, threadinfo=c7e10000 task=ce0527e0)
> Stack: 00000000 00000000 00000000 00000000 cd9ace24 00000000 00000001
> 00000000 00000005 c82fec00 c9d5fcc4 c7e11fa8 d08b33c2 00000000 00000000
> 00000000 00000000 d08b0fe0 c027e304 c027e304 c7e11fbc c01291a2 0804ea38
> 0804e008 Call Trace:
>  [<d08b33c2>] alsa_card_opl3sa2_init+0x2a/0xffffe3cc [snd_opl3sa2]
>  [<d08b0fe0>] +0x0/0x100 [snd_opl3sa2]
>  [<c01291a2>] sys_init_module+0x116/0x1ac
>  [<c010a75b>] syscall_call+0x7/0xb
>
> Code: 89 9a 44 01 00 00 8b 45 e8 89 98 60 0c 8b d0 31 c0 eb 0a 89
>
> 100% reproducible.
>
> machine is a toshiba tecra 8000 laptop (p3-500), kernel is 2.5.59 +
> vmlinux.lds.h patch + sound-firmware-load-fix + jaroslav's driver patch +
> adam's pnp.h id len patch.
>
> kernel config: no acpi, _all_ pnp options enabled. soundcore compiled in,
> alsa as module.
>
> # lspci
> 00:00.0 Host bridge: Intel Corporation 440BX/ZX - 82443BX/ZX Host bridge
> (AGP disabled) (rev 03) 00:04.0 VGA compatible controller: Neomagic
> Corporation [MagicMedia 256AV] (rev 12) 00:05.0 Bridge: Intel Corporation
> 82371AB PIIX4 ISA (rev 02)
> 00:05.1 IDE interface: Intel Corporation 82371AB PIIX4 IDE (rev 01)
> 00:05.2 USB Controller: Intel Corporation 82371AB PIIX4 USB (rev 01)
> 00:05.3 Bridge: Intel Corporation 82371AB PIIX4 ACPI (rev 02)
> 00:09.0 Communication controller: Toshiba America Info Systems FIR Port
> (rev 23) 00:0b.0 CardBus bridge: Toshiba America Info Systems ToPIC97 (rev
> 05) 00:0b.1 CardBus bridge: Toshiba America Info Systems ToPIC97 (rev 05)
>
> # lspnp
> 01 PNP0c01 memory controller: RAM
> 02 PNP0200 system peripheral: DMA controller
> 03 PNP0000 system peripheral: programmable interrupt controller
> 04 PNP0100 system peripheral: system timer
> 05 PNP0800 system peripheral: other
> 06 PNP0c04 reserved: other
> 07 PNP0303 input device: keyboard
> 08 PNP0f13 input device: mouse
> 09 PNP0b00 system peripheral: real time clock
> 0a PNP0c02 system peripheral: other
> 0b PNP0700 mass storage device: floppy
> 0e PNP0501 communications device: RS-232
> 10 PNP0401 communications device: AT parallel port
> 11 PNP0a03 bridge controller: PCI
> 14 PNP0e03 bridge controller: PCMCIA
> 15 YMH0021 multimedia controller: audio
>
> # lspnp -vv 15
> 15 YMH0021 multimedia controller: audio
>     flags: [dynamic]
>     allocated resources:
>         io 0x0220-0x0233 [16-bit decode]
>         io 0x0530-0x0537 [16-bit decode]
>         io 0x0388-0x038f [16-bit decode]
>         io 0x0330-0x0333 [16-bit decode]
>         io 0x0538-0x0539 [16-bit decode]
>         irq 5 [high edge]
>         dma 0 [8 bit] [count byte] [compat]
>         dma 1 [8 bit] [count byte] [compat]
>     possible resources:
>         [start dep fn]
>         io base 0x0220-0x0280 align 0x20 len 0x14 [16-bit decode]
>         io 0x0530-0x0537 [16-bit decode]
>         io base 0x0388-0x03b8 align 0x10 len 0x08 [16-bit decode]
>         io base 0x0300-0x0330 align 0x10 len 0x04 [16-bit decode]
>         io 0x0538-0x0539 [16-bit decode]
>         irq mask 0x0ca0 [high edge]
>         dma mask 0x000b [8 bit] [count byte] [compat]
>         dma mask 0x000b [8 bit] [count byte] [compat]
>         [start dep fn]
>         io base 0x0220-0x0280 align 0x20 len 0x14 [16-bit decode]
>         io 0x0530-0x0537 [16-bit decode]
>         io base 0x0388-0x03b8 align 0x10 len 0x08 [16-bit decode]
>         io base 0x0300-0x0330 align 0x10 len 0x04 [16-bit decode]
>         io 0x0538-0x0539 [16-bit decode]
>         irq mask 0x0ca0 [high edge]
>         dma mask 0x000b [8 bit] [count byte] [compat]
>         dma disabled [8 bit] [count byte] [compat]
>         [start dep fn]
>         io base 0x0220-0x0280 align 0x20 len 0x14 [16-bit decode]
>         io 0x0540-0x0547 [16-bit decode]
>         io base 0x0388-0x03b8 align 0x10 len 0x08 [16-bit decode]
>         io base 0x0300-0x0330 align 0x10 len 0x04 [16-bit decode]
>         io 0x0548-0x0549 [16-bit decode]
>         irq mask 0x0ca0 [high edge]
>         dma mask 0x000b [8 bit] [count byte] [compat]
>         dma mask 0x000b [8 bit] [count byte] [compat]
>         [start dep fn]
>         io base 0x0220-0x0280 align 0x20 len 0x14 [16-bit decode]
>         io 0x0540-0x0547 [16-bit decode]
>         io base 0x0388-0x03b8 align 0x10 len 0x08 [16-bit decode]
>         io base 0x0300-0x0330 align 0x10 len 0x04 [16-bit decode]
>         io 0x0548-0x0549 [16-bit decode]
>         irq mask 0x0ca0 [high edge]
>         dma mask 0x000b [8 bit] [count byte] [compat]
>         dma disabled [8 bit] [count byte] [compat]
>         [start dep fn]
>         io base 0x0220-0x0280 align 0x20 len 0x14 [16-bit decode]
>         io 0x0550-0x0557 [16-bit decode]
>         io base 0x0388-0x03b8 align 0x10 len 0x08 [16-bit decode]
>         io base 0x0300-0x0330 align 0x10 len 0x04 [16-bit decode]
>         io 0x0558-0x0559 [16-bit decode]
>         irq mask 0x0ca0 [high edge]
>         dma mask 0x000b [8 bit] [count byte] [compat]
>         dma mask 0x000b [8 bit] [count byte] [compat]
>         [start dep fn]
>         io base 0x0220-0x0280 align 0x20 len 0x14 [16-bit decode]
>         io 0x0550-0x0557 [16-bit decode]
>         io base 0x0388-0x03b8 align 0x10 len 0x08 [16-bit decode]
>         io base 0x0300-0x0330 align 0x10 len 0x04 [16-bit decode]
>         io 0x0558-0x0559 [16-bit decode]
>         irq mask 0x0ca0 [high edge]
>         dma mask 0x000b [8 bit] [count byte] [compat]
>         dma disabled [8 bit] [count byte] [compat]
>         [start dep fn]
>         io base 0x0220-0x0280 align 0x20 len 0x14 [16-bit decode]
>         io 0x0560-0x0567 [16-bit decode]
>         io base 0x0388-0x03b8 align 0x10 len 0x08 [16-bit decode]
>         io base 0x0300-0x0330 align 0x10 len 0x04 [16-bit decode]
>         io 0x0568-0x0569 [16-bit decode]
>         irq mask 0x0ca0 [high edge]
>         dma mask 0x000b [8 bit] [count byte] [compat]
>         dma mask 0x000b [8 bit] [count byte] [compat]
>         [start dep fn]
>         io base 0x0220-0x0280 align 0x20 len 0x14 [16-bit decode]
>         io 0x0560-0x0567 [16-bit decode]
>         io base 0x0388-0x03b8 align 0x10 len 0x08 [16-bit decode]
>         io base 0x0300-0x0330 align 0x10 len 0x04 [16-bit decode]
>         io 0x0568-0x0569 [16-bit decode]
>         irq mask 0x0ca0 [high edge]
>         dma mask 0x000b [8 bit] [count byte] [compat]
>         dma disabled [8 bit] [count byte] [compat]
>         [end dep fn]
>     compatible devices:
>         identifier 'OPL3-SA3 Sound System'
>
>
> if you need more info or if you want me to test patches...just ask
>
> rgds,
> -daniel