[ALSA - driver 0000860]: Wrong detection of opl3 device

[bugtrack@alsa-project.org]

A NOTE has been added to this issue.
======================================================================
<https://bugtrack.alsa-project.org/alsa-bug/view.php?id=860> 
======================================================================
Reported By:                theli
Assigned To:                
======================================================================
Project:                    ALSA - driver
Issue ID:                   860
Category:                   ISA - als100
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Distribution:               
Kernel Version:             2.6.10-ck4
======================================================================
Date Submitted:             01-29-2005 18:20 CET
Last Modified:              02-07-2005 16:13 CET
======================================================================
Summary:                    Wrong detection of opl3 device
Description: 
driver als100.c has
        acard->devmpu = pnp_request_card_device(card, id->devs[1].id,
acard->dev);
        acard->devopl = pnp_request_card_device(card, id->devs[2].id,
acard->devmpu);
in "static int __devinit snd_card_als100_pnp".

so search for opl device starts from devmpu...
and in my MF1000 opl device is 00:01.03
and opl3 is 00:01.01 so it can't be found that way ...
and devopl is NULL :(

but opl3 works when changed above code to 

        acard->devmpu = pnp_request_card_device(card, id->devs[1].id,
acard->dev);
        acard->devopl = pnp_request_card_device(card, id->devs[2].id,
acard->dev);

(search of opl3 starts from acard->dev )
and that should work in any case

PS: sorry for my english.
PPS: don't even know does anyone else use such old sound card.
PPPS: was fighting with alsa for weeks trying to get opl3 working before
decided to look in sources.
======================================================================

----------------------------------------------------------------------
 Clemens Ladisch - 02-07-05 10:50 
----------------------------------------------------------------------
Please post a Signed-off-by line for your patch (see
linux/Documentation/SubmittingPatches).

----------------------------------------------------------------------
 theli - 02-07-05 16:13 
----------------------------------------------------------------------
well i've added signed-off-by and description in patch...
but .... well, my english is far from perfect :)

(hm, wonder why i can't delete attachment, uploaded by me?)

Issue History
Date Modified  Username       Field                    Change              
======================================================================
01-29-05 18:20 theli          New Issue                                    
01-29-05 18:20 theli          Kernel Version            => 2.6.10-ck4      
01-29-05 18:42 theli          Issue Monitored: theli                       
01-29-05 19:43 theli          Note Added: 0003439                          
02-05-05 19:53 theli          File Added: als100_opl3_fix.patch                 
  
02-05-05 19:54 theli          Note Added: 0003523                          
02-07-05 10:50 Clemens LadischNote Added: 0003529                          
02-07-05 16:06 theli          File Added: als100_opl3_fixx.patch                
   
02-07-05 16:13 theli          Note Added: 0003531                          
======================================================================




-------------------------------------------------------
This SF.Net email is sponsored by: IntelliVIEW -- Interactive Reporting
Tool for open source databases. Create drag-&-drop reports. Save time
by over 75%! Publish reports on the web. Export to DOC, XLS, RTF, etc.
Download a FREE copy at http://www.intelliview.com/go/osdn_nl